From 253639162d8b9b940656928987dfa0c9ac97ae98 Mon Sep 17 00:00:00 2001
From: Fabian Kramm <fab.kramm@googlemail.com>
Date: Thu, 28 Dec 2023 15:06:28 +0100
Subject: [PATCH] refactor: add isRemote to WriteKubeConfigToSecret

---
 pkg/setup/controller_context.go   | 6 +++---
 pkg/setup/controllers.go          | 2 +-
 pkg/util/kubeconfig/kubeconfig.go | 4 ++--
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/pkg/setup/controller_context.go b/pkg/setup/controller_context.go
index 55730a4818..3defeba0d5 100644
--- a/pkg/setup/controller_context.go
+++ b/pkg/setup/controller_context.go
@@ -286,7 +286,7 @@ func CreateVClusterKubeConfig(config *clientcmdapi.Config, options *options.Virt
 	return config, nil
 }
 
-func WriteKubeConfigToSecret(ctx context.Context, currentNamespace string, currentNamespaceClient client.Client, options *options.VirtualClusterOptions, config *clientcmdapi.Config) error {
+func WriteKubeConfigToSecret(ctx context.Context, currentNamespace string, currentNamespaceClient client.Client, options *options.VirtualClusterOptions, config *clientcmdapi.Config, isRemote bool) error {
 	config, err := CreateVClusterKubeConfig(config, options)
 	if err != nil {
 		return err
@@ -334,14 +334,14 @@ func WriteKubeConfigToSecret(ctx context.Context, currentNamespace string, curre
 		}
 
 		// write the extra secret
-		err = kubeconfig.WriteKubeConfig(ctx, currentNamespaceClient, options.KubeConfigSecret, secretNamespace, config)
+		err = kubeconfig.WriteKubeConfig(ctx, currentNamespaceClient, options.KubeConfigSecret, secretNamespace, config, isRemote)
 		if err != nil {
 			return fmt.Errorf("creating %s secret in the %s ns failed: %w", options.KubeConfigSecret, secretNamespace, err)
 		}
 	}
 
 	// write the default Secret
-	return kubeconfig.WriteKubeConfig(ctx, currentNamespaceClient, kubeconfig.GetDefaultSecretName(translate.Suffix), currentNamespace, config)
+	return kubeconfig.WriteKubeConfig(ctx, currentNamespaceClient, kubeconfig.GetDefaultSecretName(translate.Suffix), currentNamespace, config, isRemote)
 }
 
 func InitControllerContext(
diff --git a/pkg/setup/controllers.go b/pkg/setup/controllers.go
index 563e9a788a..279bfe3239 100644
--- a/pkg/setup/controllers.go
+++ b/pkg/setup/controllers.go
@@ -55,7 +55,7 @@ func StartControllers(controllerContext *options.ControllerContext) error {
 	// write the kube config to secret
 	go func() {
 		wait.Until(func() {
-			err := WriteKubeConfigToSecret(controllerContext.Context, controllerContext.CurrentNamespace, controllerContext.CurrentNamespaceClient, controllerContext.Options, controllerContext.VirtualRawConfig)
+			err := WriteKubeConfigToSecret(controllerContext.Context, controllerContext.CurrentNamespace, controllerContext.CurrentNamespaceClient, controllerContext.Options, controllerContext.VirtualRawConfig, false)
 			if err != nil {
 				klog.Errorf("Error writing kube config to secret: %v", err)
 			}
diff --git a/pkg/util/kubeconfig/kubeconfig.go b/pkg/util/kubeconfig/kubeconfig.go
index 708d3b946b..20769b2e33 100644
--- a/pkg/util/kubeconfig/kubeconfig.go
+++ b/pkg/util/kubeconfig/kubeconfig.go
@@ -25,7 +25,7 @@ const (
 	CertificateKeySecretKey = "client-key"
 )
 
-func WriteKubeConfig(ctx context.Context, currentNamespaceClient client.Client, secretName, secretNamespace string, config *api.Config) error {
+func WriteKubeConfig(ctx context.Context, currentNamespaceClient client.Client, secretName, secretNamespace string, config *api.Config, isRemote bool) error {
 	out, err := clientcmd.Write(*config)
 	if err != nil {
 		return err
@@ -73,7 +73,7 @@ func WriteKubeConfig(ctx context.Context, currentNamespaceClient client.Client,
 			kubeConfigSecret.Data[CertificateKeySecretKey] = key
 
 			// set owner reference
-			if translate.Owner != nil && translate.Owner.GetNamespace() == kubeConfigSecret.Namespace {
+			if !isRemote && translate.Owner != nil && translate.Owner.GetNamespace() == kubeConfigSecret.Namespace {
 				kubeConfigSecret.OwnerReferences = translate.GetOwnerReference(nil)
 			}
 			return nil