Comment 0 for bug 1539039

Revision history for this message
Roman Sokolkov (rsokolkov) wrote :

In "extensions/cluster_upgrade/upgrade.py" inside "copy_network_config" function there is a buggy for loop.

Do not change dictionary while iterating over it.

Following patch fixes the issue.

@@ -154,7 +157,7 @@ class UpgradeHelper(object):

         new_net_manager.update(nets)
         vips = orig_net_manager.get_assigned_vips()
- for ng_name in vips:
+ for ng_name in list(vips):
             if ng_name not in (consts.NETWORKS.public,
                                consts.NETWORKS.management):
                 vips.pop(ng_name)