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é <phi...@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 >
