fix: subcommand flags in metric

sindrerh2 · ybelMekk · sindrerh2 · commit 452638f70872 · 2024-11-05T10:53:26.000+01:00
Co-authored-by: ybelmekk &lt;youssef.bel.mekki@nav.no&gt;
Co-authored-by: Carl Hedgren&lt;carl.hedgren@nav.no&gt;
diff --git a/pkg/metrics/otel.go b/pkg/metrics/otel.go
@@ -60,8 +60,9 @@ func recordCommandUsage(ctx context.Context, provider *metric.MeterProvider, fla
 	for i, f := range flags {
 		if i == 0 {
 			attributes[0] = attribute.String("command", f)
+		} else {
+			attributes[i] = attribute.String("subcommand", f)
 		}
-		attributes[i] = attribute.String("subcommand", f)
 	}
 	commandHistogram.Record(ctx, 1, m.WithAttributes(attributes...))
 }

Original file line number	Diff line number	Diff line change
`@@ -60,8 +60,9 @@ func recordCommandUsage(ctx context.Context, provider *metric.MeterProvider, fla`
`60`	`60`	`for i, f := range flags {`
`61`	`61`	`if i == 0 {`
`62`	`62`	`attributes[0] = attribute.String("command", f)`
	`63`	`+ } else {`
	`64`	`+ attributes[i] = attribute.String("subcommand", f)`
`63`	`65`	`}`
`64`		`- attributes[i] = attribute.String("subcommand", f)`
`65`	`66`	`}`
`66`	`67`	`commandHistogram.Record(ctx, 1, m.WithAttributes(attributes...))`
`67`	`68`	`}`