fixes

scionproto · Sep 23, 2019 · 97de2d4 · 97de2d4
1 parent b58197a
commit 97de2d4
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 8 deletions.
diff --git a/go/border/internal/metrics/ctrl.go b/go/border/internal/metrics/ctrl.go
@@ -50,13 +50,13 @@ type control struct {
 
 func newControl() control {
 	sub := "control"
-	intf := IntfLabels{}
-	l := intf.Labels()
+	il := IntfLabels{}
+	cl := ControlLabels{}
 	return control{
 		pkts: prom.NewCounterVec(Namespace, sub,
-			"pkts_total", "Total number of processed packets.", l),
+			"pkts_total", "Total number of processed packets.", cl.Labels()),
 		ifstate: prom.NewGaugeVec(Namespace, sub,
-			"interface_active", "Interface is active.", l),
+			"interface_active", "Interface is active.", il.Labels()),
 	}
 }
 

diff --git a/go/border/internal/metrics/process.go b/go/border/internal/metrics/process.go
@@ -49,12 +49,12 @@ type process struct {
 func newProcess() process {
 	sub := "process"
 	pl := ProcessLabels{}
-	l := pl.Labels()
+	il := IntfLabels{}
 	return process{
 		pkts: prom.NewCounterVec(Namespace, sub,
-			"pkts_total", "Total number of processed packets.", l),
+			"pkts_total", "Total number of processed packets.", pl.Labels()),
 		time: prom.NewCounterVec(Namespace, sub,
-			"time_seconds_total", "Total packet processing time.", l),
+			"time_seconds_total", "Total packet processing time.", il.Labels()),
 	}
 }
 

diff --git a/go/border/rctrl/ifstate.go b/go/border/rctrl/ifstate.go
@@ -94,9 +94,9 @@ func genIFStateReq() error {
 			errors = append(errors, common.NewBasicError("Writing IFStateReq", err, "dst", dst))
 			continue
 		}
+		logger.Debug("Sent IFStateReq", "dst", dst, "overlayDst", addr)
 		cl.Result = metrics.Success
 		metrics.Control.PktsWith(cl).Inc()
-		logger.Debug("Sent IFStateReq", "dst", dst, "overlayDst", addr)
 	}
 	return errors.ToError()
 }