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

Skip some parallel tests for OpenMPI version 5.0.5 #5151

Merged
merged 1 commit into from
Nov 27, 2024

Conversation

jhendersonHDF
Copy link
Collaborator

An issue in the 5.0.5 release of OpenMPI causes some parallel HDF5 tests to fail. The issue was resolved in the OpenMPI 5.0.6 release.

An issue in the 5.0.5 release of OpenMPI causes some parallel
HDF5 tests to fail. The issue was resolved in the OpenMPI 5.0.6
release.
@jhendersonHDF jhendersonHDF added Priority - 3. Low 🔽 Code cleanup, small feature change requests, etc. Component - Parallel Parallel HDF5 (NOT thread-safety) Component - Testing Code in test or testpar directories, GitHub workflows Type - Improvement Improvements that don't add a new feature or functionality labels Nov 26, 2024
* The issue was fixed for the 5.0.6 release.
*/
#if defined(OPEN_MPI) && defined(OMPI_MAJOR_VERSION) && (OMPI_MAJOR_VERSION == 5) && \
defined(OMPI_MINOR_VERSION) && (OMPI_MINOR_VERSION == 0) && defined(OMPI_RELEASE_VERSION) && \
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It would probably make sense to have a macro for MPI library version checking in case we need to do the same for an MPICH release in the future, but it didn't seem too worth it yet.

@lrknox lrknox merged commit 090e604 into HDFGroup:develop Nov 27, 2024
67 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component - Parallel Parallel HDF5 (NOT thread-safety) Component - Testing Code in test or testpar directories, GitHub workflows Priority - 3. Low 🔽 Code cleanup, small feature change requests, etc. Type - Improvement Improvements that don't add a new feature or functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants