All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Huth <thuth@redhat.com>
To: "Stefan Hajnoczi" <stefanha@gmail.com>,
	"Daniel P. Berrangé" <berrange@redhat.com>,
	"John Snow" <jsnow@redhat.com>,
	"Stefan Hajnoczi" <stefanha@redhat.com>
Cc: qemu-devel@nongnu.org, "Alex Bennée" <alex.bennee@linaro.org>,
	"Peter Maydell" <peter.maydell@linaro.org>,
	"Richard Henderson" <richard.henderson@linaro.org>
Subject: Re: [PATCH] tests/docker/dockerfiles: Use Fedora 41 for the python container file
Date: Tue, 22 Apr 2025 15:36:47 +0200	[thread overview]
Message-ID: <b48a034e-dbb4-48d6-a889-d0212cb085e1@redhat.com> (raw)
In-Reply-To: <CAJSP0QUAUss1aGSYbcyprzuxQv4bEQGc_Owv557hK-Te0PKSaA@mail.gmail.com>

On 22/04/2025 15.22, Stefan Hajnoczi wrote:
> On Tue, Apr 22, 2025 at 9:09 AM Daniel P. Berrangé <berrange@redhat.com> wrote:
>>
>> On Tue, Apr 22, 2025 at 02:56:26PM +0200, Thomas Huth wrote:
>>> From: Thomas Huth <thuth@redhat.com>
>>>
>>> Using "fedora:latest" now fails with Fedora 42:
>>>
>>>    Failed to resolve the transaction:
>>>    No match for argument: python3.8
>>>
>>> Switch back to Fedora 41 for the time being, to be able to still
>>> test with Python 3.8.
>>
>> This page details why it is was removed:
>>
>>    https://fedoraproject.org/wiki/Changes/RetirePython3.8
>>
>> Based on the explanation there, it seems doubtful that any of our
>> targetted build platforms have python 3.8. Thus I'd question whether
>> we should test 3.8 at all.
>>
>> IOW, rather than pin to fedora:41, it seems better to drop
>> python3.8 from the dockerfile.
> 
> I'm in favor of pinning so that the test is reproducible. A test that
> passed on a given git commit should pass again when it is re-run.

Yes, we also don't use ":latest" in any of the other tests, so pinning to a 
specific version also sounds like the better idea to me.

Anyway, if we really want to drop Python 3.8 (which we likely should do now 
indeed), we also have to update "configure" and 
docs/about/build-platforms.rst and likely some other spots... John?

  Thomas



      reply	other threads:[~2025-04-22 13:37 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-04-22 12:56 [PATCH] tests/docker/dockerfiles: Use Fedora 41 for the python container file Thomas Huth
2025-04-22 13:08 ` Daniel P. Berrangé
2025-04-22 13:22   ` Stefan Hajnoczi
2025-04-22 13:36     ` Thomas Huth [this message]

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=b48a034e-dbb4-48d6-a889-d0212cb085e1@redhat.com \
    --to=thuth@redhat.com \
    --cc=alex.bennee@linaro.org \
    --cc=berrange@redhat.com \
    --cc=jsnow@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=richard.henderson@linaro.org \
    --cc=stefanha@gmail.com \
    --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 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.