From 09b873ca69821ac2a3e306da0af0437b849d1dd8 Mon Sep 17 00:00:00 2001

From: Eduardo Otubo <otubo@redhat.com>

Date: Fri, 18 Jan 2019 16:55:36 +0100

Subject: [PATCH] net: Make sysconfig renderer compatible with Network Manager.

RH-Author: Eduardo Otubo <otubo@redhat.com>

Message-id: <20190118165536.25963-1-otubo@redhat.com>

Patchwork-id: 84052

O-Subject: [RHEL-8.0 cloud-init PATCH] net: Make sysconfig renderer compatible with Network Manager.

Bugzilla: 1602784

RH-Acked-by: Vitaly Kuznetsov <vkuznets@redhat.com>

RH-Acked-by: Mohammed Gamal <mgamal@redhat.com>

Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1602784

Brew: https://brewweb.engineering.redhat.com/brew/taskinfo?taskID=19877292

Tested by: upstream maintainers and me

commit 3861102fcaf47a882516d8b6daab518308eb3086

Author: Eduardo Otubo <otubo@redhat.com>

Date:   Fri Jan 18 15:36:19 2019 +0000

    net: Make sysconfig renderer compatible with Network Manager.

    The 'sysconfig' renderer is activated if, and only if, there's ifup and

    ifdown commands present in its search dictonary or the network-scripts

    configuration files are found. This patch adds a check for Network-

    Manager configuration file as well.

    This solution is based on the use of the plugin 'ifcfg-rh' present in

    Network-Manager and is designed to support Fedora 29 or other

    distributions that also replaced network-scripts by Network-Manager.

Signed-off-by: Eduardo Otubo <otubo@redhat.com>

Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>

---

 cloudinit/net/sysconfig.py  | 36 +++++++++++++++++++++++

 tests/unittests/test_net.py | 71 +++++++++++++++++++++++++++++++++++++++++++++

 2 files changed, 107 insertions(+)

diff --git a/cloudinit/net/sysconfig.py b/cloudinit/net/sysconfig.py

index bd81832..42291aa 100644

--- a/cloudinit/net/sysconfig.py

+++ b/cloudinit/net/sysconfig.py

@@ -10,11 +10,14 @@ from cloudinit.distros.parsers import resolv_conf

 from cloudinit import log as logging

 from cloudinit import util

+from configobj import ConfigObj

+

 from . import renderer

 from .network_state import (

     is_ipv6_addr, net_prefix_to_ipv4_mask, subnet_is_ipv6)

 LOG = logging.getLogger(__name__)

+NM_CFG_FILE = "/etc/NetworkManager/NetworkManager.conf"

 def _make_header(sep='#'):

@@ -46,6 +49,24 @@ def _quote_value(value):

         return value

+def enable_ifcfg_rh(path):

+    """Add ifcfg-rh to NetworkManager.cfg plugins if main section is present"""

+    config = ConfigObj(path)

+    if 'main' in config:

+        if 'plugins' in config['main']:

+            if 'ifcfg-rh' in config['main']['plugins']:

+                return

+        else:

+            config['main']['plugins'] = []

+

+        if isinstance(config['main']['plugins'], list):

+            config['main']['plugins'].append('ifcfg-rh')

+        else:

+            config['main']['plugins'] = [config['main']['plugins'], 'ifcfg-rh']

+        config.write()

+        LOG.debug('Enabled ifcfg-rh NetworkManager plugins')

+

+

 class ConfigMap(object):

     """Sysconfig like dictionary object."""

@@ -597,6 +618,8 @@ class Renderer(renderer.Renderer):

             netrules_content = self._render_persistent_net(network_state)

             netrules_path = util.target_path(target, self.netrules_path)

             util.write_file(netrules_path, netrules_content, file_mode)

+        if available_nm(target=target):

+            enable_ifcfg_rh(util.target_path(target, path=NM_CFG_FILE))

         # always write /etc/sysconfig/network configuration

         sysconfig_path = util.target_path(target, "etc/sysconfig/network")

@@ -608,6 +631,13 @@ class Renderer(renderer.Renderer):

 def available(target=None):

+    sysconfig = available_sysconfig(target=target)

+    nm = available_nm(target=target)

+

+    return any([nm, sysconfig])

+

+

+def available_sysconfig(target=None):

     expected = ['ifup', 'ifdown']

     search = ['/sbin', '/usr/sbin']

     for p in expected:

@@ -623,4 +653,10 @@ def available(target=None):

     return True

+def available_nm(target=None):

+    if not os.path.isfile(util.target_path(target, path=NM_CFG_FILE)):

+        return False

+    return True

+

+

 # vi: ts=4 expandtab

diff --git a/tests/unittests/test_net.py b/tests/unittests/test_net.py

index 9cf41bc..8e520f6 100644

--- a/tests/unittests/test_net.py

+++ b/tests/unittests/test_net.py

@@ -24,6 +24,7 @@ import os

 import textwrap

 import yaml

+

 DHCP_CONTENT_1 = """

 DEVICE='eth0'

 PROTO='dhcp'

@@ -1542,6 +1543,7 @@ iface eth1 inet dhcp

 class TestSysConfigRendering(CiTestCase):

+    nm_cfg_file = "/etc/NetworkManager/NetworkManager.conf"

     scripts_dir = '/etc/sysconfig/network-scripts'

     header = ('# Created by cloud-init on instance boot automatically, '

               'do not edit.\n#\n')

@@ -1853,6 +1855,75 @@ iface eth0 inet dhcp

         self.assertEqual(

             expected, dir2dict(tmp_dir)['/etc/network/interfaces'])

+    def test_check_ifcfg_rh(self):

+        """ifcfg-rh plugin is added NetworkManager.conf if conf present."""

+        render_dir = self.tmp_dir()

+        nm_cfg = util.target_path(render_dir, path=self.nm_cfg_file)

+        util.ensure_dir(os.path.dirname(nm_cfg))

+

+        # write a template nm.conf, note plugins is a list here

+        with open(nm_cfg, 'w') as fh:

+            fh.write('# test_check_ifcfg_rh\n[main]\nplugins=foo,bar\n')

+        self.assertTrue(os.path.exists(nm_cfg))

+

+        # render and read

+        entry = NETWORK_CONFIGS['small']

+        found = self._render_and_read(network_config=yaml.load(entry['yaml']),

+                                      dir=render_dir)

+        self._compare_files_to_expected(entry[self.expected_name], found)

+        self._assert_headers(found)

+

+        # check ifcfg-rh is in the 'plugins' list

+        config = sysconfig.ConfigObj(nm_cfg)

+        self.assertIn('ifcfg-rh', config['main']['plugins'])

+

+    def test_check_ifcfg_rh_plugins_string(self):

+        """ifcfg-rh plugin is append when plugins is a string."""

+        render_dir = self.tmp_path("render")

+        os.makedirs(render_dir)

+        nm_cfg = util.target_path(render_dir, path=self.nm_cfg_file)

+        util.ensure_dir(os.path.dirname(nm_cfg))

+

+        # write a template nm.conf, note plugins is a value here

+        util.write_file(nm_cfg, '# test_check_ifcfg_rh\n[main]\nplugins=foo\n')

+

+        # render and read

+        entry = NETWORK_CONFIGS['small']

+        found = self._render_and_read(network_config=yaml.load(entry['yaml']),

+                                      dir=render_dir)

+        self._compare_files_to_expected(entry[self.expected_name], found)

+        self._assert_headers(found)

+

+        # check raw content has plugin

+        nm_file_content = util.load_file(nm_cfg)

+        self.assertIn('ifcfg-rh', nm_file_content)

+

+        # check ifcfg-rh is in the 'plugins' list

+        config = sysconfig.ConfigObj(nm_cfg)

+        self.assertIn('ifcfg-rh', config['main']['plugins'])

+

+    def test_check_ifcfg_rh_plugins_no_plugins(self):

+        """enable_ifcfg_plugin creates plugins value if missing."""

+        render_dir = self.tmp_path("render")

+        os.makedirs(render_dir)

+        nm_cfg = util.target_path(render_dir, path=self.nm_cfg_file)

+        util.ensure_dir(os.path.dirname(nm_cfg))

+

+        # write a template nm.conf, note plugins is missing

+        util.write_file(nm_cfg, '# test_check_ifcfg_rh\n[main]\n')

+        self.assertTrue(os.path.exists(nm_cfg))

+

+        # render and read

+        entry = NETWORK_CONFIGS['small']

+        found = self._render_and_read(network_config=yaml.load(entry['yaml']),

+                                      dir=render_dir)

+        self._compare_files_to_expected(entry[self.expected_name], found)

+        self._assert_headers(found)

+

+        # check ifcfg-rh is in the 'plugins' list

+        config = sysconfig.ConfigObj(nm_cfg)

+        self.assertIn('ifcfg-rh', config['main']['plugins'])

+

 class TestNetplanNetRendering(CiTestCase):

-- 

1.8.3.1