We stopped clearing UpdateIdentifier because setting it to None caused
issues with template validation. However, this introduced a regression
with regards to minor updates.
We do patch stack-updates, which means that the UpdateIdentifer can
persist from previous runs. This can cause unwanted minor update on a
normal stack-update or on a scale-up, if the UpdateIdentifier is not
empty. We could prevent it on normal stack-update via some means,
e.g. persisting the last known update identifier value on the nodes and
comparing that with current value from the parameter. However, working
around this issue on scale-up would probably be considerably harder. A
good way forward seems to be to explicitly unset UpdateIdentifier as we
did before, but do it in a way that doesn't cause validation problems.
Reviewed: https:/ /review. openstack. org/336526 /git.openstack. org/cgit/ openstack/ python- tripleoclient/ commit/ ?id=88548d8742c d00c8a5a0b3c371 af7a5cff5d5df3
Committed: https:/
Submitter: Jenkins
Branch: master
commit 88548d8742cd00c 8a5a0b3c371af7a 5cff5d5df3
Author: Jiri Stransky <email address hidden>
Date: Fri Jul 1 13:48:45 2016 +0200
Unset UpdateIdentifier on deploy
We stopped clearing UpdateIdentifier because setting it to None caused
issues with template validation. However, this introduced a regression
with regards to minor updates.
We do patch stack-updates, which means that the UpdateIdentifer can
persist from previous runs. This can cause unwanted minor update on a
normal stack-update or on a scale-up, if the UpdateIdentifier is not
empty. We could prevent it on normal stack-update via some means,
e.g. persisting the last known update identifier value on the nodes and
comparing that with current value from the parameter. However, working
around this issue on scale-up would probably be considerably harder. A
good way forward seems to be to explicitly unset UpdateIdentifier as we
did before, but do it in a way that doesn't cause validation problems.
Change-Id: I7f6e3a65bb1f44 a42cc532a537f10 67bee168142
Closes-Bug: #1598124
Related-Bug: #1567385
Related-Bug: #1596640