fix: returning an error when vcluster config schema changed

Author: José Silva

pkg/cli/delete_helm.go

@@ -130,7 +130,9 @@ func DeleteHelm(ctx context.Context, platformClient platform.Client, options *De
 	vclusterConfig := &config.Config{}
 	err = yaml.Unmarshal(values, vclusterConfig)
 	if err != nil {
-		return err
+		cmd.log.Warnf("Failed to parse vcluster config from Helm values: %v. ", err)
+		cmd.log.Infof("Continuing with deletion...")
+		vclusterConfig = nil
 	}
 
 	// we have to delete the chart
@@ -203,7 +205,7 @@ func DeleteHelm(ctx context.Context, platformClient platform.Client, options *De
 	}
 
 	// if namespace sync is enabled, use cleanup handlers to handle namespace cleanup
-	if vclusterConfig.Sync.ToHost.Namespaces.Enabled {
+	if vclusterConfig != nil && vclusterConfig.Sync.ToHost.Namespaces.Enabled {
 		if err := CleanupSyncedNamespaces(ctx, cmd.Namespace, vClusterName, cmd.restConfig, cmd.kubeClient, cmd.log); err != nil {
 			return fmt.Errorf("run namespace cleanup: %w", err)
 		}