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

[8.x](backport #4722) Update labels to use kubernetes recommended labels #5759

Merged
merged 1 commit into from
Oct 11, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Oct 11, 2024

What does this PR do?

Kubernetes have not used the app or k8s-app for some time
and have provided guidance on what the recommended labels are
for some time as seen in their docs.

This commit updates all manifests/docs to use this approach.

Why is it important?

Users who choose to adopt kubernetes label best practises will not have to fork elastic manifests and can instead rely on the upstream ones directly.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in ./changelog/fragments using the changelog tool
  • I have added an integration test or an E2E test

Disruptive User Impact

Users will need to upgrade using kubectl replace as some of the labels are immutable


This is an automatic backport of pull request #4722 done by [Mergify](https://mergify.com).

Kubernetes have not used the `app` or `k8s-app` for some time
and have provided guidance on what the recommended labels are
for some time as seen in their [docs](https://kubernetes.io/docs/concepts/overview/working-with-objects/common-labels/).

This commit updates all manifests/docs to use this approach.

Co-authored-by: Julien Lind <julien.lind@elastic.co>
(cherry picked from commit f2af90f)
@mergify mergify bot requested review from a team as code owners October 11, 2024 07:32
@mergify mergify bot added the backport label Oct 11, 2024
@mergify mergify bot requested review from gizas, tetianakravchenko, blakerouse and swiatekm and removed request for a team October 11, 2024 07:32
@pierrehilbert pierrehilbert enabled auto-merge (squash) October 11, 2024 07:39
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@pierrehilbert pierrehilbert merged commit e151978 into 8.x Oct 11, 2024
15 checks passed
@pierrehilbert pierrehilbert deleted the mergify/bp/8.x/pr-4722 branch October 11, 2024 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants