Skip to content

Change the log severity leve from ERROR to NOTICE if getStatus is not supported by vendor #908

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

Merged
merged 4 commits into from
Aug 26, 2021

Conversation

stephenxs
Copy link
Contributor

Change the log severity leve from ERROR to NOTICE if getStatus is not supported by vendor

Signed-off-by: Stephen Sun [email protected]

Signed-off-by: Stephen Sun <[email protected]>
Signed-off-by: Stephen Sun <[email protected]>
Signed-off-by: Stephen Sun <[email protected]>
@kcudnik kcudnik merged commit 1020de7 into sonic-net:master Aug 26, 2021
@stephenxs stephenxs deleted the fix-error-log-get-status branch August 26, 2021 22:04
@stephenxs
Copy link
Contributor Author

@kcudnik can you cherry-pick this PR to 202106? thanks.

@kcudnik
Copy link
Collaborator

kcudnik commented Sep 3, 2021

can you prepare PR for that ?

@stephenxs
Copy link
Contributor Author

can you prepare PR for that ?

I tested it and it can be smoothly cherry picked to 202106. Can you please check?

@kcudnik
Copy link
Collaborator

kcudnik commented Sep 3, 2021

of course it cherry-pick smoothly, just wanted for you to make PR for this it's your change not mine.

@stephenxs
Copy link
Contributor Author

of course it cherry-pick smoothly, just wanted for you to make PR for this it's your change not mine.

OK. Will do.

stephenxs added a commit to stephenxs/sonic-sairedis that referenced this pull request Sep 3, 2021
kcudnik pushed a commit that referenced this pull request Dec 7, 2021
stephenxs added a commit to stephenxs/sonic-sairedis that referenced this pull request Jan 7, 2022
qiluo-msft pushed a commit that referenced this pull request Jan 22, 2022
… supported by vendor (#908) (#991)

Cherry-pick: #908 to 202012 branch.
pettershao-ragilenetworks pushed a commit to pettershao-ragilenetworks/sonic-sairedis that referenced this pull request Nov 18, 2022
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