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

Creating VLAN interface with "edit interface vlan<N>" does not set VLAN id #391

Closed
jovatn opened this issue Apr 10, 2024 · 1 comment · Fixed by #392
Closed

Creating VLAN interface with "edit interface vlan<N>" does not set VLAN id #391

jovatn opened this issue Apr 10, 2024 · 1 comment · Fixed by #392
Assignees
Labels
bug Something isn't working

Comments

@jovatn
Copy link
Contributor

jovatn commented Apr 10, 2024

When creating a VLAN interface with interface name convention "vlan" we get interface "type vlan". But as a convenience the vlan id should also be set to N. (Compare with "edit interface eth1.N" which fills in both type and vlan id)

admin@infix-00-00-00:/> configure
admin@infix-00-00-00:/config/> edit interface vlan10
admin@infix-00-00-00:/config/interface/vlan10/> show
type vlan;
admin@infix-00-00-00:/config/interface/vlan10/> set vlan id 10
admin@infix-00-00-00:/config/interface/vlan10/> show
type vlan;
vlan {
  id 10;
}
admin@infix-00-00-00:/config/interface/vlan10/> 
@mattiaswal mattiaswal self-assigned this Apr 10, 2024
@troglobit troglobit added the bug Something isn't working label Apr 10, 2024
@troglobit
Copy link
Contributor

For completeness, when using the CLI, the set vlan id 10 should not be necessary since the system can "help out" by predicting the VLAN ID based on the interface name.

@mattiaswal mattiaswal changed the title Creating VLAN interface with "edit interface vlan<N>" does not fill out "vlan id N" Creating VLAN interface with "edit interface vlan<N>" does not set VLAN id Apr 11, 2024
@mattiaswal mattiaswal mentioned this issue Apr 11, 2024
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants