sahara-dashboard uses UTC time instead of set timezone

Bug #1581435 reported by Anatolii Neliubin
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Mirantis OpenStack
Status tracked in 10.0.x
10.0.x
Invalid
Medium
MOS Sahara
7.0.x
Fix Released
High
Alexey Stupnikov
8.0.x
Won't Fix
Medium
Alexey Stupnikov
9.x
Won't Fix
Medium
MOS Sahara

Bug Description

All time values that are shown in sahara dashboard are in UTC no matter what kind of timezone we have set in settings. It affects the Data Sources, Job Execution detail views and Cluster provision steps table.

implemented in upstream: https://bugs.launchpad.net/sahara/+bug/1563419

MOS-7.0, MU3 applied

Revision history for this message
Roman Podoliaka (rpodolyaka) wrote :
summary: - sahara uses UTC time instead of set timezone
+ sahara-dashboard uses UTC time instead of set timezone
description: updated
tags: added: area-sahara
Revision history for this message
Roman Podoliaka (rpodolyaka) wrote :

No longer fixing Medium bugs in 9.0. Not sure if it's worth it to backport the fixes to previous releases.

Revision history for this message
Bug Checker Bot (bug-checker) wrote : Autochecker

(This check performed automatically)
Please, make sure that bug description contains the following sections filled in with the appropriate data related to the bug you are describing:

actual result

version

expected result

For more detailed information on the contents of each of the listed sections see https://wiki.openstack.org/wiki/Fuel/How_to_contribute#Here_is_how_you_file_a_bug

tags: added: need-info
Revision history for this message
Fabrizio Soppelsa (fsoppelsa) wrote :

Need to backport to 7.0. (customer-found)

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/horizon (openstack-ci/fuel-7.0/2015.1.0)

Fix proposed to branch: openstack-ci/fuel-7.0/2015.1.0
Change author: Vitaly Gridnev <email address hidden>
Review: https://review.fuel-infra.org/21899

Roman Rufanov (rrufanov)
tags: added: support
Revision history for this message
Alexey Stupnikov (astupnikov) wrote :

Setting it to Won't fix for MOS 8.0, since it has Medium Importance and not a customer-found bug. Plus we have other concern here: since this patch hadn't been taken into stable branches of upstream horizon, it could complicate our backporting process.

I will try to push a patch to upstream's Liberty branch.

Revision history for this message
Vitalii Gridnev (vgridnev) wrote :

Alexey, kilo is almost (or already) is EOL-ed. Also, please note that stable/liberty is Phase 2 of support of stable branches, so it means that only Critical and security bugs are accepted. So, I think that bug can't be proposed to stable branches right now.

Revision history for this message
Alexey Stupnikov (astupnikov) wrote :

Vitaly, thanks for the tip! I will send the patch to review and and will look where it will bring me.

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to openstack/horizon (openstack-ci/fuel-7.0/2015.1.0)

Reviewed: https://review.fuel-infra.org/21899
Submitter: Vitaly Sedelnik <email address hidden>
Branch: openstack-ci/fuel-7.0/2015.1.0

Commit: 7a68a456b550e78cb7a8ecc8e0f50ab3681a3fe1
Author: Vitaly Gridnev <email address hidden>
Date: Fri Jun 10 10:00:51 2016

fix timezone issues

Change-Id: Ifde5f26262747ec9ca33fcab9df0599cd311fa09
Closes-bug: 1581435

tags: added: ct-patched-7.0
tags: added: on-verification
Revision history for this message
Alexey Stupnikov (astupnikov) wrote :

This patch was merged in upstream mitaka (bug #1563419). And will be synced to MOS.

https://review.openstack.org/#/c/333587/

Changed in mos:
assignee: MOS Maintenance (mos-maintenance) → Alexey Stupnikov (astupnikov)
milestone: 9.0 → 9.1
Revision history for this message
Ekaterina Shutova (eshutova) wrote :

Verified on MOS 7.0 + mu5 updates.

Checked time on all Data Processing Tabs, especially on Data Sources, Job Execution detail views and Cluster provision steps table. For Job Execution detail views and Cluster provision steps table it is fixed now, but for Data Source Overview time is still in UTC format:
Name
    input
ID
    318238a6-2802-446b-81b9-baf2d6665080
Type
    swift
URL
    swift://es_cont/input
Description
    None
Create time
    2016-08-22T14:52:09

tags: removed: on-verification
Revision history for this message
Ekaterina Shutova (eshutova) wrote :

Verified on MOS 7.0 + mu5 updates.

Time is changed from UTC to selected timezone that we have set in settings on all sahara dashboard tabs incl. Data Source.
Name
    output
ID
    2fe78de1-4e58-49dc-b355-4c92a415673c
Type
    swift
URL
    swift://es_cont/output
Description
    None
Create time
    Aug. 24, 2016, 5:42 p.m.

Revision history for this message
Alexey Stupnikov (astupnikov) wrote :

Moving to invalid for MOS9.1 as the patch [1] was synced from stable/mitaka to 9.0/mitaka.
[1] https://review.openstack.org/#/c/333587/

Changed in mos:
status: Confirmed → Invalid
Revision history for this message
Vitalii Gridnev (vgridnev) wrote :

Why not fix committed?

Revision history for this message
Alexey Stupnikov (astupnikov) wrote :

This is the way we proceed that bugs that was fixed after sync in stable MOS branches. I think this is because we haven't committed anything and the issue is no longer there. I understand that it is also can be closed as Fix Committed, but chose to do it as usual.

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.