Error messages from nested stacks are awful

Bug #1459837 reported by Zane Bitter
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Heat
Fix Released
High
Angus Salkeld
Kilo
Fix Released
Undecided
Zane Bitter

Bug Description

Since the shift to handling nested stacks over RPC, error messages from within them have become utterly incomprehensible. The nested stack resource reports its status as:

ResourceUnknownStatus: Resource failed - Unknown status FAILED due to "..."

Which is ridiculous. The status isn't unknown, we should know exactly what FAILED means and just pass the error message along. A couple of layers of nested stacks deep (not uncommon) and we are quickly into absurdities the likes of which were previously restricted to such things as C++ linker errors:

ResourceUnknownStatus: Resource failed - Unknown status FAILED due to "Resource CREATE failed: ResourceUnknownStatus: Resource failed - Unknown status FAILED due to "Resource CREATE failed: Error: Deployment to server failed: deploy_status_code : Deployment exited with non-zero status code: 6""

It doesn't even add any useful information, like the name of the resource that failed, just noise.

Zane Bitter (zaneb)
Changed in heat:
importance: Undecided → Medium
Changed in heat:
status: New → Triaged
importance: Medium → High
Angus Salkeld (asalkeld)
Changed in heat:
assignee: nobody → Angus Salkeld (asalkeld)
milestone: none → liberty-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to heat (master)

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

Changed in heat:
status: Triaged → In Progress
tags: added: kilo-backport-potential
Changed in heat:
status: In Progress → Fix Committed
Changed in heat:
status: Fix Committed → In Progress
Thierry Carrez (ttx)
Changed in heat:
milestone: liberty-1 → liberty-2
Changed in heat:
milestone: liberty-2 → liberty-1
Changed in heat:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in heat:
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to heat (stable/kilo)

Fix proposed to branch: stable/kilo
Review: https://review.openstack.org/195854

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

Reviewed: https://review.openstack.org/195854
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=ed52c86549e65544ca9813d0da135b29d31a4460
Submitter: Jenkins
Branch: stable/kilo

commit ed52c86549e65544ca9813d0da135b29d31a4460
Author: Angus Salkeld <email address hidden>
Date: Wed Jun 24 08:52:08 2015 +1000

    Produce more meaningful exception messages in nested stacks

    This produces a nested exception like:
     'ValueError: resources.nested.resources.my_server: it is broken, sorry'

    This re-uses the path mechanism that StackValidationFailed exception
    uses.

    Cherry-picked from caa1bd860233296fa3dbf8197667f38c10719611
    Change-Id: Id5204c15ee96784e04522ab3c5a8e66900f9a1d3
    Closes-bug: 1459837

tags: added: in-stable-kilo
removed: kilo-backport-potential
Thierry Carrez (ttx)
Changed in heat:
milestone: liberty-1 → 5.0.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.