Skip to content

Sonnenblume vlan ports - #1705

Open
Koltonowski wants to merge 3 commits into
freifunk-berlin:mainfrom
Koltonowski:sonnenblume-vlan-ports
Open

Sonnenblume vlan ports#1705
Koltonowski wants to merge 3 commits into
freifunk-berlin:mainfrom
Koltonowski:sonnenblume-vlan-ports

Conversation

@Koltonowski

Copy link
Copy Markdown
Contributor

This adds optional per-network DSA port mappings.

Currently, DSA VLAN generation applies every VLAN to all dsa_ports. For devices such as here for example the AVM FRITZ!Box 7530, this means that a network with untagged: true is rendered as untagged on all LAN ports.

This change keeps the existing default behavior unchanged:
if no ports: field is set, the template still uses all dsa_ports.

If ports: is set on a network, the template uses that explicit list instead. This allows location configs to express DSA layouts such as:

  • VLAN 40: lan1:t lan2:u lan3:u lan4:u
  • VLAN 50: lan1:u

The second line also accepts already annotated entries like lan1:t / lan2:u, while preserving the old automatic :t / :u behavior for plain port names.

@Koltonowski
Koltonowski force-pushed the sonnenblume-vlan-ports branch from 994af34 to ddb6083 Compare July 10, 2026 13:10

@FFHener FFHener left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for trying to implement this feature. Unfortunately this approach doesnt work if you have different devices, e.g. a different Router and AP, as they would have different ports.

And it should fail or provide a way for swconfig devices

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