Signed-off-by: Jaganathan Anbalagan <[email protected]>

set host_tx_ready to false if gbsyncd SAI API fails.
sonic-net · Jun 30, 2022 · 3d44c37 · 3d44c37
1 parent ed306d7
commit 3d44c37
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/orchagent/portsorch.cpp b/orchagent/portsorch.cpp
@@ -1056,6 +1056,10 @@ bool PortsOrch::setPortAdminStatus(Port &port, bool state)
     }
 
     bool gbstatus = setGearboxPortsAttr(port, SAI_PORT_ATTR_ADMIN_STATE, &state);
+    if (gbstatus != true)
+    {
+        m_portStateTable.hset(port.m_alias, "host_tx_ready", "false");
+    }
 
     /* Update the state table for host_tx_ready*/
     if (state && (gbstatus == true) && (status == SAI_STATUS_SUCCESS) )
@@ -1064,12 +1068,6 @@ bool PortsOrch::setPortAdminStatus(Port &port, bool state)
         SWSS_LOG_INFO("Set admin status UP host_tx_ready to true to port pid:%" PRIx64,
                 port.m_port_id);
     } 
-    else
-    {
-        m_portStateTable.hset(port.m_alias, "host_tx_ready", "false");
-        SWSS_LOG_INFO("Set admin status %s host_tx_ready to false to port pid:%" PRIx64,
-                    state ? "UP" : "DOWN", port.m_port_id);
-    }
 
     return true;
 }