-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
experiment/bump_e2e_images.sh doesn't work #13693
Comments
the last bit is probably the relevant part:
I suspect we're not forcing the right python version |
this also fails on my linux workstation with 0.28.1 |
symlinking bazel to bazelisk results in the same issue. |
It appears that since bazel 0.25.0 the default python version for So, while this file was written for python2 (inferred from the fact that I am actively working on python2 -> python3 migration, so I will go ahead and apply the updates to this script (and |
Explicitly set python_version=PY3 for "generate_tests", #13693
thank you!
…On Wed, Jul 31, 2019 at 4:08 PM Kubernetes Prow Robot < ***@***.***> wrote:
Closed #13693 <#13693> via
#13702 <#13702>.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#13693?email_source=notifications&email_token=AAHADKZ7MJ2UN7Y6RQYI6OTQCILONA5CNFSM4IIAR4NKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOSZ6UBFY#event-2524790935>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAHADKZDJ6KVQGMRS7FHFYLQCILONANCNFSM4IIAR4NA>
.
|
on macOS with bazel 0.28.1 gives a ton of output and fails, this is what was left in my buffer:
The text was updated successfully, but these errors were encountered: