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

Add support for VLANs #145

Merged
merged 4 commits into from
Feb 19, 2024
Merged

Add support for VLANs #145

merged 4 commits into from
Feb 19, 2024

Conversation

fw2568
Copy link
Member

@fw2568 fw2568 commented Feb 16, 2024

This PR adds support for VLAN tagging of overlay provider networks.

Provider VLANs

New option vlan in network provider configuration:

network_providers:
- name: default
  type: overlay
  vlan: 10

Overlay provider network VLAN tags are attached by OVN to all egress traffic from the VMs. This is accomplished by configuring the tag to the logical switch port of the external router port that connects from the project network to the provider network.

Bridge VLANs and IP mode

The bridge adapter is not affected by the provider's VLAN setting. The default configuration for an overlay adapter also doesn't enable the bridge adapter for network communications.

For scenarios where we have few (or only one) network adapters, the bridge can now be configured to be enabled for DHCP by default. For this case, we now also have the option to configure a VLAN for the bridge adapter:

network_providers:
- name: default
  type: overlay
  vlan: 10
  bridge_name: br-pif
  bridge_options: 
   default_ip_mode: dhcp
   bridge_vlan: 11

See also #144 for static address configuration support.

new field provider_vlan to tag traffic from a network provider
to a vlan.
replaced enabled_adapters with a bridge option for
disabled bridges or dhcp enabled bridges.
@fw2568 fw2568 marked this pull request as ready for review February 16, 2024 20:33
@fw2568 fw2568 merged commit b8e33b0 into main Feb 19, 2024
3 checks passed
@fw2568 fw2568 deleted the pull/vlan branch February 19, 2024 15:13
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.

2 participants