Fix cluster name parsing logic

Author: jpraychev

internal/controller/managedmetric_controller.go

@@ -20,7 +20,6 @@ import (
 	"context"
 	"fmt"
 	"net/url"
-	"strings"
 	"time"
 
 	apierrors "k8s.io/apimachinery/pkg/api/errors"
@@ -309,11 +308,7 @@ func getClusterInfo(config *rest.Config) (string, error) {
 		return "localhost", nil
 	}
 
-	// Remove any prefix (like "kubernetes" or "kubernetes.default.svc")
-	parts := strings.Split(hostname, ".")
-	clusterName := parts[0]
-
-	return clusterName, nil
+	return hostname, 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",
+			expectedName: "kubernetes.default.svc",
 		},
 		{
 			name:         "CustomClusterName",
 			host:         "https://my-cluster-api.example.com:6443",
-			expectedName: "my-cluster-api",
+			expectedName: "my-cluster-api.example.com",
 		},
 		{
 			name:         "IPAddress",
 			host:         "https://192.168.1.1:6443",
-			expectedName: "192", // The function only extracts the first part of the IP address
+			expectedName: "192.168.1.1", // The function only extracts the first part of the IP address
 		},
 		{
 			name:         "WithPath",
 			host:         "https://kubernetes.default.svc:6443/api",
-			expectedName: "kubernetes",
+			expectedName: "kubernetes.default.svc",
 		},
 	}