Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tuning: Add support of altering the allmulticast flag #624

Merged
merged 1 commit into from
May 5, 2021

Conversation

bjosv
Copy link
Contributor

@bjosv bjosv commented Apr 28, 2021

This adds support to allow the tuning plugin to enable/disable the allmulticast mode of the interface.
When enabled it instructs the network driver to retrieve all multicast packets from the network.

Resolves #621

This adds support to allow the tuning plugin to enable/disable the
allmulticast mode of the interface. When enabled it instructs the
network driver to retrieve all multicast packets from the network.

Signed-off-by: Björn Svensson <[email protected]>
@bjosv
Copy link
Contributor Author

bjosv commented Apr 28, 2021

If the PR vishvananda/netlink#635 is merged the import of sys/unix can be avoided, but since the tuning plugin is not amongst the Windows supported plugins (i.e not in windows_only.txt) it was assumed to be ok for now.

@squeed
Copy link
Member

squeed commented Apr 28, 2021

@bjosv this looks good! Can you also file a PR at https://github.com/containernetworking/cni.dev/blob/main/content/plugins/current/meta/tuning.md adding this flag please?

@bjosv
Copy link
Contributor Author

bjosv commented Apr 29, 2021

@bjosv this looks good! Can you also file a PR at https://github.com/containernetworking/cni.dev/blob/main/content/plugins/current/meta/tuning.md adding this flag please?

@squeed Great! Filed the corresponding documentation change in containernetworking/cni.dev#84

@dcbw dcbw merged commit a5b7963 into containernetworking:master May 5, 2021
@bjosv bjosv deleted the allmulti branch May 6, 2021 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

tuning: Support changing the allmulticast flag on interface
3 participants