[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 08/10] ci, docker: update CentOS and OpenSUSE Python to non-E
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH 08/10] ci, docker: update CentOS and OpenSUSE Python to non-EOL versions |
Date: |
Wed, 22 Feb 2023 15:14:42 +0000 |
User-agent: |
Mutt/2.2.9 (2022-11-12) |
On Wed, Feb 22, 2023 at 03:37:50PM +0100, Paolo Bonzini wrote:
> Python 3.6 is at end-of-life. libvirt-ci now supports overrides for
> targets and package mappings, therefore QEMU can use newer versions
> provided by CentOS 8 (Python 3.8) and OpenSUSE 15.3 (Python 3.9).
>
> Packages that the distro does not provide are included in the image
> via PyPI; lcitool knows how to do that whenever the rpm mapping
> is empty.
>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
> docs/devel/testing.rst | 6 ++
> tests/docker/dockerfiles/centos8.docker | 22 +++---
> tests/docker/dockerfiles/opensuse-leap.docker | 22 +++---
> tests/docker/dockerfiles/ubuntu2004.docker | 2 +-
> tests/lcitool/mappings.yml | 77 +++++++++++++++++++
> tests/lcitool/targets/centos-stream-8.yml | 3 +
> tests/lcitool/targets/opensuse-leap-153.yml | 3 +
> 7 files changed, 113 insertions(+), 22 deletions(-)
> create mode 100644 tests/lcitool/mappings.yml
> create mode 100644 tests/lcitool/targets/centos-stream-8.yml
> create mode 100644 tests/lcitool/targets/opensuse-leap-153.yml
>
> diff --git a/docs/devel/testing.rst b/docs/devel/testing.rst
> index 648b7aa09137..67798dc63def 100644
> --- a/docs/devel/testing.rst
> +++ b/docs/devel/testing.rst
> @@ -481,6 +481,12 @@ the mapping to the ``libvirt-ci`` project as well:
> contains the ``mappings.yml`` update. Then add the prerequisite and
> run ``make lcitool-refresh``.
>
> +For enterprise distros that default to old, end-of-life versions of the
> +Python runtime, QEMU uses a separate set of mappings that work with more
> +recent versions. These can be found in ``tests/lcitool/mappings.yml``.
> +These should not be a problem unless the dependencies you are adding
> +is a Python library.
I'd suggest changing this last sentence to
It should not be required to modify this file unless the new
dependency being added is a Python library.
Either way,
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
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 :|
- [PATCH 05/10] configure: Look for auxiliary Python installations, (continued)
- [PATCH 05/10] configure: Look for auxiliary Python installations, Paolo Bonzini, 2023/02/22
- [PATCH 02/10] python: drop pipenv, Paolo Bonzini, 2023/02/22
- [PATCH 03/10] meson: prefer 'sphinx-build' to 'sphinx-build-3', Paolo Bonzini, 2023/02/22
- [PATCH 04/10] configure: protect against escaping venv when running Meson, Paolo Bonzini, 2023/02/22
- [PATCH 08/10] ci, docker: update CentOS and OpenSUSE Python to non-EOL versions, Paolo Bonzini, 2023/02/22
- Re: [PATCH 08/10] ci, docker: update CentOS and OpenSUSE Python to non-EOL versions,
Daniel P . Berrangé <=
- [PATCH 07/10] docs/devel: update and clarify lcitool instructions, Paolo Bonzini, 2023/02/22
- [PATCH 06/10] lcitool: update submodule, Paolo Bonzini, 2023/02/22
- [PATCH 09/10] Python: Drop support for Python 3.6, Paolo Bonzini, 2023/02/22
- [PATCH 10/10] configure: Add courtesy hint to Python version failure message, Paolo Bonzini, 2023/02/22