[pve-devel] [PATCH pve-container 0/1] vmstatus: add pressure stats

[pve-devel] [PATCH pve-container 0/1] vmstatus: add pressure stats

[Alexandre Derumier]

need pve-common patches serie

Alexandre Derumier (1):
  vmstatus: add pressure stats

 src/PVE/LXC.pm | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

-- 
2.20.1

[pve-devel] [PATCH pve-container 1/1] vmstatus: add pressure stats

[Alexandre Derumier]

---
 src/PVE/LXC.pm | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/src/PVE/LXC.pm b/src/PVE/LXC.pm
index 5cfcf28..ad1460c 100644
--- a/src/PVE/LXC.pm
+++ b/src/PVE/LXC.pm
@@ -291,6 +291,24 @@ sub vmstatus {
 	} else {
 	    $d->{cpu} = 0;
 	}
+
+	if (defined(my $pressure = $cgroups->get_pressure_stat())) {
+	    $d->{pressure} = $pressure;
+	} else {
+	    $d->{pressure} = {
+		cpu => {
+		    some => { avg10 => 10, avg60 => 0, avg300 => 0 }
+		},
+		memory => {
+		    some => { avg10 => 10, avg60 => 0, avg300 => 0 },
+		    full => { avg10 => 10, avg60 => 0, avg300 => 0 }
+		},
+		io => {
+		    some => { avg10 => 10, avg60 => 0, avg300 => 0 },
+		    full => { avg10 => 10, avg60 => 0, avg300 => 0 }
+		},
+	    };
+	}
     }
 
     my $netdev = PVE::ProcFSTools::read_proc_net_dev();
-- 
2.20.1

Re: [pve-devel] [PATCH pve-container 1/1] vmstatus: add pressure stats

[Thomas Lamprecht]

sorry for the late reply!

On 04.11.20 02:42, Alexandre Derumier wrote:
> ---
>  src/PVE/LXC.pm | 18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
> 
> diff --git a/src/PVE/LXC.pm b/src/PVE/LXC.pm
> index 5cfcf28..ad1460c 100644
> --- a/src/PVE/LXC.pm
> +++ b/src/PVE/LXC.pm
> @@ -291,6 +291,24 @@ sub vmstatus {
>  	} else {
>  	    $d->{cpu} = 0;
>  	}
> +
> +	if (defined(my $pressure = $cgroups->get_pressure_stat())) {
> +	    $d->{pressure} = $pressure;
> +	} else {
> +	    $d->{pressure} = {
> +		cpu => {
> +		    some => { avg10 => 10, avg60 => 0, avg300 => 0 }

Setting avg10 to 10 seems wrong?


> +		},
> +		memory => {
> +		    some => { avg10 => 10, avg60 => 0, avg300 => 0 },
> +		    full => { avg10 => 10, avg60 => 0, avg300 => 0 }
> +		},
> +		io => {
> +		    some => { avg10 => 10, avg60 => 0, avg300 => 0 },
> +		    full => { avg10 => 10, avg60 => 0, avg300 => 0 }
> +		},
> +	    };
> +	}
>      }
>  
>      my $netdev = PVE::ProcFSTools::read_proc_net_dev();
> 

Re: [pve-devel] [PATCH pve-container 1/1] vmstatus: add pressure stats

[aderumier]

Le samedi 06 février 2021 à 15:12 +0100, Thomas Lamprecht a écrit :
> 
> Setting avg10 to 10 seems wrong?
Sorry, I think it's was some debug value when I was testing it.


I have resend both pve-common && pve-container patches.

Thanks for the review !

[pve-devel] [PATCH-SERIES cluster/common/qemu-server/lxc/manager] add new metric stats in single rrd

[Alexandre Derumier]

Hi,

I'm still working on vm balancing/scheduling, and need some new metrics.

This patch-series add new metrics stats in rrd

- cpu,mem,io pressure for qemu/lxc/host
- hostcpu/hostmem cgroup for qemu
- ksm

as we discussed last year, theses new metrics are pushed in single rrd files,
like proxmox backup server.

The paths are :

pve2-metrics/vms/<vmid>/<metric>
pve2-metrics/nodes/<node>/<metric>
pve2-metrics/storages/<node>/<storage>/<metric>



pve-cluster:

Alexandre Derumier (1):
  add pve2-metrics rrd (single metrics)

 data/src/status.c | 51 +++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 51 insertions(+)


pve-common:

Alexandre Derumier (2):
  cgroup: get_pressure_stat: use controllers in get_path
  cgroup: get_pressure_stat: add cpu full pressure

 src/PVE/CGroup.pm | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

qemu-server:

Alexandre Derumier (3):
  vmstatus: add hostcpu value
  vmstatus: add hostmem value
  vmstatus: add pressure stats

 PVE/QemuServer.pm | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

pve-container:

Alexandre Derumier (1):
  vmstatus: add pressure stats

 src/PVE/LXC.pm | 2 ++
 1 file changed, 2 insertions(+)

pve-manager:

Alexandre Derumier (3):
  pvestatd: qemu/lxc/host : broadcast rrd pressure metrics
  pvestatd: qemu: broadcast rrd hostcpu && hostmem metrics
  pvestatd: host: broadcast rrd ksm metric

 PVE/Service/pvestatd.pm | 53 +++++++++++++++++++++++++++++++++++++++++
 1 file changed, 53 insertions(+)

-- 
2.30.2

[pve-devel] [PATCH pve-container 1/1] vmstatus: add pressure stats

[Alexandre Derumier]

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>
---
 src/PVE/LXC.pm | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/PVE/LXC.pm b/src/PVE/LXC.pm
index fe63087..af47ff9 100644
--- a/src/PVE/LXC.pm
+++ b/src/PVE/LXC.pm
@@ -299,6 +299,8 @@ sub vmstatus {
 	} else {
 	    $d->{cpu} = 0;
 	}
+
+	$d->{pressure} = $cgroups->get_pressure_stat();
     }
 
     my $netdev = PVE::ProcFSTools::read_proc_net_dev();
-- 
2.30.2