search

OP-TEE / build

Makefiles to use OP-TEE on various platforms
106 stars 209 forks source link

qemuv8: rust.exp: set timeout to 10 seconds #722

Closed jforissier closed 5 months ago

jforissier commented 5 months ago

There is no timeout specified in rust.exp. This makes the tests unreliable, because depending on which tests have been run before, the $timeout global variable may have different values. For example, it is set to 900 by qemu-check.exp (the main file), but may be overriden by trusted-keys.exp to 5 seconds. That's not enough in some cases, causing the following error in optee_os CI:

2024-01-19T11:58:23.5778720Z Running signature_verification-rs... 2024-01-19T11:58:28.3431817Z !!! Timeout: Test failed

jforissier commented 5 months ago

Could someone please Ack? I'd like to fix the CI before the release. Thanks!

jenswi-linaro commented 5 months ago

Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

jforissier commented 5 months ago

Thanks!