Resources depending on conditioned resources are created before the dependencies of the conditioned resources are completed

Bug #1700569 reported by Giulio Fidente
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Heat
Triaged
Low
Unassigned

Bug Description

In a dependency chain like "A < B < C" where B is optional (conditioned), C is created before A is complete.

I am attaching a reproducer, after the stack is complete, the even list shows that Second is created before (or together with) First.

Revision history for this message
Giulio Fidente (gfidente) wrote :
Revision history for this message
Thomas Herve (therve) wrote :

Yeah that's expected. When you depend on something not created, we ignore the error (since recently), but we don't go up the dep chain. You need to add the dependency explicitly yourself.

Revision history for this message
Giulio Fidente (gfidente) wrote :

hi Thomas, thanks again. We were indeed discussing with Steven on IRC if this is (or not) the intended behavior!

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

This is really inconvenient for template authors though - I wonder if we should consider changing how conditionally defined resources work - what if we forced their type to OS::Heat::None when the condition is false, then the chain of dependencies and other weird corner cases related to skipping the node in the graph could be avoided?

Revision history for this message
Thomas Herve (therve) wrote :

It would somewhat change the expected behaviour. We could change the pike behavior, as the condition parsing is template specific. We may need to check the impact, though.

Rico Lin (rico-lin)
Changed in heat:
status: New → Opinion
importance: Undecided → Low
status: Opinion → Triaged
Rico Lin (rico-lin)
Changed in heat:
milestone: none → no-priority-tag-bugs
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.