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

chore(chart): check CRDs available before installing prometheus resources #108

Closed

Conversation

cmontemuino
Copy link
Contributor

Prometheus resources should not be rendered unless Prometheus CRDs are available.

This changeset includes an extra option to override the apiVersion Prometheus provides.

close #107

@avishnu
Copy link
Member

avishnu commented Sep 13, 2024

There is a conflict that needs resolution.

@abhilashshetty04
Copy link
Member

Hi @cmontemuino , Thanks for thr PR. Can you please remove chart bump from your commit. We will cherry-pick your commit on PR which will have chart bump.

…rces

Prometheus  resources should not be rendered unless Prometheus CRDs are
available.

This changeset includes an extra option to override the `apiVersion` Prometheus
provides.

close openebs#107

Signed-off-by: cmontemuino <1761056+cmontemuino@users.noreply.github.com>
@cmontemuino cmontemuino force-pushed the check-prometheus-capabilities branch from 06a76b7 to 7d5c116 Compare September 16, 2024 08:51
@cmontemuino
Copy link
Contributor Author

Hi @cmontemuino , Thanks for thr PR. Can you please remove chart bump from your commit. We will cherry-pick your commit on PR which will have chart bump.

Hi @abhilashshetty04. I did a rebase against latest changes and kept file Chart.yaml unchanged.

@cmontemuino
Copy link
Contributor Author

Changes from this PR have been cherry-picked in #119

@cmontemuino cmontemuino deleted the check-prometheus-capabilities branch September 18, 2024 06:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants