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

On NRL sites, use OS zlib instead of spack-built zlib-ng #1439

Merged
merged 12 commits into from
Jan 7, 2025

Conversation

climbfuji
Copy link
Collaborator

@climbfuji climbfuji commented Jan 2, 2025

Summary

This PR updates all NRL sites to use the OS libz.so instead of the spack-built zlib-ng version. This is to address numerous problems, especially on Cray, where after loading spack-stack environments, basic tools like less and more sophisticated tools like ddt would throw many warnings or even stop working. For consistency, we do this on all NRL systems, regardless of being a Cray or not.

Included are also:

  • Additional, minor updates to the NRL site configs, most notably the update of cray-mpich on Narwhal to the latest version 8.1.26.
  • In template neptune-dev, exclude neptune-python-env with Intel Classic compilers. Yes, the time has come to say goodbye. The new versions of Python packages py-numpy and py-scipy don't build with these compilers, and it's not worth fixing those builds given that these compilers are deprecated and the replacement (%oneapi with icx, icpx, ifort) works fine. Add same conflict for LLVM (clang) compilers.
  • Add AMD AOCC compiler to tier2 site config Bounty.
  • Turn off gssapi variant for package libtircp because of a build error with latest OneAPI compilers (libtircp only needed by hdf@4, and the variant isn't needed)

This PR requires a bugfix/workaround in spack for package freetype, see JCSDA/spack#496

Testing

Applications affected

NEPTUNE applications running on NRL systems

Systems affected

NRL systems

Dependencies

Issue(s) addressed

Resolves numerous issue in the NRL Enterprise GitHub related to using spack's zlib-ng.

Resolves #1441
Resolves #1437

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.

# For TDS tusk
- spec: [email protected]
prefix: /usr
# For TDS cole
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The blueback site config will be split into separate site configs for TDS cole, tusk in a follow-up PR.

Copy link
Collaborator Author

@climbfuji climbfuji left a comment

Choose a reason for hiding this comment

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

self-review

configs/sites/tier2/blackpearl/compilers.yaml Outdated Show resolved Hide resolved
configs/sites/tier2/bounty/compilers.yaml Outdated Show resolved Hide resolved
@climbfuji climbfuji marked this pull request as ready for review January 7, 2025 16:32
@climbfuji climbfuji self-assigned this Jan 7, 2025
@climbfuji climbfuji merged commit 814186d into JCSDA:develop Jan 7, 2025
9 checks passed
@climbfuji climbfuji deleted the feature/nrl-zlib branch January 7, 2025 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
2 participants