Merge branch 'fix_otatool_serial_handoff_v5.3' into 'release/v5.3'

fix(otatool): fix flaky serial port handoff in otatool CI test (backport v5.3)

See merge request espressif/esp-idf!47716

examples/system/ota/otatool/pytest_otatool.py

@@ -1,8 +1,10 @@
 # SPDX-FileCopyrightText: 2022-2026 Espressif Systems (Shanghai) CO LTD
 # SPDX-License-Identifier: Unlicense OR CC0-1.0
+import logging
 import os
 import subprocess
 import sys
+import time
 
 import pytest
 from pytest_embedded import Dut
@@ -17,6 +19,10 @@ def _real_test_func(dut: Dut) -> None:
     # Close connection to DUT
     dut.serial.proc.close()
 
+    # Allow the OS to fully release the serial port. pytest-embedded's
+    # QueueFeederThread may still hold the port FD when close() returns.
+    time.sleep(2)
+
     script_path = os.path.join(str(os.getenv('IDF_PATH')), 'examples', 'system', 'ota', 'otatool', 'otatool_example.py')
     binary_path = ''
 
@@ -24,7 +30,23 @@ def _real_test_func(dut: Dut) -> None:
         if 'otatool.bin' in flash_file[1]:
             binary_path = flash_file[1]
             break
-    subprocess.check_call([sys.executable, script_path, '--binary', binary_path])
+
+    # Retry the subprocess to handle transient serial port contention.
+    # The otatool_example.py subprocess opens the serial port independently
+    # via esptool, and may fail if pytest-embedded's QueueFeederThread has
+    # not fully released the port file descriptor yet.
+    last_err = None
+    for attempt in range(3):
+        try:
+            subprocess.check_call([sys.executable, script_path, '--binary', binary_path])
+            return
+        except subprocess.CalledProcessError as e:
+            last_err = e
+            logging.warning('otatool subprocess attempt %d/3 failed: %s', attempt + 1, e)
+            time.sleep(3)
+
+    assert last_err is not None
+    raise last_err
 
 
 @pytest.mark.generic