[PATCH V8 04/13] spec: Move lockd plugin to a new subpackage

2023-01-11 Thread Jim Fehlig
Introduce the libvirt-daemon-plugin-lockd subpackage to provide the client-side lockd plugin for virtlockd. Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) diff

[PATCH V8 09/13] spec: Remove libvirt-daemon dependency from secondary drivers

2023-01-11 Thread Jim Fehlig
To avoid needlessly installing the monolithic daemon, replace the libvirt-daemon dependency with libvirt-daemon-common in the secondary drivers. The common subpackage contains all the utilities and files needed by the secondary drivers. Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé

[PATCH V8 11/13] spec: Remove libvirt-daemon dependency from hypervisor subpackages

2023-01-11 Thread Jim Fehlig
. Signed-off-by: Jim Fehlig Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 33 + 1 file changed, 33 insertions(+) diff --git a/libvirt.spec.in b/libvirt.spec.in index fb318ad599..06d4b3e9f3 100644 --- a/libvirt.spec.in +++ b/libvirt.spec.in @@ -823,7 +823,15

[PATCH V8 08/13] spec: Add numad dependency to qemu and lxc drivers

2023-01-11 Thread Jim Fehlig
Both drivers use numad via virNumaGetAutoPlacementAdvice. Drop the numad dependency from libvirt-daemon. Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a

[PATCH V8 13/13] NEWS: Mention split of libvirt-daemon subpackage

2023-01-11 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Andrea Bolognani --- NEWS.rst | 6 ++ 1 file changed, 6 insertions(+) diff --git a/NEWS.rst b/NEWS.rst index 768208ed03..af7d1054d4 100644 --- a/NEWS.rst +++ b/NEWS.rst @@ -39,6 +39,12 @@ v9.0.0 (unreleased) With sufficiently new QEMU (v7.1.0

[PATCH V8 02/13] spec: Move virtlogd to a new subpackage libvirt-daemon-log

2023-01-11 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 53 +++-- 1 file changed, 38 insertions(+), 15 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index bb0bfb5f35..19a3c8465c

[PATCH V8 00/13] spec: Decompose the daemon subpackage

2023-01-11 Thread Jim Fehlig
. Changes from V7: * Since the primary and secondary drivers need to be handled a bit differently when dropping the libvirt-daemon dependency, split the work into two patches Jim Fehlig (13): spec: Move virtlockd to a new subpackage libvirt-daemon-lock spec: Move virtlogd to a new

[PATCH V8 06/13] spec: Move common files and dependencies to libvirt-daemon-common

2023-01-11 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 77 ++--- 1 file changed, 48 insertions(+), 29 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index 56cf35ef06..4c2e6e3587 100644 --- a/libvirt.spec.in +++ b

[PATCH V8 03/13] spec: Move virtproxyd to a new subpackage libvirt-daemon-proxy

2023-01-11 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 68 - 1 file changed, 45 insertions(+), 23 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index 19a3c8465c..6072ac2c1c

[PATCH V8 05/13] spec: Rename the libvirt-lock-sanlock subpackage

2023-01-11 Thread Jim Fehlig
emon package. Signed-off-by: Jim Fehlig Reviewed-by: Andrea Bolognani --- docs/kbase/rpm-deployment.rst | 12 ++-- libvirt.spec.in | 11 ++- 2 files changed, 12 insertions(+), 11 deletions(-) diff --git a/docs/kbase/rpm-deployment.rst b/docs/kbase/rpm-deploy

Re: [PATCH V7 09/12] spec: Remove libvirt-daemon dependency from drivers

2023-01-11 Thread Jim Fehlig
On 1/11/23 11:21, Andrea Bolognani wrote: On Wed, Jan 11, 2023 at 05:35:01PM +, Daniel P. Berrangé wrote: On Wed, Jan 11, 2023 at 09:24:09AM -0800, Andrea Bolognani wrote: Based on the fact that virtlogd is opt-out and virtlockd is opt-in, can we leave the Requires=virtlogd.socket relations

Re: [PATCH V7 09/12] spec: Remove libvirt-daemon dependency from drivers

2023-01-10 Thread Jim Fehlig
On 1/9/23 10:42, Jim Fehlig wrote: To avoid needlessly installing the monolithic daemon, replace the libvirt-daemon dependency with libvirt-daemon-common. The common subpackage contains all the utilities (e.g. virt-admin) and files used by other daemons. Signed-off-by: Jim Fehlig Reviewed-by

[PATCH V7 10/12] spec: Remove libvirt-daemon dependency from hypervisor subpackages

2023-01-09 Thread Jim Fehlig
. Signed-off-by: Jim Fehlig Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 33 + 1 file changed, 33 insertions(+) diff --git a/libvirt.spec.in b/libvirt.spec.in index 2cc77a2622..8c610d7797 100644 --- a/libvirt.spec.in +++ b/libvirt.spec.in @@ -815,7 +815,15

[PATCH V7 12/12] NEWS: Mention split of libvirt-daemon subpackage

2023-01-09 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Andrea Bolognani --- NEWS.rst | 6 ++ 1 file changed, 6 insertions(+) diff --git a/NEWS.rst b/NEWS.rst index fecca64344..732fce59da 100644 --- a/NEWS.rst +++ b/NEWS.rst @@ -29,6 +29,12 @@ v9.0.0 (unreleased) With sufficiently new QEMU (v7.1.0

[PATCH V7 09/12] spec: Remove libvirt-daemon dependency from drivers

2023-01-09 Thread Jim Fehlig
To avoid needlessly installing the monolithic daemon, replace the libvirt-daemon dependency with libvirt-daemon-common. The common subpackage contains all the utilities (e.g. virt-admin) and files used by other daemons. Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by

[PATCH V7 11/12] kbase: Update rpm-deployment.rst with new subpackages

2023-01-09 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Andrea Bolognani --- docs/kbase/rpm-deployment.rst | 36 --- 1 file changed, 29 insertions(+), 7 deletions(-) diff --git a/docs/kbase/rpm-deployment.rst b/docs/kbase/rpm-deployment.rst index 7685ee1291..26fe1be8e6 100644

[PATCH V7 08/12] spec: Add numad dependency to qemu and lxc drivers

2023-01-09 Thread Jim Fehlig
Both drivers use numad via virNumaGetAutoPlacementAdvice. Drop the numad dependency from libvirt-daemon. Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a

[PATCH V7 07/12] spec: Add module-init-tools dependency to nodedev and lxc drivers

2023-01-09 Thread Jim Fehlig
Both the nodedev and lxc drivers can load kernel modules. Add a module-init-tools dependency to the drivers and remove the dependency from libvirt-daemon. Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 6 -- 1 file changed, 4

[PATCH V7 06/12] spec: Move common files and dependencies to libvirt-daemon-common

2023-01-09 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 77 ++--- 1 file changed, 48 insertions(+), 29 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index dd1b6a74e9..12151cb90f 100644 --- a/libvirt.spec.in +++ b

[PATCH V7 05/12] spec: Rename the libvirt-lock-sanlock subpackage

2023-01-09 Thread Jim Fehlig
emon package. Signed-off-by: Jim Fehlig Reviewed-by: Andrea Bolognani --- docs/kbase/rpm-deployment.rst | 12 ++-- libvirt.spec.in | 11 ++- 2 files changed, 12 insertions(+), 11 deletions(-) diff --git a/docs/kbase/rpm-deployment.rst b/docs/kbase/rpm-deploy

[PATCH V7 04/12] spec: Move lockd plugin to a new subpackage

2023-01-09 Thread Jim Fehlig
Introduce the libvirt-daemon-plugin-lockd subpackage to provide the client-side lockd plugin for virtlockd. Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) diff

[PATCH V7 03/12] spec: Move virtproxyd to a new subpackage libvirt-daemon-proxy

2023-01-09 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 68 - 1 file changed, 45 insertions(+), 23 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index d303cac9df..a2bc591333

[PATCH V7 02/12] spec: Move virtlogd to a new subpackage libvirt-daemon-log

2023-01-09 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 53 +++-- 1 file changed, 38 insertions(+), 15 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index 33398b2877..d303cac9df

[PATCH V7 01/12] spec: Move virtlockd to a new subpackage libvirt-daemon-lock

2023-01-09 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 61 +++-- 1 file changed, 44 insertions(+), 17 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index 6b8acf252e..33398b2877

[PATCH V7 00/12] spec: Decompose the daemon subpackage

2023-01-09 Thread Jim Fehlig
. Changes from V6: * Revert to including virtlockd and lockd plugin in the hypervisor subpackages * Add NEWS entry Jim Fehlig (12): spec: Move virtlockd to a new subpackage libvirt-daemon-lock spec: Move virtlogd to a new subpackage libvirt-daemon-log spec: Move virtproxyd to a new

Re: [PATCH V5 10/11] spec: Remove libvirt-daemon dependency from hypervisor subpackages

2023-01-09 Thread Jim Fehlig
On 1/9/23 07:54, Andrea Bolognani wrote: On Mon, Jan 09, 2023 at 02:25:44PM +, Daniel P. Berrangé wrote: I still th8ink that dropping the dep is wrong, becasue that is defeating the purpose of 'libvirt-daemon-$HYPERVISOR' packages aim todo: "installs every possible libvirt feature for th

[PATCH] NEWS: Mention split of libvirt-daemon subpackage

2023-01-05 Thread Jim Fehlig
Signed-off-by: Jim Fehlig --- NEWS.rst | 6 ++ 1 file changed, 6 insertions(+) diff --git a/NEWS.rst b/NEWS.rst index 39f508a6ce..a799ee4529 100644 --- a/NEWS.rst +++ b/NEWS.rst @@ -23,6 +23,12 @@ v9.0.0 (unreleased) With sufficiently new QEMU (v7.1.0) screenshots change format from

Re: [PATCH V6 10/11] spec: Remove libvirt-daemon dependency from hypervisor subpackages

2023-01-05 Thread Jim Fehlig
On 1/4/23 02:14, Andrea Bolognani wrote: On Tue, Jan 03, 2023 at 02:22:14PM -0700, Jim Fehlig wrote: Remove the libvirt-daemon dependency from the various libvirt-daemon- subpackages, replacing it with a set of the new sub subpackages providing the same functionality. Maybe we could add

Re: [PATCH V5 10/11] spec: Remove libvirt-daemon dependency from hypervisor subpackages

2023-01-03 Thread Jim Fehlig
On 1/3/23 12:02, Andrea Bolognani wrote: On Tue, Jan 03, 2023 at 10:57:21AM -0700, Jim Fehlig wrote: On 1/3/23 10:27, Jim Fehlig wrote: On 1/2/23 08:26, Andrea Bolognani wrote: This version is fine, but as explained elsewhere I think it would be better to have Requires: libvirt-daemon

[PATCH V6 11/11] kbase: Update rpm-deployment.rst with new subpackages

2023-01-03 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Andrea Bolognani --- docs/kbase/rpm-deployment.rst | 36 --- 1 file changed, 29 insertions(+), 7 deletions(-) diff --git a/docs/kbase/rpm-deployment.rst b/docs/kbase/rpm-deployment.rst index 7685ee1291..26fe1be8e6 100644

[PATCH V6 10/11] spec: Remove libvirt-daemon dependency from hypervisor subpackages

2023-01-03 Thread Jim Fehlig
Remove the libvirt-daemon dependency from the various libvirt-daemon- subpackages, replacing it with a set of the new sub subpackages providing the same functionality. Signed-off-by: Jim Fehlig --- libvirt.spec.in | 27 +++ 1 file changed, 27 insertions(+) diff --git a

[PATCH V6 09/11] spec: Remove libvirt-daemon dependency from drivers

2023-01-03 Thread Jim Fehlig
To avoid needlessly installing the monolithic daemon, replace the libvirt-daemon dependency with libvirt-daemon-common. The common subpackage contains all the utilities (e.g. virt-admin) and files used by other daemons. Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by

[PATCH V6 08/11] spec: Add numad dependency to qemu and lxc drivers

2023-01-03 Thread Jim Fehlig
Both drivers use numad via virNumaGetAutoPlacementAdvice. Drop the numad dependency from libvirt-daemon. Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a

[PATCH V6 07/11] spec: Add module-init-tools dependency to nodedev and lxc drivers

2023-01-03 Thread Jim Fehlig
Both the nodedev and lxc drivers can load kernel modules. Add a module-init-tools dependency to the drivers and remove the dependency from libvirt-daemon. Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 6 -- 1 file changed, 4

[PATCH V6 06/11] spec: Move common files and dependencies to libvirt-daemon-common

2023-01-03 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 77 ++--- 1 file changed, 48 insertions(+), 29 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index dd1b6a74e9..12151cb90f 100644 --- a/libvirt.spec.in +++ b

[PATCH V6 05/11] spec: Rename the libvirt-lock-sanlock subpackage

2023-01-03 Thread Jim Fehlig
emon package. Signed-off-by: Jim Fehlig Reviewed-by: Andrea Bolognani --- docs/kbase/rpm-deployment.rst | 12 ++-- libvirt.spec.in | 11 ++- 2 files changed, 12 insertions(+), 11 deletions(-) diff --git a/docs/kbase/rpm-deployment.rst b/docs/kbase/rpm-deploy

[PATCH V6 04/11] spec: Move lockd plugin to a new subpackage

2023-01-03 Thread Jim Fehlig
Introduce the libvirt-daemon-plugin-lockd subpackage to provide the client-side lockd plugin for virtlockd. Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) diff

[PATCH V6 03/11] spec: Move virtproxyd to a new subpackage libvirt-daemon-proxy

2023-01-03 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 68 - 1 file changed, 45 insertions(+), 23 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index d303cac9df..a2bc591333

[PATCH V6 02/11] spec: Move virtlogd to a new subpackage libvirt-daemon-log

2023-01-03 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 53 +++-- 1 file changed, 38 insertions(+), 15 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index 33398b2877..d303cac9df

[PATCH V6 01/11] spec: Move virtlockd to a new subpackage libvirt-daemon-lock

2023-01-03 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 61 +++-- 1 file changed, 44 insertions(+), 17 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index 6b8acf252e..33398b2877

[PATCH V6 00/11] spec: Decompose the daemon subpackage

2023-01-03 Thread Jim Fehlig
libvirtd. Changes from V5: * Remove lockd dependency from hypervisor packages * Add virtproxyd as a weak dependency to the hypervisor packages Jim Fehlig (11): spec: Move virtlockd to a new subpackage libvirt-daemon-lock spec: Move virtlogd to a new subpackage libvirt-daemon-log spec: Move

Re: [PATCH V5 10/11] spec: Remove libvirt-daemon dependency from hypervisor subpackages

2023-01-03 Thread Jim Fehlig
On 1/3/23 10:27, Jim Fehlig wrote: On 1/2/23 08:26, Andrea Bolognani wrote: On Fri, Dec 23, 2022 at 11:56:03AM -0700, Jim Fehlig wrote:   %package daemon-qemu   Summary: Server side daemon & driver required to run QEMU guests +    %if %{with_modular_daemons} +Requires: libvirt-daemon-co

Re: [PATCH V5 10/11] spec: Remove libvirt-daemon dependency from hypervisor subpackages

2023-01-03 Thread Jim Fehlig
On 1/2/23 08:26, Andrea Bolognani wrote: On Fri, Dec 23, 2022 at 11:56:03AM -0700, Jim Fehlig wrote: %package daemon-qemu Summary: Server side daemon & driver required to run QEMU guests +%if %{with_modular_daemons} +Requires: libvirt-daemon-common = %{version}-%{release} +Requ

Re: [PATCH V4 06/11] spec: Move common files and dependencies to libvirt-daemon-common

2023-01-03 Thread Jim Fehlig
On 1/2/23 07:53, Andrea Bolognani wrote: On Fri, Dec 23, 2022 at 10:57:34AM -0700, Jim Fehlig wrote: On 12/23/22 03:52, Andrea Bolognani wrote: One more thing. After your changes, libvirt-daemon still has # netcat is needed on the server side so that clients that have # libvirt

[PATCH V5 11/11] kbase: Update rpm-deployment.rst with new subpackages

2022-12-23 Thread Jim Fehlig
Signed-off-by: Jim Fehlig --- docs/kbase/rpm-deployment.rst | 39 --- 1 file changed, 32 insertions(+), 7 deletions(-) diff --git a/docs/kbase/rpm-deployment.rst b/docs/kbase/rpm-deployment.rst index 7685ee1291..2f50d04c80 100644 --- a/docs/kbase/rpm

[PATCH V5 10/11] spec: Remove libvirt-daemon dependency from hypervisor subpackages

2022-12-23 Thread Jim Fehlig
Remove the libvirt-daemon dependency from the various libvirt-daemon- subpackages, replacing it with a set of the new sub subpackages providing the same functionality. Signed-off-by: Jim Fehlig --- libvirt.spec.in | 28 1 file changed, 28 insertions(+) diff --git a

[PATCH V5 09/11] spec: Remove libvirt-daemon dependency from drivers

2022-12-23 Thread Jim Fehlig
To avoid needlessly installing the monolithic daemon, replace the libvirt-daemon dependency with libvirt-daemon-common. The common subpackage contains all the utilities (e.g. virt-admin) and files used by other daemons. Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé

[PATCH V5 08/11] spec: Add numad dependency to qemu and lxc drivers

2022-12-23 Thread Jim Fehlig
Both drivers use numad via virNumaGetAutoPlacementAdvice. Drop the numad dependency from libvirt-daemon. Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé --- libvirt.spec.in | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in

[PATCH V5 07/11] spec: Add module-init-tools dependency to nodedev and lxc drivers

2022-12-23 Thread Jim Fehlig
Both the nodedev and lxc drivers can load kernel modules. Add a module-init-tools dependency to the drivers and remove the dependency from libvirt-daemon. Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé --- libvirt.spec.in | 6 -- 1 file changed, 4 insertions(+), 2 deletions

[PATCH V5 06/11] spec: Move common files and dependencies to libvirt-daemon-common

2022-12-23 Thread Jim Fehlig
Signed-off-by: Jim Fehlig --- libvirt.spec.in | 77 ++--- 1 file changed, 48 insertions(+), 29 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index dd1b6a74e9..12151cb90f 100644 --- a/libvirt.spec.in +++ b/libvirt.spec.in @@ -429,17 +429,30

[PATCH V5 05/11] spec: Rename the libvirt-lock-sanlock subpackage

2022-12-23 Thread Jim Fehlig
emon package. Signed-off-by: Jim Fehlig Reviewed-by: Andrea Bolognani --- docs/kbase/rpm-deployment.rst | 12 ++-- libvirt.spec.in | 11 ++- 2 files changed, 12 insertions(+), 11 deletions(-) diff --git a/docs/kbase/rpm-deployment.rst b/docs/kbase/rpm-deploy

[PATCH V5 04/11] spec: Move lockd plugin to a new subpackage

2022-12-23 Thread Jim Fehlig
Introduce the libvirt-daemon-plugin-lockd subpackage to provide the client-side lockd plugin for virtlockd. Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) diff

[PATCH V5 02/11] spec: Move virtlogd to a new subpackage libvirt-daemon-log

2022-12-23 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 53 +++-- 1 file changed, 38 insertions(+), 15 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index 33398b2877..d303cac9df

[PATCH V5 00/11] spec: Decompose the daemon subpackage

2022-12-23 Thread Jim Fehlig
libvirtd. Changes from V4: - Address more review comments from Andrea Jim Fehlig (11): spec: Move virtlockd to a new subpackage libvirt-daemon-lock spec: Move virtlogd to a new subpackage libvirt-daemon-log spec: Move virtproxyd to a new subpackage libvirt-daemon-proxy spec: Move lockd plugin to

[PATCH V5 03/11] spec: Move virtproxyd to a new subpackage libvirt-daemon-proxy

2022-12-23 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 68 - 1 file changed, 45 insertions(+), 23 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index d303cac9df..a2bc591333

[PATCH V5 01/11] spec: Move virtlockd to a new subpackage libvirt-daemon-lock

2022-12-23 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 61 +++-- 1 file changed, 44 insertions(+), 17 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index 6b8acf252e..33398b2877

Re: [PATCH V4 07/11] spec: Add module-init-tools dependency to nodedev and lxc drivers

2022-12-23 Thread Jim Fehlig
On 12/23/22 03:42, Andrea Bolognani wrote: On Thu, Dec 22, 2022 at 11:03:38AM -0700, Jim Fehlig wrote: @@ -571,6 +571,8 @@ Requires: libvirt-libs = %{version}-%{release} Requires: systemd >= 185 # For managing persistent mediated devices Requires: mdevctl +# for modprobe of pci devi

Re: [PATCH V4 06/11] spec: Move common files and dependencies to libvirt-daemon-common

2022-12-23 Thread Jim Fehlig
On 12/23/22 03:52, Andrea Bolognani wrote: On Fri, Dec 23, 2022 at 04:30:32AM -0500, Andrea Bolognani wrote: On Thu, Dec 22, 2022 at 11:03:37AM -0700, Jim Fehlig wrote: -Requires: polkit >= 0.112 %if %{with_dmidecode} # For virConnectGetSysinfo Requires: dmidecode %endif #

[PATCH V4 11/11] kbase: Update rpm-deployment.rst with new subpackages

2022-12-22 Thread Jim Fehlig
Signed-off-by: Jim Fehlig --- docs/kbase/rpm-deployment.rst | 39 --- 1 file changed, 32 insertions(+), 7 deletions(-) diff --git a/docs/kbase/rpm-deployment.rst b/docs/kbase/rpm-deployment.rst index 7685ee1291..2f50d04c80 100644 --- a/docs/kbase/rpm

[PATCH V4 10/11] spec: Remove libvirt-daemon dependency from hypervisor subpackages

2022-12-22 Thread Jim Fehlig
Remove the libvirt-daemon dependency from the various libvirt-daemon- subpackages, replacing it with a set of the new sub subpackages providing the same functionality. Signed-off-by: Jim Fehlig --- libvirt.spec.in | 28 1 file changed, 28 insertions(+) diff --git a

[PATCH V4 09/11] spec: Remove libvirt-daemon dependency from drivers

2022-12-22 Thread Jim Fehlig
To avoid needlessly installing the monolithic daemon, replace the libvirt-daemon dependency with libvirt-daemon-common. The common subpackage contains all the utilities (e.g. virt-admin) and files used by other daemons. Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé

[PATCH V4 07/11] spec: Add module-init-tools dependency to nodedev and lxc drivers

2022-12-22 Thread Jim Fehlig
Both the nodedev and lxc drivers can load kernel modules. Add a module-init-tools dependency to the drivers. Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé --- libvirt.spec.in | 4 1 file changed, 4 insertions(+) diff --git a/libvirt.spec.in b/libvirt.spec.in index cf197beaca

[PATCH V4 08/11] spec: Add numad dependency to qemu and lxc drivers

2022-12-22 Thread Jim Fehlig
Both drivers use numad via virNumaGetAutoPlacementAdvice. Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé --- libvirt.spec.in | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libvirt.spec.in b/libvirt.spec.in index e69005bb42..c5adb1a2ba 100644 --- a/libvirt.spec.in +++ b

[PATCH V4 06/11] spec: Move common files and dependencies to libvirt-daemon-common

2022-12-22 Thread Jim Fehlig
Signed-off-by: Jim Fehlig --- libvirt.spec.in | 93 + 1 file changed, 56 insertions(+), 37 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index 98497f639f..cf197beaca 100644 --- a/libvirt.spec.in +++ b/libvirt.spec.in @@ -429,15 +429,11

[PATCH V4 05/11] spec: Rename the libvirt-lock-sanlock subpackage

2022-12-22 Thread Jim Fehlig
emon package. Signed-off-by: Jim Fehlig --- docs/kbase/rpm-deployment.rst | 12 ++-- libvirt.spec.in | 10 ++ 2 files changed, 12 insertions(+), 10 deletions(-) diff --git a/docs/kbase/rpm-deployment.rst b/docs/kbase/rpm-deployment.rst index 31805c2280..7685ee1291 10

[PATCH V4 03/11] spec: Move virtproxyd to a new subpackage libvirt-daemon-proxy

2022-12-22 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 56 ++--- 1 file changed, 39 insertions(+), 17 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index d303cac9df..ee12720b72

[PATCH V4 04/11] spec: Move lockd plugin to a new subpackage

2022-12-22 Thread Jim Fehlig
Introduce the libvirt-daemon-plugin-lockd subpackage to provide the client-side lockd plugin for virtlockd. Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé --- libvirt.spec.in | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/libvirt.spec.in b

[PATCH V4 02/11] spec: Move virtlogd to a new subpackage libvirt-daemon-log

2022-12-22 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 53 +++-- 1 file changed, 38 insertions(+), 15 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index 33398b2877..d303cac9df

[PATCH V4 01/11] spec: Move virtlockd to a new subpackage libvirt-daemon-lock

2022-12-22 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 61 +++-- 1 file changed, 44 insertions(+), 17 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index 6b8acf252e..33398b2877

[PATCH V4 00/11] spec: Decompose the daemon subpackage

2022-12-22 Thread Jim Fehlig
libvirtd. Changes from V3: - Address review comments, some of which I overlooked from V2 review Jim Fehlig (11): spec: Move virtlockd to a new subpackage libvirt-daemon-lock spec: Move virtlogd to a new subpackage libvirt-daemon-log spec: Move virtproxyd to a new subpackage libvirt-daemon-proxy

Re: [PATCH] libxl: adjust 'ich6' sound card name

2022-12-20 Thread Jim Fehlig
On 12/15/22 18:42, Marek Marczykowski-Górecki wrote: Xen 4.17 has strict parsing of 'soundhw' option that allows only specific values (instead of passing through any value directly to qemu's -soundhw option, it uses -device now). For 'intel-hda' audio device, it requires "hda" string. "hda" works

[PATCH V3 11/11] kbase: Update rpm-deployment.rst with new subpackages

2022-12-20 Thread Jim Fehlig
Signed-off-by: Jim Fehlig --- docs/kbase/rpm-deployment.rst | 38 --- 1 file changed, 31 insertions(+), 7 deletions(-) diff --git a/docs/kbase/rpm-deployment.rst b/docs/kbase/rpm-deployment.rst index 7685ee1291..7f38052ceb 100644 --- a/docs/kbase/rpm

[PATCH V3 10/11] spec: Remove libvirt-daemon dependency from hypervisor subpackages

2022-12-20 Thread Jim Fehlig
Remove the libvirt-daemon dependency from the various libvirt-daemon- subpackages, replacing it with a set of the new sub subpackages providing the same functionality. Signed-off-by: Jim Fehlig --- libvirt.spec.in | 22 +++--- 1 file changed, 19 insertions(+), 3 deletions

[PATCH V3 09/11] spec: Remove libvirt-daemon dependency from drivers

2022-12-20 Thread Jim Fehlig
To avoid needlessly installing the monolithic daemon, replace the libvirt-daemon dependency with libvirt-daemon-common. The common subpackage contains all the utilities (e.g. virt-admin) and files used by other daemons. Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé

[PATCH V3 08/11] spec: Add numad dependency to qemu and lxc drivers

2022-12-20 Thread Jim Fehlig
Both drivers use numad via virNumaGetAutoPlacementAdvice. Signed-off-by: Jim Fehlig --- libvirt.spec.in | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libvirt.spec.in b/libvirt.spec.in index 4a57a948cc..503549dc04 100644 --- a/libvirt.spec.in +++ b/libvirt.spec.in @@ -763,6 +763,9

[PATCH V3 07/11] spec: Add module-init-tools dependency to nodedev and lxc drivers

2022-12-20 Thread Jim Fehlig
Both the nodedev and lxc drivers can load kernel modules. Add a module-init-tools dependency to the drivers. Signed-off-by: Jim Fehlig --- libvirt.spec.in | 4 1 file changed, 4 insertions(+) diff --git a/libvirt.spec.in b/libvirt.spec.in index 9e70518566..4a57a948cc 100644 --- a

[PATCH V3 06/11] spec: Move common files and dependencies to libvirt-daemon-common

2022-12-20 Thread Jim Fehlig
Signed-off-by: Jim Fehlig --- libvirt.spec.in | 93 + 1 file changed, 56 insertions(+), 37 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index cc445df83b..9e70518566 100644 --- a/libvirt.spec.in +++ b/libvirt.spec.in @@ -429,15 +429,11

[PATCH V3 05/11] spec: Rename the libvirt-lock-sanlock subpackage

2022-12-20 Thread Jim Fehlig
emon package. Signed-off-by: Jim Fehlig --- docs/kbase/rpm-deployment.rst | 12 ++-- libvirt.spec.in | 11 +++ 2 files changed, 13 insertions(+), 10 deletions(-) diff --git a/docs/kbase/rpm-deployment.rst b/docs/kbase/rpm-deployment.rst index 31805c2280..7685ee129

[PATCH V3 04/11] spec: Move lockd plugin to a new subpackage

2022-12-20 Thread Jim Fehlig
Introduce the libvirt-daemon-plugin-lockd subpackage to provide the client-side lockd plugin for virtlockd. Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé --- libvirt.spec.in | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/libvirt.spec.in b

[PATCH V3 03/11] spec: Move virtproxyd to a new subpackage libvirt-daemon-proxy

2022-12-20 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 56 ++--- 1 file changed, 39 insertions(+), 17 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index d303cac9df..ee12720b72

[PATCH V3 02/11] spec: Move virtlogd to a new subpackage libvirt-daemon-log

2022-12-20 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 53 +++-- 1 file changed, 38 insertions(+), 15 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index 33398b2877..d303cac9df

[PATCH V3 01/11] spec: Move virtlockd to a new subpackage libvirt-daemon-lock

2022-12-20 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé Reviewed-by: Andrea Bolognani --- libvirt.spec.in | 61 +++-- 1 file changed, 44 insertions(+), 17 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index 6b8acf252e..33398b2877

[PATCH V3 00/11] spec: Decompose the daemon subpackage

2022-12-20 Thread Jim Fehlig
libvirtd. Changes from V2: * Move more files and dependencies to the libvirt-daemon-common package * Address many more review comments Jim Fehlig (11): spec: Move virtlockd to a new subpackage libvirt-daemon-lock spec: Move virtlogd to a new subpackage libvirt-daemon-log spec: Move virtproxyd to a

Re: [PATCH V2 9/9] kbase: Update rpm-deployment.rst with new subpackages

2022-12-19 Thread Jim Fehlig
On 12/15/22 08:50, Andrea Bolognani wrote: On Wed, Dec 14, 2022 at 02:55:30PM -0700, Jim Fehlig wrote: On 12/14/22 11:08, Andrea Bolognani wrote: On Tue, Dec 13, 2022 at 05:31:02PM -0700, Jim Fehlig wrote: +* libvirt-daemon-plugin-lockd + This package provides the lockd.so module, a daemon

Re: [libvirt PATCH 2/2] kbase: Reorder deployments

2022-12-14 Thread Jim Fehlig
eeded + +The only required package is the `libvirt-libs`, however, it is useful to +also install `libvirt-client`. + + IMO, these days the "Minimal features for one virt driver" is as likely or more so than client-only. But you can choose the color of the shed :-). Reviewed-by: Jim F

Re: [libvirt PATCH 1/2] kbase: Reorder sections

2022-12-14 Thread Jim Fehlig
+- 1 file changed, 67 insertions(+), 67 deletions(-) Reviewed-by: Jim Fehlig Regards, Jim diff --git a/docs/kbase/rpm-deployment.rst b/docs/kbase/rpm-deployment.rst index b5ee7fd827..d1180fb0c3 100644 --- a/docs/kbase/rpm-deployment.rst +++ b/docs/kbase/rpm-deployment.rst @@ -20,6

Re: [PATCH V2 8/9] spec: Remove libvirt-daemon dependency from hypervisor subpackages

2022-12-14 Thread Jim Fehlig
On 12/14/22 10:59, Andrea Bolognani wrote: On Tue, Dec 13, 2022 at 05:31:01PM -0700, Jim Fehlig wrote: %package daemon-qemu Summary: Server side daemon & driver required to run QEMU guests -Requires: libvirt-daemon = %{version}-%{release} +Requires: libvirt-daemon-common = %{ver

Re: [PATCH V2 4/9] spec: Move lockd plugin to a new subpackage

2022-12-14 Thread Jim Fehlig
On 12/14/22 10:47, Andrea Bolognani wrote: On Tue, Dec 13, 2022 at 05:30:57PM -0700, Jim Fehlig wrote: Introduce the libvirt-daemon-plugin-lockd subpackage to provide the client-side lockd plugin for virtlockd. The sanlock plugin also needs the directory %{_libdir}/libvirt/lock-driver, which

Re: [PATCH V2 9/9] kbase: Update rpm-deployment.rst with new subpackages

2022-12-14 Thread Jim Fehlig
On 12/14/22 14:55, Jim Fehlig wrote: On 12/14/22 11:08, Andrea Bolognani wrote: On Tue, Dec 13, 2022 at 05:31:02PM -0700, Jim Fehlig wrote: Signed-off-by: Jim Fehlig ---   docs/kbase/rpm-deployment.rst | 38 ---   1 file changed, 31 insertions(+), 7 deletions

Re: [PATCH V2 9/9] kbase: Update rpm-deployment.rst with new subpackages

2022-12-14 Thread Jim Fehlig
On 12/14/22 11:08, Andrea Bolognani wrote: On Tue, Dec 13, 2022 at 05:31:02PM -0700, Jim Fehlig wrote: Signed-off-by: Jim Fehlig --- docs/kbase/rpm-deployment.rst | 38 --- 1 file changed, 31 insertions(+), 7 deletions(-) diff --git a/docs/kbase/rpm

Re: [libvirt PATCH 2/2] spec: List more directories

2022-12-14 Thread Jim Fehlig
On 12/14/22 04:25, Andrea Bolognani wrote: On Tue, Dec 13, 2022 at 04:37:34PM -0700, Jim Fehlig wrote: On 12/13/22 03:58, Andrea Bolognani wrote: %dir %attr(0755, root, root) %{_libdir}/libvirt/ %dir %attr(0755, root, root) %{_libdir}/libvirt/connection-driver/ %dir %attr(0755, root

[PATCH V2 9/9] kbase: Update rpm-deployment.rst with new subpackages

2022-12-13 Thread Jim Fehlig
Signed-off-by: Jim Fehlig --- docs/kbase/rpm-deployment.rst | 38 --- 1 file changed, 31 insertions(+), 7 deletions(-) diff --git a/docs/kbase/rpm-deployment.rst b/docs/kbase/rpm-deployment.rst index b5ee7fd827..bf2f4f5a6b 100644 --- a/docs/kbase/rpm

[PATCH V2 8/9] spec: Remove libvirt-daemon dependency from hypervisor subpackages

2022-12-13 Thread Jim Fehlig
Remove the libvirt-daemon dependency from the various libvirt-daemon- subpackages, replacing it with a set of the new sub subpackages providing the same functionality. Signed-off-by: Jim Fehlig --- libvirt.spec.in | 21 - 1 file changed, 16 insertions(+), 5 deletions

[PATCH V2 6/9] spec: Move common files to a new subpackage libvirt-daemon-common

2022-12-13 Thread Jim Fehlig
Move virt-admin, virt-host-validate, virt-ssh-helper, libvirt-guests and miscellaneous files/directories to a new libvirt-daemon-common subpackage. These files and utilities are used by other core libvirt daemons Signed-off-by: Jim Fehlig --- libvirt.spec.in | 70

[PATCH V2 7/9] spec: Remove libvirt-daemon dependency from drivers

2022-12-13 Thread Jim Fehlig
To avoid needlessly installing the monolithic daemon, replace the libvirt-daemon dependency with libvirt-daemon-common. The common subpackage contains all the utilities (e.g. virt-admin) and files used by other daemons. Signed-off-by: Jim Fehlig --- libvirt.spec.in | 22

[PATCH V2 5/9] spec: Rename the libvirt-lock-sanlock subpackage

2022-12-13 Thread Jim Fehlig
The new name "libvirt-daemon-plugin-sanlock" provides consistency with the newly introduced "libvirt-daemon-plugin-lockd" subpackage. Signed-off-by: Jim Fehlig --- libvirt.spec.in | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/libvirt.spec.

[PATCH V2 4/9] spec: Move lockd plugin to a new subpackage

2022-12-13 Thread Jim Fehlig
sanlock plugin, allowing the libvirt-daemon dependency to be dropped. Signed-off-by: Jim Fehlig --- libvirt.spec.in | 16 +--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index 633cf05100..63b29ba6f4 100644 --- a/libvirt.spec.in

[PATCH V2 3/9] spec: Move virtproxyd to a new subpackage libvirt-daemon-proxy

2022-12-13 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé --- libvirt.spec.in | 56 ++--- 1 file changed, 39 insertions(+), 17 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index 3383011b79..633cf05100 100644 --- a/libvirt.spec.in +++ b

[PATCH V2 2/9] spec: Move virtlogd to a new subpackage libvirt-daemon-log

2022-12-13 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé --- libvirt.spec.in | 53 +++-- 1 file changed, 38 insertions(+), 15 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index 6de03ec76a..3383011b79 100644 --- a/libvirt.spec.in +++ b

[PATCH V2 1/9] spec: Move virtlockd to a new subpackage libvirt-daemon-lock

2022-12-13 Thread Jim Fehlig
Signed-off-by: Jim Fehlig Reviewed-by: Daniel P. Berrangé --- libvirt.spec.in | 61 +++-- 1 file changed, 44 insertions(+), 17 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index 31ff8ea01e..6de03ec76a 100644 --- a/libvirt.spec.in +++ b

<    1   2   3   4   5   6   7   8   9   10   >