Re: [pve-devel] [PATCH storage 1/2] Fix: 1542 - show storage utilization per pool, not per global

2017-11-16 Thread Alwin Antreich
the math manually, and you don't known the > replication factor without calling manually ceph api or commands. > > > > > - Mail original - > De: "dietmar" > À: "aderumier" , "pve-devel" > Envoyé: Mercredi 15 Novembre 2017 17:21:0

Re: [pve-devel] [PATCH storage 1/2] Fix: 1542 - show storage utilization per pool, not per global

2017-11-16 Thread Fabian Grünbichler
On Wed, Nov 15, 2017 at 03:47:27PM +0100, Alwin Antreich wrote: > Signed-off-by: Alwin Antreich > --- > PVE/Storage/RBDPlugin.pm | 40 +++- > 1 file changed, 19 insertions(+), 21 deletions(-) > > diff --git a/PVE/Storage/RBDPlugin.pm b/PVE/Storage/RBDPlugin.pm

Re: [pve-devel] [PATCH storage 1/2] Fix: 1542 - show storage utilization per pool, not per global

2017-11-15 Thread Thomas Lamprecht
comments inline. On 11/15/2017 03:47 PM, Alwin Antreich wrote: > Signed-off-by: Alwin Antreich > --- > PVE/Storage/RBDPlugin.pm | 40 +++- > 1 file changed, 19 insertions(+), 21 deletions(-) > > diff --git a/PVE/Storage/RBDPlugin.pm b/PVE/Storage/RBDPlugin.pm

Re: [pve-devel] [PATCH storage 1/2] Fix: 1542 - show storage utilization per pool, not per global

2017-11-15 Thread Alexandre DERUMIER
api or commands. - Mail original - De: "dietmar" À: "aderumier" , "pve-devel" Envoyé: Mercredi 15 Novembre 2017 17:21:05 Objet: Re: [pve-devel] [PATCH storage 1/2] Fix: 1542 - show storage utilization per pool, not per global Wouldn't that be tota

Re: [pve-devel] [PATCH storage 1/2] Fix: 1542 - show storage utilization per pool, not per global

2017-11-15 Thread Dietmar Maurer
Wouldn't that be totally confusing if you have several pools with different replication factor? > Could it be possible to use replication factor from pool config to display > used and free space ? > > (/3 if size=3 for example) > > We didn't do it before, but I think this was because we monitor

Re: [pve-devel] [PATCH storage 1/2] Fix: 1542 - show storage utilization per pool, not per global

2017-11-15 Thread Alexandre DERUMIER
- Mail original - De: "Alwin Antreich" À: "pve-devel" Envoyé: Mercredi 15 Novembre 2017 15:47:27 Objet: [pve-devel] [PATCH storage 1/2] Fix: 1542 - show storage utilization per pool, not per global Signed-off-by: Alwin Antreich ---

[pve-devel] [PATCH storage 1/2] Fix: 1542 - show storage utilization per pool, not per global

2017-11-15 Thread Alwin Antreich
Signed-off-by: Alwin Antreich --- PVE/Storage/RBDPlugin.pm | 40 +++- 1 file changed, 19 insertions(+), 21 deletions(-) diff --git a/PVE/Storage/RBDPlugin.pm b/PVE/Storage/RBDPlugin.pm index decfbf5..14386c4 100644 --- a/PVE/Storage/RBDPlugin.pm +++ b/PVE/Stor