[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

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?


Attachment: OpenPGP_0xB0DE9DD628BF132F.asc
Description: OpenPGP public key

Attachment: OpenPGP_signature
Description: OpenPGP digital signature



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.