From 3a1a7c40a13ee9f5561823a79876d88d3f5bf053 Mon Sep 17 00:00:00 2001
From: Christian Poessinger <christian@poessinger.com>
Date: Sun, 20 Feb 2022 10:36:33 +0100
Subject: interface: T4203: prevent DHCP client restart if not necessary

In the past whenever a change happened to any interface and it was configured
as a DHCP client, VyOS always had a breif outage as DHCP released the old lease
and re-aquired a new one - bad!

This commit changes the behavior that DHCP client is only restarted if any one
of the possible options one can set for DHCP client under the "dhcp-options"
node is altered.
---
 python/vyos/configdict.py | 38 +++++++++++++++++++++++++++++++++++++-
 1 file changed, 37 insertions(+), 1 deletion(-)

(limited to 'python/vyos/configdict.py')

diff --git a/python/vyos/configdict.py b/python/vyos/configdict.py
index e7f515ea9..efeb6dc1f 100644
--- a/python/vyos/configdict.py
+++ b/python/vyos/configdict.py
@@ -423,6 +423,15 @@ def get_interface_dict(config, base, ifname=''):
     bond = is_member(config, ifname, 'bonding')
     if bond: dict.update({'is_bond_member' : bond})
 
+    # Check if any DHCP options changed which require a client restat
+    for leaf_node in ['client-id', 'default-route-distance', 'host-name',
+                 'no-default-route', 'vendor-class-id']:
+        dhcp = leaf_node_changed(config, ['dhcp-options', leaf_node])
+        if dhcp:
+            dict.update({'dhcp_options_old' : dhcp})
+            # one option is suffiecient to set 'dhcp_options_old' key
+            break
+
     # Some interfaces come with a source_interface which must also not be part
     # of any other bond or bridge interface as it is exclusivly assigned as the
     # Kernels "lower" interface to this new "virtual/upper" interface.
@@ -470,6 +479,15 @@ def get_interface_dict(config, base, ifname=''):
         bridge = is_member(config, f'{ifname}.{vif}', 'bridge')
         if bridge: dict['vif'][vif].update({'is_bridge_member' : bridge})
 
+        # Check if any DHCP options changed which require a client restat
+        for leaf_node in ['client-id', 'default-route-distance', 'host-name',
+                     'no-default-route', 'vendor-class-id']:
+            dhcp = leaf_node_changed(config, ['vif', vif, 'dhcp-options', leaf_node])
+            if dhcp:
+                dict['vif'][vif].update({'dhcp_options_old' : dhcp})
+                # one option is suffiecient to set 'dhcp_options_old' key
+                break
+
     for vif_s, vif_s_config in dict.get('vif_s', {}).items():
         default_vif_s_values = defaults(base + ['vif-s'])
         # XXX: T2665: we only wan't the vif-s defaults - do not care about vif-c
@@ -495,6 +513,15 @@ def get_interface_dict(config, base, ifname=''):
         bridge = is_member(config, f'{ifname}.{vif_s}', 'bridge')
         if bridge: dict['vif_s'][vif_s].update({'is_bridge_member' : bridge})
 
+        # Check if any DHCP options changed which require a client restat
+        for leaf_node in ['client-id', 'default-route-distance', 'host-name',
+                     'no-default-route', 'vendor-class-id']:
+            dhcp = leaf_node_changed(config, ['vif-s', vif_s, 'dhcp-options', leaf_node])
+            if dhcp:
+                dict['vif_s'][vif_s].update({'dhcp_options_old' : dhcp})
+                # one option is suffiecient to set 'dhcp_options_old' key
+                break
+
         for vif_c, vif_c_config in vif_s_config.get('vif_c', {}).items():
             default_vif_c_values = defaults(base + ['vif-s', 'vif-c'])
 
@@ -521,6 +548,16 @@ def get_interface_dict(config, base, ifname=''):
             if bridge: dict['vif_s'][vif_s]['vif_c'][vif_c].update(
                 {'is_bridge_member' : bridge})
 
+            # Check if any DHCP options changed which require a client restat
+            for leaf_node in ['client-id', 'default-route-distance', 'host-name',
+                         'no-default-route', 'vendor-class-id']:
+                dhcp = leaf_node_changed(config, ['vif-s', vif_s, 'vif-c', vif_c,
+                                                  'dhcp-options', leaf_node])
+                if dhcp:
+                    dict['vif_s'][vif_s]['vif_c'][vif_c].update({'dhcp_options_old' : dhcp})
+                    # one option is suffiecient to set 'dhcp_options_old' key
+                    break
+
     # Check vif, vif-s/vif-c VLAN interfaces for removal
     dict = get_removed_vlans(config, dict)
     return dict
@@ -545,7 +582,6 @@ def get_vlan_ids(interface):
 
     return vlan_ids
 
-
 def get_accel_dict(config, base, chap_secrets):
     """
     Common utility function to retrieve and mangle the Accel-PPP configuration
-- 
cgit v1.2.3