df923a
From 6183f8a282703eaab967355a63fd5b116bb21f0b Mon Sep 17 00:00:00 2001
df923a
From: Miroslav Rezanina <mrezanin@redhat.com>
df923a
Date: Thu, 31 May 2018 19:37:55 +0200
df923a
Subject: Do not write NM_CONTROLLED=no in generated interface config  files
df923a
df923a
X-downstream-only: true
df923a
Signed-off-by: Ryan McCabe <rmccabe@redhat.com>
df923a
---
df923a
 cloudinit/net/sysconfig.py                     |  1 -
df923a
 tests/unittests/test_distros/test_netconfig.py |  4 ----
df923a
 tests/unittests/test_net.py                    | 30 --------------------------
df923a
 3 files changed, 35 deletions(-)
df923a
df923a
diff --git a/cloudinit/net/sysconfig.py b/cloudinit/net/sysconfig.py
df923a
index 39d89c4..cefb5c5 100644
df923a
--- a/cloudinit/net/sysconfig.py
df923a
+++ b/cloudinit/net/sysconfig.py
df923a
@@ -233,7 +233,6 @@ class Renderer(renderer.Renderer):
df923a
     iface_defaults = tuple([
df923a
         ('ONBOOT', True),
df923a
         ('USERCTL', False),
df923a
-        ('NM_CONTROLLED', False),
df923a
         ('BOOTPROTO', 'none'),
df923a
     ])
df923a
 
df923a
diff --git a/tests/unittests/test_distros/test_netconfig.py b/tests/unittests/test_distros/test_netconfig.py
df923a
index 1c2e45f..2f69455 100644
df923a
--- a/tests/unittests/test_distros/test_netconfig.py
df923a
+++ b/tests/unittests/test_distros/test_netconfig.py
df923a
@@ -479,7 +479,6 @@ DEVICE=eth0
df923a
 GATEWAY=192.168.1.254
df923a
 IPADDR=192.168.1.5
df923a
 NETMASK=255.255.255.0
df923a
-NM_CONTROLLED=no
df923a
 ONBOOT=yes
df923a
 TYPE=Ethernet
df923a
 USERCTL=no
df923a
@@ -496,7 +495,6 @@ USERCTL=no
df923a
 #
df923a
 BOOTPROTO=dhcp
df923a
 DEVICE=eth1
df923a
-NM_CONTROLLED=no
df923a
 ONBOOT=yes
df923a
 TYPE=Ethernet
df923a
 USERCTL=no
df923a
@@ -630,7 +628,6 @@ DEVICE=eth0
df923a
 IPV6ADDR=2607:f0d0:1002:0011::2/64
df923a
 IPV6INIT=yes
df923a
 IPV6_DEFAULTGW=2607:f0d0:1002:0011::1
df923a
-NM_CONTROLLED=no
df923a
 ONBOOT=yes
df923a
 TYPE=Ethernet
df923a
 USERCTL=no
df923a
@@ -645,7 +642,6 @@ USERCTL=no
df923a
 #
df923a
 BOOTPROTO=dhcp
df923a
 DEVICE=eth1
df923a
-NM_CONTROLLED=no
df923a
 ONBOOT=yes
df923a
 TYPE=Ethernet
df923a
 USERCTL=no
df923a
diff --git a/tests/unittests/test_net.py b/tests/unittests/test_net.py
df923a
index c12a487..95318ed 100644
df923a
--- a/tests/unittests/test_net.py
df923a
+++ b/tests/unittests/test_net.py
df923a
@@ -144,7 +144,6 @@ GATEWAY=172.19.3.254
df923a
 HWADDR=fa:16:3e:ed:9a:59
df923a
 IPADDR=172.19.1.34
df923a
 NETMASK=255.255.252.0
df923a
-NM_CONTROLLED=no
df923a
 ONBOOT=yes
df923a
 TYPE=Ethernet
df923a
 USERCTL=no
df923a
@@ -212,7 +211,6 @@ IPADDR=172.19.1.34
df923a
 IPADDR1=10.0.0.10
df923a
 NETMASK=255.255.252.0
df923a
 NETMASK1=255.255.255.0
df923a
-NM_CONTROLLED=no
df923a
 ONBOOT=yes
df923a
 TYPE=Ethernet
df923a
 USERCTL=no
df923a
@@ -302,7 +300,6 @@ IPV6ADDR_SECONDARIES="2001:DB9::10/64 2001:DB10::10/64"
df923a
 IPV6INIT=yes
df923a
 IPV6_DEFAULTGW=2001:DB8::1
df923a
 NETMASK=255.255.252.0
df923a
-NM_CONTROLLED=no
df923a
 ONBOOT=yes
df923a
 TYPE=Ethernet
df923a
 USERCTL=no
df923a
@@ -417,7 +414,6 @@ NETWORK_CONFIGS = {
df923a
                 BOOTPROTO=none
df923a
                 DEVICE=eth1
df923a
                 HWADDR=cf:d6:af:48:e8:80
df923a
-                NM_CONTROLLED=no
df923a
                 ONBOOT=yes
df923a
                 TYPE=Ethernet
df923a
                 USERCTL=no"""),
df923a
@@ -432,7 +428,6 @@ NETWORK_CONFIGS = {
df923a
                 HWADDR=c0:d6:9f:2c:e8:80
df923a
                 IPADDR=192.168.21.3
df923a
                 NETMASK=255.255.255.0
df923a
-                NM_CONTROLLED=no
df923a
                 ONBOOT=yes
df923a
                 TYPE=Ethernet
df923a
                 USERCTL=no"""),
df923a
@@ -544,7 +539,6 @@ NETWORK_CONFIGS = {
df923a
                 IPV6ADDR=2001:1::1/64
df923a
                 IPV6INIT=yes
df923a
                 NETMASK=255.255.255.0
df923a
-                NM_CONTROLLED=no
df923a
                 ONBOOT=yes
df923a
                 TYPE=Ethernet
df923a
                 USERCTL=no
df923a
@@ -745,14 +739,12 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
df923a
                 DHCPV6C=yes
df923a
                 IPV6INIT=yes
df923a
                 MACADDR=aa:bb:cc:dd:ee:ff
df923a
-                NM_CONTROLLED=no
df923a
                 ONBOOT=yes
df923a
                 TYPE=Bond
df923a
                 USERCTL=no"""),
df923a
             'ifcfg-bond0.200': textwrap.dedent("""\
df923a
                 BOOTPROTO=dhcp
df923a
                 DEVICE=bond0.200
df923a
-                NM_CONTROLLED=no
df923a
                 ONBOOT=yes
df923a
                 PHYSDEV=bond0
df923a
                 TYPE=Ethernet
df923a
@@ -768,7 +760,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
df923a
                 IPV6INIT=yes
df923a
                 IPV6_DEFAULTGW=2001:4800:78ff:1b::1
df923a
                 NETMASK=255.255.255.0
df923a
-                NM_CONTROLLED=no
df923a
                 ONBOOT=yes
df923a
                 PRIO=22
df923a
                 STP=no
df923a
@@ -778,7 +769,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
df923a
                 BOOTPROTO=none
df923a
                 DEVICE=eth0
df923a
                 HWADDR=c0:d6:9f:2c:e8:80
df923a
-                NM_CONTROLLED=no
df923a
                 ONBOOT=yes
df923a
                 TYPE=Ethernet
df923a
                 USERCTL=no"""),
df923a
@@ -795,7 +785,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
df923a
                 MTU=1500
df923a
                 NETMASK=255.255.255.0
df923a
                 NETMASK1=255.255.255.0
df923a
-                NM_CONTROLLED=no
df923a
                 ONBOOT=yes
df923a
                 PHYSDEV=eth0
df923a
                 TYPE=Ethernet
df923a
@@ -806,7 +795,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
df923a
                 DEVICE=eth1
df923a
                 HWADDR=aa:d6:9f:2c:e8:80
df923a
                 MASTER=bond0
df923a
-                NM_CONTROLLED=no
df923a
                 ONBOOT=yes
df923a
                 SLAVE=yes
df923a
                 TYPE=Ethernet
df923a
@@ -816,7 +804,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
df923a
                 DEVICE=eth2
df923a
                 HWADDR=c0:bb:9f:2c:e8:80
df923a
                 MASTER=bond0
df923a
-                NM_CONTROLLED=no
df923a
                 ONBOOT=yes
df923a
                 SLAVE=yes
df923a
                 TYPE=Ethernet
df923a
@@ -826,7 +813,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
df923a
                 BRIDGE=br0
df923a
                 DEVICE=eth3
df923a
                 HWADDR=66:bb:9f:2c:e8:80
df923a
-                NM_CONTROLLED=no
df923a
                 ONBOOT=yes
df923a
                 TYPE=Ethernet
df923a
                 USERCTL=no"""),
df923a
@@ -835,7 +821,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
df923a
                 BRIDGE=br0
df923a
                 DEVICE=eth4
df923a
                 HWADDR=98:bb:9f:2c:e8:80
df923a
-                NM_CONTROLLED=no
df923a
                 ONBOOT=yes
df923a
                 TYPE=Ethernet
df923a
                 USERCTL=no"""),
df923a
@@ -843,7 +828,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
df923a
                 BOOTPROTO=dhcp
df923a
                 DEVICE=eth5
df923a
                 HWADDR=98:bb:9f:2c:e8:8a
df923a
-                NM_CONTROLLED=no
df923a
                 ONBOOT=no
df923a
                 TYPE=Ethernet
df923a
                 USERCTL=no""")
df923a
@@ -1125,7 +1109,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
df923a
         IPV6INIT=yes
df923a
         NETMASK=255.255.255.0
df923a
         NETMASK1=255.255.255.0
df923a
-        NM_CONTROLLED=no
df923a
         ONBOOT=yes
df923a
         TYPE=Bond
df923a
         USERCTL=no
df923a
@@ -1135,7 +1118,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
df923a
         DEVICE=bond0s0
df923a
         HWADDR=aa:bb:cc:dd:e8:00
df923a
         MASTER=bond0
df923a
-        NM_CONTROLLED=no
df923a
         ONBOOT=yes
df923a
         SLAVE=yes
df923a
         TYPE=Ethernet
df923a
@@ -1153,7 +1135,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
df923a
         DEVICE=bond0s1
df923a
         HWADDR=aa:bb:cc:dd:e8:01
df923a
         MASTER=bond0
df923a
-        NM_CONTROLLED=no
df923a
         ONBOOT=yes
df923a
         SLAVE=yes
df923a
         TYPE=Ethernet
df923a
@@ -1190,7 +1171,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
df923a
                 BOOTPROTO=none
df923a
                 DEVICE=en0
df923a
                 HWADDR=aa:bb:cc:dd:e8:00
df923a
-                NM_CONTROLLED=no
df923a
                 ONBOOT=yes
df923a
                 TYPE=Ethernet
df923a
                 USERCTL=no"""),
df923a
@@ -1206,7 +1186,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
df923a
                 IPV6_DEFAULTGW=2001:1::1
df923a
                 NETMASK=255.255.255.0
df923a
                 NETMASK1=255.255.255.0
df923a
-                NM_CONTROLLED=no
df923a
                 ONBOOT=yes
df923a
                 PHYSDEV=en0
df923a
                 TYPE=Ethernet
df923a
@@ -1247,7 +1226,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
df923a
                 DEVICE=br0
df923a
                 IPADDR=192.168.2.2
df923a
                 NETMASK=255.255.255.0
df923a
-                NM_CONTROLLED=no
df923a
                 ONBOOT=yes
df923a
                 PRIO=22
df923a
                 STP=no
df923a
@@ -1261,7 +1239,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
df923a
                 HWADDR=52:54:00:12:34:00
df923a
                 IPV6ADDR=2001:1::100/96
df923a
                 IPV6INIT=yes
df923a
-                NM_CONTROLLED=no
df923a
                 ONBOOT=yes
df923a
                 TYPE=Ethernet
df923a
                 USERCTL=no
df923a
@@ -1273,7 +1250,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
df923a
                 HWADDR=52:54:00:12:34:01
df923a
                 IPV6ADDR=2001:1::101/96
df923a
                 IPV6INIT=yes
df923a
-                NM_CONTROLLED=no
df923a
                 ONBOOT=yes
df923a
                 TYPE=Ethernet
df923a
                 USERCTL=no
df923a
@@ -1347,7 +1323,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
df923a
                 HWADDR=52:54:00:12:34:00
df923a
                 IPADDR=192.168.1.2
df923a
                 NETMASK=255.255.255.0
df923a
-                NM_CONTROLLED=no
df923a
                 ONBOOT=no
df923a
                 TYPE=Ethernet
df923a
                 USERCTL=no
df923a
@@ -1357,7 +1332,6 @@ pre-down route del -net 10.0.0.0 netmask 255.0.0.0 gw 11.0.0.1 metric 3 || true
df923a
                 DEVICE=eth1
df923a
                 HWADDR=52:54:00:12:34:aa
df923a
                 MTU=1480
df923a
-                NM_CONTROLLED=no
df923a
                 ONBOOT=yes
df923a
                 TYPE=Ethernet
df923a
                 USERCTL=no
df923a
@@ -1366,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
df923a
                 BOOTPROTO=none
df923a
                 DEVICE=eth2
df923a
                 HWADDR=52:54:00:12:34:ff
df923a
-                NM_CONTROLLED=no
df923a
                 ONBOOT=no
df923a
                 TYPE=Ethernet
df923a
                 USERCTL=no
df923a
@@ -1639,7 +1612,6 @@ class TestSysConfigRendering(CiTestCase):
df923a
 BOOTPROTO=dhcp
df923a
 DEVICE=eth1000
df923a
 HWADDR=07-1C-C6-75-A4-BE
df923a
-NM_CONTROLLED=no
df923a
 ONBOOT=yes
df923a
 TYPE=Ethernet
df923a
 USERCTL=no
df923a
@@ -1759,7 +1731,6 @@ GATEWAY=10.0.2.2
df923a
 HWADDR=52:54:00:12:34:00
df923a
 IPADDR=10.0.2.15
df923a
 NETMASK=255.255.255.0
df923a
-NM_CONTROLLED=no
df923a
 ONBOOT=yes
df923a
 TYPE=Ethernet
df923a
 USERCTL=no
df923a
@@ -1780,7 +1751,6 @@ USERCTL=no
df923a
 #
df923a
 BOOTPROTO=dhcp
df923a
 DEVICE=eth0
df923a
-NM_CONTROLLED=no
df923a
 ONBOOT=yes
df923a
 TYPE=Ethernet
df923a
 USERCTL=no
df923a
-- 
df923a
1.8.3.1
df923a