Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sustainable Kibana Architecture: Temporarily allow non-snake case package names #201736

Merged

Conversation

gsoldevila
Copy link
Contributor

@gsoldevila gsoldevila commented Nov 26, 2024

Summary

In the context of Sustainable Kibana Architecture, we want to relocate plugins and packages to dedicated solutions folders.
Current CI checks enforce snake case for new package names, and we cannot afford the extra churn of renaming all packages' folders during relocation.

The PR aims at relaxing this constraint, only for the target folders of the relocation.

@gsoldevila gsoldevila added Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc release_note:skip Skip the PR/issue when compiling release notes backport:skip This commit does not require backporting labels Nov 26, 2024
@gsoldevila gsoldevila requested a review from a team as a code owner November 26, 2024 09:49
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-core (Team:Core)

@gsoldevila gsoldevila force-pushed the kbn-team-1309-relax-snake-casing-rule branch from 4bb93d5 to 39bc24e Compare November 26, 2024 10:28
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #24 / management Index patterns on aliases discover verify hits should be able to discover and verify no of hits for alias2

Metrics [docs]

✅ unchanged

@gsoldevila gsoldevila merged commit 2850f68 into elastic:main Nov 26, 2024
22 checks passed
paulinashakirova pushed a commit to paulinashakirova/kibana that referenced this pull request Nov 26, 2024
…kage names (elastic#201736)

## Summary

In the context of _Sustainable Kibana Architecture_, we want to relocate
plugins and packages to dedicated solutions folders.
Current CI checks enforce snake case for new package names, and we
cannot afford the extra churn of renaming all packages' folders during
relocation.

The PR aims at relaxing this constraint, only for the target folders of
the relocation.
@gsoldevila gsoldevila added backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) and removed backport:skip This commit does not require backporting labels Dec 9, 2024
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/12235458969

@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/12235458954

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Dec 9, 2024
…kage names (elastic#201736)

## Summary

In the context of _Sustainable Kibana Architecture_, we want to relocate
plugins and packages to dedicated solutions folders.
Current CI checks enforce snake case for new package names, and we
cannot afford the extra churn of renaming all packages' folders during
relocation.

The PR aims at relaxing this constraint, only for the target folders of
the relocation.

(cherry picked from commit 2850f68)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Dec 9, 2024
…kage names (elastic#201736)

## Summary

In the context of _Sustainable Kibana Architecture_, we want to relocate
plugins and packages to dedicated solutions folders.
Current CI checks enforce snake case for new package names, and we
cannot afford the extra churn of renaming all packages' folders during
relocation.

The PR aims at relaxing this constraint, only for the target folders of
the relocation.

(cherry picked from commit 2850f68)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Dec 9, 2024
…se package names (#201736) (#203404)

# Backport

This will backport the following commits from `main` to `8.x`:
- [Sustainable Kibana Architecture: Temporarily allow non-snake case
package names (#201736)](#201736)

<!--- Backport version: 9.4.3 -->

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

<!--BACKPORT [{"author":{"name":"Gerard
Soldevila","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-26T12:58:18Z","message":"Sustainable
Kibana Architecture: Temporarily allow non-snake case package names
(#201736)\n\n## Summary\r\n\r\nIn the context of _Sustainable Kibana
Architecture_, we want to relocate\r\nplugins and packages to dedicated
solutions folders.\r\nCurrent CI checks enforce snake case for new
package names, and we\r\ncannot afford the extra churn of renaming all
packages' folders during\r\nrelocation.\r\n\r\nThe PR aims at relaxing
this constraint, only for the target folders of\r\nthe
relocation.","sha":"2850f68406960592b71a8e8ee21b1a8831f7348b","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Core","release_note:skip","v9.0.0","backport:prev-minor"],"title":"Sustainable
Kibana Architecture: Temporarily allow non-snake case package
names","number":201736,"url":"https://github.com/elastic/kibana/pull/201736","mergeCommit":{"message":"Sustainable
Kibana Architecture: Temporarily allow non-snake case package names
(#201736)\n\n## Summary\r\n\r\nIn the context of _Sustainable Kibana
Architecture_, we want to relocate\r\nplugins and packages to dedicated
solutions folders.\r\nCurrent CI checks enforce snake case for new
package names, and we\r\ncannot afford the extra churn of renaming all
packages' folders during\r\nrelocation.\r\n\r\nThe PR aims at relaxing
this constraint, only for the target folders of\r\nthe
relocation.","sha":"2850f68406960592b71a8e8ee21b1a8831f7348b"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/201736","number":201736,"mergeCommit":{"message":"Sustainable
Kibana Architecture: Temporarily allow non-snake case package names
(#201736)\n\n## Summary\r\n\r\nIn the context of _Sustainable Kibana
Architecture_, we want to relocate\r\nplugins and packages to dedicated
solutions folders.\r\nCurrent CI checks enforce snake case for new
package names, and we\r\ncannot afford the extra churn of renaming all
packages' folders during\r\nrelocation.\r\n\r\nThe PR aims at relaxing
this constraint, only for the target folders of\r\nthe
relocation.","sha":"2850f68406960592b71a8e8ee21b1a8831f7348b"}}]}]
BACKPORT-->

Co-authored-by: Gerard Soldevila <[email protected]>
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 12, 2024
…kage names (elastic#201736)

## Summary

In the context of _Sustainable Kibana Architecture_, we want to relocate
plugins and packages to dedicated solutions folders.
Current CI checks enforce snake case for new package names, and we
cannot afford the extra churn of renaming all packages' folders during
relocation.

The PR aims at relaxing this constraint, only for the target folders of
the relocation.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) release_note:skip Skip the PR/issue when compiling release notes Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants