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(monitoring): allow disabling prometheus operator #331

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

vknabel
Copy link
Contributor

@vknabel vknabel commented Oct 11, 2024

Additional Description

In some cases CRDs and the prometheus operator will already be installed by Gardener itself leading to two conflicting instances of the operator. These changes allow disabling deploying the prometheus ooperator to solve this conflict.

Release Notes

  • introduces monitoring_prometheus_operator_crds_enabled and monitoring_prometheus_operator_enabled to be able to prevent conflicts with the Gardener Prometheus Operator. Defaults to true to retain the old behavior.

@vknabel vknabel requested a review from Gerrit91 October 11, 2024 08:49
@vknabel vknabel requested a review from a team as a code owner October 11, 2024 08:49
@vknabel vknabel force-pushed the allow-disabling-prometheus-operator branch from b5960ca to b25724d Compare October 11, 2024 08:50
@vknabel
Copy link
Contributor Author

vknabel commented Oct 11, 2024

Info: currently untested

vknabel and others added 4 commits October 17, 2024 10:59
In some cases CRDs and the prometheus operator will already be installed by Gardener itself leading to two conflicting instances of the operator. These changes allow disabling deploying the prometheus ooperator to solve this conflict.
@vknabel vknabel force-pushed the allow-disabling-prometheus-operator branch from f7ddf5a to ce0c26a Compare October 17, 2024 09:04
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.

3 participants