This is an automated email from the ASF dual-hosted git repository. casaroli pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx-apps.git
commit 5cf28b6d6e3954b5c295a86ac58f65a7218d8f81 Author: Brijesh Thakkar <[email protected]> AuthorDate: Sat Oct 24 15:25:04 2026 +0530 .github/workflows/build.yml: remove the pytest watchdog The Linux build job starts "( sleep 7200 ; ... ; pkill -f pytest )&" in the background before running cibuild.sh. It was meant to stop a hung test run, but it does not do that now that the runtime tests use NTFC: NTFC runs pytest in-process, so no process has "pytest" in its command line. The pattern only matches the "bash -c" job script and the watchdog subshell, because their script text contains the word "pytest". When it fires it kills the whole step instead of the tests. The watchdog is also redundant. NTFC bounds each test session itself with "timeout_session" in the config section of the citest config.yaml files of the nuttx tree (3600 seconds when not set, 6000 seconds for sim/citest), enforced through pytest-timeout --session-timeout. When the limit is reached NTFC stops the session, keeps its reports and exits with status 1, so the job still fails. Note that the session timeout covers the test session only, not the build. This matches the change made in apache/nuttx. Part of apache/nuttx#17717 Assisted-by: Claude Code:claude-sonnet-5-5 Signed-off-by: Brijesh Thakkar <[email protected]> --- .github/workflows/build.yml | 1 - 1 file changed, 1 deletion(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index af65867fa..b523baaf6 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -419,7 +419,6 @@ jobs: if [ "X${{matrix.boards}}" = "Xcodechecker" ]; then ./cibuild.sh -c -A -N -R --codechecker testlist/${{matrix.boards}}.dat else - ( sleep 7200 ; echo Killing pytest after timeout... ; pkill -f pytest )& ./cibuild.sh -c -A -N -R -S testlist/${{matrix.boards}}.dat fi
