skip CMIS init if config state is ConfigUndefined

For the ET8001-2FR4, if there are two breakout ports, the config status for one would be ConfigSuccess and the other would be ConfigUndefined.
edge-core · Oct 23, 2024 · d553fda · d553fda
1 parent 4a24b8d
commit d553fda
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/sonic-xcvrd/xcvrd/xcvrd.py b/sonic-xcvrd/xcvrd/xcvrd.py
@@ -1024,10 +1024,10 @@ def is_cmis_application_update_required(self, api, app_new, host_lanes_mask):
                 if dp_state[name] != 'DataPathActivated':
                     skip = False
                     break
-                #name = "ConfigStatusLane{}".format(lane + 1)
-                #if conf_state[name] != 'ConfigSuccess':
-                #    skip = False
-                #    break
+                name = "ConfigStatusLane{}".format(lane + 1)
+                if conf_state[name] != 'ConfigSuccess' and conf_state[name] != 'ConfigUndefined':
+                    skip = False
+                    break
             return (not skip)
         return True