No start and end time in transaction data

Bug #1593753 reported by Julia Aranovich
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Vitaly Kramskikh
Mitaka
Fix Released
High
Bulat Gaifullin

Bug Description

Fuel UI wants to show a list of a particular cluster deployments. Each item in the list should include deployment transaction id and the start time. BUT there is no time_start and time_end attributes in
GET /api/transactions/<transaction_id> response:

{
   "result":{},
   "uuid":"b961ee6c-bf7c-4ccc-a02e-7a0a874da67c",
   "name":"deployment",
   "cluster":1,
   "message":"Deployment of environment '1' is done.",
   "progress":100,
   "status":"ready",
   "id":7
}

time_start and time_end attributes are needed to render a timeline for a deployment.

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

expected result

steps to reproduce

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
Dmitry Klenov (dklenov)
Changed in fuel:
assignee: nobody → Fuel Sustaining (fuel-sustaining-team)
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
assignee: Fuel Sustaining (fuel-sustaining-team) → Roman Prykhodchenko (romcheg)
summary: - No start time in transaction data
+ No start and end time in transaction data
description: updated
Revision history for this message
Julia Aranovich (jkirnosova) wrote :

Priority raised to High as this issue affects enough https://blueprints.launchpad.net/fuel/+spec/ui-deployment-history development.

Changed in fuel:
importance: Medium → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

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

Changed in fuel:
status: Confirmed → In Progress
Dmitry Pyzhov (dpyzhov)
tags: added: 9.1-proposed
Dmitry Pyzhov (dpyzhov)
tags: added: feature
Changed in fuel:
assignee: Roman Prykhodchenko (romcheg) → Dmitry Guryanov (dguryanov)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/360283

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-ui (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/362123

Changed in fuel:
assignee: Dmitry Guryanov (dguryanov) → Julia Aranovich (jkirnosova)
Changed in fuel:
assignee: Julia Aranovich (jkirnosova) → Dmitry Guryanov (dguryanov)
Changed in fuel:
assignee: Dmitry Guryanov (dguryanov) → Julia Aranovich (jkirnosova)
tags: added: area-ui
Changed in fuel:
assignee: Julia Aranovich (jkirnosova) → Bulat Gaifullin (bgaifullin)
Changed in fuel:
assignee: Bulat Gaifullin (bgaifullin) → Julia Aranovich (jkirnosova)
Changed in fuel:
assignee: Julia Aranovich (jkirnosova) → Vitaly Kramskikh (vkramskikh)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-ui (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/365606

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-ui (master)

Reviewed: https://review.openstack.org/362123
Committed: https://git.openstack.org/cgit/openstack/fuel-ui/commit/?id=2996e3e6f1ca47eda408fca2820ccf897f771625
Submitter: Jenkins
Branch: master

commit 2996e3e6f1ca47eda408fca2820ccf897f771625
Author: Julia Aranovich <email address hidden>
Date: Mon Sep 5 13:09:41 2016 +0300

    Display time start in transaction menu

    Replace transaction ID in History tab menu with transaction time start
    as far as User does not use IDs on UI.

    Partial-Bug: #1593753

    Change-Id: Ieca09c4fe63a6ef2ffca1ee0e09e99e7b1391eaa

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-ui (stable/mitaka)

Reviewed: https://review.openstack.org/365606
Committed: https://git.openstack.org/cgit/openstack/fuel-ui/commit/?id=588e8567f991f2167952c708b5cea251617b76a1
Submitter: Jenkins
Branch: stable/mitaka

commit 588e8567f991f2167952c708b5cea251617b76a1
Author: Julia Aranovich <email address hidden>
Date: Mon Sep 5 13:09:41 2016 +0300

    Display time start in transaction menu

    Replace transaction ID in History tab menu with transaction time start
    as far as User does not use IDs on UI.

    Partial-Bug: #1593753

    Change-Id: Ieca09c4fe63a6ef2ffca1ee0e09e99e7b1391eaa

tags: added: in-stable-mitaka
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/349901
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=dd2413fd387da2a63eab4d8c2f94b52ed4680d3e
Submitter: Jenkins
Branch: master

commit dd2413fd387da2a63eab4d8c2f94b52ed4680d3e
Author: Roman Prykhodchenko <email address hidden>
Date: Thu Sep 1 19:32:50 2016 +0300

    Track timestamps for start and end of transactions

    In order to enable building a history of deployment transactions
    for a custer it is necessary to keep track of start and end time
    for them. This patch adds timestamp_start and timestamp_end attributes
    to transactions, makes changes to the API serializers and introduces
    appropriate database fields to store that information

    I've added times only for 'deployment' task, which is the only
    one, needed by new UI feature. At this point there is no single
    point, where task switches to error or ready state and I'd have
    to set in in many places.

    Whem everything-is-a-graph approach will be fully implemented
    it would be very easy. So let's not add unnecessary code now.

    Co-authored by: Roman Prykhodchenko <email address hidden>
    Co-authored by: Dmitry Guryanov <email address hidden>

    DocImpact
    Closes-bug: #1593753

    Change-Id: Ib206d75a8d2215f8fd6b3f89e209c6e86ed20d0f

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Julia Aranovich (jkirnosova) wrote :

UI patch has been landed to both 10th and 9th branches.

tags: removed: area-ui in-stable-mitaka
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-ui (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/365962

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-ui (stable/mitaka)

Related fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/366027

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-ui (master)

Reviewed: https://review.openstack.org/365962
Committed: https://git.openstack.org/cgit/openstack/fuel-ui/commit/?id=7383dfaf394352c9544cc847eb0bc1f4d91953b8
Submitter: Jenkins
Branch: master

commit 7383dfaf394352c9544cc847eb0bc1f4d91953b8
Author: Julia Aranovich <email address hidden>
Date: Tue Sep 6 10:41:18 2016 +0300

    Fixes for deployment history

    * don't try to format not defined task time
    * don't display an empty task attribute in task details dialog
    * don't display label for transaction with empty graph type

    Related-Bug: #1593753

    Change-Id: I7c7530aa6cce43eef874719b0a1779609bd6497b

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-ui (stable/mitaka)

Reviewed: https://review.openstack.org/366027
Committed: https://git.openstack.org/cgit/openstack/fuel-ui/commit/?id=f3c5293750c24126c3f090834edfae6698829d00
Submitter: Jenkins
Branch: stable/mitaka

commit f3c5293750c24126c3f090834edfae6698829d00
Author: Julia Aranovich <email address hidden>
Date: Tue Sep 6 10:41:18 2016 +0300

    Fixes for deployment history

    * don't try to format not defined task time
    * don't display an empty task attribute in task details dialog
    * don't display label for transaction with empty graph type

    Related-Bug: #1593753

    Change-Id: I7c7530aa6cce43eef874719b0a1779609bd6497b

tags: added: in-stable-mitaka
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (stable/mitaka)

Reviewed: https://review.openstack.org/360283
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=ecfdf2f4105a1f1805d0cec3a943600b6036df5d
Submitter: Jenkins
Branch: stable/mitaka

commit ecfdf2f4105a1f1805d0cec3a943600b6036df5d
Author: Roman Prykhodchenko <email address hidden>
Date: Thu Sep 1 19:32:50 2016 +0300

    Track timestamps for start and end of transactions

    In order to enable building a history of deployment transactions
    for a custer it is necessary to keep track of start and end time
    for them. This patch adds timestamp_start and timestamp_end attributes
    to transactions, makes changes to the API serializers and introduces
    appropriate database fields to store that information

    I've added times only for 'deployment' task, which is the only
    one, needed by new UI feature. At this point there is no single
    point, where task switches to error or ready state and I'd have
    to set in in many places.

    Whem everything-is-a-graph approach will be fully implemented
    it would be very easy. So let's not add unnecessary code now.

    Co-authored by: Roman Prykhodchenko <email address hidden>
    Co-authored by: Dmitry Guryanov <email address hidden>

    DocImpact
    Closes-bug: #1593753

    Change-Id: Ib206d75a8d2215f8fd6b3f89e209c6e86ed20d0f

tags: added: on-verification
Revision history for this message
Alexey. Kalashnikov (akalashnikov) wrote :

verified:
CUSTOM_VERSION=snapshot #229
MAGNET_LINK=magnet:?xt=urn:btih:bfec808dd71ff42c5613a3527733d9012bb1fabc&dn=MirantisOpenStack-9.0.iso&tr=http%3A%2F%2Ftracker01-bud.infra.mirantis.net%3A8080%2Fannounce&tr=http%3A%2F%2Ftracker01-scc.infra.mirantis.net%3A8080%2Fannounce&tr=http%3A%2F%2Ftracker01-msk.infra.mirantis.net%3A8080%2Fannounce&ws=http%3A%2F%2Fvault.infra.mirantis.net%2FMirantisOpenStack-9.0.iso
FUEL_QA_COMMIT=78e594c093e52853cc2678fd903bedd1fb9fb96e
UBUNTU_MIRROR_ID=ubuntu-2016-08-03-174238
CENTOS_MIRROR_ID=centos-7.2.1511-2016-05-31-083834
MOS_UBUNTU_MIRROR_ID=9.0-2016-09-06-174322
MOS_CENTOS_OS_MIRROR_ID=os-2016-06-23-135731
MOS_CENTOS_PROPOSED_MIRROR_ID=proposed-2016-09-06-174322
MOS_CENTOS_UPDATES_MIRROR_ID=updates-2016-06-23-135916
MOS_CENTOS_HOLDBACK_MIRROR_ID=holdback-2016-06-23-140047
MOS_CENTOS_HOTFIX_MIRROR_ID=hotfix-2016-07-18-162958
MOS_CENTOS_SECURITY_MIRROR_ID=security-2016-06-23-140002

tags: removed: on-verification
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-web 10.0.0rc1

This issue was fixed in the openstack/fuel-web 10.0.0rc1 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-web 10.0.0

This issue was fixed in the openstack/fuel-web 10.0.0 release.

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.