qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Fabiano Rosas <farosas@suse.de>
To: Stefan Hajnoczi <stefanha@redhat.com>
Cc: qemu-devel@nongnu.org, Peter Xu <peterx@redhat.com>
Subject: Re: Migration compatibility test broken on major version releases
Date: Wed, 23 Apr 2025 14:01:58 -0300	[thread overview]
Message-ID: <87bjsmerqx.fsf@suse.de> (raw)
In-Reply-To: <20250422154131.GA308586@fedora>

Stefan Hajnoczi <stefanha@redhat.com> writes:

> Hi Fabiano,
> The build-previous-qemu job does not work when a new major version is
> released:
> https://gitlab.com/qemu-project/qemu/-/jobs/9788294494
>

You might be using a slightly different workflow from Peter and Richard,
I don't think this ever happened before. But that's totally fine, I'll
change the job to behave better in that case.

> The previous version computation produces "v10.0.0" when testing:
>
>   $ export QEMU_PREV_VERSION="$(sed 's/\([0-9.]*\)\.[0-9]*/v␁.0/' VERSION)"
>   $ git remote add upstream https://gitlab.com/qemu-project/qemu
>   $ git fetch upstream refs/tags/$QEMU_PREV_VERSION:refs/tags/$QEMU_PREV_VERSION
>   warning: redirecting to https://gitlab.com/qemu-project/qemu.git/
>   fatal: couldn't find remote ref refs/tags/v10.0.0
>
> The CI job runs before the v10.0.0 tag is pushed to the repo. (The tag
> is only pushed once tests have passed.)
>
> Even if the tag was there and git fetch succeeded, the test would test
> migration between v10.0.0 and v10.0.0, which doesn't seem to be the
> purpose of the test.
>

Yes, but since that one commit does not have any code anyway, we've
decided to just let it run on the same version. The very next commit
will be aligned again. Still, the time-of-tag issue was indeed an
oversight.

> Please adjust the test to handle this situation. For now I will re-run
> the job after pushing the final tag (since it already passed for the
> release candidate tag).

Will do, thanks!

>
> Thanks!
>
> Stefan


  reply	other threads:[~2025-04-23 17:02 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-04-22 15:41 Migration compatibility test broken on major version releases Stefan Hajnoczi
2025-04-23 17:01 ` Fabiano Rosas [this message]
2025-04-23 22:05   ` Peter Xu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87bjsmerqx.fsf@suse.de \
    --to=farosas@suse.de \
    --cc=peterx@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).