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

[portsyncd]: add debug message if a port cannot be found in port able #1328

Merged
merged 1 commit into from
Jun 24, 2020

Conversation

lguohan
Copy link
Contributor

@lguohan lguohan commented Jun 21, 2020

What I did
add debug message if a port cannot be found in port able

Why I did it
to debug sonic-net/sonic-buildimage#4821

How I verified it

Details if related

@lguohan lguohan requested a review from prsunny June 21, 2020 21:01
@lguohan lguohan merged commit c05601c into sonic-net:master Jun 24, 2020
@lguohan lguohan deleted the port branch June 24, 2020 10:23
EdenGri pushed a commit to EdenGri/sonic-swss that referenced this pull request Feb 28, 2022
…a member from a Portchannel (sonic-net#1328)

* Validations checks while adding a member to PortChannel and removing a member from a Portchannel

Added below validation checks when user the configures a port as member a portchannel:
1. Check if the given portchannel name is valid or not
2. Check if the given portchannel name exists in CONFIG_DB
3. Check if the given port is configured with an IP address or not
4. Check if the given port is configured with with VLAN membership or not
5. Check if the given port is already member of a portchannel
6. Check if the given port speed matches the speed of existing members
7. Check if the given port MTU matches the speed of existing members

Added below validation checks when user the removes member port from a portchannel:
1. Check if the given portchannel name is valid or not
2. Check if the given portchannel name exists in exists in CONFIG_DB or not
3. Check if the given port is member of the given portchannel or not

Signed-off-by: Akhilesh Samineni <akhilesh.samineni@broadcom.com>
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