Removing redundant namespace param.

dell · Dec 19, 2024 · fee6efc · fee6efc
1 parent cfb6a41
commit fee6efc
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/controllers/csm_controller.go b/controllers/csm_controller.go
@@ -1335,7 +1335,7 @@ func (r *ContainerStorageModuleReconciler) PreChecks(ctx context.Context, cr *cs
 			return fmt.Errorf("failed powerflex validation: %v", err)
 		}
 		// zoning initially applies only to pflex
-		err = r.ZoneValidation(ctx, cr, namespace)
+		err = r.ZoneValidation(ctx, cr)
 		if err != nil {
 			return fmt.Errorf("error during zone validation: %v", err)
 		}
@@ -1529,9 +1529,9 @@ func (r *ContainerStorageModuleReconciler) GetK8sClient() kubernetes.Interface {
 	return r.K8sClient
 }
 
-func (r *ContainerStorageModuleReconciler) ZoneValidation(ctx context.Context, cr *csmv1.ContainerStorageModule, namespace string) error {
+func (r *ContainerStorageModuleReconciler) ZoneValidation(ctx context.Context, cr *csmv1.ContainerStorageModule) error {
 
-	err := drivers.ValidateZones(ctx, cr, r.Client, namespace)
+	err := drivers.ValidateZones(ctx, cr, r.Client)
 	if err != nil {
 		return fmt.Errorf("ZoneValidation failed with error: %v", err)
 	}

diff --git a/pkg/drivers/powerflex.go b/pkg/drivers/powerflex.go
@@ -347,7 +347,7 @@ func ModifyPowerflexCR(yamlString string, cr csmv1.ContainerStorageModule, fileT
 	return yamlString
 }
 
-func ValidateZones(ctx context.Context, cr *csmv1.ContainerStorageModule, ct client.Client, namespace string) error {
+func ValidateZones(ctx context.Context, cr *csmv1.ContainerStorageModule, ct client.Client) error {
 	secretName := cr.Name + "-config"
 	err := ValidateZonesInSecret(ctx, ct, cr.Namespace, secretName)
 	return err