Comment 3 for bug 851428

Cool, I'll take a look this weekend.

On Fri, Sep 16, 2011 at 5:00 PM, Scott Smith <email address hidden>wrote:

> OK, uploaded my branch with a fix.
>
> ** Branch linked: lp:~ohlol/graphite/fix-storage-schema-retention-units
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/851428
>
> Title:
> Fix retention units (sec, min, etc.) for storage schemas (located in
> whisper.py) to match those accepted by Carbon and the Webapp
>
> Status in Graphite - Enterprise scalable realtime graphing:
> New
>
> Bug description:
> In whisper.py the units available appear to be:
>
> UnitMultipliers = {
> 's' : 1,
> 'm' : 60,
> 'h' : 60 * 60,
> 'd' : 60 * 60 * 24,
> 'y' : 60 * 60 * 24 * 365,
> }
>
> These need to be fixed to match the ones accepted by the webapp and
> carbon.
>
> This includes changing the detection from an index (currently
> string[-1]) to a regex match ([0-9])([A-Za-z])
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/graphite/+bug/851428/+subscriptions
>