Skip to content

Conversation

@ethanjli
Copy link
Member

@ethanjli ethanjli commented Jan 16, 2025

This PR bumps the pallet used for the OS images from the stable release (v2024.0.0) to the master branch. This PR also takes advantage of GitHub's newly-available public beta of arm64 hosted runners for building OS images, and thus simplifies the setup scripts somewhat (following up on #416 as part of #136) and speeds up the OS image build process by ~4x (see #520 (comment) ).

@ethanjli ethanjli changed the title Bump the pallet Simplify OS build process using Github's arm64 runners & bump the pallet Jan 17, 2025
@ethanjli ethanjli changed the title Simplify OS build process using Github's arm64 runners & bump the pallet Simplify OS build process using Github's arm64 runners Jan 17, 2025
@ethanjli
Copy link
Member Author

Previously (e.g. in 58b2528 ) each OS image build job took 30-40 minutes to run. In the above commit, each one took 8-10 minutes to run. This is a ~4x reduction in build times, and it makes it feasible to bring back the merge queue!!

I still need to test the resulting images to see whether they work, and in particular whether I need to fix anything related to rfkill (which doesn't work in an unbooted container)

@ethanjli ethanjli marked this pull request as ready for review January 18, 2025 04:41
@ethanjli
Copy link
Member Author

I've confirmed in testing that we don't actually need to run rfkill unblock wifi in the OS setup scripts - the Wi-Fi hotspot appears on first boot regardless!

@ethanjli ethanjli enabled auto-merge January 18, 2025 07:01
@ethanjli ethanjli changed the title Simplify OS build process using Github's arm64 runners Simplify & speed up OS build process using Github's arm64 runners Jan 18, 2025
@ethanjli ethanjli added this pull request to the merge queue Jan 18, 2025
Merged via the queue into master with commit 132a81b Jan 18, 2025
8 checks passed
@ethanjli ethanjli deleted the feature/bump-pallet branch January 18, 2025 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants