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

[tempo-distributed] Fix global.extraEnv key support for metrics-generator statefulset #3349

Merged
merged 6 commits into from
Oct 14, 2024

Conversation

bruksnys
Copy link
Contributor

@bruksnys bruksnys commented Oct 9, 2024

Fixes global.extraEnv key support for metrics-generator when it is deployed as kind: StatefulSet

Resolves: #3346

@Sheikh-Abubaker
Copy link
Collaborator

Hey @bruksnys Thanks for the contribution! could you please address the failed CI check by running helm-docs inside charts/tempo-distributed and then we are good to go!

@bruksnys
Copy link
Contributor Author

bruksnys commented Oct 9, 2024

Hey @bruksnys Thanks for the contribution! could you please address the failed CI check by running helm-docs inside charts/tempo-distributed and then we are good to go!

Sure thing! I've already re-run the helm-docs and will be waiting for the green pipeline! 👍

@bruksnys
Copy link
Contributor Author

bruksnys commented Oct 9, 2024

@Sheikh-Abubaker All of the checks are now passing, please let me know if anything else is needed on my side. Thank you!

Copy link
Collaborator

@Sheikh-Abubaker Sheikh-Abubaker left a comment

Choose a reason for hiding this comment

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

LGTM!!

@CLAassistant
Copy link

CLAassistant commented Oct 13, 2024

CLA assistant check
All committers have signed the CLA.

@bruksnys
Copy link
Contributor Author

updated, signed CLA.

@zanhsieh zanhsieh merged commit 70dbbb7 into grafana:main Oct 14, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[tempo-distributed] Missing extraEnv for metrics-generator statefulset
4 participants