Skip to content

Bounty: Speedup CI setup to <20s using native install #35623

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

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

nkhatpe
Copy link

@nkhatpe nkhatpe commented Jul 1, 2025

This PR replaces the Docker-based environment setup with a minimal native pip-based install to dramatically reduce CI setup time.

Summary

  • Replaces eval $BUILD in .github/workflows/setup/action.yaml with native install
  • Uses GitHub Actions caching for pip
  • Only installs essential Python packages:
    • numpy, pycapnp, psutil, tqdm
  • Skips Docker pull/build entirely
  • Confirmed to work on default GitHub-hosted runners (ubuntu-latest)

Benchmark

  • Setup time: 17 seconds
  • Well under the <20s requirement for full bounty

Relevant bounty issue:

#30706

Tested in my fork:

Thank you for reviewing! I’d love to continue contributing to openpilot and take on more bounties.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thanks for contributing to openpilot! In order for us to review your PR as quickly as possible, check the following:

  • Convert your PR to a draft unless it's ready to review
  • Read the contributing docs
  • Before marking as "ready for review", ensure:
    • the goal is clearly stated in the description
    • all the tests are passing
    • the change is something we merge
    • include a route or your device' dongle ID if relevant

@adeebshihadeh
Copy link
Contributor

I assume this is a draft?

@nkhatpe
Copy link
Author

nkhatpe commented Jul 2, 2025

Yes, this is still a draft. I’m actively working on optimizing the setup and testing different caching strategies. Will mark it ready for review once it's stable.

@maxime-desroches maxime-desroches marked this pull request as draft July 2, 2025 01:30
@nkhatpe nkhatpe force-pushed the setup-native-final branch from 6e953cc to b4b16a1 Compare July 5, 2025 00:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants