Re: [pve-devel] [PATCH] add hugepages option v5

2016-06-07 Thread Alexandre DERUMIER
>>We really need a stop hook... but the question there is *how*... >>(Even if we added a hook to kvm, it would be nice to have something for >>the case where it gets SIGKILLed). For clean qemu stop (or shutdown in guest), we could use qmp-events

Re: [pve-devel] [PATCH] add hugepages option v5

2016-06-07 Thread Alexandre DERUMIER
>>But parse_numa is still in QemuServer, did you mean to include another >>hunk in this patch moving this function? Ok, my bad,here a fix (better to keep it in QemuServer, like other parse sub) --- a/PVE/QemuServer/Memory.pm +++ b/PVE/QemuServer/Memory.pm @@ -187,7 +187,7 @@ sub config {

Re: [pve-devel] [PATCH] add hugepages option v5

2016-06-07 Thread Alexandre DERUMIER
>>When adding more numaX entries to the VM's config than the host has this >>now produces an 'Use of uninitialized value' error. >>Better check for whether /sys/devices/system/node/node$numanode exists >>and throw a useful error. >>But should this even be fixed to host nodes? Without hugepages

Re: [pve-devel] [PATCH] add hugepages option v5

2016-06-07 Thread Alexandre DERUMIER
>>When not specifying numaX entries we run into this code using the >>default $dimm_size of 512, which won't work with 1G pages. Iow this >>config without any `numa0`, `numa1`, ... entries: >> hugepages: 1024 >> numa: 1 >>will error at this point about wrong memory sizes. We could probably

Re: [pve-devel] [PATCH_V2 pve-storage] Add LVM-thin storage migration

2016-06-07 Thread Alexandre DERUMIER
Hi, I think it could be great in the future to add support for online migration (live migration + storage migration). I have send the workflow last year, but never have time to implement it. - Mail original - De: "Wolfgang Link" À: "pve-devel"

[pve-devel] [PATCH_V2 pve-storage] Add LVM-thin storage migration

2016-06-07 Thread Wolfgang Link
It is now possibel to migrate LVM-thin volumes offline from one node to an other. Also LVM what is used by Qemu. --- PVE/Storage.pm | 33 + 1 file changed, 33 insertions(+) diff --git a/PVE/Storage.pm b/PVE/Storage.pm index 1192ce3..907e3b3 100755 ---

[pve-devel] [PATCH_V5 pve-container 1/5] Add the functionality of full clone

2016-06-07 Thread Wolfgang Link
--- src/PVE/API2/LXC.pm | 21 +- src/PVE/LXC.pm | 62 + 2 files changed, 72 insertions(+), 11 deletions(-) diff --git a/src/PVE/API2/LXC.pm b/src/PVE/API2/LXC.pm index 1dd9405..8c1bf4a 100644 --- a/src/PVE/API2/LXC.pm +++

[pve-devel] [PATCH_V5 pve-container 2/5] Add move_volume.

2016-06-07 Thread Wolfgang Link
Now it is possible to move the volume to an other storage. This works only when the CT is off, to keep the volume consistent. --- src/PVE/API2/LXC.pm | 132 src/PVE/CLI/pct.pm | 1 + 2 files changed, 133 insertions(+) diff --git

[pve-devel] [PATCH_V5 pve-container 3/5] clone disk is complete, so we can remove the experimental flag

2016-06-07 Thread Wolfgang Link
--- src/PVE/API2/LXC.pm | 6 -- 1 file changed, 6 deletions(-) diff --git a/src/PVE/API2/LXC.pm b/src/PVE/API2/LXC.pm index 9c932a3..71828c9 100644 --- a/src/PVE/API2/LXC.pm +++ b/src/PVE/API2/LXC.pm @@ -1107,12 +1107,6 @@ __PACKAGE__->register_method({ "you clone a

[pve-devel] [PATCH_V5 pve-container 4/5] Prevent that a CT run at cloning.

2016-06-07 Thread Wolfgang Link
If we make a linked clone the CT must be a template so it is not allowed to run. If we make a full clone, it is safer to have the CT offline. --- src/PVE/API2/LXC.pm | 11 +++ src/PVE/LXC.pm | 4 ++-- 2 files changed, 5 insertions(+), 10 deletions(-) diff --git

Re: [pve-devel] [PATCH] add hugepages option v5

2016-06-07 Thread Alexandre DERUMIER
>>General question: >>Can you give an example in what kind of situation this boosts >>performance? I'll send benchmark results soon. (Note that hugepages are required for dpdk too) >>Note that vm_sto_cleanup() literally only happens when you stop the VM >>via CLI or GUI, not when the VM

[pve-devel] [PATCH V2 pve-manager 2/3] Extend the Ceph featuring version Infernalis and Hammer.

2016-06-07 Thread Wolfgang Link
--- PVE/CLI/pveceph.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/PVE/CLI/pveceph.pm b/PVE/CLI/pveceph.pm index ce991c1..2783f24 100755 --- a/PVE/CLI/pveceph.pm +++ b/PVE/CLI/pveceph.pm @@ -73,8 +73,8 @@ __PACKAGE__->register_method ({ properties => {

[pve-devel] [PATCH V2 pve-manager 3/3] Now pveceph can also createmon with Infernalis and Hammer.

2016-06-07 Thread Wolfgang Link
Ceph changed user name from root to ceph. And for startup systemd is used instead of sysvinit. --- PVE/API2/Ceph.pm | 9 + 1 file changed, 9 insertions(+) diff --git a/PVE/API2/Ceph.pm b/PVE/API2/Ceph.pm index 58e5b35..4f85860 100644 --- a/PVE/API2/Ceph.pm +++ b/PVE/API2/Ceph.pm @@

[pve-devel] [PATCH V2 pve-manager 1/3] Add check if ceph is managed by systemd

2016-06-07 Thread Wolfgang Link
Ceph is managed since version infernalis by systemd and use ceph as user and group. --- PVE/CephTools.pm | 11 +++ 1 file changed, 11 insertions(+) diff --git a/PVE/CephTools.pm b/PVE/CephTools.pm index c7749bb..ddf4777 100644 --- a/PVE/CephTools.pm +++ b/PVE/CephTools.pm @@ -353,4

Re: [pve-devel] [PATCH] add hugepages option v5

2016-06-07 Thread Wolfgang Bumiller
General question: Can you give an example in what kind of situation this boosts performance? Code comments inline: On Sat, Jun 04, 2016 at 10:19:56AM +0200, Alexandre Derumier wrote: > changelog : rebase on last git > > vm configuration > > hugepages: (any|2|1024) > > any:

[pve-devel] [PATCH v3 container] docs: update mountpoint descriptions again

2016-06-07 Thread Fabian Grünbichler
--- rebased v2 with v1 applied ;) src/PVE/LXC/Config.pm | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/PVE/LXC/Config.pm b/src/PVE/LXC/Config.pm index d80dae0..c067e7a 100644 --- a/src/PVE/LXC/Config.pm +++ b/src/PVE/LXC/Config.pm @@ -551,9 +551,10 @@ my $mp_desc =

Re: [pve-devel] [PATCH docs 2/3] add mountpoint restore documentation

2016-06-07 Thread Dietmar Maurer
> +By default (if no mountpoint option is overridden on the command line), `pct > +restore` will recreate volume mountpoints that are found in the backed up > +configuration. All recovered mountpoints will be created on the same storage > +(provided via `-storage`, defaulting to `local`), I

Re: [pve-devel] [PATCH docs 3/3] add note about skipped bind/dev mps

2016-06-07 Thread Dietmar Maurer
applied ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] [PATCH docs 1/3] add bind mount warning

2016-06-07 Thread Dietmar Maurer
applied ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] [PATCH container] docs: update mountpoint descriptions

2016-06-07 Thread Dietmar Maurer
applied ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [PATCH v2 container] docs: update mountpoint descriptions

2016-06-07 Thread Fabian Grünbichler
--- Changed: - WARNING to NOTE - include symlink info in normal description as well src/PVE/LXC/Config.pm | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/src/PVE/LXC/Config.pm b/src/PVE/LXC/Config.pm index 6b4ec4c..c067e7a 100644 --- a/src/PVE/LXC/Config.pm +++

[pve-devel] [PATCH container] docs: update mountpoint descriptions

2016-06-07 Thread Fabian Grünbichler
--- src/PVE/LXC/Config.pm | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/PVE/LXC/Config.pm b/src/PVE/LXC/Config.pm index 6b4ec4c..d80dae0 100644 --- a/src/PVE/LXC/Config.pm +++ b/src/PVE/LXC/Config.pm @@ -208,7 +208,7 @@ my $rootfs_desc = { }, ro => {

[pve-devel] [PATCH docs 1/3] add bind mount warning

2016-06-07 Thread Fabian Grünbichler
--- pct.adoc | 6 ++ 1 file changed, 6 insertions(+) diff --git a/pct.adoc b/pct.adoc index dc9f446..014e48d 100644 --- a/pct.adoc +++ b/pct.adoc @@ -382,6 +382,12 @@ mounting mechanisms or storage technologies, it is possible to establish the FUSE mount on the Proxmox host and use a bind

[pve-devel] [PATCH docs 2/3] add mountpoint restore documentation

2016-06-07 Thread Fabian Grünbichler
--- pct.adoc | 53 + 1 file changed, 53 insertions(+) diff --git a/pct.adoc b/pct.adoc index 014e48d..38d5507 100644 --- a/pct.adoc +++ b/pct.adoc @@ -452,6 +452,45 @@ and destroy containers, and control execution (start, stop, migrate, ...).

[pve-devel] [PATCH docs 3/3] add note about skipped bind/dev mps

2016-06-07 Thread Fabian Grünbichler
--- vzdump.adoc | 3 +++ 1 file changed, 3 insertions(+) diff --git a/vzdump.adoc b/vzdump.adoc index 3276a32..00f4aa6 100644 --- a/vzdump.adoc +++ b/vzdump.adoc @@ -93,6 +93,9 @@ NOTE: `snapshot` mode requires that all backed up volumes are on a storage that supports snapshots. Using the

Re: [pve-devel] [RFC PATCH storage] fix 1012: dir: add mkdir option

2016-06-07 Thread Dietmar Maurer
applied ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] [PATCH common] remove duplicate 'set -o pipefail'

2016-06-07 Thread Dietmar Maurer
applied ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] [PATCH storage 2/2] disable jewel image features when using krdb

2016-06-07 Thread Dietmar Maurer
applied ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] [PATCH storage 1/2] add ceph version helpers

2016-06-07 Thread Dietmar Maurer
applied ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] [PATCH] add hyperv enlightments : hv_reset, hv_vpindex, hv_runtime

2016-06-07 Thread Dietmar Maurer
applied ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] [PATCH pve-storage] Add LVM-thin storage migration

2016-06-07 Thread Wolfgang Bumiller
On Tue, Jun 07, 2016 at 09:10:07AM +0200, Wolfgang Link wrote: > It is now possibel to migrate LVM-thin volumes offline from one node to an > other. > Also LVM what is used by Qemu. > --- > PVE/Storage.pm | 35 +++ > 1 file changed, 35 insertions(+) > > diff

[pve-devel] [PATCH pve-storage] Add LVM-thin storage migration

2016-06-07 Thread Wolfgang Link
It is now possibel to migrate LVM-thin volumes offline from one node to an other. Also LVM what is used by Qemu. --- PVE/Storage.pm | 35 +++ 1 file changed, 35 insertions(+) diff --git a/PVE/Storage.pm b/PVE/Storage.pm index 1192ce3..22c1a4c 100755 ---

[pve-devel] [PATCH qemu-server] Add LVM and LVMThin to QemuMigration

2016-06-07 Thread Wolfgang Link
Ofline migration on LVM and LVMThin are possible offline. --- PVE/QemuMigrate.pm | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/PVE/QemuMigrate.pm b/PVE/QemuMigrate.pm index 7b9506f..2fd307c 100644 --- a/PVE/QemuMigrate.pm +++ b/PVE/QemuMigrate.pm @@ -293,7 +293,9 @@ sub

[pve-devel] [PATCH storage 1/2] add ceph version helpers

2016-06-07 Thread Fabian Grünbichler
--- Note: this could be used in CephTools / pveceph as well if necessary PVE/Storage/RBDPlugin.pm | 30 ++ 1 file changed, 30 insertions(+) diff --git a/PVE/Storage/RBDPlugin.pm b/PVE/Storage/RBDPlugin.pm index 967ee04..2b4ff9c 100644 --- a/PVE/Storage/RBDPlugin.pm

[pve-devel] [PATCH storage 2/2] disable jewel image features when using krdb

2016-06-07 Thread Fabian Grünbichler
otherwise mapping those images will fail. disabling the features only needs to be done once per image, so it makes sense to do this when creating the images. unfortunately, the command does not work in hammer, so it needs a version check for jewel or higher. --- PVE/Storage/RBDPlugin.pm | 17