Merge pull request grafana#338 from xperimental/remove-metrics-5.6

operator: Remove unused telemetry implementation

operator/CHANGELOG.md

@@ -2,6 +2,7 @@
 
 ## Release 5.6.22
 
+- [openshift#338](https://github.com/openshift/loki/pull/338) **xperimental**: Remove unused telemetry metrics
 - [13512](https://github.com/grafana/loki/pull/13512) **xperimental**: feat(operator): Add alert for discarded samples
 
 ## Release 5.6.21

operator/internal/handlers/lokistack_create_or_update.go

@@ -24,7 +24,6 @@ import (
 	"github.com/grafana/loki/operator/internal/handlers/internal/storage"
 	"github.com/grafana/loki/operator/internal/handlers/internal/tlsprofile"
 	"github.com/grafana/loki/operator/internal/manifests"
-	"github.com/grafana/loki/operator/internal/metrics"
 	"github.com/grafana/loki/operator/internal/status"
 )
 
@@ -207,12 +206,6 @@ func CreateOrUpdateLokiStack(
 		return kverrors.New("failed to configure lokistack resources", "name", req.NamespacedName)
 	}
 
-	// 1x.extra-small is used only for development, so the metrics will not
-	// be collected.
-	if opts.Stack.Size != lokiv1.SizeOneXExtraSmall {
-		metrics.Collect(&opts.Stack, opts.Name)
-	}
-
 	return nil
 }
 

operator/main.go

@@ -22,7 +22,6 @@ import (
 	lokiv1beta1 "github.com/grafana/loki/operator/apis/loki/v1beta1"
 	lokictrl "github.com/grafana/loki/operator/controllers/loki"
 	"github.com/grafana/loki/operator/internal/config"
-	"github.com/grafana/loki/operator/internal/metrics"
 	"github.com/grafana/loki/operator/internal/validation"
 	"github.com/grafana/loki/operator/internal/validation/openshift"
 
@@ -175,9 +174,6 @@ func main() {
 		os.Exit(1)
 	}
 
-	logger.Info("registering metrics")
-	metrics.RegisterMetricCollectors()
-
 	logger.Info("Registering profiling endpoints.")
 	err = registerProfiler(mgr)
 	if err != nil {