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

[grafana] adds extraObjects as strings support #3602

Merged
merged 5 commits into from
Mar 16, 2025

Conversation

marshallford
Copy link
Contributor

@CLAassistant
Copy link

CLAassistant commented Mar 14, 2025

CLA assistant check
All committers have signed the CLA.

@marshallford marshallford force-pushed the extra-objects-string branch from 0f7297a to cf8e18e Compare March 14, 2025 16:39
@jkroepke
Copy link
Collaborator

Hi, please create a distinct PR for each chart, thanks!

@marshallford marshallford changed the title adds extraObjects as strings support to grafana and tempo-distributed charts adds extraObjects as strings support to grafana chart Mar 14, 2025
@marshallford marshallford force-pushed the extra-objects-string branch from f34f272 to 158b663 Compare March 14, 2025 18:04
Signed-off-by: Marshall Ford <[email protected]>
Signed-off-by: Marshall Ford <[email protected]>
@marshallford marshallford force-pushed the extra-objects-string branch from 158b663 to c5836eb Compare March 14, 2025 18:05
@jkroepke
Copy link
Collaborator

Please also bump the chart version, thanks!

Signed-off-by: Marshall Ford <[email protected]>
@marshallford
Copy link
Contributor Author

Hi, please create a distinct PR for each chart, thanks!

Done: #3603

@jkroepke jkroepke changed the title adds extraObjects as strings support to grafana chart [grafana] adds extraObjects as strings support Mar 14, 2025
Signed-off-by: Marshall Ford <[email protected]>
@Sheikh-Abubaker Sheikh-Abubaker merged commit 2445442 into grafana:main Mar 16, 2025
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.

4 participants