[pve-devel] Graphite.pm needs a correction

Alexey Kuzmin al.m.kuzmin at gmail.com
Wed Apr 27 13:55:26 CEST 2016


Hello,

Carbon (particularly carbon-c-relay) expects one metric per line. Current
PVE implementation breaks this rule. The following patch corrects this bug.
I also created a PR: https://github.com/proxmox/pve-manager/pull/6


Code:

--- PVE/Status/Graphite.pm.orig  2016-04-26 20:03:02.961141497 +0000
+++ PVE/Status/Graphite.pm  2016-04-26 20:03:14.541705841 +0000
@@ -102,7 +102,7 @@
  if ( ref $value eq 'HASH' ) {
  write_graphite($carbon_socket, $value, $ctime, $path);
  }else {
-  $carbon_socket->send( "$path $value $ctime" );
+  $carbon_socket->send( "$path $value $ctime\n" );
  }
  }
  $path = $oldpath;


More information about the pve-devel mailing list