c1c26e
From 0bff7d73c49043b0820d0231c9a47539287f35e3 Mon Sep 17 00:00:00 2001
c1c26e
From: Miroslav Rezanina <mrezanin@redhat.com>
c1c26e
Date: Thu, 31 May 2018 19:37:55 +0200
c1c26e
Subject: Do not write NM_CONTROLLED=no in generated interface config  files
c1c26e
c1c26e
X-downstream-only: true
c1c26e
Signed-off-by: Ryan McCabe <rmccabe@redhat.com>
c1c26e
Signed-off-by: Danilo C. L. de Paula <ddepaula@redhat.com>
c1c26e
---
c1c26e
 cloudinit/net/sysconfig.py  |  1 -
c1c26e
 tests/unittests/test_net.py | 30 ------------------------------
c1c26e
 2 files changed, 31 deletions(-)
c1c26e
c1c26e
diff --git a/cloudinit/net/sysconfig.py b/cloudinit/net/sysconfig.py
c1c26e
index 17293e1d..ae0554ef 100644
c1c26e
--- a/cloudinit/net/sysconfig.py
c1c26e
+++ b/cloudinit/net/sysconfig.py
c1c26e
@@ -250,7 +250,6 @@ class Renderer(renderer.Renderer):
c1c26e
     iface_defaults = tuple([
c1c26e
         ('ONBOOT', True),
c1c26e
         ('USERCTL', False),
c1c26e
-        ('NM_CONTROLLED', False),
c1c26e
         ('BOOTPROTO', 'none'),
c1c26e
     ])
c1c26e
 
c1c26e
diff --git a/tests/unittests/test_net.py b/tests/unittests/test_net.py
c1c26e
index 195f261c..5f1aa3e7 100644
c1c26e
--- a/tests/unittests/test_net.py
c1c26e
+++ b/tests/unittests/test_net.py
c1c26e
@@ -175,7 +175,6 @@ GATEWAY=172.19.3.254
c1c26e
 HWADDR=fa:16:3e:ed:9a:59
c1c26e
 IPADDR=172.19.1.34
c1c26e
 NETMASK=255.255.252.0
c1c26e
-NM_CONTROLLED=no
c1c26e
 ONBOOT=yes
c1c26e
 TYPE=Ethernet
c1c26e
 USERCTL=no
c1c26e
@@ -279,7 +278,6 @@ IPADDR=172.19.1.34
c1c26e
 IPADDR1=10.0.0.10
c1c26e
 NETMASK=255.255.252.0
c1c26e
 NETMASK1=255.255.255.0
c1c26e
-NM_CONTROLLED=no
c1c26e
 ONBOOT=yes
c1c26e
 TYPE=Ethernet
c1c26e
 USERCTL=no
c1c26e
@@ -407,7 +405,6 @@ IPV6ADDR_SECONDARIES="2001:DB9::10/64 2001:DB10::10/64"
c1c26e
 IPV6INIT=yes
c1c26e
 IPV6_DEFAULTGW=2001:DB8::1
c1c26e
 NETMASK=255.255.252.0
c1c26e
-NM_CONTROLLED=no
c1c26e
 ONBOOT=yes
c1c26e
 TYPE=Ethernet
c1c26e
 USERCTL=no
c1c26e
@@ -523,7 +520,6 @@ NETWORK_CONFIGS = {
c1c26e
                 BOOTPROTO=none
c1c26e
                 DEVICE=eth1
c1c26e
                 HWADDR=cf:d6:af:48:e8:80
c1c26e
-                NM_CONTROLLED=no
c1c26e
                 ONBOOT=yes
c1c26e
                 TYPE=Ethernet
c1c26e
                 USERCTL=no"""),
c1c26e
@@ -539,7 +535,6 @@ NETWORK_CONFIGS = {
c1c26e
                 IPADDR=192.168.21.3
c1c26e
                 NETMASK=255.255.255.0
c1c26e
                 METRIC=10000
c1c26e
-                NM_CONTROLLED=no
c1c26e
                 ONBOOT=yes
c1c26e
                 TYPE=Ethernet
c1c26e
                 USERCTL=no"""),
c1c26e
@@ -652,7 +647,6 @@ NETWORK_CONFIGS = {
c1c26e
                 IPV6ADDR=2001:1::1/64
c1c26e
                 IPV6INIT=yes
c1c26e
                 NETMASK=255.255.255.0
c1c26e
-                NM_CONTROLLED=no
c1c26e
                 ONBOOT=yes
c1c26e
                 TYPE=Ethernet
c1c26e
                 USERCTL=no
c1c26e
@@ -894,14 +888,12 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
c1c26e
                 DHCPV6C=yes
c1c26e
                 IPV6INIT=yes
c1c26e
                 MACADDR=aa:bb:cc:dd:ee:ff
c1c26e
-                NM_CONTROLLED=no
c1c26e
                 ONBOOT=yes
c1c26e
                 TYPE=Bond
c1c26e
                 USERCTL=no"""),
c1c26e
             'ifcfg-bond0.200': textwrap.dedent("""\
c1c26e
                 BOOTPROTO=dhcp
c1c26e
                 DEVICE=bond0.200
c1c26e
-                NM_CONTROLLED=no
c1c26e
                 ONBOOT=yes
c1c26e
                 PHYSDEV=bond0
c1c26e
                 TYPE=Ethernet
c1c26e
@@ -918,7 +910,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
c1c26e
                 IPV6_DEFAULTGW=2001:4800:78ff:1b::1
c1c26e
                 MACADDR=bb:bb:bb:bb:bb:aa
c1c26e
                 NETMASK=255.255.255.0
c1c26e
-                NM_CONTROLLED=no
c1c26e
                 ONBOOT=yes
c1c26e
                 PRIO=22
c1c26e
                 STP=no
c1c26e
@@ -928,7 +919,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
c1c26e
                 BOOTPROTO=none
c1c26e
                 DEVICE=eth0
c1c26e
                 HWADDR=c0:d6:9f:2c:e8:80
c1c26e
-                NM_CONTROLLED=no
c1c26e
                 ONBOOT=yes
c1c26e
                 TYPE=Ethernet
c1c26e
                 USERCTL=no"""),
c1c26e
@@ -945,7 +935,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
c1c26e
                 MTU=1500
c1c26e
                 NETMASK=255.255.255.0
c1c26e
                 NETMASK1=255.255.255.0
c1c26e
-                NM_CONTROLLED=no
c1c26e
                 ONBOOT=yes
c1c26e
                 PHYSDEV=eth0
c1c26e
                 TYPE=Ethernet
c1c26e
@@ -956,7 +945,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
c1c26e
                 DEVICE=eth1
c1c26e
                 HWADDR=aa:d6:9f:2c:e8:80
c1c26e
                 MASTER=bond0
c1c26e
-                NM_CONTROLLED=no
c1c26e
                 ONBOOT=yes
c1c26e
                 SLAVE=yes
c1c26e
                 TYPE=Ethernet
c1c26e
@@ -966,7 +954,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
c1c26e
                 DEVICE=eth2
c1c26e
                 HWADDR=c0:bb:9f:2c:e8:80
c1c26e
                 MASTER=bond0
c1c26e
-                NM_CONTROLLED=no
c1c26e
                 ONBOOT=yes
c1c26e
                 SLAVE=yes
c1c26e
                 TYPE=Ethernet
c1c26e
@@ -976,7 +963,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
c1c26e
                 BRIDGE=br0
c1c26e
                 DEVICE=eth3
c1c26e
                 HWADDR=66:bb:9f:2c:e8:80
c1c26e
-                NM_CONTROLLED=no
c1c26e
                 ONBOOT=yes
c1c26e
                 TYPE=Ethernet
c1c26e
                 USERCTL=no"""),
c1c26e
@@ -985,7 +971,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
c1c26e
                 BRIDGE=br0
c1c26e
                 DEVICE=eth4
c1c26e
                 HWADDR=98:bb:9f:2c:e8:80
c1c26e
-                NM_CONTROLLED=no
c1c26e
                 ONBOOT=yes
c1c26e
                 TYPE=Ethernet
c1c26e
                 USERCTL=no"""),
c1c26e
@@ -993,7 +978,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
c1c26e
                 BOOTPROTO=dhcp
c1c26e
                 DEVICE=eth5
c1c26e
                 HWADDR=98:bb:9f:2c:e8:8a
c1c26e
-                NM_CONTROLLED=no
c1c26e
                 ONBOOT=no
c1c26e
                 TYPE=Ethernet
c1c26e
                 USERCTL=no""")
c1c26e
@@ -1356,7 +1340,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
c1c26e
         MTU=9000
c1c26e
         NETMASK=255.255.255.0
c1c26e
         NETMASK1=255.255.255.0
c1c26e
-        NM_CONTROLLED=no
c1c26e
         ONBOOT=yes
c1c26e
         TYPE=Bond
c1c26e
         USERCTL=no
c1c26e
@@ -1366,7 +1349,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
c1c26e
         DEVICE=bond0s0
c1c26e
         HWADDR=aa:bb:cc:dd:e8:00
c1c26e
         MASTER=bond0
c1c26e
-        NM_CONTROLLED=no
c1c26e
         ONBOOT=yes
c1c26e
         SLAVE=yes
c1c26e
         TYPE=Ethernet
c1c26e
@@ -1388,7 +1370,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
c1c26e
         DEVICE=bond0s1
c1c26e
         HWADDR=aa:bb:cc:dd:e8:01
c1c26e
         MASTER=bond0
c1c26e
-        NM_CONTROLLED=no
c1c26e
         ONBOOT=yes
c1c26e
         SLAVE=yes
c1c26e
         TYPE=Ethernet
c1c26e
@@ -1426,7 +1407,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
c1c26e
                 BOOTPROTO=none
c1c26e
                 DEVICE=en0
c1c26e
                 HWADDR=aa:bb:cc:dd:e8:00
c1c26e
-                NM_CONTROLLED=no
c1c26e
                 ONBOOT=yes
c1c26e
                 TYPE=Ethernet
c1c26e
                 USERCTL=no"""),
c1c26e
@@ -1443,7 +1423,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
c1c26e
                 MTU=2222
c1c26e
                 NETMASK=255.255.255.0
c1c26e
                 NETMASK1=255.255.255.0
c1c26e
-                NM_CONTROLLED=no
c1c26e
                 ONBOOT=yes
c1c26e
                 PHYSDEV=en0
c1c26e
                 TYPE=Ethernet
c1c26e
@@ -1484,7 +1463,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
c1c26e
                 DEVICE=br0
c1c26e
                 IPADDR=192.168.2.2
c1c26e
                 NETMASK=255.255.255.0
c1c26e
-                NM_CONTROLLED=no
c1c26e
                 ONBOOT=yes
c1c26e
                 PRIO=22
c1c26e
                 STP=no
c1c26e
@@ -1498,7 +1476,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
c1c26e
                 HWADDR=52:54:00:12:34:00
c1c26e
                 IPV6ADDR=2001:1::100/96
c1c26e
                 IPV6INIT=yes
c1c26e
-                NM_CONTROLLED=no
c1c26e
                 ONBOOT=yes
c1c26e
                 TYPE=Ethernet
c1c26e
                 USERCTL=no
c1c26e
@@ -1510,7 +1487,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
c1c26e
                 HWADDR=52:54:00:12:34:01
c1c26e
                 IPV6ADDR=2001:1::101/96
c1c26e
                 IPV6INIT=yes
c1c26e
-                NM_CONTROLLED=no
c1c26e
                 ONBOOT=yes
c1c26e
                 TYPE=Ethernet
c1c26e
                 USERCTL=no
c1c26e
@@ -1584,7 +1560,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
c1c26e
                 HWADDR=52:54:00:12:34:00
c1c26e
                 IPADDR=192.168.1.2
c1c26e
                 NETMASK=255.255.255.0
c1c26e
-                NM_CONTROLLED=no
c1c26e
                 ONBOOT=no
c1c26e
                 TYPE=Ethernet
c1c26e
                 USERCTL=no
c1c26e
@@ -1594,7 +1569,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
c1c26e
                 DEVICE=eth1
c1c26e
                 HWADDR=52:54:00:12:34:aa
c1c26e
                 MTU=1480
c1c26e
-                NM_CONTROLLED=no
c1c26e
                 ONBOOT=yes
c1c26e
                 TYPE=Ethernet
c1c26e
                 USERCTL=no
c1c26e
@@ -1603,7 +1577,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
c1c26e
                 BOOTPROTO=none
c1c26e
                 DEVICE=eth2
c1c26e
                 HWADDR=52:54:00:12:34:ff
c1c26e
-                NM_CONTROLLED=no
c1c26e
                 ONBOOT=no
c1c26e
                 TYPE=Ethernet
c1c26e
                 USERCTL=no
c1c26e
@@ -1969,7 +1942,6 @@ class TestRhelSysConfigRendering(CiTestCase):
c1c26e
 BOOTPROTO=dhcp
c1c26e
 DEVICE=eth1000
c1c26e
 HWADDR=07-1C-C6-75-A4-BE
c1c26e
-NM_CONTROLLED=no
c1c26e
 ONBOOT=yes
c1c26e
 TYPE=Ethernet
c1c26e
 USERCTL=no
c1c26e
@@ -2090,7 +2062,6 @@ GATEWAY=10.0.2.2
c1c26e
 HWADDR=52:54:00:12:34:00
c1c26e
 IPADDR=10.0.2.15
c1c26e
 NETMASK=255.255.255.0
c1c26e
-NM_CONTROLLED=no
c1c26e
 ONBOOT=yes
c1c26e
 TYPE=Ethernet
c1c26e
 USERCTL=no
c1c26e
@@ -2111,7 +2082,6 @@ USERCTL=no
c1c26e
 #
c1c26e
 BOOTPROTO=dhcp
c1c26e
 DEVICE=eth0
c1c26e
-NM_CONTROLLED=no
c1c26e
 ONBOOT=yes
c1c26e
 TYPE=Ethernet
c1c26e
 USERCTL=no
c1c26e
-- 
c1c26e
2.20.1
c1c26e