summaryrefslogtreecommitdiffstats
path: root/src/plugins/gbp
diff options
context:
space:
mode:
authorBenoît Ganne <bganne@cisco.com>2019-09-03 17:54:21 +0200
committerAndrew Yourtchenko <ayourtch@gmail.com>2019-09-29 13:07:26 +0000
commitd89329a90f1db45846369ba554fbc1f86e55d09f (patch)
tree8a8d9fcf56240c95cc892079e9091fe3c1d1560a /src/plugins/gbp
parent8d5959f45e6e01a636cd5eea9e637776332ffada (diff)
gbp: fix set_mac() in unit tests
Do not use low-level sw_interface_set_mac_address() directly and use higher-level set_mac() instead, otherwise interface local_mac and configured mac are out-of-sync Type: fix Fixes: 25b049484f Change-Id: I9366e7afb95ef00e25a94ca3b0e6309871169878 Signed-off-by: Benoît Ganne <bganne@cisco.com> (cherry picked from commit a03d25d327ed9db40922cfcf247c3ee793c4754d)
Diffstat (limited to 'src/plugins/gbp')
-rw-r--r--src/plugins/gbp/test/test_gbp.py8
1 files changed, 2 insertions, 6 deletions
diff --git a/src/plugins/gbp/test/test_gbp.py b/src/plugins/gbp/test/test_gbp.py
index 2d7fa459440..cb426492891 100644
--- a/src/plugins/gbp/test/test_gbp.py
+++ b/src/plugins/gbp/test/test_gbp.py
@@ -872,9 +872,7 @@ class TestGBP(VppTestCase):
if epg != epgs[1] and epg != epgs[4]:
VppIpInterfaceBind(self, epg.bvi, epg.rd.t4).add_vpp_config()
VppIpInterfaceBind(self, epg.bvi, epg.rd.t6).add_vpp_config()
- self.vapi.sw_interface_set_mac_address(
- epg.bvi.sw_if_index,
- self.router_mac.packed)
+ epg.bvi.set_mac(self.router_mac)
# The BVIs are NAT inside interfaces
flags = self.config_flags.NAT_IS_INSIDE
@@ -2261,9 +2259,7 @@ class TestGBP(VppTestCase):
if epg != epgs[1]:
VppIpInterfaceBind(self, epg.bvi, epg.rd.t4).add_vpp_config()
VppIpInterfaceBind(self, epg.bvi, epg.rd.t6).add_vpp_config()
- self.vapi.sw_interface_set_mac_address(
- epg.bvi.sw_if_index,
- self.router_mac.packed)
+ epg.bvi.set_mac(self.router_mac)
if_ip4 = VppIpInterfaceAddress(self, epg.bvi, epg.bvi_ip4, 32)
if_ip6 = VppIpInterfaceAddress(self, epg.bvi, epg.bvi_ip6, 128)