-
Notifications
You must be signed in to change notification settings - Fork 865
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
mindist mapper broken on master #1623
Comments
I see it in Brice's unmerged PR too. HWLOC change makes sense, but how has it crept into master? Don't see it on 2.x. Strange... |
@jladd-mlnx Did you git bisect and find that the place where it came into master was the commit that came from #1612? |
No, just combing through closed PRs to see where it first showed up. It may have nothing to do with this commit. It shows up in Brice's unmerged HWLOC update - which would make more sense, but then I don't understand how it would have leaked in master. Not sure I will have time to chase much down today. If I get a few spare cycles, I can bisect. |
Discussed on teleconf today: @rhc54 is going to have a look, but will need help in testing it because he doesn't have any HCAs. |
@jladd-mlnx @Di0gen Can you temporarily disable the mindist test while we know that it is broken? It's making all other PR's fail the Mellanox Jenkins test. We can re-enable it when mindist gets fixed. |
Done. I plan to re-enable it in one week. On Wed, May 4, 2016 at 6:58 AM, Jeff Squyres [email protected]
|
@jladd-mlnx does mellanox Jenkins explicitly configure ompi with external pmix ? |
Yes, that would certainly be true. However, someone on the call looked back at the logs and claimed it started with a different PR, so there is some uncertainty here. |
A bunch of Mellanox Jenkins runs have been failing on master with this kind of error:
In #1612 (comment), @jladd-mlnx reported that the PMIx external component changes (i.e., the changes from that PR) seem to be what broke this test.
@rhc54 Can you investigate?
The text was updated successfully, but these errors were encountered: