search

Tendrl / specifications

Tendrl specs go here
GNU Lesser General Public License v3.0
6 stars 16 forks source link

Gluster metrics #206

Closed anmolbabu closed 6 years ago

anmolbabu commented 7 years ago

Gluster metrics

tendrl-bug-id: Tendrl/specifications#188 Signed-off-by: anmolbabu anmolbudugutta@gmail.com

anmolbabu commented 7 years ago

This is WIP spec with some unknowns: status wise counters for bricks: This brick status is availble in get-state for the brick belonging to the node on which the get-state command is fired.. So there's no way collectd residing on a node can push this counter. The best that can be done from collectd's perspective is to push brick status as a integral number(as ceph-metrics does) to graphite and a way to aggregate this at grafana needs to be found.. Note This is not a mere count query. As the actual status is number encoded in the metric value and not in metric name.. Encoding it in metric name makes no sense as when the status changes, the previous metric and the current metric both co-exist leading to unnecessary confusions

@cloudbehl Please add the grafana side of things either to the same spec or to a different one..

@r0h4n @brainfunked @shtripat @nthomas-redhat @anivargi Please review

anmolbabu commented 7 years ago

https://github.com/Tendrl/specifications/issues/213#issuecomment-318001791

nnDarshan commented 7 years ago

Looks good to me

anmolbabu commented 7 years ago

@r0h4n @nthomas-redhat @shtripat @brainfunked As discussedI have trimmed this spec to deal with milestone1 part I'll raise Gluster Metrics-milestone 2 once this is merged Please review..