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

[prometheus-json-exporter] Add default action to ServiceMonitor #4591

Merged

Conversation

yahel2410
Copy link
Contributor

@yahel2410 yahel2410 commented Jun 7, 2024

What this PR does / why we need it

Add the default action: replace to ServiceAccount.
Otherwise ArgoCD shows constant diff on the ServiceAccount resource. See image

image

Checklist

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

@yahel2410 yahel2410 requested review from xiu and zanhsieh as code owners June 7, 2024 00:17
@yahel2410 yahel2410 closed this Jun 7, 2024
@yahel2410 yahel2410 reopened this Jun 7, 2024
@yahel2410 yahel2410 force-pushed the json-exporter-add-action-replace branch from 74952bc to 687e9a3 Compare June 7, 2024 07:54
@elihaz
Copy link

elihaz commented Jun 7, 2024

@xiu @zanhsieh We really need it, can you review this 🙏

@zanhsieh zanhsieh merged commit 31342d2 into prometheus-community:main Jun 8, 2024
4 checks passed
nerzhul pushed a commit to nerzhul/helm-charts that referenced this pull request Jun 9, 2024
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