cloud-init-dev team mailing list archive

[Andreas Karis <akaris@xxxxxxxxxx>]

Andreas Karis has proposed merging ~akaris/cloud-init:bug1696176 into cloud-init:master.

Commit message:
RHEL/CentOS: Fix default routes for IPv4/IPv6 configuration

Since f38fa41317602908139aa96e930b634f65e39555 , default routes
get added to both ifcfg-* and route-* and route6-* files.
Default routes should only go to ifcfg-* files, otherwise the
information is redundant.

LP: #1696176

Requested reviews:
  cloud-init commiters (cloud-init-dev)
Related bugs:
  Bug #1696176 in cloud-init: "Add default routes for IPv4 and IPv6 only to ifcfg-* files and not to route-* or route6-* files in sysconfig"
  https://bugs.launchpad.net/cloud-init/+bug/1696176

For more details, see:
https://code.launchpad.net/~akaris/cloud-init/+git/cloud-init/+merge/325174
-- 
Your team cloud-init commiters is requested to review the proposed merge of ~akaris/cloud-init:bug1696176 into cloud-init:master.

diff --git a/cloudinit/net/sysconfig.py b/cloudinit/net/sysconfig.py
index 58c5713..3ab6b45 100644
--- a/cloudinit/net/sysconfig.py
+++ b/cloudinit/net/sysconfig.py
@@ -372,11 +372,13 @@ class Renderer(renderer.Renderer):
                     nm_key = 'NETMASK%s' % route_cfg.last_idx
                     addr_key = 'ADDRESS%s' % route_cfg.last_idx
                     route_cfg.last_idx += 1
-                for (old_key, new_key) in [('gateway', gw_key),
-                                           ('netmask', nm_key),
-                                           ('network', addr_key)]:
-                    if old_key in route:
-                        route_cfg[new_key] = route[old_key]
+                    # add default routes only to ifcfg files, not 
+                    # to route-* or route6-*
+                    for (old_key, new_key) in [('gateway', gw_key),
+                                               ('netmask', nm_key),
+                                               ('network', addr_key)]:
+                        if old_key in route:
+                            route_cfg[new_key] = route[old_key]
 
     @classmethod
     def _render_bonding_opts(cls, iface_cfg, iface):