Skip to content

Commit

Permalink
fix cache key generated from data workflow
Browse files Browse the repository at this point in the history
  • Loading branch information
zacharyburnett committed Oct 4, 2023
1 parent 05c9861 commit 2c6ef24
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/data.yml
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ jobs:
key: webbpsf-${{ steps.retrieve_webbpsf_hash.outputs.hash }}
- id: cache_key
name: construct combined cache key
run: echo "key=data-${{ steps.retrieve_webbpsf_hash.outputs.hash }}-" >> $GITHUB_OUTPUT
run: echo "key=data-${{ steps.retrieve_webbpsf_hash.outputs.hash }}" >> $GITHUB_OUTPUT
- name: cache combined data directory
uses: actions/cache@v3
with:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/roman_ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ jobs:
DD_GIT_COMMIT_SHA: ${{ github.sha }}
DD_GIT_BRANCH: ${{ github.ref_name }}
cache-path: ${{ needs.data.outputs.path }}
cache-key: data-${{ needs.data.outputs.data_hash }}-${{ needs.data.outputs.crds_context }}
cache-key: ${{ needs.data.outputs.cache_key }}-${{ needs.data.outputs.crds_context }}
cache-restore-keys: ${{ needs.data.outputs.cache_key }}
envs: |
- linux: py39-oldestdeps-cov
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/roman_ci_cron.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ jobs:
CRDS_CLIENT_RETRY_COUNT: 3
CRDS_CLIENT_RETRY_DELAY_SECONDS: 20
cache-path: ${{ needs.data.outputs.crds_path }}
cache-key: data-${{ needs.data.outputs.webbpsf_hash }}-${{ needs.data.outputs.crds_context }}
cache-key: ${{ needs.data.outputs.cache_key }}-${{ needs.data.outputs.crds_context }}
cache-restore-keys: ${{ needs.data.outputs.cache_key }}
envs: |
- macos: py39
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/tests_devdeps.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ jobs:
CRDS_CLIENT_RETRY_COUNT: 3
CRDS_CLIENT_RETRY_DELAY_SECONDS: 20
cache-path: ${{ needs.data.outputs.crds_path }}
cache-key: data-${{ needs.data.outputs.webbpsf_hash }}-${{ needs.data.outputs.crds_context }}
cache-key: ${{ needs.data.outputs.cache_key }}-${{ needs.data.outputs.crds_context }}
cache-restore-keys: ${{ needs.data.outputs.cache_key }}
envs: |
- linux: py39-devdeps
Expand Down

0 comments on commit 2c6ef24

Please sign in to comment.