Re: [pve-devel] [PATCH container] start/fix #1478: add check for unsupported config

2017-08-21 Thread Fabian Grünbichler
On Fri, Aug 18, 2017 at 04:30:33PM +0200, 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

Re: [pve-devel] [PATCH container] fix os-release mapping for similar flavoured distros

2017-08-21 Thread Fabian Grünbichler
On Fri, Aug 18, 2017 at 02:59:33PM +0200, Thomas Lamprecht wrote: > This avoid failing the check where the auto detected type differs > from the configs ostype. > > Signed-off-by: Thomas Lamprecht > --- > src/PVE/LXC/Setup.pm | 10 -- > 1 file changed, 8

[pve-devel] applied: [PATCH qemu-server v2 0/3] Typo fix, refactoring and new check

2017-08-21 Thread Fabian Grünbichler
On Mon, Aug 21, 2017 at 11:47:15AM +0200, Philip Abernethy wrote: > First patch splits the nokvm->kvm patch into two parts to separate the > function renaming, according to your suggestion. > Second patch refactors $nokvm to $kvm, simplifying the definition of the > variable and getting rid of

[pve-devel] applied: [PATCH common v2] tools: add pipe_socket_to_command

2017-08-21 Thread Fabian Grünbichler
___ pve-devel mailing list pve-devel@pve.proxmox.com https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] applied: [PATCH cluster 2/2] pvecm mtunnel: factor out run command

2017-08-21 Thread Fabian Grünbichler
___ pve-devel mailing list pve-devel@pve.proxmox.com https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [PATCH qemu-server v2 0/3] Typo fix, refactoring and new check

2017-08-21 Thread Philip Abernethy
First patch splits the nokvm->kvm patch into two parts to separate the function renaming, according to your suggestion. Second patch refactors $nokvm to $kvm, simplifying the definition of the variable and getting rid of unnecessary inversions. Third patch introduces a new check to catch

[pve-devel] [PATCH qemu-server v2 1/3] Refactor HyperV enlightenments

2017-08-21 Thread Philip Abernethy
Fixes a typo in the function name and removes the $nokvm parameter, as it's only used to immideately exit the function. Instead calling the function conditionally. --- PVE/QemuServer.pm | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/PVE/QemuServer.pm

[pve-devel] [PATCH qemu-server v2 3/3] Fix #1125. Check for KVM support before starting VM

2017-08-21 Thread Philip Abernethy
Moved the check to the beginning of the function. VMs configured to use KVM won't start if KVM is not available. VMs not configured to use KVM will start regardless. --- PVE/QemuServer.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm

[pve-devel] [PATCH qemu-server v2 2/3] Refactors $nokvm to $kvm

2017-08-21 Thread Philip Abernethy
for improved readability and consistency with the option name. --- PVE/QemuServer.pm | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index ee5e467..d6ddb1d 100644 --- a/PVE/QemuServer.pm +++ b/PVE/QemuServer.pm @@

[pve-devel] [PATCH manager 2/2] qemu/create: use 'virtio-scsi-single' if 'iothread' is set

2017-08-21 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht CC: Nils --- www/manager6/qemu/CreateWizard.js | 2 +- www/manager6/qemu/HDEdit.js | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/www/manager6/qemu/CreateWizard.js

[pve-devel] [PATCH manager 1/2] qemu/HDEdit: whitespace cleanup

2017-08-21 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht --- www/manager6/qemu/HDEdit.js | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/www/manager6/qemu/HDEdit.js b/www/manager6/qemu/HDEdit.js index 8f5c9dbe..8384083e 100644 ---