[minigraph.py]: Parse VLAN MAC address from minigraph when present (s…

…onic-net#5726)
santhosh-kt · Feb 25, 2021 · 4683d45 · 4683d45
1 parent c52631f
commit 4683d45
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 1 deletion.
diff --git a/src/sonic-config-engine/minigraph.py b/src/sonic-config-engine/minigraph.py
@@ -383,6 +383,10 @@ def parse_dpg(dpg, hname):
  vdhcpserver_list = vintfdhcpservers.split(';')
  vlan_attributes['dhcp_servers'] = vdhcpserver_list
 
+ vlanmac = vintf.find(str(QName(ns, "MacAddress")))
+ if vlanmac != None:
+ vlan_attributes['mac'] = vlanmac.text
+
  sonic_vlan_name = "Vlan%s" % vlanid
  if sonic_vlan_name != vintfname:
  vlan_attributes['alias'] = vintfname

diff --git a/src/sonic-config-engine/tests/simple-sample-graph-case.xml b/src/sonic-config-engine/tests/simple-sample-graph-case.xml
@@ -134,6 +134,7 @@
  <VlanID>1000</VlanID>
  <Tag>1000</Tag>
  <Subnets>192.168.0.0/27</Subnets>
+ <MacAddress>00:aa:bb:cc:dd:ee</MacAddress>
  </VlanInterface>
  </VlanInterfaces>
  <IPInterfaces>

diff --git a/src/sonic-config-engine/tests/test_minigraph_case.py b/src/sonic-config-engine/tests/test_minigraph_case.py
@@ -83,7 +83,7 @@ def test_minigraph_vlans(self):
  output = self.run_script(argument)
  self.assertEqual(
  utils.to_dict(output.strip()),
- utils.to_dict("{'Vlan1000': {'alias': 'ab1', 'dhcp_servers': ['192.0.0.1', '192.0.0.2'], 'vlanid': '1000'}}")
+ utils.to_dict("{'Vlan1000': {'alias': 'ab1', 'dhcp_servers': ['192.0.0.1', '192.0.0.2'], 'vlanid': '1000', 'mac': '00:aa:bb:cc:dd:ee' }}")
  )
 
  def test_minigraph_vlan_members(self):