-
Notifications
You must be signed in to change notification settings - Fork 580
[orchagent]: Remove updating APP_DB port oper_status logic from orchagent #192
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
Conversation
This is to address the issue sonic-net/sonic-buildimage#456. The logic is already integrated in portsyncd which will update the oper_stauts of a port once the port changes its oper_status. |
it is better to use orchagent to update the oper_status instead of using portsyncd |
…s in portsyncd - Front panel ports' oper_status should be notified from the SAI instead of getting it from kernel via portsyncd. Thus in this commit, oper_status is removed from the portsyncd. - VLAN and LAG oper_status on the contrary need to be updated from the protocol daemon, the kernel and the teamd. Signed-off-by: Shuotian Cheng <[email protected]>
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you update the commit message?
commit message:
|
Signed-off-by: Sihui Han <[email protected]>
* Add support for logrotate in sairedis * Fix attribute id access * Fix logrotate variable name * Address comments
Signed-off-by: Shuotian Cheng [email protected]