From 8902d2614a066766d27c5bea15f03ebd097fe1d8 Mon Sep 17 00:00:00 2001 From: Tomofumi Hayashi Date: Wed, 7 Aug 2019 13:52:44 +0900 Subject: [PATCH] Remove link Down/Up in MAC address change to prevent route flush --- plugins/meta/tuning/tuning.go | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/plugins/meta/tuning/tuning.go b/plugins/meta/tuning/tuning.go index 9ea7c2c4..d62734cf 100644 --- a/plugins/meta/tuning/tuning.go +++ b/plugins/meta/tuning/tuning.go @@ -93,15 +93,7 @@ func changeMacAddr(ifName string, newMacAddr string) error { return fmt.Errorf("failed to get %q: %v", ifName, err) } - err = netlink.LinkSetDown(link) - if err != nil { - return fmt.Errorf("failed to set %q down: %v", ifName, err) - } - err = netlink.LinkSetHardwareAddr(link, addr) - if err != nil { - return fmt.Errorf("failed to set %q address to %q: %v", ifName, newMacAddr, err) - } - return netlink.LinkSetUp(link) + return netlink.LinkSetHardwareAddr(link, addr) } func updateResultsMacAddr(config TuningConf, ifName string, newMacAddr string) {