[pve-devel] [PATCH Storage] Newly created LVM volumes are now created with a number of stripes equal to the number of physical volumes in a volume group.

2017-07-26 Thread Martin Lablans
Signed-off-by: Martin Lablans --- PVE/Storage/LVMPlugin.pm | 19 ++- 1 file changed, 18 insertions(+), 1 deletion(-) diff --git a/PVE/Storage/LVMPlugin.pm b/PVE/Storage/LVMPlugin.pm index 9633e4c..cfb18e4 100644 --- a/PVE/Storage/LVMPlugin.pm +++ b/PVE/Storage/LVMPlugin.pm @@ -2

[pve-devel] [PATCH Storage] LVM Striping

2017-07-26 Thread Martin Lablans
Dear all, this patch will change the LVM storage plugin to create striped rather than linear logical volumes, which can multiply the throughput for volume groups backed by several controllers or network paths. The number of stripes is set equal to the number of physical volumes in the volume

[pve-devel] applied: [PATCH manager] add missing '-' to service name pattern

2017-07-26 Thread Wolfgang Bumiller
applied On Wed, Jul 26, 2017 at 04:46:16PM +0200, Dominik Csapak wrote: > since ids can contain '-', we have to include them in the pattern > for the service names > > Signed-off-by: Dominik Csapak > --- > PVE/API2/Ceph.pm | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --

[pve-devel] [PATCH manager] add missing '-' to service name pattern

2017-07-26 Thread Dominik Csapak
since ids can contain '-', we have to include them in the pattern for the service names Signed-off-by: Dominik Csapak --- PVE/API2/Ceph.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/PVE/API2/Ceph.pm b/PVE/API2/Ceph.pm index c8ccab8a..eb8f14f3 100644 --- a/PVE/API2/Ce

[pve-devel] [PATCH container 1/2] Add support for cgroups ns subgroup

2017-07-26 Thread Tobias Böhm
For privileged containers a ns subgroup is created by lxc. Currently it is not used, except by pvestatd, but not for reading stats from. With this patch presence of ns subgroup is checked and if present values are read from and written to this subgroup. Signed-off-by: Tobias Böhm --- src/PVE/LXC

[pve-devel] [PATCH container 0/2] Fix blkio bytes stats

2017-07-26 Thread Tobias Böhm
Currently, the cgroup paths used are always /lxc/. For privileged containers there is the ns subgroup created by lxc. For CPU and mem stats this doesn't matter, but for blkio stats the proper values are only available in the subgroup. In order to handle things correctly, a subroutine has been extra

[pve-devel] [PATCH container 2/2] Fix block stats bytes counter

2017-07-26 Thread Tobias Böhm
Instead of setting the bytes count, sum it This way stats from all devices are accumulated, instead of only stats from the last device in the file are set. Signed-off-by: Tobias Böhm --- src/PVE/LXC.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/PVE/LXC.pm b/src/P

[pve-devel] [PATCH manager 1/3] CreateWizard: remove title from confirm tabpanel

2017-07-26 Thread Thomas Lamprecht
It isn't needed as its obvious that the table below show the configuration. It just takes away space. Signed-off-by: Thomas Lamprecht --- www/manager6/lxc/CreateWizard.js | 1 - www/manager6/qemu/CreateWizard.js | 1 - 2 files changed, 2 deletions(-) diff --git a/www/manager6/lxc/CreateWizard.

[pve-devel] [PATCH manager 2/3] CreateWizard: allow adding notes during guest creation

2017-07-26 Thread Thomas Lamprecht
Add a text area to the end of the creation wizard, to allow adding some notes to this guest. We can set this through the API already on creation so no big changes needed. Add the text area in the confirmation panel, as there is the point where it's "sure" which task the VM will do, if there's any.

[pve-devel] [PATCH manager 0/3] CreateWizard allow setting Notes and remove a title

2017-07-26 Thread Thomas Lamprecht
See: vs for a CreateWizard screenshot comparison of before/after I'd understand if the less visible rows are a concern, I'd increase the overall wizard size from 620x450 to 700x510 (keep aspect ratio) to address this, it would then

[pve-devel] [PATCH manager 3/3] window.Wizard: increase default size

2017-07-26 Thread Thomas Lamprecht
The current size was chosen with the older, more compact, classic theme. The Crisp theme, now in use, has a bit more spaces and bigger font sizes to adapt to the increased pixel density of current displays. Increase the size a bit (~13%) but try to keep the aspect ratio. Signed-off-by: Thomas Lamp

[pve-devel] applied: [PATCH manager v3 0/3] ceph-mgr integration

2017-07-26 Thread Wolfgang Bumiller
applied whole series On Wed, Jul 26, 2017 at 11:11:50AM +0200, Dominik Csapak wrote: > since ceph luminous sees ceph-mgr as a first class daemon, (and does > not autogenerate/autostart it anymore), we want to generate one with every > monitor we create > > for users upgrading from jewel and advan

[pve-devel] applied: [PATCH manager v3 1/5] FileSelector: adapt combogrid total and column width

2017-07-26 Thread Wolfgang Bumiller
applied whole series On Tue, Jul 25, 2017 at 04:26:56PM +0200, Thomas Lamprecht wrote: > By default the "Size" column ellipsed a lot entries, which is just > annoying. Adapt it so that all sizes can be viewed comfortably. > > Further increase the total width of this combogrid, a lot less could >

[pve-devel] applied: [PATCH container] fix #1253: show ssh key fingerprints again

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

Re: [pve-devel] [PATCH qemu-server v2 1/2] remove unused obsolete vmtar

2017-07-26 Thread Thomas Lamprecht
any comments on this series? On 06/14/2017 03:52 PM, Thomas Lamprecht wrote: This was used by the pre-VMA backup methods, as the new backup process was introduced in PVE 2.3 (04.03.2013) and this would be only used for creating new backups it can be removed safely. Fixes Debian stretch build on

Re: [pve-devel] [PATCH storage v3 1/2] storage_info: return also disabled storages

2017-07-26 Thread Thomas Lamprecht
any comments on this series? On 06/27/2017 01:42 PM, Thomas Lamprecht wrote: in the Storage/Status API call we have a 'enabled' param which had no effect because storage_info only returned enabled one way or the other. This affected also `pvesm status` which uses the Storage/Status API call. S

Re: [pve-devel] [PATCH manager v3 1/5] FileSelector: adapt combogrid total and column width

2017-07-26 Thread Dominik Csapak
series looks good to me, this will be a nice improvement :) i hope we can enable this in the future for more combogrids ___ pve-devel mailing list pve-devel@pve.proxmox.com https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [PATCH manager v3 3/3] whitespace cleanup

2017-07-26 Thread Dominik Csapak
and intendation fix Signed-off-by: Dominik Csapak --- PVE/API2/Ceph.pm | 120 +++ 1 file changed, 60 insertions(+), 60 deletions(-) diff --git a/PVE/API2/Ceph.pm b/PVE/API2/Ceph.pm index 13b156f6..c8ccab8a 100644 --- a/PVE/API2/Ceph.pm +++ b/P

[pve-devel] [PATCH manager v3 2/3] pveceph: add createmgr/destroymgr commands

2017-07-26 Thread Dominik Csapak
this patch adds the create-/destroymgr commands to the api and pveceph, so that advanced users can split monitor and manager daemons Signed-off-by: Dominik Csapak --- changes from v2: * changed the remaining instances of 'mgrid' to 'id' for destroymgr PVE/API2/Ceph.pm | 87 +++

[pve-devel] [PATCH manager v3 1/3] pveceph: create mgr with mon, use nodename for id

2017-07-26 Thread Dominik Csapak
we now want to add a ceph-mgr daemon to every node where a ceph-mon daemon runs, as per ceph documentation recommendation, because in luminous the mgr daemons will not be automatically created/started with a monitor anymore we also give the createmon an optional id parameter, so that one can set a

[pve-devel] [PATCH manager v3 0/3] ceph-mgr integration

2017-07-26 Thread Dominik Csapak
since ceph luminous sees ceph-mgr as a first class daemon, (and does not autogenerate/autostart it anymore), we want to generate one with every monitor we create for users upgrading from jewel and advanced users, we also introduce createmgr and destroymgr changelogs are in the individual patches

Re: [pve-devel] [PATCH manager v2 1/3] pveceph: create mgr with mon, use nodename for id

2017-07-26 Thread Thomas Lamprecht
besides one minor nitpick below it looks good to me now. On 07/26/2017 10:42 AM, Dominik Csapak wrote: we now want to add a ceph-mgr daemon to every node where a ceph-mon daemon runs, as per ceph documentation recommendation, because in luminous the mgr daemons will not be automatically created/

Re: [pve-devel] [PATCH manager v2 2/3] pveceph: add createmgr/destroymgr commands

2017-07-26 Thread Thomas Lamprecht
The s/mgrid/id/ renaming was only done partially, see minor comments below. Once fixed the rest looks good for me now. On 07/26/2017 10:42 AM, Dominik Csapak wrote: this patch adds the create-/destroymgr commands to the api and pveceph, so that advanced users can split monitor and manager daemon

[pve-devel] [PATCH manager v2 1/3] pveceph: create mgr with mon, use nodename for id

2017-07-26 Thread Dominik Csapak
we now want to add a ceph-mgr daemon to every node where a ceph-mon daemon runs, as per ceph documentation recommendation, because in luminous the mgr daemons will not be automatically created/started with a monitor anymore we also give the createmon an optional id parameter, so that one can set a

[pve-devel] [PATCH manager v2 2/3] pveceph: add createmgr/destroymgr commands

2017-07-26 Thread Dominik Csapak
this patch adds the create-/destroymgr commands to the api and pveceph, so that advanced users can split monitor and manager daemons Signed-off-by: Dominik Csapak --- changes from v1: * corrected the id regex * optimized the id assignment and dropped the unnecessary die * removed the hunk which b

[pve-devel] [PATCH manager v2 3/3] whitespace cleanup

2017-07-26 Thread Dominik Csapak
and intendation fix Signed-off-by: Dominik Csapak --- changes from v1: none PVE/API2/Ceph.pm | 120 +++ 1 file changed, 60 insertions(+), 60 deletions(-) diff --git a/PVE/API2/Ceph.pm b/PVE/API2/Ceph.pm index a499bd3a..310ace63 100644 --- a/P

[pve-devel] [PATCH manager v2 0/3] ceph-mgr integration

2017-07-26 Thread Dominik Csapak
since ceph luminous sees ceph-mgr as a first class daemon, (and does not autogenerate/autostart it anymore), we want to generate one with every monitor we create for users upgrading from jewel and advanced users, we also introduce createmgr and destroymgr changelogs are in the individual patches

Re: [pve-devel] [PATCH manager 2/3] pveceph: add createmgr/destroymgr commands

2017-07-26 Thread Thomas Lamprecht
some comments inline On 07/25/2017 05:14 PM, Dominik Csapak wrote: this patch adds the create-/destroymgr commands to the api and pveceph, so that advanced users can split monitor and manager daemons Signed-off-by: Dominik Csapak --- PVE/API2/Ceph.pm | 105 +

Re: [pve-devel] [PATCH manager 1/3] pveceph: create mgr with mon, use nodename for id

2017-07-26 Thread Thomas Lamprecht
some comments inline On 07/25/2017 05:14 PM, Dominik Csapak wrote: we now want to add a ceph-mgr daemon to every node where a ceph-mon daemon runs, as per ceph documentation recommendation, because in luminous the mgr daemons will not be automatically created/started with a monitor anymore we a