Re: [pve-devel] [PATCH container v3] close #1478: add check for unsupported config

2017-08-23 Thread Thomas Lamprecht
On 08/23/2017 04:27 PM, Philip Abernethy wrote: Adds a check if an unprivileged container is configured to use quota on any of its mountpoints. If so an understandable error message is given. Ideally I'd like to catch those configurations on the GUI, too, to avoid users just running into it. ---

Re: [pve-devel] [PATCH container v3] close #1478: add check for unsupported config

2017-08-23 Thread Dietmar Maurer
I wonder if this simplified version works: if ($conf->{unprivileged}) { PVE::LXC::Config->foreach_mountpoint($conf, sub { my ($ms, $mountpoint) = @_; die "Quotas are not supported by unprivileged containers.\n" if $mountpoint->{q

[pve-devel] [PATCH container v3] close #1478: add check for unsupported config

2017-08-23 Thread Philip Abernethy
Adds a check if an unprivileged container is configured to use quota on any of its mountpoints. If so an understandable error message is given. Ideally I'd like to catch those configurations on the GUI, too, to avoid users just running into it. --- Done. GUI fix coming up. src/PVE/API2/LXC/Status.