[pve-devel] ACK: [PATCH manager] Reflect changed output for 'ceph pg dump -f json'.

[Fabian Grünbichler]

looks good, I will apply this once 12.2.0 is out and we have a
(hopefully stable) final output schema for the various ceph status and
health commands.

On Wed, Aug 23, 2017 at 08:27:45PM +0200, Waschbüsch IT-Services GmbH wrote:
> Reflect changed output for 'ceph pg dump -f json'.
> 
> Signed-off-by: Martin Waschbüsch <service at waschbuesch.it>
> ---
> Without this patch, all osds will show a latency of 0.
> Sadly, that is not true. :-)
> 
> PVE/API2/Ceph.pm | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/PVE/API2/Ceph.pm b/PVE/API2/Ceph.pm
> index c4d6ffcb..f8ee8b21 100644
> --- a/PVE/API2/Ceph.pm
> +++ b/PVE/API2/Ceph.pm
> @@ -122,7 +122,7 @@ __PACKAGE__->register_method ({
> 		$new->{total_space} = ($stat->{kb} || 1) * 1024;
> 		$new->{bytes_used} = ($stat->{kb_used} || 0) * 1024;
> 		$new->{percent_used} = ($new->{bytes_used}*100)/$new->{total_space};
> -		if (my $d = $stat->{fs_perf_stat}) {
> +		if (my $d = $stat->{perf_stat}) {
> 		    $new->{commit_latency_ms} = $d->{commit_latency_ms};
> 		    $new->{apply_latency_ms} = $d->{apply_latency_ms};
> 		}
> -- 
> 2.11.0
> 
> _______________________________________________
> pve-devel mailing list
> pve-devel at pve.proxmox.com
> https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel