fix(kubernetes): fix context override not reflecting in the summary

Author: gabe565

internal/kubernetes/client.go

@@ -58,11 +58,24 @@ func NewConfigLoader(kubeconfig, context string) clientcmd.ClientConfig {
 }
 
 func NewClient(kubeconfig, context, namespace string) (KubeClient, error) {
+	client := KubeClient{
+		Context:   context,
+		Namespace: namespace,
+	}
+
 	configLoader := NewConfigLoader(kubeconfig, context)
-	var client KubeClient
 
-	if rawConfig, err := configLoader.RawConfig(); err == nil {
-		client.Context = rawConfig.CurrentContext
+	if client.Context == "" {
+		if rawConfig, err := configLoader.RawConfig(); err == nil {
+			client.Context = rawConfig.CurrentContext
+		}
+	}
+
+	if client.Namespace == "" {
+		var err error
+		if client.Namespace, _, err = configLoader.Namespace(); err != nil {
+			return client, err
+		}
 	}
 
 	var err error
@@ -72,14 +85,6 @@ func NewClient(kubeconfig, context, namespace string) (KubeClient, error) {
 
 	client.ClientConfig.UserAgent = rest.DefaultKubernetesUserAgent()
 
-	if namespace == "" {
-		if client.Namespace, _, err = configLoader.Namespace(); err != nil {
-			return client, err
-		}
-	} else {
-		client.Namespace = namespace
-	}
-
 	if client.ClientSet, err = kubernetes.NewForConfig(client.ClientConfig); err != nil {
 		return client, err
 	}
@@ -88,5 +93,5 @@ func NewClient(kubeconfig, context, namespace string) (KubeClient, error) {
 		return client, err
 	}
 
-	return client, err
+	return client, nil
 }