Closed Bug 759552 Opened 12 years ago Closed 12 years ago

pencil: add support for epoch seconds to from= parameter

Categories

(Cloud Services :: Operations: Metrics/Monitoring, task)

Product:
Cloud Services
Component:
Operations: Metrics/Monitoring
Platform:
x86
macOS
Type:
task
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: Atoll, Assigned: whd)

Details

(Whiteboard: [qa+]) 

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: nobody → wdawson
https://github.com/fetep/pencil/commit/9d0ee0385edd7be19b514e64d3050d2b9a0ad4be
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [qa+]
You need to log in before you can comment on or make changes to this bug.