-
Notifications
You must be signed in to change notification settings - Fork 64
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
Add openmpi install to CI #551
Conversation
@grantfirl @mkavulich @scrasmussen |
That's great! I know this is pedantic but we'll probably want to include that in one of the upcoming ufs prs, even though this clearly doesn't affect anything but CI. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@mkavulich This should not have been merged into main. All PRs targeting the main branch need to go through host model testing before getting the okay to merge. |
I created ufs-community/ufs-weather-model#2216 to include this change. No need to roll back I think because this PR shouldn't affect anything in the UFS. But let's try to avoid merging into main w/o full testing (see also @peverwhee's issue about creating a develop branch with a smaller testing harness) |
This PR adds to the CCPP Prebuild CI test to install open-mpi v4.1.6, which is now a requirement for CCPP.