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

Vlan status enhancement #316

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Vlan status enhancement #316

wants to merge 14 commits into from

Conversation

jihaix
Copy link

@jihaix jihaix commented Dec 21, 2018

  • New CLI command to set vlan's admin-status
    • config vlan adsmin-status <up|down>
  • Make Vlanxx's oper_status be relavent to its admin_status and member ports' oper_status.

Signed-off-by: xiongjihai@baidu.com

@msftclas
Copy link

msftclas commented Dec 21, 2018

CLA assistant check
All CLA requirements met.

@jipanyang
Copy link
Contributor

I think the cause of mismatch between vlan member oper status and vlan status is that only one Bridge is being used by all vlans.

As long as there is one slave netdev(port or lag) of the Bridge is up operationally, the oper status of Bridge will be up, then Vlan is up (oper status).

Probably we should create one dedicated bridge for each Vlan, then the oper status of VLAN will automatically reflect the oper status of bridge which is controlled by vlan member status with current Linux behavior?

@yxieca yxieca force-pushed the master branch 2 times, most recently from 8498931 to 8837dc2 Compare April 15, 2022 16:51
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.

None yet

3 participants