switch back to old hostname parsing logic

Author: jpraychev

internal/controller/managedmetric_controller.go

@@ -20,6 +20,7 @@ import (
 	"context"
 	"fmt"
 	"net/url"
+	"strings"
 	"time"
 
 	apierrors "k8s.io/apimachinery/pkg/api/errors"
@@ -308,7 +309,12 @@ func getClusterInfo(config *rest.Config) (string, error) {
 		return "localhost", nil
 	}
 
-	return hostname, nil
+	// Remove any prefix (like "kubernetes" or "kubernetes.default.svc")
+	parts := strings.Split(hostname, ".")
+	clusterName := parts[0]
+
+	return clusterName, nil
+
 }
 
 // OrchestratorFactory is a function type for creating orchestrators

internal/controller/metric_controller_helpers_test.go

@@ -45,22 +45,22 @@ func TestGetClusterInfo(t *testing.T) {
 		{
 			name:         "KubernetesService",
 			host:         "https://kubernetes.default.svc:6443",
-			expectedName: "kubernetes.default.svc",
+			expectedName: "kubernetes",
 		},
 		{
 			name:         "CustomClusterName",
 			host:         "https://my-cluster-api.example.com:6443",
-			expectedName: "my-cluster-api.example.com",
+			expectedName: "my-cluster-api",
 		},
 		{
 			name:         "IPAddress",
 			host:         "https://192.168.1.1:6443",
-			expectedName: "192.168.1.1", // The function only extracts the first part of the IP address
+			expectedName: "192", // The function only extracts the first part of the IP address
 		},
 		{
 			name:         "WithPath",
 			host:         "https://kubernetes.default.svc:6443/api",
-			expectedName: "kubernetes.default.svc",
+			expectedName: "kubernetes",
 		},
 	}