-
Notifications
You must be signed in to change notification settings - Fork 15k
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
ci: fixup diagnose_goma_log.py call #40131
Conversation
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.
The shebang for goma_ctl.py
already specifies python3
, but doesn't hurt to explicitly specify it. The shebang for diagnose_goma_log.py
should be updated upstream since it requires python3
now.
No Release Notes |
I have automatically backported this PR to "27-x-y", please check out #40149 |
I have automatically backported this PR to "25-x-y", please check out #40150 |
I have automatically backported this PR to "26-x-y", please check out #40151 |
Description of Change
The call to diagnose_goma_log.py after builds was failing with the error:
eg see https://app.circleci.com/pipelines/github/electron/electron/75261/workflows/0ea342e8-4faa-43bc-a706-9f59312f2c10/jobs/1616734/parallel-runs/0/steps/0-127.
This PR explictly runs that utility with python3 which fixes the issue.
Checklist
Release Notes
Notes: none