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

Log level change from ERR to INFO for fetch systemports issue #1632

Merged
merged 1 commit into from
Feb 10, 2021

Conversation

vaibhavhd
Copy link
Contributor

What I did
Fix: sonic-net/sonic-buildimage#6727

Why I did it
SAI attribute SAI_SWITCH_ATTR_NUMBER_OF_SYSTEM_PORTS is not supported on many platforms, and the call to getSystemPorts() throws error.
This error is not harmful at this state, and therefore the log level is changed from ERROR to INFO.

How I verified it

Details if related

@lguohan lguohan merged commit aaa7bf2 into sonic-net:master Feb 10, 2021
DavidZagury pushed a commit to DavidZagury/sonic-swss that referenced this pull request Mar 4, 2021
…net#1632)

SAI attribute SAI_SWITCH_ATTR_NUMBER_OF_SYSTEM_PORTS is not supported on many platforms, and the call to getSystemPorts() throws error.
This error is not harmful at this state, and therefore the log level is changed from ERROR to INFO.
@radha-danda
Copy link

@lguohan, can this PR me cherry-picked to 202012 branch?

@yxieca
Copy link
Contributor

yxieca commented Apr 8, 2021

@vaibhavhd please create an PR for 202012 branch. This change cannot be cherry-picked cleanly.

@vaibhavhd vaibhavhd deleted the system-port-log-fix branch April 8, 2021 22:52
@vaibhavhd
Copy link
Contributor Author

@vaibhavhd please create an PR for 202012 branch. This change cannot be cherry-picked cleanly.

This fix is not needed on 202012 branch, as the change introducing systemports init is not in the 202012 branch yet - #1431

raphaelt-nvidia pushed a commit to raphaelt-nvidia/sonic-swss that referenced this pull request Oct 5, 2021
…net#1632)

SAI attribute SAI_SWITCH_ATTR_NUMBER_OF_SYSTEM_PORTS is not supported on many platforms, and the call to getSystemPorts() throws error.
This error is not harmful at this state, and therefore the log level is changed from ERROR to INFO.
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.

PortsOrch init error: SAI attr SAI_SWITCH_ATTR_NUMBER_OF_SYSTEM_PORTS not supported
5 participants