forked from openstack/kayobe
-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Synchronise yoga with upstream #218
Closed
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Match ML2/OVS agents only when OVN is not enabled. Change-Id: Ia2ba7414202de6750b18ada47e7a5bb0b636e7b3 (cherry picked from commit d9ee59e)
Change-Id: I73fad29af6a57b29afea46947d1ae90546253a2a (cherry picked from commit 1e2ac5e)
Using the raw module will honor ssh args. The slight change behaviour is that it will print an unreachable message and show an unreachable task in the summary. Change-Id: I371e8a583c439264a88dbc4bdae14c472955a063 (cherry picked from commit ffbd1a7)
If inline Ansible vault encryption is used to define an encrypted variable in kayobe-config, running 'kayobe configuration dump -l <host>' fails with the following: Failed to decode config dump YAML file /tmp/tmp_fg1bv_j/localhost.yml: ConstructorError(None, None, "could not determine a constructor for the tag '!vault'", <yaml.error.Mark object at 0x7f1e5c7404c0>) This change fixes the error by using the Ansible YAML loader which supports the vault tag. Any vault encrypted variables are sanitised in the dump output. Note that variables in vault encrypted files are not sanitised. Change-Id: I4830500d3c927b0689b6f0bca32c28137916420b Closes-Bug: #2031390 (cherry picked from commit 78702d0)
The image from 20220913 has been removed from cloud.centos.org. Change-Id: I8db8414987b9ae85f794d09f5eadff80b41a9c57
Since using the to_bool function in more places in I3a5a43fef9c3d68d0db02be12b9f892c437e513d, we are now more strict about the result of the variable dump. If there are no controllers in the inventory, the result will not be a valid boolean and the to_bool function will exit non-zero. This change fixes the issue by running against localhost, which should always be in the inventory. Change-Id: Idcfd9d335f11f6c4d676033128d207f62b363ee9 (cherry picked from commit 0f1f009)
This variable was configuring enable_host_ntp in kolla-ansible, which was removed in the Xena release. Change-Id: I737598d3bbe40b933b4b727eccc3b2a76ed97cdb (cherry picked from commit 8da5472)
Change-Id: I57fe9a6a5037adc341617bd35d3df83c81acc5b0 Signed-off-by: Maksim Malchuk <[email protected]> (cherry picked from commit 46bba8e)
It only covers the internal API VIP, since we don't have a separate external API in the development environment. Change-Id: I28f2b43ddb8bad6097e690e766f0348b1b8a296b (cherry picked from commit 804c1cb)
Change-Id: I42192ff8c2f251855a11bc5306e402707f851325
Change-Id: I828d3685201f344f14d886ed10da28886da124d1
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR contains a snapshot of yoga from upstream.