Re: [PATCH 1/2] CI: switch to the new runner

[Anton Mikanovich <amikan@ilbers.de>]

13.07.2022 14:20, Henning Schild wrote:
> Am Tue, 12 Jul 2022 20:32:36 +0300
> schrieb Anton Mikanovich <amikan@ilbers.de>:
>
>> Switch CI from legacy runner removed in Avocado 97.0 to the nrunner.
>> To make it run serialized limit maximum task count to 1.
>>
>> Signed-off-by: Anton Mikanovich <amikan@ilbers.de>
>> ---
>>   scripts/ci_build.sh | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/scripts/ci_build.sh b/scripts/ci_build.sh
>> index b1ec691..fa7208c 100755
>> --- a/scripts/ci_build.sh
>> +++ b/scripts/ci_build.sh
>> @@ -143,5 +143,5 @@ export VIRTUAL_ENV="./"
>>   set -x
>>   
>>   avocado $VERBOSE run "$TESTSUITE_DIR/citest.py" \
>> -    -t $TAGS --test-runner=runner --disable-sysinfo \
>> +    -t $TAGS --nrunner-max-parallel-tasks=1 --disable-sysinfo \
> Why 1 and not 42 or nproc? Because our tests are written in a way that
> they build onto each other, so we lack modularization and are pretty
> much still on a monolith like back in the shell age?
>
> Henning
>
>>       -p quiet=$QUIET -p cross=$CROSS_BUILD -p time_to_wait=$TIMEOUT

We need this fix to make nrunner work like previous legacy runner until 
tests
dependencies will be integrated into Avocado so we can rebuild our test 
cases
to be executed in parallel.
See: https://github.com/avocado-framework/avocado/issues/5205