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

Ignore bridge port id in fdb consolidated event if value is NULL #319

Merged
merged 1 commit into from
May 14, 2018

Conversation

kcudnik
Copy link
Collaborator

@kcudnik kcudnik commented May 9, 2018

No description provided.

@kcudnik kcudnik requested a review from lguohan May 9, 2018 10:09
Copy link
Contributor

@lguohan lguohan left a comment

Choose a reason for hiding this comment

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

:shipit:

@lguohan lguohan merged commit 7c29ecb into sonic-net:master May 14, 2018
@kcudnik kcudnik deleted the portfix branch May 16, 2018 13:23
pettershao-ragilenetworks pushed a commit to pettershao-ragilenetworks/sonic-sairedis that referenced this pull request Nov 18, 2022
pettershao-ragilenetworks pushed a commit to pettershao-ragilenetworks/sonic-sairedis that referenced this pull request Nov 18, 2022
jianyuewu pushed a commit to jianyuewu/sonic-sairedis that referenced this pull request Feb 7, 2025
…onic-net#319)

* Fix pfcwd start_default: Use config_db to get the active port list. Remove
the use of counters_db because start_default can be called on a device where no
pfcwd is enabled, so the only source we can reply on to get the port
info is the config_db not counters_db.

Signed-off-by: Wenda Ni <wenni@microsoft.com>

* Address comments: remove function get_active_ports_from_configdb, inline the operation

Signed-off-by: Wenda <wenni@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants