[PATCH v4 3/3] NEWS: mention removal of sysconfig

Olaf Hering posted 3 patches 4 years, 1 month ago
There is a newer version of this series
[PATCH v4 3/3] NEWS: mention removal of sysconfig
Posted by Olaf Hering 4 years, 1 month ago
Signed-off-by: Olaf Hering <olaf@aepfle.de>
---
 NEWS.rst | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/NEWS.rst b/NEWS.rst
index 4d1a1501ef..ce324afb8a 100644
--- a/NEWS.rst
+++ b/NEWS.rst
@@ -36,6 +36,16 @@ v8.0.0 (unreleased)
 
 * **Improvements**
 
+  * packaging: sysconfig files no longer installed
+
+    libvirt used to provide defaults in various /etc/sysconfig/ files, such
+    as /etc/sysconfig/libvirt. Since these files are owned by the admin, this
+    made it difficult to change built-in defaults in case such file was
+    modified by the admin. The built-in defaults are now part of the provided
+    systemd unit files, such as libvirtd.service. These unit files continue
+    to parse sysconfig files, in case they are created by the admin and filled
+    with the desired key=value pairs.
+
 * **Bug fixes**
 
 

Re: [PATCH v4 3/3] NEWS: mention removal of sysconfig
Posted by Jim Fehlig 4 years, 1 month ago
On 12/14/21 08:37, Olaf Hering wrote:
> Signed-off-by: Olaf Hering <olaf@aepfle.de>
> ---
>   NEWS.rst | 10 ++++++++++
>   1 file changed, 10 insertions(+)
> 
> diff --git a/NEWS.rst b/NEWS.rst
> index 4d1a1501ef..ce324afb8a 100644
> --- a/NEWS.rst
> +++ b/NEWS.rst
> @@ -36,6 +36,16 @@ v8.0.0 (unreleased)
>   
>   * **Improvements**
>   
> +  * packaging: sysconfig files no longer installed
> +
> +    libvirt used to provide defaults in various /etc/sysconfig/ files, such
> +    as /etc/sysconfig/libvirt. Since these files are owned by the admin, this

Small nit: should be /etc/sysconfig/libvirtd.

Jim

> +    made it difficult to change built-in defaults in case such file was
> +    modified by the admin. The built-in defaults are now part of the provided
> +    systemd unit files, such as libvirtd.service. These unit files continue
> +    to parse sysconfig files, in case they are created by the admin and filled
> +    with the desired key=value pairs.
> +
>   * **Bug fixes**
>   
>   
>