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

Multiple commits #1556

Merged
merged 10 commits into from
Oct 14, 2022
Merged

Multiple commits #1556

merged 10 commits into from
Oct 14, 2022

Conversation

rhc54
Copy link
Contributor

@rhc54 rhc54 commented Oct 14, 2022

Support query of pset membership

Ensure we pass pset membership when registering the job. Support
backend queries of membership. Add example that exercises the code.

Signed-off-by: Ralph Castain rhc@pmix.org
(cherry picked from commit f756956)

Cleanup grpcomm cruft

Remove unused functions and entry points

Signed-off-by: Ralph Castain rhc@pmix.org
(cherry picked from commit 9e833e1)

remove some unused vars

Signed-off-by: Thomas Naughton naughtont@ornl.gov
(cherry picked from commit 793e3bc)

squash warnings

Signed-off-by: Thomas Naughton naughtont@ornl.gov
(cherry picked from commit 77c9359)

unused var

Signed-off-by: Thomas Naughton naughtont@ornl.gov
(cherry picked from commit 10b256e)

more unused vars

Signed-off-by: Thomas Naughton naughtont@ornl.gov
(cherry picked from commit e45def7)

remove unused vars and fix rc/ret typo

Signed-off-by: Thomas Naughton naughtont@ornl.gov
(cherry picked from commit f80a882)

misc unused var cleanups

Signed-off-by: Thomas Naughton naughtont@ornl.gov
(cherry picked from commit 9c48d62)

Reduce debugger confusion

Having two options that apply to all ranks and another
nearly-identical one that only applies to specified ranks
is confusing. It also is questionable as to whether it makes
any sense to have some ranks run free while others are told
to stop on exec.

So just simplify things and have all three options be boolean.

Signed-off-by: Ralph Castain rhc@pmix.org
(cherry picked from commit 004d0e8)

Pickup missing changes

Bring the branch up to target version

Signed-off-by: Ralph Castain rhc@pmix.org

bot:notacherrypick

rhc54 and others added 10 commits October 14, 2022 07:36
Ensure we pass pset membership when registering the job. Support
backend queries of membership. Add example that exercises the code.

Signed-off-by: Ralph Castain <rhc@pmix.org>
(cherry picked from commit f756956)
Remove unused functions and entry points

Signed-off-by: Ralph Castain <rhc@pmix.org>
(cherry picked from commit 9e833e1)
Signed-off-by: Thomas Naughton <naughtont@ornl.gov>
(cherry picked from commit 793e3bc)
Signed-off-by: Thomas Naughton <naughtont@ornl.gov>
(cherry picked from commit 77c9359)
Signed-off-by: Thomas Naughton <naughtont@ornl.gov>
(cherry picked from commit 10b256e)
Signed-off-by: Thomas Naughton <naughtont@ornl.gov>
(cherry picked from commit e45def7)
Signed-off-by: Thomas Naughton <naughtont@ornl.gov>
(cherry picked from commit f80a882)
Signed-off-by: Thomas Naughton <naughtont@ornl.gov>
(cherry picked from commit 9c48d62)
Having two options that apply to all ranks and another
nearly-identical one that only applies to specified ranks
is confusing. It also is questionable as to whether it makes
any sense to have some ranks run free while others are told
to stop on exec.

So just simplify things and have all three options be boolean.

Signed-off-by: Ralph Castain <rhc@pmix.org>
(cherry picked from commit 004d0e8)
Bring the branch up to target version

Signed-off-by: Ralph Castain <rhc@pmix.org>
@rhc54 rhc54 merged commit 34735ca into openpmix:v3.0 Oct 14, 2022
@rhc54 rhc54 deleted the cmr30/up branch October 14, 2022 15:12
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.

2 participants