raintank / worldping-api

Worldping Backend Service
Other
25 stars 18 forks source link

don't require exact timestamps in alerting data responses #37

Closed Dieterbe closed 8 years ago

Dieterbe commented 8 years ago

grafana alert executor has sanity checks for the returned data and looks for timestamps the way data was originally saved. but we've since added logic to MT to quantize all timestamps. and want to query MT. I think we can just remove this check from the executor and assume that the output returned by "graphite" (whether graphite-api or MT directly) is correct in that regard.

Dieterbe commented 8 years ago

why is this closed?

woodsaj commented 8 years ago

it was fixed 7months ago. https://github.com/raintank/worldping-api/commit/2489ba7b4f93af4299ac9aae4186704b5d6e30e8

woodsaj commented 8 years ago

Actual a commit is https://github.com/raintank/worldping-api/commit/913a5406e9f695a9a3578d0f2b7e5769cbb29d5e