Skip to content

Sync config_machines.xml to E3SM 4487cd1 and update Frontier GNU Spack externals#408

Merged
xylar merged 3 commits intomainfrom
copilot/update-cime-machine-config-another-one
Apr 16, 2026
Merged

Sync config_machines.xml to E3SM 4487cd1 and update Frontier GNU Spack externals#408
xylar merged 3 commits intomainfrom
copilot/update-cime-machine-config-another-one

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 16, 2026

Daily drift detected changes in supported machine config for bebop,
frontier, and improv. This PR syncs
mache/cime_machine_config/config_machines.xml to upstream E3SM revision
4487cd1c5daf8ab1de910a9d0c74be4b8e813bfb and updates the affected Frontier
Spack templates for module/version drift.

  • config_machines.xml upstream sync

    • Replaced mache/cime_machine_config/config_machines.xml with generated
      upstream content from E3SM 4487cd1c5daf8ab1de910a9d0c74be4b8e813bfb.
    • Removed the temporary generated file
      upstream_config_machines.xml from the repo root.
  • Frontier Spack template alignment

    • Updated:
      • mache/spack/templates/frontier_craygnu_mpich.yaml
      • mache/spack/templates/frontier_craygnu-mphipcc_mpich.yaml
    • Aligned GNU toolchain/module and external package versions with upstream
      Frontier drift in the synced machine configuration:
      • cpe/24.11cpe/25.09
      • gcc-native/12.3gcc-native/14.2
      • hdf5 1.14.3.31.14.3.7
      • netcdf-c 4.9.0.154.9.2.1
      • parallel-netcdf 1.12.3.151.12.3.19
    • Per review feedback, Frontier prefix: paths keep gnu/12.3 (not
      gnu/14.2) for these externals.
    • Clarification: Frontier template updates were based on Frontier config
      drift, not on crux machine settings.
# frontier_craygnu_mpich.yaml
- spec: hdf5@1.14.3.7~cxx+fortran+hl~java+mpi+shared
  prefix: /opt/cray/pe/hdf5-parallel/1.14.3.7/gnu/12.3
- spec: netcdf-c@4.9.2.1+mpi~parallel-netcdf
  prefix: /opt/cray/pe/netcdf-hdf5parallel/4.9.2.1/gnu/12.3

Checklist

  • User's Guide has been updated if needed
  • Developer's Guide has been updated if needed
  • API documentation lists any new or modified class, method, or function
  • Documentation builds cleanly and changes look as expected
  • Tests pass and new features are covered by tests
  • PR description includes a summary and any relevant issue references
  • Testing comment, if appropriate, in the PR documents testing used to verify the changes

Fixes #407

Copilot AI linked an issue Apr 16, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Update config_machines based on upstream changes Sync config_machines.xml to E3SM 4487cd1 and update Frontier GNU Spack externals Apr 16, 2026
Copilot AI requested a review from xylar April 16, 2026 18:35
Comment thread mache/spack/templates/frontier_craygnu-mphipcc_mpich.yaml Outdated
Comment thread mache/spack/templates/frontier_craygnu-mphipcc_mpich.yaml Outdated
Comment thread mache/spack/templates/frontier_craygnu-mphipcc_mpich.yaml Outdated
Comment thread mache/spack/templates/frontier_craygnu-mphipcc_mpich.yaml Outdated
@xylar
Copy link
Copy Markdown
Collaborator

xylar commented Apr 16, 2026

@copilot, you are making changes to Frontier files based on changes to config_machines.xml in for the machine crux. Why are you doing that?

@xylar
Copy link
Copy Markdown
Collaborator

xylar commented Apr 16, 2026

@copilot, sorry, my mistake. There were, indeed, frontier changes. Your changes now look right to me. Again, sorry.

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 16, 2026

@copilot, you are making changes to Frontier files based on changes to config_machines.xml in for the machine crux. Why are you doing that?

Good catch — those Frontier template edits were not intended to be driven by crux. In cf10241, I synced config_machines.xml and then updated Frontier templates to match Frontier module/version drift in that same upstream sync. The follow-up 88924b8 kept the Frontier prefix paths at gnu/12.3 per your review feedback.

Copy link
Copy Markdown
Collaborator

@xylar xylar 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 look good!

@xylar xylar marked this pull request as ready for review April 16, 2026 18:52
@xylar
Copy link
Copy Markdown
Collaborator

xylar commented Apr 16, 2026

Testing

I was able to deploy all 7 spack environments for Polaris for the 6 compilers on Frontier with these changes.

@xylar xylar merged commit fdc6072 into main Apr 16, 2026
11 checks passed
@xylar xylar deleted the copilot/update-cime-machine-config-another-one branch April 16, 2026 19:30
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.

Daily config_machines drift detected

2 participants