Re: [RFC PATCH 1/2] gitlab-ci: Step in to maintain the fedora-i386-cross runner

[Philippe Mathieu-Daudé]

On 12/14/20 1:35 PM, Thomas Huth wrote:
> On 11/12/2020 18.17, Philippe Mathieu-Daudé wrote:
>> As I am interested in using the runner based on the fedora-i386-cross
>> docker image, add a smoke test job to be sure this image is usable,
>> and set the JOB_MAINTAINER_NAME/JOB_MAINTAINER_EMAIL variables so
>> other developers can contact me in case there is a problem with the
>> public runner based on this image.
>>
>> We will eventually send a patch to make script/checkpatch.pl parse
>> the Gitlab YAML jobs to understand the JOB_MAINTAINER_NAME and
>> JOB_MAINTAINER_EMAIL variables.
>>
>> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
>> ---
>>  .gitlab-ci.d/crossbuilds.yml | 15 +++++++++++++++
>>  1 file changed, 15 insertions(+)
>>
>> diff --git a/.gitlab-ci.d/crossbuilds.yml b/.gitlab-ci.d/crossbuilds.yml
>> index bd6473a75a7..864cad7cec5 100644
>> --- a/.gitlab-ci.d/crossbuilds.yml
>> +++ b/.gitlab-ci.d/crossbuilds.yml
>> @@ -39,6 +39,13 @@
>>        ../configure --enable-werror $QEMU_CONFIGURE_OPTS --disable-system
>>      - make -j$(expr $(nproc) + 1) all check-build
>>  
>> +.cross_sanity_check_job:
>> +  stage: build
>> +  image: $CI_REGISTRY_IMAGE/qemu/$IMAGE:latest
>> +  timeout: 3m
>> +  script:
>> +    - /bin/true
> 
> I guess it makes more sense if I respin my patch here instead:
> 
> https://www.mail-archive.com/qemu-devel@nongnu.org/msg762919.html
> 
> ?

I think these are different. The idea of testing /bin/true is to be
able to make the distinction between a broken runner and a broken
test.

> 
>  Thomas
>