-
Notifications
You must be signed in to change notification settings - Fork 129
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: Revive the prometheus plugin #582
Conversation
d8d740f
to
c2e8f01
Compare
Excited to see this PR! Will try it out soon :) |
Tests are flaky, i saw some "address already in use" in one of them. |
c2e8f01
to
9a0a86a
Compare
Interesting, looks like this is only broken on 23.11 |
Gonna update the CI to add 24.08 and remove 23.11, then this should be good to go |
b6f7a3c
to
78b17da
Compare
78b17da
to
ee6e482
Compare
Sure, the plugin just exposes the |
Taking a look at this, I have a raspiblitz running cln, going to clone the prometheus plugin branch in and run prometheus and point it at the endpoint. |
I was able to install prometheus and get it to ingest the metrics and display them on port 9100. I had some difficulties with poetry and raspiblitz. Since raspiblitz does everything with |
ee6e482
to
7c565dd
Compare
We have been playing with a number of possible python installation options:
Ideally we'd have something that works everywhere. Maybe a runner script that analyzes the env, adjusts it and then runs the interpreter could be useful, but at that point why not use |
Hmm |
re-ACK 7c565dd |
I also added myself as maintainer in the
CODEOWNERS
file.