Skip to content
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

Ignore errors when creating/using source and binary caches #684

Merged

Conversation

climbfuji
Copy link
Collaborator

@climbfuji climbfuji commented Jul 21, 2023

Summary

Ignore errors when creating/updating/using source and build caches. Needed as long as JCSDA/spack#295 and #672 are not merged (because some checksums changed).

After JCSDA/spack#295 and #672 are merged (some time after my vacation), we can revert most of these workarounds and clean up the caches on the CI platforms.

Testing

none

Applications affected

CI only

Systems affected

CI only

Dependencies

n/a

Issue(s) addressed

n/a

Checklist

  • This PR addresses one issue/problem/enhancement, or has a very good reason for not doing so.
  • These changes have been tested on the affected systems and applications.
  • All dependency PRs/issues have been resolved and this PR can be merged.

@climbfuji climbfuji changed the title Ignore errors when creating source caches Ignore errors when creating source caches, comment out some other stuff Jul 21, 2023
@climbfuji climbfuji changed the title Ignore errors when creating source caches, comment out some other stuff Ignore errors when creating source caches and other stuff Jul 21, 2023
@climbfuji climbfuji force-pushed the bugfix/ci_ignore_source_checksum_errors branch from f94f291 to 07cc1ae Compare July 22, 2023 00:03
@climbfuji climbfuji changed the title Ignore errors when creating source caches and other stuff Ignore errors when creating/using source and binary caches Jul 22, 2023
@climbfuji climbfuji marked this pull request as ready for review July 22, 2023 02:53
@climbfuji climbfuji self-assigned this Jul 22, 2023
@climbfuji climbfuji added the INFRA JEDI Infrastructure label Jul 22, 2023
Copy link
Collaborator

@ulmononian ulmononian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚇

@srherbener srherbener merged commit 4b0e18a into JCSDA:develop Jul 25, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
INFRA JEDI Infrastructure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants