Skip to content
This repository has been archived by the owner on Sep 30, 2022. It is now read-only.

Shmem fixes (Don't merge until #1041 is merged) #1329

Closed
wants to merge 10 commits into from
Closed

Shmem fixes (Don't merge until #1041 is merged) #1329

wants to merge 10 commits into from

Conversation

karasevb
Copy link
Member

@karasevb karasevb commented Aug 20, 2016

Cherry-picked from open-mpi/ompi#1867 and #1041
Don't merge until #1041 is merged.

@mellanox-github
Copy link

Test FAILed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-pr/2104/ for details.

@jladd-mlnx
Copy link
Member

@artpol84 Please have a look.

@artpol84
Copy link
Contributor

bot:milestone:v2.1.0

@ompiteam-bot ompiteam-bot added this to the v2.1.0 milestone Aug 22, 2016
@artpol84
Copy link
Contributor

bot:label:WIP-DNM

@jsquyres
Copy link
Member

bot:mellanox:retest

@mellanox-github
Copy link

Test FAILed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-pr/2129/ for details.

@artpol84
Copy link
Contributor

@jsquyres this PR is ok, but the base PR is failing.

@jsquyres
Copy link
Member

jsquyres commented Aug 26, 2016

@artpol84 Ah, you mean this one won't pass CI until #1041 is merged?

@artpol84
Copy link
Contributor

@jsquyres right!

@artpol84
Copy link
Contributor

At least I hope so - we'll see once #1041 will be merged and this commit will get a chance to be verified.

@jsquyres
Copy link
Member

bot:mellanox:retest

@mellanox-github
Copy link

Test FAILed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-pr/2169/ for details.

@jsquyres
Copy link
Member

@karasevb @artpol84 Getting a compile failure in in SPML UCX.

@artpol84
Copy link
Contributor

@jsquyres this is expected:

we depend on #1041 which in turn depends on #913. Won't get compiled untill those 2 are in.

@jsquyres
Copy link
Member

@artpol84 Ok

@jsquyres
Copy link
Member

The ompi-release repo is now closed; all release branches have been moved to the ompi repo (https://github.com/open-mpi/ompi). Please re-open this PR on the corresponding branch on the ompi repo.

@artpol84
Copy link
Contributor

ported to ompi (open-mpi/ompi#2127).
Please close this one to avoid confusion.

@jsquyres jsquyres closed this Sep 27, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants