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

Fast actions #478

Merged
merged 22 commits into from
Dec 17, 2024
Merged

Fast actions #478

merged 22 commits into from
Dec 17, 2024

Conversation

ChristopherHX
Copy link
Owner

@ChristopherHX ChristopherHX commented Dec 15, 2024

Reduce startup delay by optimizing the bootstrapping.

Current estimate 50% of time needed to run the test suite.

40min workflow reduced to 20min, windows takes the most time eventually makes sense to split this into a parallel job.

@ChristopherHX ChristopherHX marked this pull request as draft December 15, 2024 22:01
@ChristopherHX
Copy link
Owner Author

ChristopherHX commented Dec 15, 2024

  • 2:30 minutes saved in regular non docker actions test suite
  • apply this to ado pipelines as well then we are much faster
    • 20:00 minutes to 10:00 minutes for external runner optimization
    • 10:00 minutes saved
  • respect parallel runner e.g. multiple folders more semaphoreslim slots
  • ctrl+c should not stop server

@ChristopherHX
Copy link
Owner Author

ctrl+c handing costs again 10minutes so 30min only 10min less than legacy.

The change gid trick must be refactored...

@ChristopherHX
Copy link
Owner Author

Reduced test from 40min to 25min with ctrl+c handling

  • pull windows docker image in parallel
  • use spawn command.

@ChristopherHX ChristopherHX marked this pull request as ready for review December 17, 2024 21:36
@ChristopherHX ChristopherHX enabled auto-merge (squash) December 17, 2024 21:46
@ChristopherHX ChristopherHX merged commit 036aa1d into main Dec 17, 2024
38 of 39 checks passed
@ChristopherHX ChristopherHX deleted the fast-actions branch December 18, 2024 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant