commit 3443d01e09e079801b0660c82f5b57b230cc97d6
Author: Stan Lagun <email address hidden>
Date: Fri Apr 29 07:52:56 2016 -0700
Revert the destroy execution order
Restores back the order of .destroy execution so that
child objects get destroyed first and Environment get
destroyed last.
Both orders have their cons and pros. With
the current order Environment and thus Heat stack
are delete first so if the app sends a command to
murano-agent in its destroy method it will never
receive an answer since the VM of the agent was
already terminated. This broke Kubernetes and might
broke other apps as well. However the downside is that
apps are going to perform uninstall of the software
on VM that is going to be terminated anyway which
takes time. But until we implement a controllable
resource deallocation this looks like a safer approach.
Change-Id: I8265a69938658e89e4fc36824502c75a75a75c66
Closes-Bug: #1576703
(cherry picked from commit 440b4b104996337bf07be75ffb412ddf5adc9a5d)
Reviewed: https:/ /review. openstack. org/312508 /git.openstack. org/cgit/ openstack/ murano/ commit/ ?id=3443d01e09e 079801b0660c82f 5b57b230cc97d6
Committed: https:/
Submitter: Jenkins
Branch: stable/mitaka
commit 3443d01e09e0798 01b0660c82f5b57 b230cc97d6
Author: Stan Lagun <email address hidden>
Date: Fri Apr 29 07:52:56 2016 -0700
Revert the destroy execution order
Restores back the order of .destroy execution so that
child objects get destroyed first and Environment get
destroyed last.
Both orders have their cons and pros. With
the current order Environment and thus Heat stack
are delete first so if the app sends a command to
murano-agent in its destroy method it will never
receive an answer since the VM of the agent was
already terminated. This broke Kubernetes and might
broke other apps as well. However the downside is that
apps are going to perform uninstall of the software
on VM that is going to be terminated anyway which
takes time. But until we implement a controllable
resource deallocation this looks like a safer approach.
Change-Id: I8265a69938658e 89e4fc36824502c 75a75a75c66 bf07be75ffb412d df5adc9a5d)
Closes-Bug: #1576703
(cherry picked from commit 440b4b104996337