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

sessions: add smoke test #12893

Merged
merged 2 commits into from
Oct 31, 2024
Merged

Conversation

hppritcha
Copy link
Member

and add to mac os github CI

Signed-off-by: Howard Pritchard [email protected]
(cherry picked from commit 9109c33)

we need this PR before another one with a bug fix can be pushed to v5.0.x

@github-actions github-actions bot added this to the v5.0.6 milestone Oct 29, 2024
@hppritcha hppritcha requested a review from janjust October 29, 2024 18:12
examples/hello_sessions_c.c Outdated Show resolved Hide resolved
examples/hello_sessions_c.c Show resolved Hide resolved
@hppritcha
Copy link
Member Author

this PR will not pass CI until #12883 is merged into the v5.0.x branch.

@hppritcha hppritcha force-pushed the pr12871_back_to_v50x branch from 912239d to 0d3fe46 Compare October 30, 2024 18:33
@hppritcha hppritcha requested a review from jsquyres October 30, 2024 18:34
@jsquyres
Copy link
Member

Amusing -- your new test is failing on this PR. 😆

@janjust
Copy link
Contributor

janjust commented Oct 30, 2024

this PR will not pass CI until #12883 is merged into the v5.0.x branch.

done, merged

and add to mac os github CI

Signed-off-by: Howard Pritchard <[email protected]>
(cherry picked from commit 9109c33)
when there are no more refs to opal util layer.

Update sessions smoke test to illustrate the issue
in case this change gets reverted or undone by some
future commit.

examples: add return 0 to sessions smoke test

Signed-off-by: Howard Pritchard <[email protected]>
(cherry picked from commit 70e51fe)
(cherry picked from commit 6cf4dbe)
@hppritcha hppritcha force-pushed the pr12871_back_to_v50x branch from 0d3fe46 to be59665 Compare October 30, 2024 22:06
@janjust janjust merged commit c87f646 into open-mpi:v5.0.x Oct 31, 2024
15 checks passed
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