-
Notifications
You must be signed in to change notification settings - Fork 347
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
T6949: adds blackbox exporter #4255
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
👍 |
11 tasks
nvollmar
requested review from
dmbaturin,
sarthurdev,
jestabro,
sever-sever,
c-po,
fett0,
nicolas-fort and
zdc
as code owners
December 24, 2024 11:45
c-po
requested changes
Dec 24, 2024
nvollmar
force-pushed
the
T6949
branch
2 times, most recently
from
December 24, 2024 21:13
14395ba
to
9aa9553
Compare
nvollmar
force-pushed
the
T6949
branch
7 times, most recently
from
December 25, 2024 11:28
22133a8
to
eef3a85
Compare
c-po
requested changes
Dec 26, 2024
nvollmar
force-pushed
the
T6949
branch
4 times, most recently
from
December 26, 2024 15:19
d23407e
to
b444bbc
Compare
c-po
requested changes
Dec 26, 2024
nvollmar
force-pushed
the
T6949
branch
2 times, most recently
from
December 26, 2024 15:41
fbc98d5
to
90e98d2
Compare
c-po
approved these changes
Dec 26, 2024
The smoke tests won't work until the vyos-build changes are in the image. |
Do we need to update debian/control? |
auto-merge was automatically disabled
December 30, 2024 06:10
Head branch was pushed to by a user without write access
Aadded |
CI integration ❌ failed! Details
|
sever-sever
approved these changes
Dec 31, 2024
Integration failed by not this feature |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Change Summary
Adds Prometheus Blackbox Exporter
Types of changes
Related Task(s)
Related PR(s)
Component(s) name
monitoring
Proposed changes
Adding Blackbox Exporter allows Prometheus to probe endpoints over HTTP, HTTPS, DNS, TCP, ICMP and gRPC.
This can deliver metrics about upstream connectivity, dns service responsiveness etc.
How to test
Enable blackbox_exporter
Then check service is running
Smoketest result
Checklist: