fix: add unlock on error path

FourFifthsCode · FourFifthsCode · commit 4d8f5cb10846 · 2025-06-09T10:08:09.000-04:00
diff --git a/providers/kubeconfig/provider.go b/providers/kubeconfig/provider.go
@@ -214,6 +214,7 @@ func (p *Provider) handleSecret(ctx context.Context, secret *corev1.Secret, mgr
 
 		log.Info("Cluster already exists, updating it")
 		if err := p.removeCluster(clusterName); err != nil {
+			p.lock.RUnlock()
 			return fmt.Errorf("failed to remove existing cluster: %w", err)
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -214,6 +214,7 @@ func (p Provider) handleSecret(ctx context.Context, secret corev1.Secret, mgr`
`214`	`214`
`215`	`215`	`log.Info("Cluster already exists, updating it")`
`216`	`216`	`if err := p.removeCluster(clusterName); err != nil {`
	`217`	`+ p.lock.RUnlock()`
`217`	`218`	`return fmt.Errorf("failed to remove existing cluster: %w", err)`
`218`	`219`	`}`
`219`	`220`	`}`