Skip to content

Remove parallel-netcdf dependency except for jedi-mpas-env/unified-dev/skylab-dev#1621

Merged
climbfuji merged 4 commits intoJCSDA:developfrom
climbfuji:feature/parallel_netcdf_jedi_mpas_env_only
Apr 30, 2025
Merged

Remove parallel-netcdf dependency except for jedi-mpas-env/unified-dev/skylab-dev#1621
climbfuji merged 4 commits intoJCSDA:developfrom
climbfuji:feature/parallel_netcdf_jedi_mpas_env_only

Conversation

@climbfuji
Copy link
Copy Markdown
Collaborator

@climbfuji climbfuji commented Apr 29, 2025

Summary

Remove parallel-netcdf dependency from base-env. Instead, add it to jedi-mpas-env and enable the variant +pnetcdf for parallelio in jedi-mpas-env (the default for this variant is ~pnetcdf).

This means that environments using template neptune-dev concretize without parallel-netcdf, but any template that includes jedi-mpas-env (e.g. unified-dev, skylab-dev) concretize with parallel-netcdf.

I added neptune-dev as an additional template to the Ubuntu GNU runner, but had to disable the xnrl variant since this package isn't publicly available. For neptune-dev, parallel-netcdf is removed, for unified-dev and skylab-dev, it is included as before.

Testing

I tested this on my laptop and I also confirmed that the CI builds behave as expected (see https://github.com/JCSDA/spack-stack/actions/runs/14740976173/job/41378625127?pr=1621)

Applications affected

None - NEPTUNE doesn't use parallel-netcdf, and the other environments in use will still have it.

Systems affected

None

Dependencies

None

Issue(s) addressed

Closes #1613

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.

… and enable variant +pnetcdf for parallelio in jedi-mpas-env
@climbfuji climbfuji self-assigned this Apr 29, 2025
@climbfuji climbfuji marked this pull request as ready for review April 30, 2025 02:14
@climbfuji climbfuji moved this to In Progress in spack-stack-2.0.x (2025 Q4) Apr 30, 2025
@climbfuji climbfuji merged commit 8605551 into JCSDA:develop Apr 30, 2025
8 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in spack-stack-2.0.x (2025 Q4) Apr 30, 2025
@climbfuji climbfuji deleted the feature/parallel_netcdf_jedi_mpas_env_only branch April 30, 2025 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects

Development

Successfully merging this pull request may close these issues.

Make parallel-netcdf an optional dependency, only compiled if MPAS is used

2 participants