Merge pull request #395 from klueska/support-namespace-change-in-upgrade

Allow ComputeDomain resources to be removed from any namespace

Author: klueska

cmd/compute-domain-controller/daemonset.go

@@ -79,7 +79,6 @@ func NewDaemonSetManager(config *ManagerConfig, getComputeDomain GetComputeDomai
 	factory := informers.NewSharedInformerFactoryWithOptions(
 		config.clientsets.Core,
 		informerResyncPeriod,
-		informers.WithNamespace(config.driverNamespace),
 		informers.WithTweakListOptions(func(opts *metav1.ListOptions) {
 			opts.LabelSelector = metav1.FormatLabelSelector(labelSelector)
 		}),
@@ -317,6 +316,11 @@ func (m *DaemonSetManager) onAddOrUpdate(ctx context.Context, obj any) error {
 		return fmt.Errorf("failed to cast to DaemonSet")
 	}
 
+	// Only process events from the driver namespace
+	if d.Namespace != m.config.driverNamespace {
+		return nil
+	}
+
 	klog.Infof("Processing added or updated DaemonSet: %s/%s", d.Namespace, d.Name)
 
 	cd, err := m.getComputeDomain(d.Labels[computeDomainLabelKey])

cmd/compute-domain-controller/resourceclaimtemplate.go

@@ -81,7 +81,6 @@ func newBaseResourceClaimTemplateManager(config *ManagerConfig, getComputeDomain
 	factory := informers.NewSharedInformerFactoryWithOptions(
 		config.clientsets.Core,
 		informerResyncPeriod,
-		informers.WithNamespace(namespace),
 		informers.WithTweakListOptions(func(opts *metav1.ListOptions) {
 			opts.LabelSelector = metav1.FormatLabelSelector(labelSelector)
 		}),