Re: [PATCH v2 2/2] tools/xenstore: set open file descriptor limit for xenstored

[Juergen Gross <jgross@xxxxxxxx>]

On 11.06.21 07:46, Olaf Hering wrote:
> Am Fri, 11 Jun 2021 07:01:31 +0200
> schrieb Juergen Gross <jgross@xxxxxxxx>:
>
> > Why? You realize that above is a comment just documenting the default?
> That depends on the context. See 
> https://bugzilla.opensuse.org/show_bug.cgi?id=1185682 for a reason why it 
> should become an empty variable. But yes, we can patch that one too.
Isn't that a bug in fillup or the related rpm-macro? A variable set in
the existing /etc/sysconfig/xencommons file only should be preserved.

In general I think we should be consistent in the file.

In case there is no downside for other distributions I'd recommend to
switch all variables to your suggested pattern.

If there are disadvantages for others we should keep the current
pattern as changing it now would break existing installations.

Any thoughts?


Juergen

Attachment: OpenPGP_0xB0DE9DD628BF132F.asc
Description: OpenPGP public key

Attachment: OpenPGP_signature
Description: OpenPGP digital signature