search

snuids / trafficlights-panel

A Grafana version of the Kibana Traffic Lights Panel
MIT License
22 stars 13 forks source link

feat: add setting noValueNumber to turn off the light and display N/A when metric is equal to this value #29

Closed slickArgonian closed 2 years ago

slickArgonian commented 2 years ago

Sometimes the monitored metric doesn't have a value. This can be not a problem and we don't want to give a false information by putting a color on the light and displaying a default and false numeric value, so I propose the feature of turning off the light.

snuids commented 2 years ago

Merci pour le fix. C'est marrant j'ai pas mal bossé en Algo Genetic en début de carrière. Je connaissais pas jenetics.

Je n'ai plus vraiment le temps de m'occuper des plugins Grafana donc si tu as envie d'être responsable avec moi de celui-ci. Je t'invite comme commiter atitré.

Bonne journée.

slickArgonian commented 2 years ago

J'ai pas mal joué avec les algos génétiques en projet persos et pendant mes études, j'ai pu utiliser Jenetics pendant un stage. Très solide pour en faire avec Java.

Je veux bien en être responsable avec toi, mais je ne pense pas avoir une activité forte sur le plugin. En tout cas, je vais essayer de release la nouvelle version sur grafana plugins.

snuids commented 2 years ago

Voilà tu es invitée.

J'ai deux autres plugins grafana. J'en ai un que j'ai signé pour l'avoir dans la bibliothèque grafana. C'est celui avec le svg. C'est un peu la croix et la bannière pour tout aligner et que le plugin soit listé chez eux. Si tu as le courage c'est cool. Si tu as des questions je l'ai déjà fait.

A+

Arnaud

slickArgonian commented 2 years ago

Je suis en train de regarder la procédure pour mettre à jour le plugin (https://github.com/grafana/grafana-plugin-repository#maintain-your-plugin) mais comme tu es propriétaire de celui-ci je ne peux rien faire. Est-ce que je le publie comme nouveau plugin par moi-même ou bien tu prends le temps de le mettre à jour ? J'ai les infos prêtes pour remplir les cases si il faut.

snuids commented 2 years ago

Arg je peux vite remplir les infos si tu les as

snuids commented 2 years ago
  Grafana Labs Support (Grafana) Nov 10, 2021, 10:51 AM EST Community Pipeline Completed: WarningPlugin has warnings but can be reviewed.Community Pipeline ReportPlugin : snuids-trafficlights-panelVersion: 1.5.0Current State: https://grafana.com/plugins/snuids-trafficlights-panelWARNING: 0 (manifest:) Rule 0: warning (unsigned-plugin) unsigned plugin

Bref faut attendre qu'ils valident.

A+

slickArgonian commented 2 years ago

Up, grafana a-t-il abandonné la mise à jour?

snuids commented 2 years ago

Nope, ils ont répondu cela:

Marcus Olsson (Grafana) Nov 29, 2021, 9:18 AM EST Hi Arnaud,I've reviewed and approved your submission, but we need you to sign the plugin before we can publish it.For more information on how to sign a plugin, refer to Sign a plugin.Let me know if you have any questions!Thanks,Marcus

snuids commented 2 years ago

En gros il faut se faire la procédure ici:

https://grafana.com/docs/grafana/latest/developers/plugins/sign-a-plugin/

Je voulais faire ça pendant les vacances de Noel, mais j'ai oublié. J'avais déjà fait cela pour mon plugin SVG. C'est assez lourd.