Skip to content

ursa: update openmpi install hash, remove system tar external#1614

Merged
climbfuji merged 40 commits intoJCSDA:developfrom
rickgrubin-noaa:ursa_update_openmpi
Apr 28, 2025
Merged

ursa: update openmpi install hash, remove system tar external#1614
climbfuji merged 40 commits intoJCSDA:developfrom
rickgrubin-noaa:ursa_update_openmpi

Conversation

@rickgrubin-noaa
Copy link
Copy Markdown
Collaborator

Summary

On the 'official' ursa host (no longer TDS), the provided external package for openmpi ([email protected]) now has a different install path (different hash) from the TDS -- update to that external package's value for OPENMPI_ROOT.

Also removed the reference to the system tar command as an external package, given previous issues around using the system-supplied version.

Testing

Rebuilt spack-stack and successfully ran UFS WM control_c48 test.

Applications affected

n/a

Systems affected

ursa

Dependencies

n/a

Issue(s) addressed

None

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.

Remove spurious tab
Copy link
Copy Markdown
Collaborator

@RatkoVasic-NOAA RatkoVasic-NOAA left a comment

Choose a reason for hiding this comment

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

These changes should go to releaase:1.9.0 as well.

@rickgrubin-noaa
Copy link
Copy Markdown
Collaborator Author

These changes should go to releaase:1.9.0 as well.

Separate PR or cherry-pick the commit to develop?

@climbfuji
Copy link
Copy Markdown
Collaborator

cherry-pick please

@climbfuji climbfuji enabled auto-merge (squash) April 28, 2025 14:37
@climbfuji
Copy link
Copy Markdown
Collaborator

The macos binary cache needs to be rebuilt, we can ignore the failure for this PR.

@climbfuji climbfuji disabled auto-merge April 28, 2025 15:42
@climbfuji climbfuji merged commit a706c80 into JCSDA:develop Apr 28, 2025
7 of 8 checks passed
@rickgrubin-noaa rickgrubin-noaa deleted the ursa_update_openmpi branch October 2, 2025 14:32
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.

3 participants