SM-Sequencing: ensure that post_provsion is
enabled after all the other roles have been enabled
Partial-Bug: #1461398
There is a possibility of race condition while enabling post_provision and
other roles (storage, tsn, toragent). Currently SM may enable post_provision
first and write into the hieradata file and next iteration it may pick other
roles, enable them and again update the file. Effectively hieradata file
changes are not correctly sequenced. During enabling post_provision and other
roles, puppet may read the file and skip other roles
FIX:
===
ensure post_provision is in last of the list of iteration. This ensures
post_provision changes will be enabled in last, after all other roles has been
enabled.
TESTING:
1. Prasad: provisioned base contrail and verified post_provision is
in end of role_sequence. Verified provisioned is successful.
2. Provisioned storage roles and verfied post_provision is in end of
role_sequence. Verified provisioning is successful.
Reviewed: https:/ /review. opencontrail. org/11504 github. org/Juniper/ contrail- server- manager/ commit/ 7c29330c68c3228 613c22d3d229b79 9f04626214
Committed: http://
Submitter: Zuul
Branch: R2.20
commit 7c29330c68c3228 613c22d3d229b79 9f04626214
Author: root <email address hidden>
Date: Thu Jun 11 08:43:22 2015 -0700
SM-Sequencing: ensure that post_provsion is
enabled after all the other roles have been enabled
Partial-Bug: #1461398
There is a possibility of race condition while enabling post_provision and
other roles (storage, tsn, toragent). Currently SM may enable post_provision
first and write into the hieradata file and next iteration it may pick other
roles, enable them and again update the file. Effectively hieradata file
changes are not correctly sequenced. During enabling post_provision and other
roles, puppet may read the file and skip other roles
FIX:
===
ensure post_provision is in last of the list of iteration. This ensures
post_provision changes will be enabled in last, after all other roles has been
enabled.
TESTING:
1. Prasad: provisioned base contrail and verified post_provision is
in end of role_sequence. Verified provisioned is successful.
2. Provisioned storage roles and verfied post_provision is in end of
role_sequence. Verified provisioning is successful.
Change-Id: I8e83abfabff1cc 69f70ebfd2b7b6d 9eccfc31205