[202012] [minigraph] Remove SLB and bgp monitor peers for storage bac…

…kend (#12262) Why I did it This is to backport #12251 to 202012 Signed-off-by: Neetha John <[email protected]>
sonic-net · Oct 10, 2022 · 1b99c60 · 1b99c60
1 parent 3d8fbf4
commit 1b99c60
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/src/sonic-config-engine/minigraph.py b/src/sonic-config-engine/minigraph.py
@@ -1595,6 +1595,11 @@ def parse_xml(filename, platform=None, port_config_file=None, asic_name=None, hw
     if is_storage_device:
         results['DEVICE_METADATA']['localhost']['storage_device'] = "true"
 
+    # remove bgp monitor and slb peers for storage backend
+    if is_storage_device and 'BackEnd' in current_device['type']:
+        results['BGP_MONITORS'] = {}
+        results['BGP_PEER_RANGE'] = {}
+
     results['VLAN'] = vlans
     results['VLAN_MEMBER'] = vlan_members
 

diff --git a/src/sonic-config-engine/tests/test_cfggen.py b/src/sonic-config-engine/tests/test_cfggen.py
@@ -767,6 +767,14 @@ def verify_sub_intf(self, **kwargs):
             output = self.run_script(argument)
             self.assertEqual(output.strip(), "")
 
+            # SLB and BGP Monitor table does not exist
+            argument = '-m "' + graph_file + '" -p "' + self.port_config + '" -v "BGP_PEER_RANGE"'
+            output = self.run_script(argument)
+            self.assertEqual(output.strip(), "{}")
+            argument = '-m "' + graph_file + '" -p "' + self.port_config + '" -v "BGP_MONITORS"'
+            output = self.run_script(argument)
+            self.assertEqual(output.strip(), "{}")
+
             # ACL_TABLE should not contain EVERFLOW related entries
             argument = '-m "' + graph_file + '" -p "' + self.port_config + '" -v "ACL_TABLE"'
             output = self.run_script(argument)