Merge pull request containers#11404 from Luap99/ip-del

Drop dependency on iproute

libpod/network/devices.go

@@ -2,12 +2,11 @@ package network
 
 import (
 	"fmt"
-	"os/exec"
 
 	"github.com/containers/common/pkg/config"
 	"github.com/containers/podman/v3/pkg/util"
-	"github.com/containers/podman/v3/utils"
 	"github.com/sirupsen/logrus"
+	"github.com/vishvananda/netlink"
 )
 
 // GetFreeDeviceName returns a device name that is unused; used when no network
@@ -52,12 +51,9 @@ func GetFreeDeviceName(config *config.Config) (string, error) {
 
 // RemoveInterface removes an interface by the given name
 func RemoveInterface(interfaceName string) error {
-	// Make sure we have the ip command on the system
-	ipPath, err := exec.LookPath("ip")
+	link, err := netlink.LinkByName(interfaceName)
 	if err != nil {
 		return err
 	}
-	// Delete the network interface
-	_, err = utils.ExecCmd(ipPath, []string{"link", "del", interfaceName}...)
-	return err
+	return netlink.LinkDel(link)
 }

libpod/network/network.go

@@ -194,8 +194,9 @@ func removeNetwork(config *config.Config, name string) error {
 				return errors.Wrapf(err, "failed to get live network names")
 			}
 			if util.StringInSlice(interfaceName, liveNetworkNames) {
-				if err := RemoveInterface(interfaceName); err != nil {
-					return errors.Wrapf(err, "failed to delete the network interface %q", interfaceName)
+				if err = RemoveInterface(interfaceName); err != nil {
+					// only log the error, it is not fatal
+					logrus.Infof("failed to remove network interface %s: %v", interfaceName, err)
 				}
 			}
 		}