graphite-dev team mailing list archive

[Nicholas Leskiw <question167755@xxxxxxxxxxxxxxxxxxxxx>]

Question #167755 on Graphite changed:
https://answers.launchpad.net/graphite/+question/167755

Nicholas Leskiw posted a new comment:
This is not part of the rendering code.

in whisper.py lines 634 - 635:

634   if untilTime > now:
635     untilTime = now

You can change this in your install, but I take no responsibility if
this breaks things horribly. I think that simply commenting this out may
seem to work at first, then give you a bunch of invalid data when the
whisper file writes to the end and wraps back around.

As such, please don't submit any changes to trunk for this.

A system would need to be engineered that returns a bunch of None values
into valueList ahead of time.time() for an appropriate period if whisper
detects untilTime > now.

Might I just suggest using a relative time period of the last 24 hours
instead? Maybe split the 6 timeShift lines into a separate graph?

-- 
You received this question notification because you are a member of
graphite-dev, which is an answer contact for Graphite.