[PATCH for-9.1 0/9] Switch to glib URI parsing code

Thomas Huth posted 9 patches 1 month ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20240328140607.2433889-1-thuth@redhat.com
Maintainers: "Alex Bennée" <alex.bennee@linaro.org>, "Philippe Mathieu-Daudé" <philmd@linaro.org>, Thomas Huth <thuth@redhat.com>, Wainer dos Santos Moschetta <wainersm@redhat.com>, Beraldo Leal <bleal@redhat.com>, Kevin Wolf <kwolf@redhat.com>, Hanna Reitz <hreitz@redhat.com>, Eric Blake <eblake@redhat.com>, Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>, Peter Lieven <pl@kamp.de>, "Richard W.M. Jones" <rjones@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, "Marc-André Lureau" <marcandre.lureau@redhat.com>, "Daniel P. Berrangé" <berrange@redhat.com>, Michael Roth <michael.roth@amd.com>, Konstantin Kostiuk <kkostiuk@redhat.com>
meson.build                                   |   16 +-
include/glib-compat.h                         |   27 +-
include/qemu/uri.h                            |   99 --
block/gluster.c                               |   63 +-
block/nbd.c                                   |   66 +-
block/nfs.c                                   |  102 +-
block/ssh.c                                   |   69 +-
qga/commands-posix-ssh.c                      |    4 +-
util/uri.c                                    | 1466 -----------------
.gitlab-ci.d/buildtest.yml                    |   16 +-
.gitlab-ci.d/container-core.yml               |    4 +-
.../{centos8.docker => centos9.docker}        |   34 +-
tests/docker/dockerfiles/ubuntu2004.docker    |  157 --
tests/lcitool/libvirt-ci                      |    2 +-
tests/lcitool/mappings.yml                    |   20 -
tests/lcitool/refresh                         |    3 +-
tests/vm/centos                               |    4 +-
util/meson.build                              |    2 +-
18 files changed, 200 insertions(+), 1954 deletions(-)
delete mode 100644 include/qemu/uri.h
delete mode 100644 util/uri.c
rename tests/docker/dockerfiles/{centos8.docker => centos9.docker} (82%)
delete mode 100644 tests/docker/dockerfiles/ubuntu2004.docker
[PATCH for-9.1 0/9] Switch to glib URI parsing code
Posted by Thomas Huth 1 month ago
In the QEMU 9.1 development cycle, we can drop the support for
Ubuntu 20.04 and CentOS 8 since the following major versions of
these distributions are available since 2 years already.

This allows us to bump the minimum version of glib to 2.66 which
comes with a nice set of URI parsing functions. By switching to
these parsing functions, we can finally drop our own URI parsing
code in util/uri.c.

NB: We also need to update some of the custom runners in our CI
environment first (since they still use Ubuntu 20.04).

Thomas Huth (9):
  tests: Remove Ubuntu 20.04 container
  tests/lcitool/libvirt-ci: Update to the latest master branch
  tests: Update our CI to use CentOS Stream 9 instead of 8
  Bump minimum glib version to v2.66
  block/gluster: Use URI parsing code from glib
  block/nbd: Use URI parsing code from glib
  block/nfs: Use URI parsing code from glib
  block/ssh: Use URI parsing code from glib
  util/uri: Remove the old URI parsing code

 meson.build                                   |   16 +-
 include/glib-compat.h                         |   27 +-
 include/qemu/uri.h                            |   99 --
 block/gluster.c                               |   63 +-
 block/nbd.c                                   |   66 +-
 block/nfs.c                                   |  102 +-
 block/ssh.c                                   |   69 +-
 qga/commands-posix-ssh.c                      |    4 +-
 util/uri.c                                    | 1466 -----------------
 .gitlab-ci.d/buildtest.yml                    |   16 +-
 .gitlab-ci.d/container-core.yml               |    4 +-
 .../{centos8.docker => centos9.docker}        |   34 +-
 tests/docker/dockerfiles/ubuntu2004.docker    |  157 --
 tests/lcitool/libvirt-ci                      |    2 +-
 tests/lcitool/mappings.yml                    |   20 -
 tests/lcitool/refresh                         |    3 +-
 tests/vm/centos                               |    4 +-
 util/meson.build                              |    2 +-
 18 files changed, 200 insertions(+), 1954 deletions(-)
 delete mode 100644 include/qemu/uri.h
 delete mode 100644 util/uri.c
 rename tests/docker/dockerfiles/{centos8.docker => centos9.docker} (82%)
 delete mode 100644 tests/docker/dockerfiles/ubuntu2004.docker

-- 
2.44.0
MAINTAINERS tweak [was: [PATCH for-9.1 0/9] Switch to glib URI parsing code]
Posted by Eric Blake 2 weeks, 1 day ago
[Trying Peter Lieven's alternate address...]

On Thu, Mar 28, 2024 at 03:05:57PM +0100, Thomas Huth wrote:
> In the QEMU 9.1 development cycle, we can drop the support for
> Ubuntu 20.04 and CentOS 8 since the following major versions of
> these distributions are available since 2 years already.

Every time I've replied to any message in this thread, I've gotten a
response:

| +Your message to pl@kamp.de couldn't be delivered.
| kamp.de couldn't confirm that your message was sent from a trusted location.
| eblake  Office 365      pl
| Action Required                 Recipient
| SPF validation error
| 
| How to Fix It
| Your organization's email admin will have to diagnose and fix your domain's email settings. Please forward this message to your
| +email admin.
| 
| ________________________________
| 
| More Info for Email Admins
| Status code: 550 5.7.23
| 
| This error occurs when Sender Policy Framework (SPF) validation for the sender's domain fails. If you're the sender's email
| +admin, make sure the SPF records for your domain at your domain registrar are set up correctly. Office 365 supports only one
| +SPF record (a TXT record that defines SPF) for your domain. Include the following domain name: spf.protection.outlook.com. If
| +you have a hybrid configuration (some mailboxes in the cloud, and some mailboxes on premises) or if you're an Exchange Online
| +Protection standalone customer, add the outbound IP address of your on-premises servers to the TXT record.

Red Hat IT says that it is unlikely to be Red Hat's SPF settings, and
suspects that it is instead something caused by whatever Peter is
using to bounce mail from his alias Peter Lieven <pl@kamp.de> to his
Office 365 account.  As I appear to be unable to contact Peter (even
my use of direct email, bypassing the list, and using a personal
account instead of my Red Hat email) about this issue, I'm wondering
if Peter is still an active contributor to the project.

But while typing this email, to see if RBD, iSCSI, and NFS need a new
entry in MAINTAINERS, I did a search through the list archives, where
the last email I found from Peter was
https://lists.gnu.org/archive/html/qemu-devel/2023-01/msg00574.html,
which asked to update MAINTAINERS to his new address, and that has not
made it in so far...

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.
Virtualization:  qemu.org | libguestfs.org