Hi Evgeni,

On Sun, Apr 23, 2017 at 11:45:07AM +0200, Evgeni Golov wrote:
> > Attached is a patch for lxc that configures this sysctl setting to a higher
> > value.  I don't know if this is the "Debian way" or if some other solution
> > is preferred.
> 
> Your patch looks quite similar to what was added in LXD in Ubuntu recently:
>  
> https://github.com/lxc/lxd-pkg-ubuntu/commit/333e2671a91861b949a2b990459656a3b065e5fb
>  https://bugs.launchpad.net/juju/+bug/1602192

Ah, indeed, I had found the launchpad bug but not the patch.

> I think we also should add something like
>  sysctl -q -p /etc/sysctl.d/lxc.conf || true
> to the configure target of postinst, to actually load the file directly,
> instead of waiting for a reboot?

Good idea!

> However, I am not 100% sure that packages should alter sysctl settings without
> telling the admin.

Well, that was the meaning of my question about the "Debian way" :)

The only two packages I found doing this are bit-babbler and ufw, so it
does not exactly seem like a widespread practice.

Baptiste

Reply via email to