From 44ff25177cf6335281c4fc972d0b6f175566d10b Mon Sep 17 00:00:00 2001 From: Doug Goldstein Date: Mon, 24 Mar 2025 15:31:13 -0500 Subject: [PATCH] fix(prometheus): prevent constant sync cycling with ArgoCD The comments in the default values.yaml are wrong for the helm chart for this and one of the maintainers pointed out the correct values at: https://github.com/prometheus-community/helm-charts/issues/4500#issuecomment-2724199565 --- operators/monitoring/values.yaml | 12 ++---------- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/operators/monitoring/values.yaml b/operators/monitoring/values.yaml index 4a18df03d..70517d413 100644 --- a/operators/monitoring/values.yaml +++ b/operators/monitoring/values.yaml @@ -54,18 +54,10 @@ prometheusOperator: enabled: true annotations: - argocd.argoproj.io/hook: PreSync + argocd.argoproj.io/hook: PostSync argocd.argoproj.io/hook-delete-policy: HookSucceeded patch: annotations: - argocd.argoproj.io/hook: PreSync + argocd.argoproj.io/hook: PostSync argocd.argoproj.io/hook-delete-policy: HookSucceeded - - mutatingWebhookConfiguration: - annotations: - argocd.argoproj.io/hook: PreSync - - validatingWebhookConfiguration: - annotations: - argocd.argoproj.io/hook: PreSync