Add period label to AutoloadDuration metric (#19)

Author: cristinaleonr

handler/handler.go

@@ -63,11 +63,11 @@ func (c *Client) Load(w http.ResponseWriter, r *http.Request) {
 		t := time.Now()
 		err := c.processDatatype(ctx, dt, opts)
 		if err != nil {
-			metrics.AutoloadDuration.WithLabelValues(dt.Experiment, dt.Name, "error").Observe(time.Since(t).Seconds())
+			metrics.AutoloadDuration.WithLabelValues(dt.Experiment, dt.Name, opts.period, "error").Observe(time.Since(t).Seconds())
 			errs = append(errs, fmt.Sprintf("failed to autoload %s.%s: %s", dt.Experiment, dt.Name, err.Error()))
 			continue
 		}
-		metrics.AutoloadDuration.WithLabelValues(dt.Experiment, dt.Name, "OK").Observe(time.Since(t).Seconds())
+		metrics.AutoloadDuration.WithLabelValues(dt.Experiment, dt.Name, opts.period, "OK").Observe(time.Since(t).Seconds())
 	}
 
 	if len(errs) != 0 {

metrics/metrics.go

@@ -19,7 +19,7 @@ var (
 				10000, 21500, 46400,
 			},
 		},
-		[]string{"experiment", "datatype", "status"},
+		[]string{"experiment", "datatype", "period", "status"},
 	)
 
 	// BigQueryOperationsTotal counts the number of create, update, and load operations

metrics/metrics_test.go

@@ -3,7 +3,7 @@ package metrics
 import "testing"
 
 func TestLintMetrics(t *testing.T) {
-	AutoloadDuration.WithLabelValues("experiment", "datatype", "status")
+	AutoloadDuration.WithLabelValues("experiment", "datatype", "period", "status")
 	BigQueryOperationsTotal.WithLabelValues("experiment", "datatype", "operation", "status")
 	LoadedDates.WithLabelValues("experiment", "datatype", "period", "status")
 }