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/add servicemonitor #116

Merged
merged 4 commits into from
Jul 30, 2023
Merged

Conversation

nobbs
Copy link
Contributor

@nobbs nobbs commented Jul 29, 2023

Description of the change

Adds a ServiceMonitor resource if the enabled and the cluster supports it. This will instruct a deployed Prometheus instance in the cluster to scrape Uptime Kuma metrics.

Benefits

Uptime Kuma metrics automatically show up in Prometheus.

Possible drawbacks

None known.

Applicable issues

Additional information

The ServiceMonitor template will check if the cluster does have the monitoring.coreos.com/v1 API capability and only create the object if that check succeeds. This will prevent deployment failure with serviceMonitor.enabled: true in clusters missing the Prometheus monitoring stack.

Checklist

  • Chart version bumped in Chart.yaml according to semver.
  • Variables are documented in the README.md

Signed-off-by: Alexej Disterhoft <alexej@disterhoft.de>
Signed-off-by: Alexej Disterhoft <alexej@disterhoft.de>
Signed-off-by: Alexej Disterhoft <alexej@disterhoft.de>
Signed-off-by: Alexej Disterhoft <alexej@disterhoft.de>
@nobbs nobbs requested a review from dirsigler as a code owner July 29, 2023 20:45
@dirsigler
Copy link
Owner

Crazy good addition. Thanks a lot @nobbs !

@dirsigler dirsigler merged commit 0fea3d0 into dirsigler:main Jul 30, 2023
4 checks passed
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.

2 participants