Skip to content

Commit 5df11f5

Browse files
authored
[syncd] Lower bulk missing api message level from error to info (sonic-net#687)
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
1 parent dc73a1d commit 5df11f5

File tree

1 file changed

+11
-11
lines changed

1 file changed

+11
-11
lines changed

syncd/VendorSai.cpp

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -668,7 +668,7 @@ sai_status_t VendorSai::bulkCreate(
668668

669669
if (!ptr)
670670
{
671-
SWSS_LOG_ERROR("create bulk not supported from SAI, object_type = %s", sai_serialize_object_type(object_type).c_str());
671+
SWSS_LOG_INFO("create bulk not supported from SAI, object_type = %s", sai_serialize_object_type(object_type).c_str());
672672
return SAI_STATUS_NOT_SUPPORTED;
673673
}
674674

@@ -727,7 +727,7 @@ sai_status_t VendorSai::bulkRemove(
727727

728728
if (!ptr)
729729
{
730-
SWSS_LOG_ERROR("remove bulk not supported from SAI, object_type = %s", sai_serialize_object_type(object_type).c_str());
730+
SWSS_LOG_INFO("remove bulk not supported from SAI, object_type = %s", sai_serialize_object_type(object_type).c_str());
731731
return SAI_STATUS_NOT_SUPPORTED;
732732
}
733733

@@ -767,7 +767,7 @@ sai_status_t VendorSai::bulkCreate(
767767

768768
if (!m_apis.route_api->create_route_entries)
769769
{
770-
SWSS_LOG_ERROR("create_route_entries is not supported");
770+
SWSS_LOG_INFO("create_route_entries is not supported");
771771
return SAI_STATUS_NOT_SUPPORTED;
772772
}
773773

@@ -794,7 +794,7 @@ sai_status_t VendorSai::bulkCreate(
794794

795795
if (!m_apis.fdb_api->create_fdb_entries)
796796
{
797-
SWSS_LOG_ERROR("create_fdb_entries is not supported");
797+
SWSS_LOG_INFO("create_fdb_entries is not supported");
798798
return SAI_STATUS_NOT_SUPPORTED;
799799
}
800800

@@ -821,7 +821,7 @@ sai_status_t VendorSai::bulkCreate(
821821

822822
if (!m_apis.nat_api->create_nat_entries)
823823
{
824-
SWSS_LOG_ERROR("create_nat_entries is not supported");
824+
SWSS_LOG_INFO("create_nat_entries is not supported");
825825
return SAI_STATUS_NOT_SUPPORTED;
826826
}
827827

@@ -848,7 +848,7 @@ sai_status_t VendorSai::bulkRemove(
848848

849849
if (!m_apis.route_api->remove_route_entries)
850850
{
851-
SWSS_LOG_ERROR("remove_route_entries is not supported");
851+
SWSS_LOG_INFO("remove_route_entries is not supported");
852852
return SAI_STATUS_NOT_SUPPORTED;
853853
}
854854

@@ -872,7 +872,7 @@ sai_status_t VendorSai::bulkRemove(
872872

873873
if (!m_apis.fdb_api->remove_fdb_entries)
874874
{
875-
SWSS_LOG_ERROR("remove_fdb_entries is not supported");
875+
SWSS_LOG_INFO("remove_fdb_entries is not supported");
876876
return SAI_STATUS_NOT_SUPPORTED;
877877
}
878878

@@ -895,7 +895,7 @@ sai_status_t VendorSai::bulkRemove(
895895

896896
if (!m_apis.nat_api->remove_nat_entries)
897897
{
898-
SWSS_LOG_ERROR("remove_nat_entries is not supported");
898+
SWSS_LOG_INFO("remove_nat_entries is not supported");
899899
return SAI_STATUS_NOT_SUPPORTED;
900900
}
901901

@@ -921,7 +921,7 @@ sai_status_t VendorSai::bulkSet(
921921

922922
if (!m_apis.route_api->set_route_entries_attribute)
923923
{
924-
SWSS_LOG_ERROR("set_route_entries_attribute is not supported");
924+
SWSS_LOG_INFO("set_route_entries_attribute is not supported");
925925
return SAI_STATUS_NOT_SUPPORTED;
926926
}
927927

@@ -946,7 +946,7 @@ sai_status_t VendorSai::bulkSet(
946946

947947
if (!m_apis.fdb_api->set_fdb_entries_attribute)
948948
{
949-
SWSS_LOG_ERROR("set_fdb_entries_attribute is not supported");
949+
SWSS_LOG_INFO("set_fdb_entries_attribute is not supported");
950950
return SAI_STATUS_NOT_SUPPORTED;
951951
}
952952

@@ -971,7 +971,7 @@ sai_status_t VendorSai::bulkSet(
971971

972972
if (!m_apis.nat_api->set_nat_entries_attribute)
973973
{
974-
SWSS_LOG_ERROR("set_nat_entries_attribute is not supported");
974+
SWSS_LOG_INFO("set_nat_entries_attribute is not supported");
975975
return SAI_STATUS_NOT_SUPPORTED;
976976
}
977977

0 commit comments

Comments
 (0)