Remove if and log messages for event_publish (#1)

sonic-net · Sep 14, 2022 · 460a7ec · 460a7ec
1 parent 74ff2dc
commit 460a7ec
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 9 deletions.
diff --git a/orchagent/crmorch.cpp b/orchagent/crmorch.cpp
@@ -772,9 +772,7 @@ void CrmOrch::checkCrmThresholds()
                 SWSS_LOG_WARN("%s THRESHOLD_EXCEEDED for %s %u%% Used count %u free count %u",
                               res.name.c_str(), threshType.c_str(), percentageUtil, cnt.usedCounter, cnt.availableCounter);
 
-                if (0 != event_publish(m_events_handle, "chk_crm_threshold", &params)) {
-                    SWSS_LOG_WARN("Failed to publish event for if-state");
-                }
+                event_publish(m_events_handle, "chk_crm_threshold", &params);
 
                 res.exceededLogCounter++;
             }

diff --git a/orchagent/pfcwdorch.cpp b/orchagent/pfcwdorch.cpp
@@ -952,9 +952,7 @@ void PfcWdSwOrch<DropHandler, ForwardHandler>::report_pfc_storm(
             id,
             entry->portId);
 
-    if (0 != event_publish(m_events_handle, "pfc-storm", &params)) {
-        SWSS_LOG_WARN("Failed to publish event for pfc-storm");
-    }
+    event_publish(m_events_handle, "pfc-storm", &params);
 }
 
 template <typename DropHandler, typename ForwardHandler>

diff --git a/orchagent/portsorch.cpp b/orchagent/portsorch.cpp
@@ -2540,9 +2540,7 @@ bool PortsOrch::setHostIntfsOperStatus(const Port& port, bool isUp) const
             isUp ? "UP" : "DOWN", port.m_alias.c_str());
 
     event_params_t params = {{"ifname",port.m_alias},{"status",isUp ? "up" : "down"}};
-    if (0 != event_publish(m_events_handle, "if-state", &params)) {
-        SWSS_LOG_WARN("Failed to publish event for if-state");
-    }
+    event_publish(m_events_handle, "if-state", &params);
     return true;
 }