From 8d30bc61e06269d71646ab25f5d5605b4d24b321 Mon Sep 17 00:00:00 2001 From: Jeremy Wall Date: Tue, 5 Jan 2021 21:05:35 -0500 Subject: [PATCH] On second thought dont store IP as a label --- src/icmp.rs | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/icmp.rs b/src/icmp.rs index a955fb8..a9375b6 100644 --- a/src/icmp.rs +++ b/src/icmp.rs @@ -72,10 +72,10 @@ pub fn start_echo_loop( r.elapsed.as_millis(), ); ping_counter - .with(&prometheus::labels! {"result" => "ok", "domain" => domain_name, "ip" => &resolved}) + .with(&prometheus::labels! {"result" => "ok", "domain" => domain_name}) .inc(); ping_latency_guage - .with(&prometheus::labels! {"domain" => domain_name, "ip" => &resolved}) + .with(&prometheus::labels! {"domain" => domain_name}) .set(r.elapsed.as_millis() as i64); } EkkoResponse::UnreachableResponse((_, ref _code)) => { @@ -83,7 +83,7 @@ pub fn start_echo_loop( error!("{:?}", r); info!("Restarting our sender"); ping_counter - .with(&prometheus::labels! {"result" => "unreachable", "domain" => domain_name, "ip" => &resolved}) + .with(&prometheus::labels! {"result" => "unreachable", "domain" => domain_name}) .inc(); let resolved = resolve_host_address(domain_name); let mut new_sender = Ekko::with_target(&resolved).unwrap(); @@ -92,19 +92,19 @@ pub fn start_echo_loop( } EkkoResponse::ExceededResponse(_) => { ping_counter - .with(&prometheus::labels! {"result" => "timeout", "domain" => domain_name, "ip" => &resolved}) + .with(&prometheus::labels! {"result" => "timeout", "domain" => domain_name}) .inc(); } _ => { ping_counter - .with(&prometheus::labels! {"result" => "err", "domain" => domain_name, "ip" => &resolved}) + .with(&prometheus::labels! {"result" => "err", "domain" => domain_name}) .inc(); error!("{:?}", r); } }, Err(e) => { ping_counter - .with(&prometheus::labels! {"result" => "err", "domain" => domain_name, "ip" => &resolved}) + .with(&prometheus::labels! {"result" => "err", "domain" => domain_name}) .inc(); error!("Ping send to domain: {} address: {} failed: {:?}, Trying again later", domain_name, &resolved, e); }