Change sai log level before discovery to prevent SAI ERR spam to the …

…log (sonic-net#316)
dmytroxshevchuk · May 14, 2018 · 39a5d14 · 39a5d14
1 parent 177f768
commit 39a5d14
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 1 deletion.
diff --git a/syncd/syncd.cpp b/syncd/syncd.cpp
@@ -3054,6 +3054,21 @@ void set_sai_api_loglevel()
     }
 }
 
+void set_sai_api_log_min_prio(const std::string &prioStr)
+{
+    SWSS_LOG_ENTER();
+
+    /*
+     * We start from 1 since 0 is SAI_API_UNSPECIFIED.
+     */
+
+    for (uint32_t idx = 1; idx < sai_metadata_enum_sai_api_t.valuescount; ++idx)
+    {
+        const auto& api_name = sai_metadata_enum_sai_api_t.valuesnames[idx];
+        saiLoglevelNotify(api_name, prioStr);
+    }
+}
+
 void performWarmRestart()
 {
     SWSS_LOG_ENTER();

diff --git a/syncd/syncd.h b/syncd/syncd.h
@@ -113,4 +113,7 @@ sai_status_t processBulkEvent(
         _In_ sai_common_api_t api,
         _In_ const swss::KeyOpFieldsValuesTuple &kco);
 
+void set_sai_api_log_min_prio(
+        _In_ const std::string &prio);
+
 #endif // __SYNCD_H__
diff --git a/syncd/syncd_saiswitch.cpp b/syncd/syncd_saiswitch.cpp
@@ -937,8 +937,9 @@ void SaiSwitch::helperDiscover()
 
     {
         SWSS_LOG_TIMER("discover");
-
+        set_sai_api_log_min_prio("SAI_LOG_LEVEL_CRITICAL");
         saiDiscover(m_switch_rid, m_discovered_rids);
+        set_sai_api_log_min_prio("SAI_LOG_LEVEL_NOTICE");
     }
 
     SWSS_LOG_NOTICE("discovered objects count: %zu", m_discovered_rids.size());