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

[kube-prometheus-stack] add annotations to crds #3942

Closed
wants to merge 5 commits into from

Conversation

JuanRamino
Copy link
Contributor

@JuanRamino JuanRamino commented Oct 27, 2023

What this PR does / why we need it

I need to add annotations to crds keeping them from values

Which issue this PR fixes

(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

  • fixes #

Special notes for your reviewer

Checklist

  • DCO signed
  • Chart Version bumped
  • Title of the PR starts with chart name (e.g. [prometheus-couchdb-exporter])

Signed-off-by: juan ramino <[email protected]>
Signed-off-by: juan ramino <[email protected]>
Copy link

stale bot commented Dec 15, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Any further update will cause the issue/pull request to no longer be considered stale. Thank you for your contributions.

@jkroepke
Copy link
Member

Hi, crds have to stay in crd folder and templating is not supported.

@jkroepke jkroepke closed this Jan 16, 2024
@srekkas
Copy link

srekkas commented Sep 12, 2024

But other chart prometheus-operator-crds have templating for annotations. Why kube-prometheus-stack chart can't have it ??

@jkroepke
Copy link
Member

Because helm does not support it???!?!?!?!!?!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants