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

Update compiler/MPI entries in site configs for the UE #480

Merged
merged 15 commits into from
Feb 22, 2023

Conversation

ulmononian
Copy link
Collaborator

@ulmononian ulmononian commented Feb 21, 2023

As @climbfuji mentioned in #478, the compiler and MPI entry formatting in the sites/packages.yaml files needs to be adjusted for the UE (to properly support both Intel-latest and GNU installs). Further, for sites on which the global-workflow is supported, Intel-18 is needed, so a few sites' (Jet, Hera, Orion, S4) packages.yaml and compilers.yaml files need to be updated to include proper paths to/info. about their respective Intel-18 installations.

I will start with these updates on the following sites: Jet, Hera, Cheyenne, Orion, and Gaea. Perhaps @climbfuji can comment as to what the plan is for installation and testing plan is on the Parallelworks clusters (Azure, AWS, GCP); if UE will be installed there, I can update those too.

I do not have access to the remaining sites from https://spack-stack.readthedocs.io/en/latest/Platforms.html, so I can't check the compiler/MPI info there.

Copy link
Collaborator

@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.

This looks good to me. I'll update site configs that you do not have access to in a follow-up PR, possibly as part of the R2D2 MYSQL PR.

@ulmononian ulmononian marked this pull request as ready for review February 22, 2023 17:52
@ulmononian
Copy link
Collaborator Author

@climbfuji thanks for looking this over. i did not test any of this, but most of it was just re-arranging what was already present plus adding the intel 18 info (which i based on the core intel modules used in the respective global-workflow modules).

@climbfuji
Copy link
Collaborator

I think it would be good to test this as part of the R2D2 mysql PR, which requires updates to every site config. I am fine with merging your updates as they are, or pulling them into my PR.

@ulmononian
Copy link
Collaborator Author

ulmononian commented Feb 22, 2023

@climbfuji feel free to merge them as is from this pr. i will follow up with your r2d2 updates.

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.

None yet

2 participants