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

Don't treat ATTR_NOT_IMPLEMENTED and ATTR_NOT_SUPPORTED as errors #320

Merged
merged 15 commits into from
May 16, 2018
Merged
Changes from all commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
3cd7a6e
Add packaging for syncd-rpc
pavel-shirshov Apr 14, 2017
2de6cbe
Merge remote-tracking branch 'upstream/master'
pavel-shirshov Apr 21, 2017
6f14bbe
Merge remote-tracking branch 'upstream/master'
pavel-shirshov May 1, 2017
d0b8098
Merge branch 'master' of https://github.com/Azure/sonic-sairedis
pavel-shirshov Nov 17, 2017
9ae5f1a
Merge branch 'master' of https://github.com/Azure/sonic-sairedis
pavel-shirshov Nov 21, 2017
24bca8c
Merge branch 'master' of https://github.com/Azure/sonic-sairedis
pavel-shirshov Nov 30, 2017
bb6412f
Merge branch 'master' of https://github.com/Azure/sonic-sairedis
pavel-shirshov Jan 10, 2018
4887827
Merge branch 'master' of https://github.com/Azure/sonic-sairedis
pavel-shirshov Jan 18, 2018
e147269
Merge branch 'master' of https://github.com/Azure/sonic-sairedis
pavel-shirshov Feb 15, 2018
e4fffd2
Merge branch 'master' of https://github.com/Azure/sonic-sairedis
pavel-shirshov Mar 14, 2018
918f371
Merge branch 'master' of https://github.com/Azure/sonic-sairedis
pavel-shirshov Apr 3, 2018
548d4da
Merge branch 'master' of https://github.com/Azure/sonic-sairedis
pavel-shirshov Apr 12, 2018
7a31238
Merge branch 'master' of https://github.com/Azure/sonic-sairedis
pavel-shirshov Apr 16, 2018
2d259fa
Merge branch 'master' of https://github.com/Azure/sonic-sairedis
pavel-shirshov May 15, 2018
91d45ea
Don't treat ATTR_NOT_IMPLEMENTED and ATTR_NOT_SUPPORTED as errors
pavel-shirshov May 16, 2018
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion meta/sai_meta.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -3644,7 +3644,9 @@ sai_status_t meta_sai_set_fdb_entry(
#define META_LOG_STATUS(s)\
if (s == SAI_STATUS_SUCCESS) \
SWSS_LOG_DEBUG("get status: %s", sai_serialize_status(s).c_str()); \
else if (s == SAI_STATUS_BUFFER_OVERFLOW) \
else if (s == SAI_STATUS_BUFFER_OVERFLOW \
|| SAI_STATUS_IS_ATTR_NOT_IMPLEMENTED(s) \
|| SAI_STATUS_IS_ATTR_NOT_SUPPORTED(s)) \
SWSS_LOG_INFO("get status: %s", sai_serialize_status(s).c_str()); \
else \
SWSS_LOG_ERROR("get status: %s", sai_serialize_status(s).c_str());
Expand Down