Error storing stack not reflected in status

Bug #1273796 reported by Steven Hardy
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Heat
Confirmed
Medium
Steven Hardy

Bug Description

There are some things (notably creating at trust, and in future creating a stack-user project) which can fail when doing parser.Stack.store, but we currently don't catch them and store the stack anway with a failed status, so you end up with:

+--------------------------------------+------------+--------------+----------------------+
| id | stack_name | stack_status | creation_time |
+--------------------------------------+------------+--------------+----------------------+
| facd83a1-e3fc-43ea-90b4-b2e20d2a05b2 | ua1001 | _ | 2014-01-28T17:46:06Z |
| aec6a937-9f0c-42a9-ab86-a6cedd48e9ab | ua1002 | _ | 2014-01-28T17:47:03Z |
+--------------------------------------+------------+--------------+----------------------+

Which is, uh, not the most informative ;)

Steven Hardy (shardy)
Changed in heat:
assignee: nobody → Steven Hardy (shardy)
milestone: none → icehouse-3
status: New → In Progress
importance: Undecided → Medium
Thierry Carrez (ttx)
Changed in heat:
milestone: icehouse-3 → icehouse-rc1
Revision history for this message
Steve Baker (steve-stevebaker) wrote :

Will there be something to review in time for rc1?

Revision history for this message
Steven Hardy (shardy) wrote :

> Will there be something to review in time for rc1?

Hopefully yes, I started working on a fix but still need to sort out tests.

Revision history for this message
Steve Baker (steve-stevebaker) wrote :

I need to kick this now so this isn't a release blocker

Changed in heat:
milestone: icehouse-rc1 → next
Revision history for this message
Angus Salkeld (asalkeld) wrote :

I'd suggest converting the trust user into an implicit real resource. Basically add a keystone::trust_user resource type and add it to all incomming templates. We will get proper error reporting for free. also we could potentially not need to add it for all templates (resources like autoscaling could depend on this resource and we only create it if has dependancies).

Revision history for this message
Angus Salkeld (asalkeld) wrote :

Steven, didn't we commit something that fixes this? Is there anything more to do here?

Changed in heat:
status: In Progress → Triaged
Revision history for this message
Angus Salkeld (asalkeld) wrote :
Changed in heat:
status: Triaged → Invalid
status: Invalid → In Progress
status: In Progress → Confirmed
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.