argument mismatch fix, undefined instance fix, correct GetClientForMa…

…nagedCluster to GetClientForCluster
projectsveltos · May 14, 2024 · 3ce2647 · 3ce2647
1 parent 2a6c3c5
commit 3ce2647
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 11 deletions.
diff --git a/internal/commands/loglevel/set.go b/internal/commands/loglevel/set.go
@@ -133,14 +133,19 @@ Description:
         return updateDebuggingConfigurationInManaged(ctx, logSeverity, component, namespace, clusterName, clusterType)
     }
 
-    instance := utils.GetAccessInstance()
-    dc, err := instance.GetDebuggingConfiguration(ctx)
-    if err != nil {
-        if apierrors.IsNotFound(err) {
-            return fmt.Errorf("DebuggingConfiguration not found")
-        }
-        return err
-    }
-
-    return updateDebuggingConfiguration(ctx, logSeverity, component, dc)
+	instance := utils.GetAccessInstance()
+	dc, err := instance.GetDebuggingConfiguration(ctx)
+	if err != nil {
+		if apierrors.IsNotFound(err) {
+			return fmt.Errorf("DebuggingConfiguration not found")
+		}
+		return err
+	}
+
+	cc, err := collectLogLevelConfiguration(ctx, dc)  // pass dc
+	if err != nil {
+		return err
+	}
+
+    return updateLogLevelConfiguration(ctx, spec, dc)  // Pass dc
 }
diff --git a/internal/commands/loglevel/utils.go b/internal/commands/loglevel/utils.go
@@ -66,6 +66,7 @@ func updateLogLevelConfiguration(
         Configuration: spec,
     }
 
+    instance := utils.GetAccessInstance()  // Define instance
     return instance.UpdateDebuggingConfiguration(ctx, dc)
 }
 
@@ -80,7 +81,7 @@ func updateDebuggingConfigurationInManaged(
 ) error {
     // get a client for the managed cluster using namespace, clusterName, and clusterType
     instance := utils.GetAccessInstance()
-    client, err := instance.GetClientForCluster(namespace, clusterName, clusterType)
+    client, err := instance.GetClientForCluster(namespace, clusterName, clusterType)  // Correct method name
     if err != nil {
         return err
     }