On Tue, Feb 06, 2024 at 08:11:58AM +0000, Daniel P. Berrangé wrote: > On Tue, Feb 06, 2024 at 02:31:50PM +0800, pet...@redhat.com wrote: > > From: Peter Xu <pet...@redhat.com> > > > > The new build-previous-qemu job relies on QEMU release tag being present, > > while that may not be always true for personal git repositories since by > > default tag is not pushed. The job can fail on those CI kicks, as reported > > by Peter Maydell. > > > > Fix it by fetching the tags remotely from the official repository, as > > suggested by Dan. > > > > [1] https://lore.kernel.org/r/zcc9sckj7vvqe...@redhat.com > > > > Reported-by: Peter Maydell <peter.mayd...@linaro.org> > > Suggested-by: Daniel P. Berrangé <berra...@redhat.com> > > Signed-off-by: Peter Xu <pet...@redhat.com> > > --- > > .gitlab-ci.d/buildtest.yml | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/.gitlab-ci.d/buildtest.yml b/.gitlab-ci.d/buildtest.yml > > index 79bbc8585b..df48c9d31d 100644 > > --- a/.gitlab-ci.d/buildtest.yml > > +++ b/.gitlab-ci.d/buildtest.yml > > @@ -189,6 +189,8 @@ build-previous-qemu: > > TARGETS: x86_64-softmmu aarch64-softmmu > > before_script: > > - export QEMU_PREV_VERSION="$(sed 's/\([0-9.]*\)\.[0-9]*/v\1.0/' > > VERSION)" > > + - git remote add upstream https://gitlab.com/qemu-project/qemu > > + - git fetch upstream $QEMU_PRRV_VERSION > > Typo^^^
Interestingly, this typo won't affect the function, as QEMU_PRRV_VERSION will be an empty string, and this cmd will simply pull in all the tags.. After the fix, it should only fetch the tag that we need. > > > - git checkout $QEMU_PREV_VERSION > > after_script: > > - mv build build-previous > > With that typo fixed, then > > Reviewed-by: Daniel P. Berrangé <berra...@redhat.com> Thanks. -- Peter Xu