-
Notifications
You must be signed in to change notification settings - Fork 562
Set allow_broken_conditionals configuration variable at ansible.cfg.
#9634
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
Conversation
dcb8f36 to
e01229f
Compare
|
Actually, the environment variable name was: @bhavya01 What do you think? |
Sounds good! @ysiraichi Do you think you can fix the failing CPU test? Doesn't have to be this PR. I'll look into updating the tpu runner image |
|
TPU CI is also updated. Rerunning the tests should work now |
e01229f to
6d3df2d
Compare
ALLOW_BROKEN_CONDITIONALS variable before calling ansible-playbook.allow_broken_conditionals configuration variable at ansible.cfg.
zhanyong-wan
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
|
I'm skipping the failing CPU tests. |
|
@bhavya01 Since you requested a change after approving, you have to approve it once more. Otherwise, I can't merge this PR. |
Currently, CI is breaking with:
This is likely because the installed
ansibleis now on version2.19.2. The error is documented here.In order to quickly fix it, I have introducedALLOW_BROKEN_CONDITIONALS=1environment variable beforeansible-playbookexecutions.Key Changes:
allow_broken_conditionals = trueto ansible configuration fileansible.cfg