Re: [PATCH v2 6/6] python/machine: remove unused sock_dir argument

2023-07-25 Thread Daniel P . Berrangé
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é > 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

Re: [PATCH v2 6/6] python/machine: remove unused sock_dir argument

2023-07-25 Thread John Snow
On Tue, Jul 25, 2023 at 2:26 PM Daniel P. Berrangé 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 a

Re: [PATCH v2 6/6] python/machine: remove unused sock_dir argument

2023-07-25 Thread Daniel P . Berrangé
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 > temporar

[PATCH v2 6/6] python/machine: remove unused sock_dir argument

2023-07-25 Thread John Snow
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 sock