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

Update schizo framework #2365

Closed
wants to merge 1 commit into from
Closed

Update schizo framework #2365

wants to merge 1 commit into from

Conversation

rhc54
Copy link
Contributor

@rhc54 rhc54 commented Nov 4, 2016

Update the schizo framework to support checking the environment during startup. This allows better detection of singleton scenarios, and provides a mechanism by which we can recognize and support SLURM-imposed bindings.

Discussed with @jsquyres for pre-approval - addresses problem reported on the user list:

https://mail-archive.com/users@lists.open-mpi.org/msg30212.html

Signed-off-by: Ralph Castain rhc@open-mpi.org

@rhc54 rhc54 added this to the v2.1.0 milestone Nov 4, 2016
@hppritcha
Copy link
Member

I'd like to do some testing in SLURM and ALPS environments before merging.

orte_init. This allows schizo to better determine singleton operations
in various environments, and to detect SLURM-side binding has occurred
to avoid ORTE attempting to override it.

Signed-off-by: Ralph Castain <rhc@open-mpi.org>
@hppritcha hppritcha self-requested a review February 6, 2017 20:53
@jsquyres jsquyres assigned rhc54 and unassigned jsquyres Feb 6, 2017
@jsquyres
Copy link
Member

jsquyres commented Feb 6, 2017

@rhc54 Sorry -- we let this get stale, and it now has conflicts. Can you resolve?

@hppritcha
Copy link
Member

actually I can't test this right now because I don't know where to pull the branch from. I'd rather do that then pull from the patch file, esp. since there are conflicts with v2.x

@jsquyres
Copy link
Member

Yo @rhc54 -- this is a new one to me: GitHub can't find the repo from which you created this PR (!). Do you need to re-create this PR? (there's conflicts that need to be fixed, too)

@rhc54
Copy link
Contributor Author

rhc54 commented Feb 13, 2017

This has sat so long it might have been done off the old release repo - I'll just kill it and forget it.

@rhc54 rhc54 closed this Feb 13, 2017
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