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

Feature: update BBE to version 0.23.0 #384

Merged
merged 1 commit into from
Dec 13, 2022
Merged

Feature: update BBE to version 0.23.0 #384

merged 1 commit into from
Dec 13, 2022

Conversation

mem
Copy link
Contributor

@mem mem commented Dec 13, 2022

This exposes a new metric, probe_dns_query_succeeded, which is set to 1 after the query is sent to the DNS server and the server responds. This allows the user to differentiate between cases when the server failed to respond (and the probe is reported as having failed) vs the server responding with somehting that causes the probe to fail. In other words, probe_success set to 0 but probe_dns_query_succeeded not set means the DNS server failed to respond, while probe_success set to 0 and probe_dns_query_succeeded set to 1 means the server responded but the probe failed for a different reason (e.g. the record does not exist).

Signed-off-by: Marcelo E. Magallon marcelo.magallon@grafana.com

This exposes a new metric, probe_dns_query_succeeded, which is set to 1
after the query is sent to the DNS server and the server responds. This
allows the user to differentiate between cases when the server failed to
respond (and the probe is reported as having failed) vs the server
responding with somehting that causes the probe to fail. In other words,
probe_success set to 0 but probe_dns_query_succeeded not set means the
DNS server failed to respond, while probe_success set to 0 and
probe_dns_query_succeeded set to 1 means the server responded but the
probe failed for a different reason (e.g. the record does not exist).

Signed-off-by: Marcelo E. Magallon <marcelo.magallon@grafana.com>
@mem mem requested a review from a team as a code owner December 13, 2022 23:15
@mem mem merged commit b8437ea into main Dec 13, 2022
@mem mem deleted the update_bbe_v0.23.0 branch December 13, 2022 23:32
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.

None yet

1 participant