Skip to content

Commit

Permalink
Merge pull request #75 from youtalk/merge-2pr
Browse files Browse the repository at this point in the history
  • Loading branch information
youtalk authored Jul 12, 2024
2 parents 59668c3 + 98b0bbd commit 5e41da6
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 13 deletions.
5 changes: 3 additions & 2 deletions .github/actions/docker-build-and-push/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@ runs:
- name: Setup Docker Buildx
uses: docker/setup-buildx-action@v3

# TODO(youtalk): Remove obsolete "cache-" restore-keys
- name: Restore ccache
uses: actions/cache/restore@v4
with:
Expand All @@ -53,8 +54,8 @@ runs:
restore-keys: |
ccache-${{ inputs.platform }}-${{ inputs.name }}-
ccache-${{ inputs.platform }}-
cache-${{ inputs.platform }}-${{ inputs.name }}- # TODO(youtalk): Remove obsolete cache key
cache-${{ inputs.platform }}- # TODO(youtalk): Remove obsolete cache key
cache-${{ inputs.platform }}-${{ inputs.name }}-
cache-${{ inputs.platform }}-
- name: Restore apt-get
uses: actions/cache/restore@v4
Expand Down
22 changes: 12 additions & 10 deletions .github/actions/docker-build/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -34,23 +34,24 @@ runs:
vcs import src < autoware.repos
shell: bash

# TODO(youtalk): Remove obsolete "cache-" restore-keys
- name: Cache ccache
uses: actions/cache@v4
if: ${{ inputs.name == 'no-cuda' }}
if: ${{ inputs.name == 'no-cuda' && github.ref == 'refs/heads/main' }}
id: cache-ccahce
with:
path: |
root-ccache
key: cache-${{ inputs.platform }}-${{ inputs.name }}-${{ hashFiles('src/**/*.cpp') }}
key: ccache-${{ inputs.platform }}-${{ inputs.name }}-${{ hashFiles('src/**/*.cpp') }}
restore-keys: |
ccache-${{ inputs.platform }}-${{ inputs.name }}-
ccache-${{ inputs.platform }}-
cache-${{ inputs.platform }}-${{ inputs.name }}- # TODO(youtalk): Remove obsolete cache key
cache-${{ inputs.platform }}- # TODO(youtalk): Remove obsolete cache key
cache-${{ inputs.platform }}-${{ inputs.name }}-
cache-${{ inputs.platform }}-
- name: Cache apt-get
uses: actions/cache@v4
if: ${{ inputs.name == 'no-cuda' }}
if: ${{ inputs.name == 'no-cuda' && github.ref == 'refs/heads/main' }}
id: cache-apt-get
with:
path: |
Expand All @@ -61,22 +62,23 @@ runs:
apt-get-${{ inputs.platform }}-${{ inputs.name }}-
apt-get-${{ inputs.platform }}-
# TODO(youtalk): Remove obsolete "cache-" restore-keys
- name: Restore ccache
uses: actions/cache/restore@v4
if: ${{ inputs.name != 'no-cuda' }}
if: ${{ inputs.name != 'no-cuda' || github.ref != 'refs/heads/main' }}
with:
path: |
root-ccache
key: cache-${{ matrix.platform }}-${{ matrix.name }}-${{ hashFiles('src/**/*.cpp') }}
key: cache-${{ inputs.platform }}-${{ inputs.name }}-${{ hashFiles('src/**/*.cpp') }}
restore-keys: |
ccache-${{ inputs.platform }}-${{ inputs.name }}-
ccache-${{ inputs.platform }}-
cache-${{ inputs.platform }}-${{ inputs.name }}- # TODO(youtalk): Remove obsolete cache key
cache-${{ inputs.platform }}- # TODO(youtalk): Remove obsolete cache key
cache-${{ inputs.platform }}-${{ inputs.name }}-
cache-${{ inputs.platform }}-
- name: Restore apt-get
uses: actions/cache/restore@v4
if: ${{ inputs.name != 'no-cuda' }}
if: ${{ inputs.name != 'no-cuda' || github.ref != 'refs/heads/main' }}
with:
path: |
var-cache-apt
Expand Down
4 changes: 3 additions & 1 deletion .github/workflows/health-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,9 @@ jobs:

load-env:
needs: label-check
if: ${{ needs.label-check.outputs.result == 'true' }}
if: ${{ needs.label-check.outputs.result == 'true' ||
github.event_name == 'schedule' ||
github.event_name == 'workflow_dispatch' }}
uses: ./.github/workflows/load-env.yaml

docker-build:
Expand Down

0 comments on commit 5e41da6

Please sign in to comment.