All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Daniel P. Berrangé" <berrange@redhat.com>
To: John Snow <jsnow@redhat.com>
Cc: "Alex Bennée" <alex.bennee@linaro.org>,
	qemu-devel <qemu-devel@nongnu.org>,
	"Michael Roth" <michael.roth@amd.com>,
	"Alexander Bulekov" <alxndr@bu.edu>,
	"Qiuhao Li" <Qiuhao.Li@outlook.com>,
	"Marc-André Lureau" <marcandre.lureau@redhat.com>,
	"Philippe Mathieu-Daudé" <philmd@linaro.org>,
	"Markus Armbruster" <armbru@redhat.com>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	qemu-arm <qemu-arm@nongnu.org>,
	"Pavel Dovgalyuk" <pavel.dovgaluk@ispras.ru>,
	"Darren Kenny" <darren.kenny@oracle.com>,
	"Stefan Hajnoczi" <stefanha@redhat.com>,
	"Bandan Das" <bsd@redhat.com>, "Cleber Rosa" <crosa@redhat.com>,
	"Peter Maydell" <peter.maydell@linaro.org>,
	"Bastian Koppelmann" <kbastian@mail.uni-paderborn.de>,
	"Yonggang Luo" <luoyonggang@gmail.com>,
	"Li-Wen Hsu" <lwhsu@freebsd.org>,
	"Thomas Huth" <thuth@redhat.com>,
	"Beraldo Leal" <bleal@redhat.com>,
	"Ed Maste" <emaste@freebsd.org>,
	"Wainer dos Santos Moschetta" <wainersm@redhat.com>
Subject: Re: [PATCH 07/12] testing: update ubuntu2004 to ubuntu2204
Date: Fri, 17 Feb 2023 17:14:40 +0000	[thread overview]
Message-ID: <Y++2AK5cDgCGkpVN@redhat.com> (raw)
In-Reply-To: <CAFn=p-b8nK8pNWk3WZgVqjUMx27E4ekeypjGpyk2nrRz=UmNJg@mail.gmail.com>

On Fri, Feb 17, 2023 at 11:35:44AM -0500, John Snow wrote:
> On Thu, Feb 16, 2023, 2:44 PM Daniel P. Berrangé <berrange@redhat.com>
> wrote:
> 
> > On Thu, Feb 16, 2023 at 01:15:30PM -0500, John Snow wrote:
> > > On Wed, Feb 15, 2023 at 2:25 PM Alex Bennée <alex.bennee@linaro.org>
> > wrote:
> > > >
> > > > The 22.04 LTS release has been out for almost a year now so its time
> > > > to update all the remaining images to the current LTS. We can also
> > > > drop some hacks we need for older clang TSAN support.
> > >
> > > We still support Ubuntu 20.04 until 2024 though, don't we? Is it safe
> > > to not test this platform?
> > >
> > > I've long been uncertain about what our policy actually is for docker
> > > tests, if we want to test every platform we support or only some of
> > > them; and if it's only some of them, when do we choose the older and
> > > when do we choose the newer?
> >
> > Ideally we would test both the oldest & newest versions of each
> > distro we support. Practically though, we're compromised by the
> > limited CI resources available.
> >
> 
> Yes, understood.
> 
> 
> > Dropping older Ubuntu images is a reasonable tradeoff, since we
> > still have Debian images covered in CI. Debian can be thought
> > of as an older version of Ubuntu to some extent, giving coverage
> > that will mitigate the risks of dropping 20.04.
> >
> 
> Okay, I'll take your word for that. I am not personally familiar with how
> much those distros diverge; I know Ubuntu is debian-based but that's the
> extent of my knowledge as I don't daily-drive either.
> 
> So, firstly:
> 
> Reviewed-by: John Snow <jsnow@redhat.com>
> 
> because I suspect we all have our reasons and I also agree testing newer is
> generally of higher value than testing older.
> 
> However, would it be possible to keep the older Ubuntu test as a manual
> execution that we could invoke at will, only during RC testing phase? If
> it's not a lot of work, I could even check that in myself as a follow-up if
> it isn't unwanted.
> 
> I find that "oldest version of x" is quite useful to me for testing Python
> stuff in particular, as that ecosystem moves pretty fast. It'd be mighty
> convenient to me in particular to keep an old Ubuntu test around to run
> manually as needed.
> 
> (Heck, even if it wasn't on CI at all but was just a container I could run
> locally, that would still be quite useful.)
> 
> Whaddaya think?

It would be pretty trivial to have tests/docker/dockerfiles contain
Dockerfiles for *every* supported distro version we have, and then
only build & test a subset in CI. It would merely suggest that we
change our naming convention so the dockerfiles in that dir include
the version. Basically adopting the standard libvirt-ci naming
convention for targets of $OSNAME-$OSVERSION:

$ lcitool targets
almalinux-8
almalinux-9
alpine-315
alpine-316
alpine-edge
centos-stream-8
centos-stream-9
debian-10
debian-11
debian-sid
fedora-36
fedora-37
fedora-rawhide
freebsd-12
freebsd-13
freebsd-current
macos-12
macos-13
opensuse-leap-153
opensuse-leap-154
opensuse-tumbleweed
ubuntu-1804
ubuntu-2004
ubuntu-2204

Contributors can then use 'make docker-XXXX' to run build tests
locally on specific distros when they need to test something
that isn't covered by default in out gating CI


With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

  reply	other threads:[~2023-02-17 17:14 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-02-15 19:25 [PATCH 00/12] testing/next: docker, avocado, unit, Alex Bennée
2023-02-15 19:25 ` [PATCH 01/12] gitlab: tweak and filter ninja output to reduce build noise Alex Bennée
2023-02-16  7:32   ` Thomas Huth
2023-02-15 19:25 ` [PATCH 02/12] tests/avocado: retire the Aarch64 TCG tests from boot_linux.py Alex Bennée
2023-02-22 15:22   ` Philippe Mathieu-Daudé
2023-02-15 19:25 ` [PATCH 03/12] tests: add socat dependency for tests Alex Bennée
2023-02-15 19:52   ` Philippe Mathieu-Daudé
2023-02-15 19:25 ` [PATCH 04/12] tests: be a bit more strict cleaning up fifos Alex Bennée
2023-02-15 20:44   ` Philippe Mathieu-Daudé
2023-02-15 20:51   ` Richard Henderson
2023-02-16  7:34   ` Thomas Huth
2023-02-15 19:25 ` [PATCH 05/12] gitlab: reduce default verbosity of cirrus run Alex Bennée
2023-02-16  7:37   ` Thomas Huth
2023-02-16  8:02     ` Alex Bennée
2023-02-16  8:15       ` Thomas Huth
2023-02-15 19:25 ` [PATCH 06/12] gitlab: extend custom runners with base_job_template Alex Bennée
2023-02-16  7:39   ` Thomas Huth
2023-02-15 19:25 ` [PATCH 07/12] testing: update ubuntu2004 to ubuntu2204 Alex Bennée
2023-02-15 20:53   ` Richard Henderson
2023-02-16  7:55   ` Thomas Huth
2023-02-16 18:15   ` John Snow
2023-02-16 19:44     ` Daniel P. Berrangé
2023-02-17 16:35       ` John Snow
2023-02-17 17:14         ` Daniel P. Berrangé [this message]
2023-02-17 17:20           ` John Snow
2023-02-22 15:11           ` Philippe Mathieu-Daudé
2023-02-15 19:25 ` [PATCH 08/12] tests: skip the nios2 replay_kernel test Alex Bennée
2023-02-15 20:47   ` Philippe Mathieu-Daudé
2023-02-15 20:59     ` Richard Henderson
2023-02-22 15:01       ` Philippe Mathieu-Daudé
2023-02-15 20:54   ` Richard Henderson
2023-02-15 19:25 ` [PATCH 09/12] tests: add tuxrun baseline test to avocado Alex Bennée
2023-02-22 15:06   ` Philippe Mathieu-Daudé
2023-02-15 19:25 ` [PATCH 10/12] tests/docker: Use binaries for debian-tricore-cross Alex Bennée
2023-02-15 20:47   ` Philippe Mathieu-Daudé
2023-02-15 19:25 ` [PATCH 11/12] cirrus.yml: Improve the windows_msys2_task Alex Bennée
2023-02-22 15:32   ` Philippe Mathieu-Daudé
2023-02-15 19:25 ` [PATCH 12/12] tests: ensure we export job results for some cross builds Alex Bennée
2023-02-16  7:59   ` Thomas Huth

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=Y++2AK5cDgCGkpVN@redhat.com \
    --to=berrange@redhat.com \
    --cc=Qiuhao.Li@outlook.com \
    --cc=alex.bennee@linaro.org \
    --cc=alxndr@bu.edu \
    --cc=armbru@redhat.com \
    --cc=bleal@redhat.com \
    --cc=bsd@redhat.com \
    --cc=crosa@redhat.com \
    --cc=darren.kenny@oracle.com \
    --cc=emaste@freebsd.org \
    --cc=jsnow@redhat.com \
    --cc=kbastian@mail.uni-paderborn.de \
    --cc=luoyonggang@gmail.com \
    --cc=lwhsu@freebsd.org \
    --cc=marcandre.lureau@redhat.com \
    --cc=michael.roth@amd.com \
    --cc=pavel.dovgaluk@ispras.ru \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=philmd@linaro.org \
    --cc=qemu-arm@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.com \
    --cc=thuth@redhat.com \
    --cc=wainersm@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.