Deployment history should contain message of a failed task

Bug #1597470 reported by Julia Aranovich
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Invalid
High
Georgy Kibardin
Mitaka
Invalid
High
Georgy Kibardin

Bug Description

Deployment history data (GET /api/transactions/<transaction_id>/deployment_history response) should contain 'message' attribute at least for a failed deployment task (task status is 'error').

A possible solution is to keep this error message into existing task's.
At the same time, 'custom' attribute.
But this 'custom' attribute should be merged into root task attributes and doesn't exist in the API response.

description: updated
tags: added: area-ui
Revision history for this message
Julia Aranovich (jkirnosova) wrote :

Raised to high as far as 'message' is critical information for a failed task (deployment).

Changed in fuel:
importance: Medium → High
tags: removed: area-ui
Changed in fuel:
assignee: Fuel Sustaining (fuel-sustaining-team) → Bulat Gaifullin (bgaifullin)
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/375513

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

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

commit 8ee56f4dccb313574deb8017fb6f2b1280187c28
Author: Bulat Gaifullin <email address hidden>
Date: Fri Sep 23 17:12:13 2016 +0300

    Merge property 'custom' of deployment_history with root

    The custom property was designed to store all attributes
    witch does not define in model. And durring serialization
    this properties should be added to root like other properties.

    Partial-Bug: 1597470
    Change-Id: I42c6a9ad1d8f7f62a5ab542a4877d69cae7a3cd5

Changed in fuel:
assignee: Bulat Gaifullin (bgaifullin) → Georgy Kibardin (gkibardin)
Revision history for this message
Georgy Kibardin (gkibardin) wrote :

The information you need is already in place, you just need to:
/api/transactions/<id>/deployment_history?include_summary=1
and the message is 'error_msg' inside of 'summary'.

Changed in fuel:
status: In Progress → Won't Fix
status: Won't Fix → Invalid
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.