fix: include nodegroups in COMPLETE status in cluster update job (#427)

Co-authored-by: okozachenko1203 <okozachenko1203@users.noreply.github.com>
vexxhost · Sep 6, 2024 · b50b6dc · b50b6dc
1 parent eafbf71
commit b50b6dc
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/magnum_cluster_api/driver.py b/magnum_cluster_api/driver.py
@@ -430,6 +430,8 @@ def update_nodegroups_status(
             ):
                 continue
 
+            node_groups.append(node_group)
+
             md = objects.MachineDeployment.for_node_group_or_none(
                 self.k8s_api, cluster, node_group
             )
@@ -499,8 +501,6 @@ def update_nodegroups_status(
                 node_group.save()
                 continue
 
-            node_groups.append(node_group)
-
         return node_groups
 
     @cluster_lock_wrapper