[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 6/6] python/machine: remove unused sock_dir argument
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH v2 6/6] python/machine: remove unused sock_dir argument |
Date: |
Tue, 25 Jul 2023 19:36:35 +0100 |
User-agent: |
Mutt/2.2.9 (2022-11-12) |
On Tue, Jul 25, 2023 at 02:33:36PM -0400, John Snow wrote:
> On Tue, Jul 25, 2023 at 2:26 PM Daniel P. Berrangé <berrange@redhat.com>
> wrote:
> >
> > On Tue, Jul 25, 2023 at 02:03:37PM -0400, John Snow wrote:
> > > By using a socketpair for all of the sockets managed by the VM class and
> > > its extensions, we don't need the sock_dir argument anymore, so remove
> > > it.
> > >
> > > We only added this argument so that we could specify a second, shorter
> > > temporary directory for cases where the temp/log dirs were "too long" as
> > > a socket name on macOS. We don't need it for this class now. In one
> > > case, avocado testing takes over responsibility for creating an
> > > appropriate sockdir.
> > >
> > > Signed-off-by: John Snow <jsnow@redhat.com>
> > > ---
> > > python/qemu/machine/machine.py | 18 ------------------
> > > python/qemu/machine/qtest.py | 5 +----
> > > tests/avocado/acpi-bits.py | 5 +----
> > > tests/avocado/avocado_qemu/__init__.py | 2 +-
> > > tests/avocado/machine_aspeed.py | 5 ++++-
> > > tests/qemu-iotests/iotests.py | 2 +-
> > > tests/qemu-iotests/tests/copy-before-write | 3 +--
> > > 7 files changed, 9 insertions(+), 31 deletions(-)
> >
> > Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
> >
>
> Thanks!
>
> I don't know if we want this for *this* cycle or not, it's "only
> testing code" and it should hopefully be harmless. If it makes the
> tests more reliable, it might be worth it. I don't have strong
> feelings one way or the other, we've lived without it for so long
> as-is.
>
> I'll see what Peter says.
Although it does affect end users, the biggest impact is our own
CI. Once the release is out it would help CI on stable trees,
but the big win is CI on master.
I'd verge towards skipping this during freeze and applying to
master after release. Then propose cherry-pick to stable once
it has had some soak time in our real 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 :|
- Re: [PATCH v2 4/6] python/machine: use socketpair() for console connections, (continued)
[PATCH v2 5/6] python/machine: use socketpair() for qtest connection, John Snow, 2023/07/25
[PATCH v2 6/6] python/machine: remove unused sock_dir argument, John Snow, 2023/07/25
Re: [PATCH v2 0/6] python/machine: use socketpair() for console socket, Peter Maydell, 2023/07/27