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

feat(helm/operator): statefulset labels and annotations #7688

Merged
merged 3 commits into from
Apr 25, 2024

Conversation

nromriell
Copy link
Contributor

Updates eck-operator helm chart to support annotations and labels on the statefulset for the operator. This is useful for telemetry and tracking.

Adds two new keys to values
statefulsetAnnotations -- defines the annotations that should be added to the operator StatefulSet
statefulsetLabels -- defines additional labels that should be added to the operator StatefulSet

And then updates deploy/eck-operator/templates/statefulset.yaml to support these new values

Signed-off-by: nromriell <nateromriell@gmail.com>
@botelastic botelastic bot added the triage label Apr 4, 2024
@thbkrkr thbkrkr added the >feature Adds or discusses adding a feature to the product label Apr 4, 2024
@botelastic botelastic bot removed the triage label Apr 4, 2024
Copy link
Contributor

@thbkrkr thbkrkr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@thbkrkr
Copy link
Contributor

thbkrkr commented Apr 25, 2024

buildkite test this

@thbkrkr thbkrkr enabled auto-merge (squash) April 25, 2024 16:27
@thbkrkr thbkrkr disabled auto-merge April 25, 2024 16:27
@thbkrkr thbkrkr enabled auto-merge (squash) April 25, 2024 16:28
@thbkrkr thbkrkr changed the title feat(helm): statefulset labels and annotations feat(helm/operator): statefulset labels and annotations Apr 25, 2024
@thbkrkr thbkrkr merged commit d1d4a48 into elastic:main Apr 25, 2024
5 checks passed
@thbkrkr
Copy link
Contributor

thbkrkr commented Apr 25, 2024

Thank you Nathan!

@nromriell
Copy link
Contributor Author

Thanks @thbkrkr!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>feature Adds or discusses adding a feature to the product :helm-charts v2.13.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants