Skip to content
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

Use git directly instead of actions/checkout on Windows #13186

Merged
merged 13 commits into from
Apr 20, 2023

Conversation

GMNGeoffrey
Copy link
Contributor

@GMNGeoffrey GMNGeoffrey commented Apr 20, 2023

The GitHub-provided actions/checkout action is for some reason
unusably slow on the large managed Windows runners. We assumed this was
because of some tricky IO issue or something, but I decide to just try
directly using git commands to see and lo the checkout time goes from
10 minutes to 1.5 🚀

With the caching improvements from
#13183, this gets the Windows build
down under 10 minutes, which means we can run it on presubmit (left for
a future PR).

Part of #11009

Tested:
Enabled this workflow on push to my branch:
https://github.com/openxla/iree/actions/runs/4750681034/jobs/8439091687

skip-ci: this only affects the Windows job, which isn't run on presubmit

@GMNGeoffrey GMNGeoffrey added infrastructure Relating to build systems, CI, or testing platform/windows 🚪 Windows-specific build, execution, benchmarking, and deployment labels Apr 20, 2023
@GMNGeoffrey GMNGeoffrey requested a review from ScottTodd as a code owner April 20, 2023 05:24
Copy link
Member

@ScottTodd ScottTodd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I saw high variance in checkout times, so I'm hoping that speedup wasn't a fluke 🤞 . Let's watch the timing in postsubmit runs a bit before turning this on for presubmits too.

Base automatically changed from gcmn-gcs-cache to main April 20, 2023 16:26
@GMNGeoffrey GMNGeoffrey enabled auto-merge (squash) April 20, 2023 16:30
@GMNGeoffrey GMNGeoffrey merged commit 0d82968 into main Apr 20, 2023
@GMNGeoffrey GMNGeoffrey deleted the gcmn-windows-git-checkout branch April 20, 2023 16:30
@ScottTodd
Copy link
Member

ScottTodd commented Apr 20, 2023

Still seeing high variance, unfortunately. This might be a net improvement though.

@GMNGeoffrey
Copy link
Contributor Author

It's getting monotonically slower (⊙ _◎)

jpienaar pushed a commit that referenced this pull request May 1, 2023
The GitHub-provided `actions/checkout` action is for some reason
unusably slow on the large managed Windows runners. We assumed this was
because of some tricky IO issue or something, but I decide to just try
directly using `git` commands to see and lo the checkout time goes from
10 minutes to 1.5 🚀 

With the caching improvements from
#13183, this gets the Windows build
down under 10 minutes, which means we can run it on presubmit (left for
a future PR).

Part of #11009

Tested:
Enabled this workflow on push to my branch:
https://github.com/openxla/iree/actions/runs/4750681034/jobs/8439091687

skip-ci: this only affects the Windows job, which isn't run on presubmit
NatashaKnk pushed a commit to NatashaKnk/iree that referenced this pull request Jul 6, 2023
…3186)

The GitHub-provided `actions/checkout` action is for some reason
unusably slow on the large managed Windows runners. We assumed this was
because of some tricky IO issue or something, but I decide to just try
directly using `git` commands to see and lo the checkout time goes from
10 minutes to 1.5 🚀 

With the caching improvements from
iree-org#13183, this gets the Windows build
down under 10 minutes, which means we can run it on presubmit (left for
a future PR).

Part of iree-org#11009

Tested:
Enabled this workflow on push to my branch:
https://github.com/openxla/iree/actions/runs/4750681034/jobs/8439091687

skip-ci: this only affects the Windows job, which isn't run on presubmit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure Relating to build systems, CI, or testing platform/windows 🚪 Windows-specific build, execution, benchmarking, and deployment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants