Kwollectgen

commentaires plutôt cosmétiques:

pour la liste des métriques dans data/, peut-etre les présenter sous forme de hash avec: name => { 'description' => ..., ... } sinon pour source.protocol, peut-etre network_equipment plutot que netdevice. netdevice fait un peu penser à carte réseau

Ok pour netdevice => network_equipement mais pour le nom des métriques ça fait un peu lourd non ? par exemple là on à "netdevice_ifacein_bytes_total". Sinon network_ifacein_bytes_total ?

ah oui pour le nom, network suffit

added 1 commit

• 85ee3b03 - [fix] Do not use netdevice name

Compare with previous version

Corrigé pour netdevice.

Pour le array vs hash, si ce n'est que cosmétique, moi j'aurai tendance à préférer la liste ;)

En plus, c'est raccord avec ce qu'on a pour les disques et les ifaces réseau et on avait regretté recemment d'avoir utilisé un hash pour les gpus...

added 1 commit

• 43fda8f9 - [fix] Add schema validation for metrics

Compare with previous version

Du coup ok pour merger en l'état ? (au pire tant que c'est pas annoncé, on peut tjs changer)

oui pas de pb

closed

mergé après rebase