-
Notifications
You must be signed in to change notification settings - Fork 291
Add support for building rubik image #2110
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
Merged
Merged
Changes from 4 commits
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
4bb9bf7
build for rubik
samfreund 4e344d0
fake url for testing
samfreund 73bbcb1
oops should be raw
samfreund a87a48d
oh boy: moment of truth
samfreund 39d6174
switch to alt script for testing
samfreund a0c14c1
rerun CI
samfreund 5504d76
set script path
samfreund a50e322
update script URL
samfreund 8f5a58a
don't run on PRs
samfreund 5b1686e
more logging in armrunner.sh
samfreund 3357455
REVERT PRIOR TO MERGE
samfreund 97cd224
comprehensive debugging
samfreund 7a24567
reset URLs
samfreund 6b0b7cf
bump image version
samfreund eb764b9
Discard changes to scripts/armrunner.sh
samfreund e925c44
Merge branch 'main' into build-rubik-image
samfreund 6821a4d
REVERT BEFORE MERGE
samfreund dab898e
document flashing process
samfreund c9e78bc
Revert "REVERT BEFORE MERGE"
samfreund 6500330
bump image version
samfreund 75f9608
[docs] link to install instructions
samfreund fcc8308
test image build
samfreund 25fadbb
fix docs links
samfreund 4d1715f
remove testing bypass
samfreund 1920cf9
add warning about user and pw
samfreund 1d98660
warn about caching
samfreund af67d07
document ssh creds
samfreund ac2f53b
bump image version
samfreund 4e87696
add LL4 to quick-install
samfreund File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -10,7 +10,7 @@ concurrency: | |
| cancel-in-progress: true | ||
|
|
||
| env: | ||
| IMAGE_VERSION: v2025.0.4 | ||
| IMAGE_VERSION: Dev | ||
|
|
||
| jobs: | ||
|
|
||
|
|
@@ -517,8 +517,34 @@ jobs: | |
| with: | ||
| name: image-${{ matrix.image_suffix }} | ||
| path: photonvision*.xz | ||
| build-rubik-image: | ||
| needs: [build-package] | ||
|
|
||
| # if: ${{ github.event_name != 'pull_request' }} | ||
samfreund marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
|
|
||
| runs-on: ubuntu-24.04 | ||
| name: "Build image - Rubik Pi 3" | ||
|
|
||
| steps: | ||
| - name: Checkout code | ||
| uses: actions/checkout@v4 | ||
| with: | ||
| fetch-depth: 0 | ||
| - uses: actions/download-artifact@v4 | ||
| with: | ||
| name: jar-LinuxArm64 | ||
| - name: Generate image | ||
| run: | | ||
| wget https://raw.githubusercontent.com/PhotonVision/photon-image-modifier/refs/tags/$IMAGE_VERSION/mount_rubikpi3.sh | ||
| chmod +x mount_rubikpi3.sh | ||
| ./mount_rubikpi3.sh https://github.com/PhotonVision/photon-image-modifier/releases/download/$IMAGE_VERSION/photonvision_rubikpi3.tar.xz scripts/armrunner.sh | ||
| - uses: actions/upload-artifact@v4 | ||
| name: Upload image | ||
| with: | ||
| name: image-rubikpi3 | ||
| path: photonvision*.xz | ||
| release: | ||
| needs: [build-photonlib-vendorjson, build-package, build-image, combine] | ||
| needs: [build-photonlib-vendorjson, build-package, build-image, build-rubik-image, combine] | ||
| runs-on: ubuntu-22.04 | ||
| steps: | ||
| # Download all fat JARs | ||
|
|
@@ -576,6 +602,13 @@ jobs: | |
| if: startsWith(github.ref, 'refs/tags/v') | ||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| - uses: softprops/[email protected] | ||
samfreund marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| with: | ||
| files: | | ||
| **/*tar.xz | ||
| if: startsWith(github.ref, 'refs/tags/v') | ||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| - name: Create Vendor JSON Repo PR | ||
| uses: wpilibsuite/vendor-json-repo/.github/actions/add_vendordep@main | ||
| with: | ||
|
|
||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.