Increase retry times to wait MST device ready

The 10s waiting time is not enough occasionally, so increase it to 60s. Error message will be printed once exceed 10s so we can still detect this unnomal behavior even it get ready eventually. Signed-off-by: Yuanzhe, Liu <[email protected]>
yuazhe · Nov 15, 2024 · 2b9bf23 · 2b9bf23
1 parent 1649851
commit 2b9bf23
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/platform/mellanox/mlnx-fw-upgrade.j2 b/platform/mellanox/mlnx-fw-upgrade.j2
@@ -184,7 +184,8 @@ function UnlockStateChange() {
 }
 
 function WaitForDevice() {
-    local -i QUERY_RETRY_COUNT_MAX="10"
+    local -i QUERY_RETRY_COUNT_MAX="60"
+    local -i QUERY_RETRY_WARNING="10"
     local -i QUERY_RETRY_COUNT="0"
     local SPC_MST_DEV
     local QUERY_RC=""
@@ -196,7 +197,11 @@ function WaitForDevice() {
         [[ ("${QUERY_RETRY_COUNT}" -lt "${QUERY_RETRY_COUNT_MAX}") && ("${QUERY_RC}" != "${EXIT_SUCCESS}") ]] || break
         sleep 1s
         ((QUERY_RETRY_COUNT++))
-        LogInfo "Retrying MST device query ${QUERY_RETRY_COUNT}"
+        if [[ ("${QUERY_RETRY_COUNT}" -lt "${QUERY_RETRY_WARNING}") ]]; then
+            LogInfo "Retrying MST device query ${QUERY_RETRY_COUNT}"
+        else
+            LogError "Retrying MST device query ${QUERY_RETRY_COUNT}"
+        fi
     done
 
     if [[ "${QUERY_RC}" != "${EXIT_SUCCESS}" ]]; then