Skip to content

Commit

Permalink
[syncd] Lower bulk missing api message level from error to info (#687)
Browse files Browse the repository at this point in the history
there is command line switch for syncd which was added recently "-l" i think, and it enables use for bulk vendor sai api, previously by default we used 1by1, so actually those error messages are kind a justified that they are in ERROR, since in syncd command line right now we explicitly ask for bulk api
  • Loading branch information
kcudnik authored Nov 1, 2020
1 parent dc73a1d commit 5df11f5
Showing 1 changed file with 11 additions and 11 deletions.
22 changes: 11 additions & 11 deletions syncd/VendorSai.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -668,7 +668,7 @@ sai_status_t VendorSai::bulkCreate(

if (!ptr)
{
SWSS_LOG_ERROR("create bulk not supported from SAI, object_type = %s", sai_serialize_object_type(object_type).c_str());
SWSS_LOG_INFO("create bulk not supported from SAI, object_type = %s", sai_serialize_object_type(object_type).c_str());
return SAI_STATUS_NOT_SUPPORTED;
}

Expand Down Expand Up @@ -727,7 +727,7 @@ sai_status_t VendorSai::bulkRemove(

if (!ptr)
{
SWSS_LOG_ERROR("remove bulk not supported from SAI, object_type = %s", sai_serialize_object_type(object_type).c_str());
SWSS_LOG_INFO("remove bulk not supported from SAI, object_type = %s", sai_serialize_object_type(object_type).c_str());
return SAI_STATUS_NOT_SUPPORTED;
}

Expand Down Expand Up @@ -767,7 +767,7 @@ sai_status_t VendorSai::bulkCreate(

if (!m_apis.route_api->create_route_entries)
{
SWSS_LOG_ERROR("create_route_entries is not supported");
SWSS_LOG_INFO("create_route_entries is not supported");
return SAI_STATUS_NOT_SUPPORTED;
}

Expand All @@ -794,7 +794,7 @@ sai_status_t VendorSai::bulkCreate(

if (!m_apis.fdb_api->create_fdb_entries)
{
SWSS_LOG_ERROR("create_fdb_entries is not supported");
SWSS_LOG_INFO("create_fdb_entries is not supported");
return SAI_STATUS_NOT_SUPPORTED;
}

Expand All @@ -821,7 +821,7 @@ sai_status_t VendorSai::bulkCreate(

if (!m_apis.nat_api->create_nat_entries)
{
SWSS_LOG_ERROR("create_nat_entries is not supported");
SWSS_LOG_INFO("create_nat_entries is not supported");
return SAI_STATUS_NOT_SUPPORTED;
}

Expand All @@ -848,7 +848,7 @@ sai_status_t VendorSai::bulkRemove(

if (!m_apis.route_api->remove_route_entries)
{
SWSS_LOG_ERROR("remove_route_entries is not supported");
SWSS_LOG_INFO("remove_route_entries is not supported");
return SAI_STATUS_NOT_SUPPORTED;
}

Expand All @@ -872,7 +872,7 @@ sai_status_t VendorSai::bulkRemove(

if (!m_apis.fdb_api->remove_fdb_entries)
{
SWSS_LOG_ERROR("remove_fdb_entries is not supported");
SWSS_LOG_INFO("remove_fdb_entries is not supported");
return SAI_STATUS_NOT_SUPPORTED;
}

Expand All @@ -895,7 +895,7 @@ sai_status_t VendorSai::bulkRemove(

if (!m_apis.nat_api->remove_nat_entries)
{
SWSS_LOG_ERROR("remove_nat_entries is not supported");
SWSS_LOG_INFO("remove_nat_entries is not supported");
return SAI_STATUS_NOT_SUPPORTED;
}

Expand All @@ -921,7 +921,7 @@ sai_status_t VendorSai::bulkSet(

if (!m_apis.route_api->set_route_entries_attribute)
{
SWSS_LOG_ERROR("set_route_entries_attribute is not supported");
SWSS_LOG_INFO("set_route_entries_attribute is not supported");
return SAI_STATUS_NOT_SUPPORTED;
}

Expand All @@ -946,7 +946,7 @@ sai_status_t VendorSai::bulkSet(

if (!m_apis.fdb_api->set_fdb_entries_attribute)
{
SWSS_LOG_ERROR("set_fdb_entries_attribute is not supported");
SWSS_LOG_INFO("set_fdb_entries_attribute is not supported");
return SAI_STATUS_NOT_SUPPORTED;
}

Expand All @@ -971,7 +971,7 @@ sai_status_t VendorSai::bulkSet(

if (!m_apis.nat_api->set_nat_entries_attribute)
{
SWSS_LOG_ERROR("set_nat_entries_attribute is not supported");
SWSS_LOG_INFO("set_nat_entries_attribute is not supported");
return SAI_STATUS_NOT_SUPPORTED;
}

Expand Down

0 comments on commit 5df11f5

Please sign in to comment.