diff --git a/pkg/controller/cyclenoderequest/transitioner/transitions.go b/pkg/controller/cyclenoderequest/transitioner/transitions.go index e0e1e45..d2e9238 100644 --- a/pkg/controller/cyclenoderequest/transitioner/transitions.go +++ b/pkg/controller/cyclenoderequest/transitioner/transitions.go @@ -524,7 +524,7 @@ func (t *CycleNodeRequestTransitioner) transitionHealing() (reconcile.Result, er if err != nil { return t.transitionToFailed(err) } - + if !nodeExists { t.rm.LogEvent(t.cycleNodeRequest, "HealingNodes", "Node does not exist, skip healing node: %s", node.Name) @@ -533,13 +533,11 @@ func (t *CycleNodeRequestTransitioner) transitionHealing() (reconcile.Result, er // try and re-attach the nodes, if any were un-attached t.rm.LogEvent(t.cycleNodeRequest, "AttachingNodes", "Attaching instances to nodes group: %v", node.Name) - alreadyAttached, err := nodeGroups.AttachInstance(node.ProviderID, node.NodeGroupName) - if alreadyAttached { + if alreadyAttached, err := nodeGroups.AttachInstance(node.ProviderID, node.NodeGroupName); alreadyAttached { t.rm.LogEvent(t.cycleNodeRequest, "AttachingNodes", "Skip re-attaching instances to nodes group: %v, err: %v", node.Name, err) - } - if err != nil { + } else if err != nil { return t.transitionToFailed(err) }