Closed jgleissner closed 6 years ago
Hey @jgleissner - is there a matching change to master? I don't see one pending / merged.
Hi @kiall, I've opened a PR against master: https://github.com/kubic-project/velum/pull/636
@jgleissner : The PR for master was merged. Please backport your fixes to this PR. Thanks.
Hi @jordimassaguerpla , I've cherry-picked the remaining commits from the master PR.
I saw some errors when testing the cluster in jenkins. I just retriggered the tests to make sure these are not random.
I've added a call_runner function to salt interface and added a call to sync_pillar runner in cloud_cluster.rb. This is required to activate the external pillar, which is normally done at orchestration time, but for creating cluster nodes in the cloud, the pillar is needed before that. See also https://github.com/kubic-project/velum/issues/399 for details.
I've also added a sleep(1) after the refresh_pillar, as we ran into the issue that the succeeding salt-cloud still works on the old pillar data, and this seems to work around it. I know that this is a bit ugly and perhaps can be fixed better in salt directly, but I haven't had any luck yet with narrowing the issue down.