Skip to content

Conversation

@climbfuji
Copy link
Collaborator

@climbfuji climbfuji commented Jan 8, 2026

Description

  1. Update and add paths-ignore for all GitHub actions workflows.
  2. util/nrl/batch_install.sh: ignore errors when updating cargo mirrors, since this is can be unreliable.

Dependencies

None

Issues addressed

None

Applications affected

None

Systems affected

NRL systems, GitHub actions

Testing

  • CI: Note whether the automatic tests (GitHub actions tests that run automatically for every commit) pass or not
    • GitHub actions CI tests pass
    • GitHub actions CI tests do not pass (provide explanation)
    • GitHub actions CI tests skipped (provide explanation if necessary)
  • New tests added: List and describe any new tests added to GitHub actions
    • ...
  • Additional testing: Add information on any additional tests conducted
    • ...

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.
  • All necessary updates to the spack-stack wiki will be made when this PR is merged

@climbfuji climbfuji self-assigned this Jan 8, 2026
@climbfuji climbfuji force-pushed the feature/ci_container branch from bac0d16 to dd45584 Compare January 8, 2026 14:38
@climbfuji climbfuji changed the title Re-enable CI testing for container builds Add paths-ignore for GitHub actions, ignore cargo mirror update errors in NRL batch install scripts Jan 8, 2026
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I updated this workflow while at it, even though it is currently not used. An attempt to re-enable GitHub actions testing for containers failed miserably. Since the containers builds will see a major overhaul for spack-stack-2.1.0, I am not going to invest time to fix these CI container builds now.

@climbfuji climbfuji marked this pull request as ready for review January 8, 2026 14:42
@climbfuji climbfuji enabled auto-merge (squash) January 8, 2026 16:10
@climbfuji climbfuji disabled auto-merge January 8, 2026 20:31
@climbfuji climbfuji merged commit 260c2fa into JCSDA:develop Jan 8, 2026
6 checks passed
@climbfuji climbfuji deleted the feature/ci_container branch January 8, 2026 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants