-
-
Notifications
You must be signed in to change notification settings - Fork 241
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
WIP: data/Debian.yaml, add package- and service-names for Debian-based prometheus-packages #432
WIP: data/Debian.yaml, add package- and service-names for Debian-based prometheus-packages #432
Conversation
e3c0bc6
to
265f2a0
Compare
… with install_method=url)
265f2a0
to
35db014
Compare
@unki can you take a look at the failing tests? |
Need your advice here. The situation is - even before my #430 - that a lot of exporters are using When the Shall I...
|
i think that would be the way to go. if i remember correctly, that's how Arch does it. and thanks for tackling this! i've got profiles here to work around this problem, would be great to get rid of them. |
In general I think we should always rely on packages if some are available. Arch Linux packages a few exporters and I implemented that in this module as well. |
Dear @unki, thanks for the PR! This is Vox Pupuli Tasks, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase? You can find my sourcecode at voxpupuli/vox-pupuli-tasks |
@unki is still relevant? |
I will close this due to inactivity, please feel free to re-open if you still want to work on this |
Together with #430 this PR adapts the Debian-defaults regarding package- and service-names like they are actually present in Debian. e.g. all exporters are prefixed with
prometheus-
.https://packages.debian.org/search?keywords=prometheus&searchon=names&suite=stable§ion=all