Increase Coin timeout for ctest command under sanitized testruns

Reason is that after changing the sanitized testruns to quiet, we have been hitting the 15min output-inactivity timeout. Set it to the same value as the total workitem timeout, effectively disabling it. Pick-to: 6.6 6.5 Change-Id: I13c2d97b25cf338873bb89ef1ed45d58e3cdd170 Reviewed-by: Toni Saario <[email protected]> Reviewed-by: Dimitrios Apostolou <[email protected]>
qt · Oct 27, 2023 · 333650c · 333650c
1 parent ac03977
commit 333650c
Showing 1 changed file with 17 additions and 1 deletion.
diff --git a/coin/instructions/cmake_run_ctest_enforce_exit_code.yaml b/coin/instructions/cmake_run_ctest_enforce_exit_code.yaml
@@ -69,12 +69,28 @@ instructions:
  property: features
  contains_value: AbortTestingOnFirstFailure
  - !include "{{qt/qtbase}}/coin_module_test_android_start_emulator.yaml"
+
+ - type: EnvironmentVariable
+ variableName: COIN_COMMAND_OUTPUT_TIMEOUT
+ variableValue: "900"
+ disable_if:
+ condition: property
+ property: features
+ contains_value: UseAddressSanitizer
+ - type: EnvironmentVariable
+ variableName: COIN_COMMAND_OUTPUT_TIMEOUT
+ variableValue: "10800"
+ enable_if:
+ condition: property
+ property: features
+ contains_value: UseAddressSanitizer
+
  - type: ExecuteCommand
  command: "{{.Env.TESTS_ENV_PREFIX}} ctest {{.Env.CTEST_ARGS}}"
  executeCommandArgumentSplitingBehavior: SplitAfterVariableSubstitution
  ignoreExitCode: false
  maxTimeInSeconds: 10800
- maxTimeBetweenOutput: 900
+ maxTimeBetweenOutput: "{{.Env.COIN_COMMAND_OUTPUT_TIMEOUT}}"
  userMessageOnFailure: >
  Failed to run tests.