Skip to content

Conversation

@BenjaminPelletier
Copy link
Member

Currently, our CI is extremely inefficient and unfriendly to upstream resources because we build the same image 16 times in parallel. This PR attempts to fix this problem, and also optimize a few additional aspects of our CI Actions.

@BenjaminPelletier BenjaminPelletier force-pushed the build-image-once branch 2 times, most recently from a304ff5 to 50ad9bd Compare October 10, 2025 21:35
@BenjaminPelletier BenjaminPelletier marked this pull request as ready for review October 10, 2025 21:57
Copy link
Contributor

@barroco barroco left a comment

Choose a reason for hiding this comment

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

Looks good to me except dev-checks change. Please review the remark before merging

- uses: actions/checkout@v5
- name: Checkout on Mac
run: echo "Project successfully cloned on ${{ runner.os }}. See `Set up Job` stage for more details about the Runner."
ubuntu:
Copy link
Contributor

Choose a reason for hiding this comment

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

Why was that removed ?

Copy link
Member Author

Choose a reason for hiding this comment

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

This is redundant to all of the other checks since they all use checkout on ubuntu

@BenjaminPelletier BenjaminPelletier merged commit 844ad4c into interuss:main Oct 15, 2025
21 checks passed
github-actions bot added a commit that referenced this pull request Oct 15, 2025
@BenjaminPelletier BenjaminPelletier deleted the build-image-once branch October 15, 2025 17:41
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.

2 participants