From 2bc82b7147cc598d1b210d0dcbc5697a5f3a49b1 Mon Sep 17 00:00:00 2001 From: Eric Engestrom Date: Wed, 24 Jul 2024 11:42:28 +0200 Subject: [PATCH] ci/baremetal: fix logic for retrying boot when it failed Contrary to what the original commit said, this is actually still used (see .gitlab-ci/bare-metal/poe-powered.sh:205), and the boot retry logic has been broken ever since, exacerbating the rpi farm boot problems. Fixes: 97b2afa16aeeceff53fb ("ci/bare-metal: Drop the 2 vs 1 exit code from poe_run.") Part-of: --- .gitlab-ci/bare-metal/poe_run.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.gitlab-ci/bare-metal/poe_run.py b/.gitlab-ci/bare-metal/poe_run.py index 0220acad545..98e5d4e35d2 100755 --- a/.gitlab-ci/bare-metal/poe_run.py +++ b/.gitlab-ci/bare-metal/poe_run.py @@ -64,7 +64,7 @@ class PoERun: if not boot_detected: self.print_error( "Something wrong; couldn't detect the boot start up sequence") - return 1 + return 2 self.logger.create_job_phase("test") for line in self.ser.lines(timeout=self.test_timeout, phase="test"):