Skip to content

Commit

Permalink
Fix organization casing in docker workflow (#271) (#272)
Browse files Browse the repository at this point in the history
* Testing lowercase repo name for cache

* Remove dev branch from workflow and fix cron schedule

* Use lowercase repo for cache-from in PR build as well.

---------

Co-authored-by: Aaron Marburg <amarburg@uw.edu>
(cherry picked from commit eaa8ec1)

Co-authored-by: Evan Palmer <evanp922@gmail.com>
  • Loading branch information
mergify[bot] and evan-palmer authored Aug 23, 2024
1 parent 46bf8e0 commit 25bfbd7
Showing 1 changed file with 5 additions and 6 deletions.
11 changes: 5 additions & 6 deletions .github/workflows/docker.yaml
Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
name: Docker

on:
# Disable schedule for preliminary testing
# schedule:
# - cron: "0 17 * * 6"
schedule:
- cron: "0 17 * * 6"
push:
branches:
- iron
Expand Down Expand Up @@ -114,8 +113,8 @@ jobs:
${{ steps.meta-desktop-nvidia.outputs.bake-file }}
push: ${{ env.PUSH }}
set: |
*.cache-from=type=registry,ref=ghcr.io/${{ github.repository }}:cache-${{ matrix.ROS_DISTRO }}
*.cache-to=type=registry,mode=max,ref=ghcr.io/${{ github.repository }}:cache-${{ matrix.ROS_DISTRO }}
*.cache-from=type=registry,ref=ghcr.io/${{ steps.lowercase-repo.outputs.repository }}:cache-${{ matrix.ROS_DISTRO }}
*.cache-to=type=registry,mode=max,ref=ghcr.io/${{ steps.lowercase-repo.outputs.repository }}:cache-${{ matrix.ROS_DISTRO }}
# Pull request builds are not cached; and only built for AMD64
- if: github.event_name == 'pull_request'
Expand All @@ -130,5 +129,5 @@ jobs:
./docker-bake.hcl
set: |
*.platform=linux/amd64
*.cache-from=type=registry,ref=ghcr.io/${{ github.repository }}:cache-${{ matrix.ROS_DISTRO }}
*.cache-from=type=registry,ref=ghcr.io/${{ steps.lowercase-repo.outputs.repository }}:cache-${{ matrix.ROS_DISTRO }}
*.cache-to=

0 comments on commit 25bfbd7

Please sign in to comment.