Comment 6 for bug 1461398

Reviewed: https://review.opencontrail.org/11504
Committed: http://github.org/Juniper/contrail-server-manager/commit/7c29330c68c3228613c22d3d229b799f04626214
Submitter: Zuul
Branch: R2.20

commit 7c29330c68c3228613c22d3d229b799f04626214
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: I8e83abfabff1cc69f70ebfd2b7b6d9eccfc31205