pencil: add support for epoch seconds to from= parameter

RESOLVED FIXED

Status

Cloud Services
Operations: Metrics/Monitoring
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: atoll, Assigned: whd)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [qa+])

(Reporter)

Description

6 years ago
http://pencil-global.phx.weave.mozilla.com/dash/scl2/syncstorage/sync_zeus_qps_5xx?start=1338270028&duration=1+hour

This graph should show me May 28th at 22:30 PST8PDT (-0700).  Instead it appears to show months of time.


I think this is related to passing epoch times to Chronic.parse().  Pencil code [1] says:

@stime = Chronic.parse(start)

Which ends up in the Chronic tokenizer [2], which maybe doesn't understand epoch times?


[1] https://github.com/fetep/pencil/blob/a32a998/lib/pencil.rb#L43
[2] https://github.com/mojombo/chronic/blob/0a8193c/lib/chronic/chronic.rb#L286
(Assignee)

Updated

6 years ago
Assignee: nobody → wdawson
(Assignee)

Comment 1

6 years ago
https://github.com/fetep/pencil/commit/9d0ee0385edd7be19b514e64d3050d2b9a0ad4be
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [qa+]
You need to log in before you can comment on or make changes to this bug.