airshipit / treasuremap

Reference Airship manifests, CICD, and reference architecture.
http://openstack.org
Apache License 2.0
52 stars 39 forks source link

VRRP Cleanup #121

Closed eak13 closed 3 years ago

eak13 commented 3 years ago

To correct the issue raised on VRRP seems to be from the json6902patch merge construct. The below attributes to be added to the preKubeadmCommands as a part of the list but it has ended up concatenating both the lines which is leading to the issue.

The fix is to put and additional add op for the second line like below.

Also remove the apt-get install/update as it should be addressed in image builder #10 . This needs to be remove this line as part of the fix.

manojkva commented 3 years ago

Please assign this to me

eak13 commented 3 years ago

@manojkva done

michaelfix commented 3 years ago

For documentation only - Manoj's PS is https://review.opendev.org/c/airship/treasuremap/+/784790

manojkva commented 3 years ago

Can be closed as the fix is merged.

eak13 commented 3 years ago

Closing per merge