containernetworking / plugins

Some reference and example networking plugins, maintained by the CNI team.
Apache License 2.0
2.17k stars 778 forks source link

documentation seems a bit unclear #958

Closed nathanaelle closed 10 months ago

nathanaelle commented 10 months ago

hi,

this part of the documentation seems unclear for https://www.cni.dev/plugins/current/main/bridge/

vlan (int, optional): assign VLAN tag. Defaults to none. preserveDefaultVlan (boolean, optional): indicates whether the default vlan must be preserved on the veth end connected to the bridge. Defaults to true. vlanTrunk (list, optional): assign VLAN trunk tag. Defaults to none.

how to : – allow packets for/from a vlan and keep incoming tagged aka bridge vlan add vid x dev x ? – allow packets for/from a vlan and untag incoming aka bridge vlan add vid x dev x untagged ? – force outgoing untagged packet to be tagged aka bridge vlan add vid x dev x pvid ? – allow packets for/from a vlan and untag incoming and force outgoing untaged aka bridge vlan add vid x dev x pvid untagged ?

I hope you will understand my issue is not about understanding vlans or linux, but about understanding the doc for transposing a wellknown standard linux command to manage bridges and vlans to a cni configuration

mlguerrero12 commented 10 months ago

@nathanaelle, please have a look at https://github.com/containernetworking/cni.dev/pull/134

there is also another issue about the same. https://github.com/containernetworking/plugins/issues/939

nathanaelle commented 10 months ago

thank you :)