Skip to content
This repository has been archived by the owner on Jan 12, 2024. It is now read-only.

add support for metrics-endpoint provider #10

Merged
merged 13 commits into from
Jan 20, 2023
Merged

Conversation

lucabello
Copy link
Contributor

Issue

The Mimir charm is currently missing the metrics-endpoint provider which would allow it to be scraped by Prometheus.

Solution

Implement the relation provider in Mimir.

Testing Instructions

Deploy Mimir and Prometheus, relate them over the new interface and verify that Prometheus receives the metrics correctly.

@lucabello lucabello marked this pull request as ready for review January 17, 2023 17:20
requirements.txt Outdated Show resolved Hide resolved
@rbarry82
Copy link
Contributor

Can we just drop it entirely? Wait 10 min :D

tests/integration/test_self_monitoring.py Outdated Show resolved Hide resolved
tests/integration/helpers.py Outdated Show resolved Hide resolved
tests/integration/test_self_monitoring.py Show resolved Hide resolved
src/charm.py Outdated Show resolved Hide resolved
@lucabello
Copy link
Contributor Author

I'm currently putting the recording-rules.yaml file in the prometheus_alert_folder; that file will need to be moved in the appropriate folder after canonical/prometheus-k8s-operator#428 is resolved.

@lucabello lucabello merged commit cf49198 into main Jan 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants