fix: make MachineSetNodeController handle machinesets without clusters

Ignore such machine sets instead of failing, as fail leads to all other machine sets being skipped. Fixes: #208 Signed-off-by: Artem Chernyshev <[email protected]>
siderolabs · May 6, 2024 · 89fa1ad · 89fa1ad
1 parent fa3c9ff
commit 89fa1ad
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/internal/backend/runtime/omni/controllers/omni/machine_set_node.go b/internal/backend/runtime/omni/controllers/omni/machine_set_node.go
@@ -227,6 +227,7 @@ func (ctrl *MachineSetNodeController) reconcileMachineSet(
 	return nil
 }
 
+//nolint:gocognit
 func (ctrl *MachineSetNodeController) createNodes(
 	ctx context.Context,
 	r controller.Runtime,
@@ -249,6 +250,10 @@ func (ctrl *MachineSetNodeController) createNodes(
 
 	cluster, err := safe.ReaderGetByID[*omni.Cluster](ctx, r, clusterName)
 	if err != nil {
+		if state.IsNotFoundError(err) {
+			return nil
+		}
+
 		return err
 	}