AlexIndustrial/mesa
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

ci/poe-powered: rename --test-timeout to --test-timeout-minutes to be more explicit

Browse Source 
Due to how python's argparse works, anyone anywhere else that would
continue passing `--test-timeout` would not see any change, it will
transparently work for them too.

Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/30409>
This commit is contained in:
Eric Engestrom
2024-07-29 12:46:43 +02:00
parent 56ea4e4fa6
commit 1e5fb15afa
4 changed files with 10 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
.gitlab-ci/bare-metal/poe-powered.sh
View File

@@ -199,7 +199,7 @@ while [ $((ATTEMPTS--)) -gt 0 ]; do
--dev="$BM_SERIAL" \
--powerup="$BM_POWERUP" \
--powerdown="$BM_POWERDOWN" \
--test-timeout ${TEST_PHASE_TIMEOUT:-20}
--test-timeout-minutes ${TEST_PHASE_TIMEOUT_MINUTES:-20}
ret=$?
if [ $ret -eq 2 ]; then

4
.gitlab-ci/bare-metal/poe_run.py
View File

@@ -109,12 +109,12 @@ def main():
parser.add_argument('--powerdown', type=str,
help='shell command for powering off', required=True)
parser.add_argument(
'--test-timeout', type=int, help='Test phase timeout (minutes)', required=True)
'--test-timeout-minutes', type=int, help='Test phase timeout (minutes)', required=True)
args = parser.parse_args()
logger = CustomLogger("job_detail.json")
logger.update_dut_time("start", None)
poe = PoERun(args, args.test_timeout * 60, logger)
poe = PoERun(args, args.test_timeout_minutes * 60, logger)
retval = poe.run()
poe.logged_system(args.powerdown)