Skip to content

Commit

Permalink
Merge pull request #669 from containernetworking/jell/per_if_sysctl
Browse files Browse the repository at this point in the history
Allow setting sysctls on a particular interface
  • Loading branch information
matthewdupre authored Nov 24, 2021
2 parents 0920090 + c16cff9 commit 57e95c5
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions plugins/meta/tuning/tuning.go
Original file line number Diff line number Diff line change
Expand Up @@ -325,6 +325,11 @@ func cmdAdd(args *skel.CmdArgs) error {

err = ns.WithNetNSPath(args.Netns, func(_ ns.NetNS) error {
for key, value := range tuningConf.SysCtl {
// If the key contains `IFNAME` - substitute it with args.IfName
// to allow setting sysctls on a particular interface, on which
// other operations (like mac/mtu setting) are performed
key = strings.Replace(key, "IFNAME", args.IfName, 1)

fileName := filepath.Join("/proc/sys", strings.Replace(key, ".", "/", -1))

// Refuse to modify sysctl parameters that don't belong
Expand Down

0 comments on commit 57e95c5

Please sign in to comment.