-
Notifications
You must be signed in to change notification settings - Fork 566
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
.
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
ansible
is now on version2.19.2
. The error is documented here.In order to quickly fix it, I have introducedALLOW_BROKEN_CONDITIONALS=1
environment variable beforeansible-playbook
executions.Key Changes:
allow_broken_conditionals = true
to ansible configuration fileansible.cfg