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

Check that requests belong to similar sessions #12691

Merged
merged 1 commit into from
Jul 18, 2024

Conversation

bosilca
Copy link
Member

@bosilca bosilca commented Jul 18, 2024

Consistent way to check that an array of requests belong to the same session or are predefined. Skip all requests generated by components that do not support instances (generalized requests, file, win).

Replaces #12525.

Consistent way to check that an array of requests belong to the same
session or are predefined. Skip all requests generated by components
that do not support instances (generalized requests, file, win).

Replaces open-mpi#12525.

Signed-off-by: George Bosilca <[email protected]>
@bosilca bosilca added the bug label Jul 18, 2024
@bosilca bosilca added this to the v5.0.4 milestone Jul 18, 2024
@bosilca bosilca self-assigned this Jul 18, 2024
@wenduwan wenduwan removed this from the v5.0.4 milestone Jul 18, 2024
Copy link
Member

@edgargabriel edgargabriel left a comment

Choose a reason for hiding this comment

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

this PR seems to fix the issue that I observed, tested with pml, file, and win requests

@edgargabriel edgargabriel added this to the v5.0.5 milestone Jul 18, 2024
@bosilca bosilca merged commit a6d5b36 into open-mpi:main Jul 18, 2024
16 checks passed
@bosilca bosilca deleted the topic/request_single_instance_fix branch July 18, 2024 18:43
@edgargabriel
Copy link
Member

@bosilca can you backport this to the 5.,0.x branch please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants