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: update recommended buildkite VM #27728
Conversation
@@ -54,6 +54,8 @@ Write-Host "Installing Git for Windows." | |||
Invoke-WebRequest -Uri https://github.com/git-for-windows/git/releases/download/v2.19.1.windows.1/Git-2.19.1-64-bit.exe -OutFile git.exe | |||
.\git.exe /VERYSILENT /NORESTART /NOCANCEL /SP- /CLOSEAPPLICATIONS /RESTARTAPPLICATIONS /COMPONENTS="icons,ext\reg\shellhere,assoc,assoc_sh" /DIR="C:\git" | |||
Add-Path "C:\git\bin" | |||
# Sleep for 15s while git is installed. Trying to remove the git.exe before it finishes install causes an error. |
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.
yuck, how do you know that 15s is long enough? can we somehow check whether the thing is done and sleep more if it's not?
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.
You're right, I do not know if 15s is always enough. I checked all the available flags for the git installer executable (https://github.com/git-for-windows/git/wiki/Silent-or-Unattended-Installation) and couldn't really find something that blocks on waiting.
The Bazel folks install it via choco
(https://github.com/bazelbuild/continuous-integration/blob/322aa74d4ce07ead7f9560b773712f3c27c8deb9/buildkite/setup-windows.ps1#L82-L86) so I think they avoid this problem.
Which of course begs the question of how chocolatey does it. The files they use can be seen at https://chocolatey.org/packages/git.install/2.20.1. Nothing really obvious catches my attention in their powershell scripts.
I think I need to test this script better overall.
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Issue Number: part of #27508
What is the new behavior?
Updated information on how to provision buildkite VM.
Does this PR introduce a breaking change?
Other information