ODS et perfdata avec des espaces ? bis

[boyerf]

Suite au thread (fermé) de Dontkishoot sur "ODS et perfdata avec des espaces ? "
Je suis en train d'essayer de grapher avec un plugin de Manubullon (check_snmp_storage) mais je ne remonte pas ni les metrics ni les donnees dans data_bin (par contre la declaration est bonne dans index_data). J'ai bien viré la quote dans les carateres macro interdit...

As tu une autre piste ?

[DonKiShoot]

Vas dans le bug track j'y ai mis la solution :

http://bugs.oreon-project.org/?do=details&id=399

[boyerf]

Ok, j'ai éffectué ta modif mais apres re-test, j'ai toujours le même problème...
Est ce la seule ligne que tu as modifiée ?

Autrement, j'ai bien purgé mes 2 services de test dans ODS et ils sont bien recréés mais sans metric (et donc sans data). J'ai bien vérifié, les données de perf arrivent bien dans le fichier perfdata avec des simples quotes...

Hum y'a une c.... quelque part...:confused:

[DonKiShoot]

J'ai ajouté un quote dans la regex et ajouté une subsitution à $data[0]

Il y a 2 modif à faire puis :

stoper ODS

purger le service en panne

redemarrer ODS

[boyerf]

Oups... je n'avais pas vu le ' dans la regex.... desole.
Mais tu rajoutes seulement ' ?
ou faut il rajouter \' ?

Bon je vais retester (merci pour ton aide !)

[boyerf]

Bon, dans mes metric, il me prend toujours le dernier mot du metric...
Dans 'CPU - Minimal' il me prend comme metric : minimal

ATTENTION, j'ai vu un effet pervers de ta modif. Pour ceux qui utilisent des pluggins Centreon qui envoient des données Perfparse et Centreon en donnée de performance du type :
données_format_perfparse|Données_format_centreon

Du coup, pour ce type de pluggins, les données perfparse sont prisent en compte et plus les données centreon...

Du coup, ça foire toutes les courbes etablient avec les data centreon !

[DonKiShoot]

Je savais pas que ca existait des plugins qui sortaient des perfdata classique et des perfdatas centreons :eek:

PS: Peut-être rajouté dans la regex : '\ ' (sans les quotes) pour prendre en compte les espaces.

Je vérifie chez moi car ça semblait fonctionner sans ça même si j'y avais pensé.

[DonKiShoot]

Exact j'ai le même bug .
Je vais tenter ma modif .

if ($tab =~ /([a-zA-Z0-9\_\-\/\.\:'\ ]+)\=([0-9\.\,\-]+)([a-zA-Z0-9\_\-\/\\\%]*)[\;]*([0-9\.\,\-]*)[\;]*([0-9\.\,\-]*)[\;]*([0-9\.\,\-]*)[\;]*([0-9\.\,\-]*)/){
if (!defined($3)){$3 = "";}
if (!defined($4)){$4 = "";}
if (!defined($5)){$5 = "";}
@data = ($1, $2, $3, $4, $5); # metric, value, unit, warn, critical
}
if ($1 && defined($2)){
# Check if metric is known...
$data[0] =~ s/\//#S#/g;
$data[0] =~ s/\./\-/g;
$data[0] =~ s/\,/\-/g;
$data[0] =~ s/\:/\-/g;
$data[0] =~ s/\'//g;

[boyerf]

./check_centreon_nt -H xx.xx.xx.xx -p 12489 -v USEDDISKSPACE -s password -l C -w 75 -c 95

C: - total: 19,53 Gb - utilisé: 11,20 Gb (57 pct) - libre 8,34 Gb (43 pct) | 'C: Espace utilisé'=11,20Gb;14,65;18,56;0.00;19,53 |total=19,53Mo used=11,20Mo free=8,34Mo

Regarde le retour de ce pluggins.....

[DonKiShoot]

En effet il est louche ce plugin mais il ne devait pas fonctionner pour la partie perfdata car

'C: Espace utilisé'=

ODS ne sait pas le traiter sans ma modif.

PS: Son unité est le Gb cela me semble fort étonnant pour un espace disque.
Soit Go soit GB mais pas Gb.
Soit GigaOctet soit GigaByte mais pas Gigabit.

[boyerf]

En effet il est louche ce plugin mais il ne devait pas fonctionner pour la partie perfdata car

'C: Espace utilisé'=

ODS ne sait pas le traiter sans ma modif.

C'est bien ce que je te dis... il est traité avec ta modif
Ce qui fait que tous ceux qui utilisent des pluggins Centreon vont avoir des gros problemes de courbes...

[DonKiShoot]

ODS est censé traité les perfdatas officiels, cela me laisse perplexe c'est data supplémentaire derrière les perfdatas normal !!!

[DonKiShoot]

Bon en rajoutant un espace dans la regex j'arrive à obtenir à peu près tout le nom de la métric mais il y a encore une partie de tronquer.
Je vais chercher où elle est tronquer.

[boyerf]

Ca parrait dingue, mais ca fonctionne comme ça avec le Check_centreon_nt

[DonKiShoot]

Je crois que c bon maintenant mais ca oblige à un controle supplémentaire.
J'ai ajouté le traitement ci dessous pour spliter les espaces de façon différentes selon si nous sommes en présence de quotes ou non :

my @Tab;
if ( $_[0] =~ /\s'/ ) {
@Tab = split(' \'', $_[0]);
} else {
@Tab = split(' ', $_[0]);
}

foreach my $tab (@Tab) {
# Cut perfdata
if ($tab =~ /([a-zA-Z0-9\_\-\/\.\:'\s]+)\=([0-9\.\,\-]+)([a-zA-Z0-9\_\-\/\\\%]*)[\;]*([0-9\.\,\-]*)[\;]*([0-9\.\,\-]*)[\;]*([0-9\.\,\-]*)[\;]*([0-9\.\,\-]*)/){
if (!defined($3)){$3 = "";}
if (!defined($4)){$4 = "";}
if (!defined($5)){$5 = "";}
@data = ($1, $2, $3, $4, $5); # metric, value, unit, warn, critical
}
if ($1 && defined($2)){
# Check if metric is known...
$data[0] =~ s/\//#S#/g;
$data[0] =~ s/\./\-/g;
$data[0] =~ s/\,/\-/g;
$data[0] =~ s/\:/\-/g;
$data[0] =~ s/\'//g;