Simplify canary-exporter label check to "at least these labels" (#4562)

* Simplify canary-exporter label check to "at least these labels"

* Remove count check

* Remove now-unneeded sort

Author: oleg-kozlyuk-grafana

cmd/profilecli/canary_exporter_probes.go

@@ -9,7 +9,6 @@ import (
 	"math"
 	"net/http"
 	"net/url"
-	"slices"
 	"strings"
 	"time"
 
@@ -493,14 +492,7 @@ func (ce *canaryExporter) testLabelNames(ctx context.Context, now time.Time) err
 		return err
 	}
 
-	expectedLabelCount := 12
-	if len(respQuery.Msg.Names) != expectedLabelCount {
-		level.Error(logger).Log("msg", "received an invalid number of labels", "expected", expectedLabelCount, "received", len(respQuery.Msg.Names), "labels", strings.Join(respQuery.Msg.Names, ","))
-		return fmt.Errorf("expected %d label names, got %d", expectedLabelCount, len(respQuery.Msg.Names))
-	}
-
 	labelNames := respQuery.Msg.Names
-	slices.Sort(labelNames)
 
 	expectedLabelNames := []string{
 		model.LabelNameProfileName,
@@ -510,15 +502,24 @@ func (ce *canaryExporter) testLabelNames(ctx context.Context, now time.Time) err
 		model.LabelNameServiceNamePrivate,
 		model.LabelNameType,
 		model.LabelNameUnit,
-		"ingestion_method",
-		"instance",
-		"job",
 		"service.name",
 		model.LabelNameServiceName,
 	}
 
-	if !slices.Equal(labelNames, expectedLabelNames) {
-		return fmt.Errorf("expected label names to be %s, got %s", expectedLabelNames, labelNames)
+	// Use map as set for O(1) lookups
+	labelNamesSet := make(map[string]struct{}, len(labelNames))
+	for _, label := range labelNames {
+		labelNamesSet[label] = struct{}{}
+	}
+
+	missingLabels := []string{}
+	for _, expectedLabel := range expectedLabelNames {
+		if _, exists := labelNamesSet[expectedLabel]; !exists {
+			missingLabels = append(missingLabels, expectedLabel)
+		}
+	}
+	if len(missingLabels) > 0 {
+		return fmt.Errorf("missing expected labels: %s", missingLabels)
 	}
 
 	return nil