From: "Daniel P. Berrange" <berrange@redhat.com>
To: "Philippe Mathieu-Daudé" <f4bug@amsat.org>
Cc: "Fam Zheng" <fzheng@redhat.com>,
"Alex Bennée" <alex.bennee@linaro.org>,
"Paolo Bonzini" <pbonzini@redhat.com>,
qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH 0/7] docker: update Ubuntu and Fedora images, deprecate old ones
Date: Wed, 17 Jan 2018 14:04:41 +0000 [thread overview]
Message-ID: <20180117140441.GO19227@redhat.com> (raw)
In-Reply-To: <19661b12-2ac7-5253-3d6a-44c8994c18cc@amsat.org>
On Wed, Jan 17, 2018 at 10:58:21AM -0300, Philippe Mathieu-Daudé wrote:
> On 01/17/2018 07:25 AM, Daniel P. Berrange wrote:
> > On Wed, Jan 17, 2018 at 10:26:36AM +0800, Fam Zheng wrote:
> >> On 01/12/2018 08:49 PM, Philippe Mathieu-Daudé wrote:
> >>> Hi,
> >>>
> >>> This series is to be clearer about which upstream version we are using.
> >>>
> >>> All "FROM distrib:latest" entries have now been removed and replaced by
> >>> explicit "FROM distrib:version" ones.
> >>>
> >>> To keep backward compatibility, a warning is displayed to the user,
> >>> suggesting which correct base image to use.
> >>>
> >>> To be consistent, we remove the deprecated images of the "make docker" output.
> >>
> >> Changing image names when a new release comes out is not maintainable for
> >> Fedora and Ubuntu because of the fast pace. Therefore I prefer Paolo's
> >> simple patch.
> >
> > The flipside is that if we release QEMU 2.12 with fedora:27, then for as long
> > as the 2.12 stable branch exists, dockers tests run on that branch will have a
> > consistent target and will be unlikely to break. If we use fedora:latest,
> > then every time a new Fedora comes out there's non-trivial chance that docker
> > builds on all the stable branches will all break due to some new glibc change.
> > Sticking with explicit named versions is a big win in this respect IMHO.
>
> What happened to me twice is checking an older QEMU tag while bisecting
> and being unable to run 'make docker-test' because the older QEMU tag
> use 'latest' which is not the current 'latest' from the Docker library,
> and I had to manually change the tag to the version 'latest' was
> pointing to at the time of the QEMU tag release.
>
> On each stable release we need to freeze the current tag in a new
> Dockerfile, and keep the generic Dockerfile following the 'latest' tag.
>
> Once tagged different that 'latest' the Dockerfile isn't suppose to
> change. So we only maintain the 'latest' version.
>
> Fam/Alex/Daniel do you agree with this approach?
I don't see a benefit to that as opposed to just updating git master
to the most recent Fedora stable release every 6 months. In fact it is
more work, because it'll require special action by whomever actually
makes the releases.
Even in git master the use of 'latest' is causing pain. A patch series
of mine passed when i run it in docker because it pulled down fresh
f26 image to my machine, but it failed in patchew because patchew had
latest resolving to a previously cached f25 image. Using explicit
versions all the time will avoid this kind of caching problem.
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 :|
next prev parent reply other threads:[~2018-01-17 14:05 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-01-12 12:49 [Qemu-devel] [PATCH 0/7] docker: update Ubuntu and Fedora images, deprecate old ones Philippe Mathieu-Daudé
2018-01-12 12:49 ` [Qemu-devel] [PATCH 1/7] docker: sort images list displayed by 'make docker' Philippe Mathieu-Daudé
2018-01-12 12:49 ` [Qemu-devel] [PATCH 2/7] docker: do not display deprecated images in 'make docker' help Philippe Mathieu-Daudé
2018-01-17 2:10 ` Fam Zheng
2018-01-17 13:38 ` Philippe Mathieu-Daudé
2018-01-12 12:49 ` [Qemu-devel] [PATCH 3/7] docker: add the Ubuntu Trusty base image Philippe Mathieu-Daudé
2018-01-17 2:12 ` Fam Zheng
2018-01-17 13:42 ` Philippe Mathieu-Daudé
2018-01-12 12:49 ` [Qemu-devel] [PATCH 4/7] docker: do not use Trusty APT source in Xenial Philippe Mathieu-Daudé
2018-01-17 2:13 ` Fam Zheng
2018-01-12 12:49 ` [Qemu-devel] [PATCH 5/7] docker: warn users to use ubuntu16.04 for the LTS Xenial image Philippe Mathieu-Daudé
2018-01-12 12:49 ` [Qemu-devel] [PATCH 6/7] docker: add the Fedora 27 base image Philippe Mathieu-Daudé
2018-01-12 12:50 ` [Qemu-devel] [PATCH 7/7] docker: warn users to use newer fedora25/fedora27 " Philippe Mathieu-Daudé
2018-01-12 13:10 ` [Qemu-devel] [PATCH 0/7] docker: update Ubuntu and Fedora images, deprecate old ones no-reply
2018-01-15 14:01 ` Philippe Mathieu-Daudé
2018-01-17 2:26 ` Fam Zheng
2018-01-17 10:25 ` Daniel P. Berrange
2018-01-17 13:58 ` Philippe Mathieu-Daudé
2018-01-17 14:04 ` Daniel P. Berrange [this message]
2018-01-17 14:12 ` Philippe Mathieu-Daudé
2018-01-17 13:47 ` Philippe Mathieu-Daudé
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=20180117140441.GO19227@redhat.com \
--to=berrange@redhat.com \
--cc=alex.bennee@linaro.org \
--cc=f4bug@amsat.org \
--cc=fzheng@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
/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.