[pve-devel] applied: [PATCH manager] fix nodeselector when no element in the tree is selected

2017-08-22 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 manager 1/2] qemu/HDEdit: whitespace cleanup

2017-08-22 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+cleanup: [PATCH manager 2/2] qemu/create: use 'virtio-scsi-single' if 'iothread' is set

2017-08-22 Thread Fabian Grünbichler
replaced CC: with Reported-by: in commit message ___ pve-devel mailing list pve-devel@pve.proxmox.com https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [Bug 1484] RFC: Improve swap support (zswap, zram & etc)

2017-08-22 Thread Timofey Titovets
Hi list, By suggestion from: Fabian Grünbichler, i try open topic on the dev list. So, my suggestion (copy paste from issue): since 3.11 kernel Linux support zswap, and since 3.15 kernel linux support zram. (i'm a creator of https://github.com/Nefelim4ag/systemd-swap) What i can suggest (That jus

[pve-devel] [PATCH manager 1/4] remove unused variables

2017-08-22 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht --- www/manager6/qemu/CreateWizard.js | 2 -- 1 file changed, 2 deletions(-) diff --git a/www/manager6/qemu/CreateWizard.js b/www/manager6/qemu/CreateWizard.js index aab78479..126925e2 100644 --- a/www/manager6/qemu/CreateWizard.js +++ b/www/manager6/qemu/CreateW

[pve-devel] [PATCH manager 2/4] kvm_ostype: move to store-like format

2017-08-22 Thread Thomas Lamprecht
move from the key => value format to one where all versions from a base OS type can be directly used as data for a store. We will rely on this in the next patch to allow a easy transition to combobox UI for the OS type selector. We also avoid having the base type multiple times. Signed-off-by: T

[pve-devel] [PATCH manager 4/4] wizard: merge CD/DVD and OS panels

2017-08-22 Thread Thomas Lamprecht
We had a single column for the CD/DVD panel, fill the second column with the now in size reduced OS type selector, as ISO selection and OS type selection are tied together. Reduces steps needed for creating a VM. Signed-off-by: Thomas Lamprecht --- www/manager6/qemu/CDEdit.js | 6 +-

[pve-devel] [PATCH manager 0/4] Switch OS type selector to combobox and merge with CD/DVD selector

2017-08-22 Thread Thomas Lamprecht
See: for a screenshot of the new merged panel. First two patches are refactoring, last two patches to the switch to combobox and the merge of the panels, respectively. I'm not to attached to specific visible names or headings of UI components, and would like to ge

[pve-devel] [PATCH manager 3/4] OSType edit: switch to combobox

2017-08-22 Thread Thomas Lamprecht
Move from the radiogroup to two comboboxes, where the base OS and then a specific Version can be choosen. Avoids multiple occurences of strings like "Microsoft Windows" and saves a lot of space - which will be used to merge this panel with the ISO selector panel in the create wizard. Signed-off-b

Re: [pve-devel] [Bug 1484] RFC: Improve swap support (zswap, zram & etc)

2017-08-22 Thread Eneko Lacunza
Hi Timofey, El 22/08/17 a las 11:45, Timofey Titovets escribió: Hi list, By suggestion from: Fabian Grünbichler, i try open topic on the dev list. So, my suggestion (copy paste from issue): since 3.11 kernel Linux support zswap, and since 3.15 kernel linux support zram. (i'm a creator of https

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

2017-08-22 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. --- Applied your suggestions. $unprivileged doesn't us

Re: [pve-devel] [Bug 1484] RFC: Improve swap support (zswap, zram & etc)

2017-08-22 Thread Timofey Titovets
2017-08-22 12:58 GMT+03:00 Eneko Lacunza : > Hi Timofey, > > El 22/08/17 a las 11:45, Timofey Titovets escribió: >> >> Hi list, >> By suggestion from: Fabian Grünbichler, >> i try open topic on the dev list. >> >> So, my suggestion (copy paste from issue): >> since 3.11 kernel Linux support zswap,

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

2017-08-22 Thread Dominik Csapak
On 08/22/2017 12:32 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 v2] start/fix #1478: add check for unsupported config

2017-08-22 Thread Fabian Grünbichler
On Tue, Aug 22, 2017 at 01:26:14PM +0200, Dominik Csapak wrote: > On 08/22/2017 12:32 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 tho

[pve-devel] [PATCH installer] fix #1479: detect missing KVM support

2017-08-22 Thread Fabian Grünbichler
Signed-off-by: Fabian Grünbichler --- proxinstall | 8 1 file changed, 8 insertions(+) diff --git a/proxinstall b/proxinstall index 509430f..a04d98a 100755 --- a/proxinstall +++ b/proxinstall @@ -2971,6 +2971,14 @@ sub create_intro_view { cleanup_view (); +eval { + my

[pve-devel] Disksize without limit ZFS (refquota)

2017-08-22 Thread Detlef Bracker
Hi, Why proxmox create zfs container with refquota or others without set of refquota! Or is that possible, that old containers was set with refquota and new created not? Possible one update has change the settings for container creating or with API with refquota and via GUi with none? I dont

[pve-devel] [PATCH container v2 0/4] improve ostype checks and small updates

2017-08-22 Thread Thomas Lamprecht
v2 of the "fix os-release mapping for similar flavoured distros" patch. First patch is a small cleanup, second and third improve the CT ostype checks and clean them up where possible - addressing the comments from Fabian on v1. Fourth patch uses ostype for Fedora's setup and adds support for Fedor

[pve-devel] [PATCH container v2 1/4] remove unused Data::Dumper

2017-08-22 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht --- src/PVE/LXC.pm | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/PVE/LXC.pm b/src/PVE/LXC.pm index 3beba8c..e3433fb 100644 --- a/src/PVE/LXC.pm +++ b/src/PVE/LXC.pm @@ -26,8 +26,6 @@ use PVE::Syscall; use PVE::LXC::Config; use Time::HiRes qw (gettimeof

[pve-devel] [PATCH container v2 2/4] fix os-release mapping for similar flavoured distros

2017-08-22 Thread Thomas Lamprecht
This avoid failing the check where the auto detected type differs from the configs ostype. Signed-off-by: Thomas Lamprecht --- changes v1 -> v2: * replace regex with simple checks to already available info, this moves the fixup into "autodetect_type" where it fits better * warn not die on the

[pve-devel] [PATCH container v2 3/4] update_lxc_config: remove ostype check

2017-08-22 Thread Thomas Lamprecht
We have altogether three calls to this method, all in API context with already sanitizied ostype. Signed-off-by: Thomas Lamprecht --- src/PVE/LXC.pm | 19 --- 1 file changed, 8 insertions(+), 11 deletions(-) diff --git a/src/PVE/LXC.pm b/src/PVE/LXC.pm index e3433fb..241c5b2 100

[pve-devel] [PATCH container v2 4/4] setup/fedora: use os_release and support 26

2017-08-22 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht --- src/PVE/LXC/Setup/Fedora.pm | 16 +++- 1 file changed, 3 insertions(+), 13 deletions(-) diff --git a/src/PVE/LXC/Setup/Fedora.pm b/src/PVE/LXC/Setup/Fedora.pm index 70aab62..23851d5 100644 --- a/src/PVE/LXC/Setup/Fedora.pm +++ b/src/PVE/LXC/Setup/F

Re: [pve-devel] Disksize without limit ZFS (refquota)

2017-08-22 Thread Dominik Csapak
hi, can you reproduce that? e.g. can you create container via the gui and not have the refquota set? if yes, what is your pveversion -v ? ___ pve-devel mailing list pve-devel@pve.proxmox.com https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] Disksize without limit ZFS (refquota)

2017-08-22 Thread Detlef Bracker
Dear Dominik, we have the problem in all hosts, but not in all containers! It looks like, that this was the machines, we had backup before on other host and restored in the new machine via the normal proxmox backup - restore - process. But not sure and can been long time ago - 6 months?! We se