Fix timezones in line chart

Bug #1359690 reported by Ladislav Smola
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Medium
Ladislav Smola

Bug Description

Ceilometer gives us sample with UTC datetime, we are just passing that to the line chart, to it should recognise the UTC.

Hover detail should be presented in local time for users.

There might be more enhancements into this, like configurable timezones, but we need at least some quick fix that will work in J. In future we need to figure out how to configure timezones in Ceilometer and how to configure the same in Horizon.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

Fix proposed to branch: master
Review: https://review.openstack.org/115942

Changed in horizon:
assignee: nobody → Ladislav Smola (lsmola)
status: New → In Progress
David Lyle (david-lyle)
Changed in horizon:
importance: Undecided → Medium
milestone: none → juno-rc1
David Lyle (david-lyle)
Changed in horizon:
milestone: juno-rc1 → kilo-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/115942
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=d787eb5e9a4dda7a2a6a493bebff166952ed2640
Submitter: Jenkins
Branch: master

commit d787eb5e9a4dda7a2a6a493bebff166952ed2640
Author: Ladislav Smola <email address hidden>
Date: Thu Aug 21 13:18:44 2014 +0200

    Fix timezones in line chart

    Ceilometer gives us UTC and works with UTC, we should honor that
    in line chart, cause we are just passing samples there.

    Hover detail of sample should show localized time.

    Change-Id: I83c23a13eb27b71e71268bc7b99c0ce0fdd1d59a
    Closes-Bug: #1359690

Changed in horizon:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (stable/juno)

Fix proposed to branch: stable/juno
Review: https://review.openstack.org/134551

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (stable/juno)

Reviewed: https://review.openstack.org/134551
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=9b11e2e7fa3b786617ae83bdb0e801bdf13f5688
Submitter: Jenkins
Branch: stable/juno

commit 9b11e2e7fa3b786617ae83bdb0e801bdf13f5688
Author: Ladislav Smola <email address hidden>
Date: Thu Aug 21 13:18:44 2014 +0200

    Fix timezones in line chart

    Ceilometer gives us UTC and works with UTC, we should honor that
    in line chart, cause we are just passing samples there.

    Hover detail of sample should show localized time.

    Change-Id: I83c23a13eb27b71e71268bc7b99c0ce0fdd1d59a
    Closes-Bug: #1359690
    (cherry picked from commit d787eb5e9a4dda7a2a6a493bebff166952ed2640)

tags: added: in-stable-juno
Thierry Carrez (ttx)
Changed in horizon:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in horizon:
milestone: kilo-1 → 2015.1.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.