[orchagent] Ensure Path Tracing is supported before configuring it in…

… SAI Reject Path Tracing Interface ID and Timestamp Template configuration when Path Tracing is not supported Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>
sonic-net · Oct 26, 2023 · 06b7459 · 06b7459
1 parent 653a2e2
commit 06b7459
Showing 1 changed file with 32 additions and 0 deletions.
diff --git a/orchagent/portsorch.cpp b/orchagent/portsorch.cpp
@@ -865,6 +865,14 @@ bool PortsOrch::addPortBulk(const std::vector<PortConfig> &portList)
 
         if (cit.pt_intf_id.is_set)
         {
+            if (!m_isPathTracingSupported)
+            {
+                SWSS_LOG_WARN(
+                    "Failed to set Path Tracing Interface ID: Path Tracing is not supported by the switch"
+                );
+                continue;
+            }
+
             /*
              * First, let's check the Path Tracing Interface ID configured for the port.
              *
@@ -911,6 +919,14 @@ bool PortsOrch::addPortBulk(const std::vector<PortConfig> &portList)
 
         if (cit.pt_timestamp_template.is_set)
         {
+            if (!m_isPathTracingSupported)
+            {
+                SWSS_LOG_WARN(
+                    "Failed to set Path Tracing Timestamp Template: Path Tracing is not supported by the switch"
+                );
+                continue;
+            }
+
             attr.id = SAI_PORT_ATTR_PATH_TRACING_TIMESTAMP_TYPE;
             attr.value.u16 = cit.pt_timestamp_template.value;
             attrList.push_back(attr);
@@ -4215,6 +4231,14 @@ void PortsOrch::doPortTask(Consumer &consumer)
 
                 if (pCfg.pt_intf_id.is_set)
                 {
+                    if (!m_isPathTracingSupported)
+                    {
+                        SWSS_LOG_WARN(
+                            "Failed to set Path Tracing Interface ID: Path Tracing is not supported by the switch"
+                        );
+                        continue;
+                    }
+
                     if (p.m_pt_intf_id != pCfg.pt_intf_id.value)
                     {
                         /*
@@ -4277,6 +4301,14 @@ void PortsOrch::doPortTask(Consumer &consumer)
 
                 if (pCfg.pt_timestamp_template.is_set)
                 {
+                    if (!m_isPathTracingSupported)
+                    {
+                        SWSS_LOG_WARN(
+                            "Failed to set Path Tracing Timestamp Template: Path Tracing is not supported by the switch"
+                        );
+                        continue;
+                    }
+
                     if (p.m_pt_timestamp_template != pCfg.pt_timestamp_template.value)
                     {
                         if (!setPortPtTimestampTemplate(p, pCfg.pt_timestamp_template.value))