Skip to content

ci/lava: Use CI_JOB_TIMEOUT instead of separate variable

Valentine Burley requested to merge Valentine/mesa:timeout-lava into main

The CI_JOB_TIMEOUT variable is the GitLab-defined job timeout in seconds. Use this variable in LAVA instead of the separate JOB_TIMEOUT, which was intended to represent the test phase timeout (job timeout minus 5 minutes), but was often overlooked.

I've verified that the CI_JOB_TIMEOUT is correctly parsed and that a longer job doesn't get cut short either.

I decided to competely delete JOB_TIMEOUT, because it was both confusing and error-prone. There should be no reason to go with a value different than CI_JOB_TIMEOUT - 5 minutes.

Edited by Valentine Burley

Merge request reports

Loading