Skip to content

Commit

Permalink
[8.x] SKA: Fix kebab-case issues in obs-ux-logs packages (#211347) (#…
Browse files Browse the repository at this point in the history
…211383)

# Backport

This will backport the following commits from `main` to `8.x`:
- [SKA: Fix kebab-case issues in obs-ux-logs packages
(#211347)](#211347)

<!--- Backport version: 9.6.4 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Gerard
Soldevila","email":"gerard.soldevila@elastic.co"},"sourceCommit":{"committedDate":"2025-02-17T08:08:38Z","message":"SKA:
Fix kebab-case issues in obs-ux-logs packages (#211347)\n\n##
Summary\r\n\r\nThis PR aims at relocating some of the Kibana modules
(plugins and\r\npackages) into a new folder structure, according to the
_Sustainable\r\nKibana Architecture_ initiative.\r\n\r\n>
[!IMPORTANT]\r\n> * We kindly ask you to:\r\n> * Manually fix the errors
in the error section below (if there are\r\nany).\r\n> * Search for the
`packages[\\/\\\\]` and `plugins[\\/\\\\]` patterns in the\r\nsource
code (Babel and Eslint config files), and update
them\r\nappropriately.\r\n> * Manually
review\r\n`.buildkite/scripts/pipelines/pull_request/pipeline.ts` to
ensure that\r\nany CI pipeline customizations continue to be correctly
applied after\r\nthe changed path names\r\n> * Review all of the updated
files, specially the `.ts` and `.js` files\r\nlisted in the sections
below, as some of them contain relative paths\r\nthat have been
updated.\r\n> * Think of potential impact of the move, including tooling
and\r\nconfiguration files that can be pointing to the relocated
modules. E.g.:\r\n> * customised eslint rules\r\n> * docs pointing to
source code\r\n\r\n> [!NOTE]\r\n> * This PR has been
auto-generated.\r\n> * Any manual contributions will be lost if the
'relocate' script is\r\nre-run.\r\n> * Try to obtain the missing reviews
/ approvals before applying manual\r\nfixes, and/or keep your changes in
a .patch / git stash.\r\n> * Please
use\r\n[#sustainable_kibana_architecture](https://elastic.slack.com/archives/C07TCKTA22E)\r\nSlack
channel for feedback.\r\n\r\nAre you trying to rebase this PR to solve
merge conflicts? Please follow\r\nthe steps
describe\r\n[here](https://elastic.slack.com/archives/C07TCKTA22E/p1734019532879269?thread_ts=1734019339.935419&cid=C07TCKTA22E).\r\n\r\n####
1 packages(s) are going to be relocated:\r\n\r\n| Id | Target folder
|\r\n| -- | ------------- |\r\n| `@kbn/logs-overview` |
`x-pack/platform/packages/shared/logs-overview`\r\n|\r\n\r\n\r\n<details
>\r\n<summary>Updated
references</summary>\r\n\r\n```\r\n./package.json\r\n./packages/kbn-ts-projects/config-paths.json\r\n./src/platform/packages/private/kbn-repo-packages/package-map.json\r\n./tsconfig.base.json\r\n./x-pack/.i18nrc.json\r\n./x-pack/platform/packages/shared/logs-overview/jest.config.js\r\n./yarn.lock\r\n.github/CODEOWNERS\r\n```\r\n\r\n</details><details
>\r\n<summary>Updated relative
paths</summary>\r\n\r\n```\r\nx-pack/platform/packages/shared/logs-overview/jest.config.js:10\r\nx-pack/platform/packages/shared/logs-overview/tsconfig.json:2\r\n```\r\n\r\n</details>","sha":"0ecbbd52becb2d48150c837149d54125ddd5c6e3","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:obs-ux-logs","backport:version","v9.1.0","v8.19.0"],"title":"SKA:
Fix kebab-case issues in obs-ux-logs
packages","number":211347,"url":"https://github.com/elastic/kibana/pull/211347","mergeCommit":{"message":"SKA:
Fix kebab-case issues in obs-ux-logs packages (#211347)\n\n##
Summary\r\n\r\nThis PR aims at relocating some of the Kibana modules
(plugins and\r\npackages) into a new folder structure, according to the
_Sustainable\r\nKibana Architecture_ initiative.\r\n\r\n>
[!IMPORTANT]\r\n> * We kindly ask you to:\r\n> * Manually fix the errors
in the error section below (if there are\r\nany).\r\n> * Search for the
`packages[\\/\\\\]` and `plugins[\\/\\\\]` patterns in the\r\nsource
code (Babel and Eslint config files), and update
them\r\nappropriately.\r\n> * Manually
review\r\n`.buildkite/scripts/pipelines/pull_request/pipeline.ts` to
ensure that\r\nany CI pipeline customizations continue to be correctly
applied after\r\nthe changed path names\r\n> * Review all of the updated
files, specially the `.ts` and `.js` files\r\nlisted in the sections
below, as some of them contain relative paths\r\nthat have been
updated.\r\n> * Think of potential impact of the move, including tooling
and\r\nconfiguration files that can be pointing to the relocated
modules. E.g.:\r\n> * customised eslint rules\r\n> * docs pointing to
source code\r\n\r\n> [!NOTE]\r\n> * This PR has been
auto-generated.\r\n> * Any manual contributions will be lost if the
'relocate' script is\r\nre-run.\r\n> * Try to obtain the missing reviews
/ approvals before applying manual\r\nfixes, and/or keep your changes in
a .patch / git stash.\r\n> * Please
use\r\n[#sustainable_kibana_architecture](https://elastic.slack.com/archives/C07TCKTA22E)\r\nSlack
channel for feedback.\r\n\r\nAre you trying to rebase this PR to solve
merge conflicts? Please follow\r\nthe steps
describe\r\n[here](https://elastic.slack.com/archives/C07TCKTA22E/p1734019532879269?thread_ts=1734019339.935419&cid=C07TCKTA22E).\r\n\r\n####
1 packages(s) are going to be relocated:\r\n\r\n| Id | Target folder
|\r\n| -- | ------------- |\r\n| `@kbn/logs-overview` |
`x-pack/platform/packages/shared/logs-overview`\r\n|\r\n\r\n\r\n<details
>\r\n<summary>Updated
references</summary>\r\n\r\n```\r\n./package.json\r\n./packages/kbn-ts-projects/config-paths.json\r\n./src/platform/packages/private/kbn-repo-packages/package-map.json\r\n./tsconfig.base.json\r\n./x-pack/.i18nrc.json\r\n./x-pack/platform/packages/shared/logs-overview/jest.config.js\r\n./yarn.lock\r\n.github/CODEOWNERS\r\n```\r\n\r\n</details><details
>\r\n<summary>Updated relative
paths</summary>\r\n\r\n```\r\nx-pack/platform/packages/shared/logs-overview/jest.config.js:10\r\nx-pack/platform/packages/shared/logs-overview/tsconfig.json:2\r\n```\r\n\r\n</details>","sha":"0ecbbd52becb2d48150c837149d54125ddd5c6e3"}},"sourceBranch":"main","suggestedTargetBranches":["9.0","8.x"],"targetPullRequestStates":[{"branch":"9.0","label":"v9.0.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/211347","number":211347,"mergeCommit":{"message":"SKA:
Fix kebab-case issues in obs-ux-logs packages (#211347)\n\n##
Summary\r\n\r\nThis PR aims at relocating some of the Kibana modules
(plugins and\r\npackages) into a new folder structure, according to the
_Sustainable\r\nKibana Architecture_ initiative.\r\n\r\n>
[!IMPORTANT]\r\n> * We kindly ask you to:\r\n> * Manually fix the errors
in the error section below (if there are\r\nany).\r\n> * Search for the
`packages[\\/\\\\]` and `plugins[\\/\\\\]` patterns in the\r\nsource
code (Babel and Eslint config files), and update
them\r\nappropriately.\r\n> * Manually
review\r\n`.buildkite/scripts/pipelines/pull_request/pipeline.ts` to
ensure that\r\nany CI pipeline customizations continue to be correctly
applied after\r\nthe changed path names\r\n> * Review all of the updated
files, specially the `.ts` and `.js` files\r\nlisted in the sections
below, as some of them contain relative paths\r\nthat have been
updated.\r\n> * Think of potential impact of the move, including tooling
and\r\nconfiguration files that can be pointing to the relocated
modules. E.g.:\r\n> * customised eslint rules\r\n> * docs pointing to
source code\r\n\r\n> [!NOTE]\r\n> * This PR has been
auto-generated.\r\n> * Any manual contributions will be lost if the
'relocate' script is\r\nre-run.\r\n> * Try to obtain the missing reviews
/ approvals before applying manual\r\nfixes, and/or keep your changes in
a .patch / git stash.\r\n> * Please
use\r\n[#sustainable_kibana_architecture](https://elastic.slack.com/archives/C07TCKTA22E)\r\nSlack
channel for feedback.\r\n\r\nAre you trying to rebase this PR to solve
merge conflicts? Please follow\r\nthe steps
describe\r\n[here](https://elastic.slack.com/archives/C07TCKTA22E/p1734019532879269?thread_ts=1734019339.935419&cid=C07TCKTA22E).\r\n\r\n####
1 packages(s) are going to be relocated:\r\n\r\n| Id | Target folder
|\r\n| -- | ------------- |\r\n| `@kbn/logs-overview` |
`x-pack/platform/packages/shared/logs-overview`\r\n|\r\n\r\n\r\n<details
>\r\n<summary>Updated
references</summary>\r\n\r\n```\r\n./package.json\r\n./packages/kbn-ts-projects/config-paths.json\r\n./src/platform/packages/private/kbn-repo-packages/package-map.json\r\n./tsconfig.base.json\r\n./x-pack/.i18nrc.json\r\n./x-pack/platform/packages/shared/logs-overview/jest.config.js\r\n./yarn.lock\r\n.github/CODEOWNERS\r\n```\r\n\r\n</details><details
>\r\n<summary>Updated relative
paths</summary>\r\n\r\n```\r\nx-pack/platform/packages/shared/logs-overview/jest.config.js:10\r\nx-pack/platform/packages/shared/logs-overview/tsconfig.json:2\r\n```\r\n\r\n</details>","sha":"0ecbbd52becb2d48150c837149d54125ddd5c6e3"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
  • Loading branch information
gsoldevila authored Feb 17, 2025
1 parent 7293166 commit 361ca2f
Show file tree
Hide file tree
Showing 50 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion .github/CODEOWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -600,7 +600,7 @@ src/platform/packages/shared/kbn-logging @elastic/kibana-core
src/platform/packages/shared/kbn-logging-mocks @elastic/kibana-core
x-pack/platform/plugins/shared/logs_data_access @elastic/obs-ux-logs-team
x-pack/solutions/observability/plugins/logs_explorer @elastic/obs-ux-logs-team
x-pack/platform/packages/shared/logs_overview @elastic/obs-ux-logs-team
x-pack/platform/packages/shared/logs-overview @elastic/obs-ux-logs-team
x-pack/platform/plugins/shared/logs_shared @elastic/obs-ux-logs-team
x-pack/platform/plugins/private/logstash @elastic/logstash
src/platform/packages/private/kbn-managed-content-badge @elastic/kibana-visualizations
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -632,7 +632,7 @@
"@kbn/logging-mocks": "link:src/platform/packages/shared/kbn-logging-mocks",
"@kbn/logs-data-access-plugin": "link:x-pack/platform/plugins/shared/logs_data_access",
"@kbn/logs-explorer-plugin": "link:x-pack/solutions/observability/plugins/logs_explorer",
"@kbn/logs-overview": "link:x-pack/platform/packages/shared/logs_overview",
"@kbn/logs-overview": "link:x-pack/platform/packages/shared/logs-overview",
"@kbn/logs-shared-plugin": "link:x-pack/platform/plugins/shared/logs_shared",
"@kbn/logstash-plugin": "link:x-pack/platform/plugins/private/logstash",
"@kbn/managed-content-badge": "link:src/platform/packages/private/kbn-managed-content-badge",
Expand Down
4 changes: 2 additions & 2 deletions tsconfig.base.json
Original file line number Diff line number Diff line change
Expand Up @@ -1194,8 +1194,8 @@
"@kbn/logs-data-access-plugin/*": ["x-pack/platform/plugins/shared/logs_data_access/*"],
"@kbn/logs-explorer-plugin": ["x-pack/solutions/observability/plugins/logs_explorer"],
"@kbn/logs-explorer-plugin/*": ["x-pack/solutions/observability/plugins/logs_explorer/*"],
"@kbn/logs-overview": ["x-pack/platform/packages/shared/logs_overview"],
"@kbn/logs-overview/*": ["x-pack/platform/packages/shared/logs_overview/*"],
"@kbn/logs-overview": ["x-pack/platform/packages/shared/logs-overview"],
"@kbn/logs-overview/*": ["x-pack/platform/packages/shared/logs-overview/*"],
"@kbn/logs-shared-plugin": ["x-pack/platform/plugins/shared/logs_shared"],
"@kbn/logs-shared-plugin/*": ["x-pack/platform/plugins/shared/logs_shared/*"],
"@kbn/logstash-plugin": ["x-pack/platform/plugins/private/logstash"],
Expand Down
2 changes: 1 addition & 1 deletion x-pack/.i18nrc.json
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@
"xpack.observability_onboarding": "solutions/observability/plugins/observability_onboarding",
"xpack.observabilityShared": "solutions/observability/plugins/observability_shared",
"xpack.observabilityLogsOverview": [
"platform/packages/shared/logs_overview/src/components"
"platform/packages/shared/logs-overview/src/components"
],
"xpack.osquery": [
"platform/plugins/shared/osquery"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,5 @@
module.exports = {
preset: '@kbn/test',
rootDir: '../../../../..',
roots: ['<rootDir>/x-pack/platform/packages/shared/logs_overview'],
roots: ['<rootDir>/x-pack/platform/packages/shared/logs-overview'],
};
2 changes: 1 addition & 1 deletion yarn.lock
Original file line number Diff line number Diff line change
Expand Up @@ -6242,7 +6242,7 @@
version "0.0.0"
uid ""

"@kbn/logs-overview@link:x-pack/platform/packages/shared/logs_overview":
"@kbn/logs-overview@link:x-pack/platform/packages/shared/logs-overview":
version "0.0.0"
uid ""

Expand Down

0 comments on commit 361ca2f

Please sign in to comment.