Re: [pve-devel] [PATCH storage v2 1/2] fix #1895: use json for 'rbd ls -l' and 'rbd info'

2018-09-06 Thread Alwin Antreich
On Thu, Sep 06, 2018 at 11:35:42AM +0200, Dominik Csapak wrote: > since ceph changed the plain output format for 12.2.8 > we have to change the code anyway, and when were at it, > we can change it to the (hopefully) more robust json output > > Co-authored-by: Alwin Antreich > Signed-off-by: Domin

[pve-devel] applied: [PATCH storage v2 2/2] remove unused function

2018-09-06 Thread Thomas Lamprecht
On 9/6/18 11:35 AM, Dominik Csapak wrote: > since the json output gives the sizes in bytes, we do not > need to convert anymore > > Signed-off-by: Dominik Csapak > --- > new in v2 > > PVE/Storage/RBDPlugin.pm | 7 --- > 1 file changed, 7 deletions(-) > > diff --git a/PVE/Storage/RBDPlugin.

[pve-devel] applied: [PATCH storage v2 1/2] fix #1895: use json for 'rbd ls -l' and 'rbd info'

2018-09-06 Thread Thomas Lamprecht
On 9/6/18 11:35 AM, Dominik Csapak wrote: > since ceph changed the plain output format for 12.2.8 > we have to change the code anyway, and when were at it, > we can change it to the (hopefully) more robust json output > > Co-authored-by: Alwin Antreich > Signed-off-by: Dominik Csapak applied, t

[pve-devel] applied: [PATCH http-server 0/2] fix Build-Depends, use https for Homepage

2018-09-06 Thread Thomas Lamprecht
On 9/6/18 11:43 AM, Rhonda D'Vine wrote: > unzip is needed in the Build-Depends. While adding it I noticed that > it makes sense to switch the Homepage URL directly to https. > > Rhonda D'Vine (2): > Add unzip to Build-Depends > Use https for Homepage URL > > debian/control | 3 ++- > 1 fil

[pve-devel] [PATCH http-server 0/2] fix Build-Depends, use https for Homepage

2018-09-06 Thread Rhonda D'Vine
unzip is needed in the Build-Depends. While adding it I noticed that it makes sense to switch the Homepage URL directly to https. Rhonda D'Vine (2): Add unzip to Build-Depends Use https for Homepage URL debian/control | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) -- 2.11.0 ___

[pve-devel] [PATCH http-server 2/2] Use https for Homepage URL

2018-09-06 Thread Rhonda D'Vine
Signed-off-by: Rhonda D'Vine --- debian/control | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/debian/control b/debian/control index cf9a753..e3bc4c8 100644 --- a/debian/control +++ b/debian/control @@ -6,7 +6,7 @@ Build-Depends: debhelper (>= 9), perl (>= 5.1

[pve-devel] [PATCH http-server 1/2] Add unzip to Build-Depends

2018-09-06 Thread Rhonda D'Vine
Signed-off-by: Rhonda D'Vine --- debian/control | 1 + 1 file changed, 1 insertion(+) diff --git a/debian/control b/debian/control index bb01640..cf9a753 100644 --- a/debian/control +++ b/debian/control @@ -4,6 +4,7 @@ Priority: optional Maintainer: Proxmox Support Team Build-Depends: debhelp

Re: [pve-devel] [PATCH storage v2 1/2] fix #1895: use json for 'rbd ls -l' and 'rbd info'

2018-09-06 Thread Dominik Csapak
On 09/06/2018 11:35 AM, Dominik Csapak wrote: since ceph changed the plain output format for 12.2.8 we have to change the code anyway, and when were at it, we can change it to the (hopefully) more robust json output Co-authored-by: Alwin Antreich Signed-off-by: Dominik Csapak --- changes from

[pve-devel] [PATCH storage v2 2/2] remove unused function

2018-09-06 Thread Dominik Csapak
since the json output gives the sizes in bytes, we do not need to convert anymore Signed-off-by: Dominik Csapak --- new in v2 PVE/Storage/RBDPlugin.pm | 7 --- 1 file changed, 7 deletions(-) diff --git a/PVE/Storage/RBDPlugin.pm b/PVE/Storage/RBDPlugin.pm index 6c0e617..c3b0cd9 100644 ---

[pve-devel] [PATCH storage v2 1/2] fix #1895: use json for 'rbd ls -l' and 'rbd info'

2018-09-06 Thread Dominik Csapak
since ceph changed the plain output format for 12.2.8 we have to change the code anyway, and when were at it, we can change it to the (hopefully) more robust json output Co-authored-by: Alwin Antreich Signed-off-by: Dominik Csapak --- changes from v1: * incorporated the change from alwins patch

[pve-devel] applied: [PATCH lxc 1/1] Add debian/SOURCE to docs to install

2018-09-06 Thread Thomas Lamprecht
On 9/6/18 9:22 AM, Rhonda D'Vine wrote: > The lxc packages were missing the SOURCE file in the docs. > > Signed-off-by: Rhonda D'Vine > --- > debian/docs | 1 + > debian/lxc-pve-dev.docs | 1 + > 2 files changed, 2 insertions(+) > create mode 100644 debian/lxc-pve-dev.docs > > diff

[pve-devel] [PATCH lxc 0/1] Add debian/SOURCE to docs to install

2018-09-06 Thread Rhonda D'Vine
The lxc packages were missing the SOURCE file. This patch adds them. Rhonda D'Vine (1): Add debian/SOURCE to docs to install debian/docs | 1 + debian/lxc-pve-dev.docs | 1 + 2 files changed, 2 insertions(+) create mode 100644 debian/lxc-pve-dev.docs -- 2.11.0 ___

[pve-devel] [PATCH lxc 1/1] Add debian/SOURCE to docs to install

2018-09-06 Thread Rhonda D'Vine
The lxc packages were missing the SOURCE file in the docs. Signed-off-by: Rhonda D'Vine --- debian/docs | 1 + debian/lxc-pve-dev.docs | 1 + 2 files changed, 2 insertions(+) create mode 100644 debian/lxc-pve-dev.docs diff --git a/debian/docs b/debian/docs index e845566..310840d 10