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

Bridge vlan_filtering support #8941

Closed
Tracked by #8484
e3b0c442 opened this issue Jun 26, 2024 · 1 comment · Fixed by #8950
Closed
Tracked by #8484

Bridge vlan_filtering support #8941

e3b0c442 opened this issue Jun 26, 2024 · 1 comment · Fixed by #8950
Assignees

Comments

@e3b0c442
Copy link

Feature Request

Add ability to configure VLAN filtering on a host bridge interface.

Description

I'm trying to use KubeVirt on my Talos cluster, in a setup that has certain workload types separated by VLAN. Prior to Talos, this would be accomplished by setting the vlan_filtering field on a host bridge, and then using the vlan field with the bridge CNI in a multus NetworkAttachmentDefinition to allow the bridge to handle the VLAN filtering. However, Talos does not currently allow me to affect this configuration.

There are two unergonomic workarounds to the situation (outside of using an external CNI plugin):

  • create a privileged pod with host networking to create and configure the bridge on each node
  • create and maintain VLAN interfaces off the bridge for each VLAN, then have multus create its own bridges off of these vlan interfaces

Ideally this would be a parameter in MachineConfig.network.

Thanks!

@stijoh
Copy link

stijoh commented Jun 29, 2024

Second this!!

@smira smira self-assigned this Jun 30, 2024
smira added a commit to smira/talos that referenced this issue Jun 30, 2024
Fixes siderolabs#8941

Signed-off-by: Andrey Smirnov <andrey.smirnov@siderolabs.com>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 31, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants