Fix the Fec Mode Setting of gbsyncd (#2430)

abdosi · web-flow · commit f52a7b1c3419 · 2022-08-30T14:35:12.000-07:00
Why I did: PR: #2400 made change to pass <string> as argument to API setPortFecMode but did not updated the corresponding gbsyncd API call
diff --git a/orchagent/portsorch.cpp b/orchagent/portsorch.cpp
@@ -1290,7 +1290,7 @@ bool PortsOrch::setPortFec(Port &port, string &mode)
 
     SWSS_LOG_NOTICE("Set port %s FEC mode %s", port.m_alias.c_str(), mode.c_str());
 
-    setGearboxPortsAttr(port, SAI_PORT_ATTR_FEC_MODE, &mode);
+    setGearboxPortsAttr(port, SAI_PORT_ATTR_FEC_MODE, &port.m_fec_mode);
 
     return true;
 }
@@ -7869,4 +7869,4 @@ void PortsOrch::doTask(swss::SelectableTimer &timer)
     {
         m_port_state_poller->stop();
     }
-}
+}

Original file line number	Diff line number	Diff line change
`@@ -1290,7 +1290,7 @@ bool PortsOrch::setPortFec(Port &port, string &mode)`
`1290`	`1290`
`1291`	`1291`	`SWSS_LOG_NOTICE("Set port %s FEC mode %s", port.m_alias.c_str(), mode.c_str());`
`1292`	`1292`
`1293`		`- setGearboxPortsAttr(port, SAI_PORT_ATTR_FEC_MODE, &mode);`
	`1293`	`+ setGearboxPortsAttr(port, SAI_PORT_ATTR_FEC_MODE, &port.m_fec_mode);`
`1294`	`1294`
`1295`	`1295`	`return true;`
`1296`	`1296`	`}`
`@@ -7869,4 +7869,4 @@ void PortsOrch::doTask(swss::SelectableTimer &timer)`
`7869`	`7869`	`{`
`7870`	`7870`	`m_port_state_poller->stop();`
`7871`	`7871`	`}`
`7872`		`-}`
	`7872`	`+}`