ITQC/containernetworking-plugins
24
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 34 Wiki Activity

Merge pull request #364 from s1061123/fix/removeifdown

Browse Source 
Remove link Down/Up in MAC address change to prevent route flush
This commit is contained in:
Casey Callendrello 2019-08-07 17:39:02 +02:00 committed by GitHub
commit e9d511c5bc
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
plugins/meta/tuning/tuning.go
View File

@ -93,15 +93,7 @@ func changeMacAddr(ifName string, newMacAddr string) error {
return fmt.Errorf("failed to get %q: %v", ifName, err) return fmt.Errorf("failed to get %q: %v", ifName, err)
} }
err = netlink.LinkSetDown(link) return netlink.LinkSetHardwareAddr(link, addr)
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)
} }
func updateResultsMacAddr(config TuningConf, ifName string, newMacAddr string) { func updateResultsMacAddr(config TuningConf, ifName string, newMacAddr string) {