[pve-devel] applied: [PATCH pve-client v3] Add a simple implementation of list

2018-06-11 Thread Dietmar Maurer
applied ___ pve-devel mailing list pve-devel@pve.proxmox.com https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [PATCH pve-client] Config.pm: add new defaults sections

2018-06-11 Thread Dietmar Maurer
And implement a new command to setup defaults. Signed-off-by: Dietmar Maurer --- PVE/APIClient/Commands/help.pm | 2 + PVE/APIClient/Commands/remote.pm | 11 +-- PVE/APIClient/Config.pm | 189 --- pveclient| 2 + 4

[pve-devel] applied: [PATCH client 1/3] config: typo fixup

2018-06-11 Thread Dietmar Maurer
applied > On June 11, 2018 at 11:30 AM Wolfgang Bumiller wrote: > > > Signed-off-by: Wolfgang Bumiller > --- > PVE/APIClient/Config.pm | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/PVE/APIClient/Config.pm b/PVE/APIClient/Config.pm > index 8fa7691..25ed56b 100644 >

[pve-devel] applied: [PATCH client 3/3] lxc enter: improve escape key handling

2018-06-11 Thread Dietmar Maurer
applied ___ pve-devel mailing list pve-devel@pve.proxmox.com https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] [PATCH client 2/3] config: use more xdg compliant search path

2018-06-11 Thread Dietmar Maurer
I don't really like the idea to use different file names: .config client .pveclient ??? > +# If XDG_CONFIG_HOME is not set or empty, its default is $HOME/.config > +my $base = $xdg || "$home/.config"; > + > +my $filename = "$base/pve/client"; > +return $filename if -e

[pve-devel] need help to package frr routing software in proxmox way

2018-06-11 Thread Alexandre DERUMIER
Hi, I'm currently working to package frr https://github.com/FRRouting/frr in proxmox way (with git submodule) official debian build infos are here: https://github.com/FRRouting/frr/blob/8d659e9168063897f5a475f78162f0e63889c964/doc/developer/packaging-debian.rst But I'm not sure how to

Re: [pve-devel] need help to package frr routing software in proxmox way

2018-06-11 Thread Thomas Lamprecht
Hi, On 6/11/18 8:15 AM, Alexandre DERUMIER wrote: > Hi, > > I'm currently working to package frr > https://github.com/FRRouting/frr > > in proxmox way (with git submodule) > > > official debian build infos are here: > >

[pve-devel] applied: [PATCH qemu-server 1/2] cloud-init: nocloud v1: set ip and netmask instead of cidr

2018-06-11 Thread Thomas Lamprecht
On 6/6/18 4:13 PM, Wolfgang Bumiller wrote: > because of centos7's broken cloud-init version > > Signed-off-by: Wolfgang Bumiller > --- > PVE/QemuServer/Cloudinit.pm | 15 --- > 1 file changed, 12 insertions(+), 3 deletions(-) > applied both patches

Re: [pve-devel] need help to package frr routing software in proxmox way

2018-06-11 Thread René Jochum
Hi Alexandre, do you know Bird? http://bird.network.cz/ I had some success with it in the past. Kind regards, René On 2018-06-11 08:15, Alexandre DERUMIER wrote: > Hi, > > I'm currently working to package frr > https://github.com/FRRouting/frr > > in proxmox way (with git submodule) > > >

Re: [pve-devel] [PATCH pve-client] Add 'lxc list'

2018-06-11 Thread Dietmar Maurer
comments inline > On June 8, 2018 at 9:44 AM René Jochum wrote: > > > --- > PVE/APIClient/Commands/lxc.pm | 82 > ++- > PVE/APIClient/Helpers.pm | 14 > 2 files changed, 95 insertions(+), 1 deletion(-) > > diff --git

Re: [pve-devel] [PATCH qemu-server 3/8] implement set-user-password guest agent api call

2018-06-11 Thread Thomas Lamprecht
On 6/7/18 1:16 PM, Dominik Csapak wrote: > this executes the guest agent command 'set-user-password' > with which one can change the password of an existing user in the vm > > Signed-off-by: Dominik Csapak > --- > PVE/API2/Qemu/Agent.pm | 68 > --

Re: [pve-devel] [PATCH qemu-server 2/8] add Agent helper package

2018-06-11 Thread Thomas Lamprecht
Looks OK, few nits inline On 6/7/18 1:16 PM, Dominik Csapak wrote: > Signed-off-by: Dominik Csapak > --- > PVE/API2/Qemu/Agent.pm | 5 ++--- > PVE/QemuServer/Agent.pm | 48 > PVE/QemuServer/Makefile | 1 + > 3 files changed, 51 insertions(+),

[pve-devel] applied: [PATCH qemu-server 1/8] refactor Makefile for PVE/QemuServer/ directory

2018-06-11 Thread Thomas Lamprecht
On 6/7/18 1:16 PM, Dominik Csapak wrote: > this makes it easier to add new files there > > Signed-off-by: Dominik Csapak > --- > PVE/QemuServer/Makefile | 16 +--- > 1 file changed, 9 insertions(+), 7 deletions(-) > > diff --git a/PVE/QemuServer/Makefile b/PVE/QemuServer/Makefile >

[pve-devel] [PATCH pve-common] api_dump: add $raw_dump options

2018-06-11 Thread Dietmar Maurer
Allow to return the original tree with all refs. We use this with our new pveclient which needs the full api definition. Keeping refs makes it possible to store the tree more efficiently. Signed-off-by: Dietmar Maurer --- src/PVE/RESTHandler.pm | 21 - 1 file changed, 12

Re: [pve-devel] [PATCH qemu-server 4/8] add 'set-user-password' to qm

2018-06-11 Thread Thomas Lamprecht
On 6/7/18 1:16 PM, Dominik Csapak wrote: > and implement read_password for qm.pm > > Signed-off-by: Dominik Csapak > --- > PVE/CLI/qm.pm | 10 ++ > 1 file changed, 10 insertions(+) > > diff --git a/PVE/CLI/qm.pm b/PVE/CLI/qm.pm > index c017a59..ee52c4b 100755 > --- a/PVE/CLI/qm.pm >

[pve-devel] [PATCH client 1/3] config: typo fixup

2018-06-11 Thread Wolfgang Bumiller
Signed-off-by: Wolfgang Bumiller --- PVE/APIClient/Config.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/PVE/APIClient/Config.pm b/PVE/APIClient/Config.pm index 8fa7691..25ed56b 100644 --- a/PVE/APIClient/Config.pm +++ b/PVE/APIClient/Config.pm @@ -42,7 +42,7 @@ my

[pve-devel] [PATCH storage v3 1/5] Add missing dependency to ceph-common

2018-06-11 Thread Alwin Antreich
Signed-off-by: Alwin Antreich --- debian/control | 1 + 1 file changed, 1 insertion(+) diff --git a/debian/control b/debian/control index 2cf585a..908dd24 100644 --- a/debian/control +++ b/debian/control @@ -27,6 +27,7 @@ Depends: cstream, thin-provisioning-tools, udev,

[pve-devel] [PATCH client 3/3] lxc enter: improve escape key handling

2018-06-11 Thread Wolfgang Bumiller
The escape key should not be passed through, this now only happens when pressing the escape key twice (as it is with tmux, screen, ssh, etc.). Also prepared the code to handle a configurable escape key, and restructured the handling to more easily allow adding more keys. For instance we could add

[pve-devel] [PATCH client 2/3] config: use more xdg compliant search path

2018-06-11 Thread Wolfgang Bumiller
Read configs from XDG_CONFIG_HOME/pveclient/config first. Use a default of $HOME/.config for XDG_CONFIG_HOME if it is not set, and only then try $HOME/.pveclient. Signed-off-by: Wolfgang Bumiller --- PVE/APIClient/Config.pm | 21 ++--- 1 file changed, 18 insertions(+), 3

[pve-devel] [PATCH manager v3 2/2] Add cephfs to allowed storages for vzdump backup

2018-06-11 Thread Alwin Antreich
Signed-off-by: Alwin Antreich --- PVE/VZDump.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/PVE/VZDump.pm b/PVE/VZDump.pm index a0376ef9..7fc69f98 100644 --- a/PVE/VZDump.pm +++ b/PVE/VZDump.pm @@ -223,7 +223,7 @@ sub storage_info { die "can't use storage type

[pve-devel] [PATCH manager v3 1/2] Cephfs storage wizard

2018-06-11 Thread Alwin Antreich
Add internal and external storage wizard for cephfs Signed-off-by: Alwin Antreich --- www/manager6/Makefile | 1 + www/manager6/Utils.js | 10 ++ www/manager6/storage/CephFSEdit.js | 71 ++ 3 files changed, 82 insertions(+)

[pve-devel] [PATCH storage v3 4/5] Call methods from CephStorageTools package

2018-06-11 Thread Alwin Antreich
Replace methods with those merged into CephStorageTools.pm Signed-off-by: Alwin Antreich --- PVE/Storage/RBDPlugin.pm | 55 +--- 1 file changed, 5 insertions(+), 50 deletions(-) diff --git a/PVE/Storage/RBDPlugin.pm b/PVE/Storage/RBDPlugin.pm index

[pve-devel] [PATCH storage v3 5/5] Add simple keyring check for cephfs/rbd

2018-06-11 Thread Alwin Antreich
Signed-off-by: Alwin Antreich --- PVE/CephStorageTools.pm | 33 + 1 file changed, 33 insertions(+) diff --git a/PVE/CephStorageTools.pm b/PVE/CephStorageTools.pm index 4bbfc55..59c7be9 100644 --- a/PVE/CephStorageTools.pm +++ b/PVE/CephStorageTools.pm @@ -5,6

[pve-devel] [PATCH storage v3 3/5] Merge RBD and CephFS code into a helper module

2018-06-11 Thread Alwin Antreich
Some methods for connecting to a ceph cluster are the same for RBD and CephFS, these are merged into the helper modules. Signed-off-by: Alwin Antreich --- PVE/CephStorageTools.pm | 55 + PVE/Makefile| 1 + 2 files changed, 56

[pve-devel] [PATCH storage v3 2/5] Cephfs storage plugin

2018-06-11 Thread Alwin Antreich
- ability to mount through kernel and fuse client - allow mount options - get MONs from ceph config if not in storage.cfg - allow the use of ceph config with fuse client Signed-off-by: Alwin Antreich --- PVE/API2/Storage/Config.pm | 2 +- PVE/Storage.pm | 2 +

[pve-devel] [PATCH v3 storage/manager] Cephfs storage plugin

2018-06-11 Thread Alwin Antreich
This patch series is an update and adds the Cephfs to our list of storages. You can mount the storage through the kernel or fuse client. The plugin for now allows all content formats, but this needs further testing. Config and keyfile locations are the same as in the RBD plugin. Example entry:

[pve-devel] [PATCH pve-client v3] Add a simple implementation of list

2018-06-11 Thread René Jochum
Signed-off-by: René Jochum --- This removes all the logic from previous patches where I quieried /config from each VM/LXC. PVE/APIClient/Commands/help.pm | 2 ++ PVE/APIClient/Commands/list.pm | 58 ++ PVE/APIClient/Commands/lxc.pm | 21

Re: [pve-devel] [PATCH qemu-server 5/8] implement agent exec api call

2018-06-11 Thread Thomas Lamprecht
On 6/7/18 1:16 PM, Dominik Csapak wrote: > this imitates the qemu-guest-agent interface > with an 'exec' api call which returns a pid > and an 'exec-status' api call which takes a pid > > the command for the exec call is given as an 'alist' > which means that when using we have to give the

Re: [pve-devel] [PATCH qemu-server 6/8] add exec(-status) to qm

2018-06-11 Thread Thomas Lamprecht
On 6/7/18 1:16 PM, Dominik Csapak wrote: > on the commandline the implementation for exec is a bit different > because there we want (by default) to wait for the result, > as opposed to the api, where it is enough to return the pid and > let the client handle the polling > > this behaviour is

Re: [pve-devel] need help to package frr routing software in proxmox way

2018-06-11 Thread Alexandre DERUMIER
>>do you know Bird? http://bird.network.cz/ I had some success with it in >>the past. yes,sure. but I need frr for vxlan - bgp evpn. (This is the only opensource implementation) - Mail original - De: "Rene Jochum" À: "pve-devel" Envoyé: Lundi 11 Juin 2018 10:18:11 Objet: Re:

Re: [pve-devel] need help to package frr routing software in proxmox way

2018-06-11 Thread Alexandre DERUMIER
> They are some bootstraping to generate the /debian directory with build rules. >>what needs to be done here, roughly? >>Isn't it enough to make a softlink from debian -> debianpkg in the >>build directory? I'm currently looking inside the bootstrap, but I seem that some files are generated

Re: [pve-devel] [PATCH qemu-server 7/8] implement file-read api call via guest-agent

2018-06-11 Thread Thomas Lamprecht
On 6/7/18 1:16 PM, Dominik Csapak wrote: > this api call reads a file via the guest agent, > (in 1MB chunks) but is limited to 16MiB (for now) > > if the file is bigger, the output gets truncated and a > 'truncated' flag is set in the return object > > Signed-off-by: Dominik Csapak > --- >

Re: [pve-devel] [PATCH qemu-server 8/8] implement file-write via guest-agent in the api

2018-06-11 Thread Thomas Lamprecht
On 6/7/18 1:16 PM, Dominik Csapak wrote: > writes the given content to the file > > the size is at the moment limited by the max post size of the > pveproxy/daemon > > Signed-off-by: Dominik Csapak > --- > PVE/API2/Qemu/Agent.pm | 51 > ++ > 1