moira icon indicating copy to clipboard operation
moira copied to clipboard

[ plotting ] Empty plots are possible

Open kamaev opened this issue 6 years ago • 8 comments

Moira renders metric plots with every notification (if delivery channel supports this). However, from time to time we see empty images in notifications. Empty images can be caused by bugs in our code or by incorrect or non-existent data in metric storage. Bugs must be fixed, and in case of incorrect or non-existent data we should not send an image at all.

Steps to fix:

  • Do not use buff.bytes() for NotificationPackage when Render() fails, use []byte{} instead.
  • [limits] Validate ranges by using math.IsInf.
  • [limits/resolveLimits] Remove or refactor MinAndMaxOfTime to provide valid and different from/to's.
  • Add tests for different retention intervals.

kamaev avatar Feb 12 '19 19:02 kamaev

metric.txt Metric with 10s retention interval (default is 60s) on which we reproduced "infinite {x,y}-range delta" error bug.

borovskyav avatar Feb 12 '19 19:02 borovskyav

Main goal is to avoid errors like infinite {x,y}-range delta from wcharczuk/go-chart

kamaev avatar Feb 28 '19 12:02 kamaev

Hi i would like to work on this issue.is anyone working on this?

gane5hvarma avatar Mar 03 '19 14:03 gane5hvarma

@gane5hvarma OK, you can take this task.

beevee avatar Mar 04 '19 06:03 beevee

@beevee I'll be happy to take this one.

dr0pdb avatar Mar 02 '20 23:03 dr0pdb

@beevee I'll be happy to take this one.

Yeah, sure you can tackle this. But we don't assign tasks to new contributors anymore, since they disappear more often than not. Send us a pull request, and we'll review it.

beevee avatar Mar 03 '20 05:03 beevee

@srv-twry are you still on it ? If not, then i picking this up

hv7214 avatar Mar 09 '20 07:03 hv7214

Sure feel free. I've closed my PR

dr0pdb avatar Mar 09 '20 15:03 dr0pdb