From 5df11f5e545e61d907bc21cda8f939b243868b4d Mon Sep 17 00:00:00 2001 From: Kamil Cudnik Date: Sun, 1 Nov 2020 08:54:25 +0100 Subject: [PATCH] [syncd] Lower bulk missing api message level from error to info (#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 --- syncd/VendorSai.cpp | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/syncd/VendorSai.cpp b/syncd/VendorSai.cpp index 03b92e6cb..bc978ab2c 100644 --- a/syncd/VendorSai.cpp +++ b/syncd/VendorSai.cpp @@ -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; } @@ -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; } @@ -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; } @@ -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; } @@ -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; } @@ -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; } @@ -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; } @@ -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; } @@ -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; } @@ -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; } @@ -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; }