Re: [pve-devel] [PATCH] Document PVE::Storage::Plugin public methods

2017-01-16 Thread Jasmin J.
Hi! THX for describing it, it would have been very helpful when I was writing my DRBD8 plugin. On 01/16/2017 09:46 PM, Dmitry Petuhov wrote: +# public +# returns $path to image/directory to be passed to qemu/lxc. sub path { my ($class, $scfg, $volname, $storeid, $snapname) = @_;

[pve-devel] Question: open virtual appliance

2017-01-16 Thread Gustaf Ankarloo
Are there any plans of making a import function for OVA/OVF ? I know it's doable with a lot of manual tinkering Best Regards Gustaf Ankarloo ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] applied: [PATCH pve-manager 0/9] use new libpve-http-server-perl package

2017-01-16 Thread Wolfgang Bumiller
applied On Mon, Jan 16, 2017 at 10:59:30AM +0100, Dietmar Maurer wrote: > Dietmar Maurer (9): > use new libpve-http-server-perl package > pveproxy.pm: use PVE::APIServer::AnyEvent::add_dirs() > remove www/bootstrap completely (now in pve-http-server) > pveproxy.pm: set title to 'Proxmox

[pve-devel] [PATCH] Document PVE::Storage::Plugin public methods

2017-01-16 Thread Dmitry Petuhov
Add comments for base class's methods to be reloaded in custom storage plugins. Also add comment at end of file with exmaples of methods that must be in child classes, but not present in base class. Signed-off-by: Dmitry Petuhov --- Described returning values where

[pve-devel] [PATCH pve-manager 8/9] HTTPServer.pm: rewrite rest_handler (simplify code)

2017-01-16 Thread Dietmar Maurer
Signed-off-by: Dietmar Maurer --- PVE/HTTPServer.pm | 125 ++ 1 file changed, 51 insertions(+), 74 deletions(-) diff --git a/PVE/HTTPServer.pm b/PVE/HTTPServer.pm index 299b816..31ac293 100755 --- a/PVE/HTTPServer.pm +++

[pve-devel] [PATCH pve-manager 9/9] PVE/HTTPServer: do not return cookie_name from auth_handler

2017-01-16 Thread Dietmar Maurer
This is no longer required. Signed-off-by: Dietmar Maurer --- PVE/HTTPServer.pm | 1 - 1 file changed, 1 deletion(-) diff --git a/PVE/HTTPServer.pm b/PVE/HTTPServer.pm index 31ac293..0fae3e2 100755 --- a/PVE/HTTPServer.pm +++ b/PVE/HTTPServer.pm @@ -105,7 +105,6 @@ sub

[pve-devel] [PATCH pve-manager 1/9] use new libpve-http-server-perl package

2017-01-16 Thread Dietmar Maurer
Signed-off-by: Dietmar Maurer --- PVE/API2/Formatter/Bootstrap.pm | 238 -- PVE/API2/Formatter/HTML.pm | 288 --- PVE/API2/Formatter/Makefile | 17 - PVE/API2/Formatter/Standard.pm | 142 PVE/API2/Makefile |1 - PVE/HTTPServer.pm

[pve-devel] [PATCH pve-manager 5/9] HTTPServer.pm: call $rpcenv-init() when necessary.

2017-01-16 Thread Dietmar Maurer
This code was removed from the generic server, so we need to add it here. cleanup follows ... Signed-off-by: Dietmar Maurer --- PVE/HTTPServer.pm | 21 - 1 file changed, 20 insertions(+), 1 deletion(-) diff --git a/PVE/HTTPServer.pm b/PVE/HTTPServer.pm

[pve-devel] [PATCH pve-manager 7/9] remove base_handler_class - directly use PVE::API2 instead

2017-01-16 Thread Dietmar Maurer
Signed-off-by: Dietmar Maurer --- PVE/HTTPServer.pm | 6 +- PVE/Service/pvedaemon.pm | 1 - PVE/Service/pveproxy.pm | 1 - PVE/Service/spiceproxy.pm | 1 - 4 files changed, 1 insertion(+), 8 deletions(-) diff --git a/PVE/HTTPServer.pm b/PVE/HTTPServer.pm

[pve-devel] [PATCH pve-manager 6/9] HTTPServer.pm: call rpcenv->init_request earlier.

2017-01-16 Thread Dietmar Maurer
Because we access file from /etc/pve/ in auth_handler. Signed-off-by: Dietmar Maurer --- PVE/HTTPServer.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/PVE/HTTPServer.pm b/PVE/HTTPServer.pm index 5ae0e97..ab8526b 100755 --- a/PVE/HTTPServer.pm +++

[pve-devel] [PATCH pve-manager 4/9] pveproxy.pm: set title to 'Proxmox VE API'

2017-01-16 Thread Dietmar Maurer
Signed-off-by: Dietmar Maurer --- PVE/Service/pveproxy.pm | 1 + 1 file changed, 1 insertion(+) diff --git a/PVE/Service/pveproxy.pm b/PVE/Service/pveproxy.pm index c354888..a115dcd 100755 --- a/PVE/Service/pveproxy.pm +++ b/PVE/Service/pveproxy.pm @@ -76,6 +76,7 @@ sub

[pve-devel] [PATCH pve-manager 2/9] pveproxy.pm: use PVE::APIServer::AnyEvent::add_dirs()

2017-01-16 Thread Dietmar Maurer
Signed-off-by: Dietmar Maurer --- PVE/Service/pveproxy.pm | 13 + 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/PVE/Service/pveproxy.pm b/PVE/Service/pveproxy.pm index fe133d5..c354888 100755 --- a/PVE/Service/pveproxy.pm +++

[pve-devel] [PATCH pve-manager 0/9] use new libpve-http-server-perl package

2017-01-16 Thread Dietmar Maurer
Dietmar Maurer (9): use new libpve-http-server-perl package pveproxy.pm: use PVE::APIServer::AnyEvent::add_dirs() remove www/bootstrap completely (now in pve-http-server) pveproxy.pm: set title to 'Proxmox VE API' HTTPServer.pm: call $rpcenv-init() when necessary. HTTPServer.pm: call

[pve-devel] applied: [PATCH qemu-server] Set zero $size and continue if volume_resize() returns false

2017-01-16 Thread Wolfgang Bumiller
On Mon, Jan 16, 2017 at 10:45:10AM +0300, Dmitry Petuhov wrote: > Needed for some storages with backing block devices to do oonlinr resize. > > Signed-off-by: Dmitry Petuhov > --- > PVE/QemuServer.pm | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

[pve-devel] applied: [PATCH pve-storage] Fix RBD resize with krbd option enabled.

2017-01-16 Thread Wolfgang Bumiller
On Mon, Jan 16, 2017 at 10:45:09AM +0300, Dmitry Petuhov wrote: > With krbd we resize volume and tell QemuSever to notify running QEMU > with zero $size by returning undef. > > Signed-off-by: Dmitry Petuhov > --- > This patch depends on > [PATCH qemu-server] Set zero