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

Add UCX support #718

Merged
merged 9 commits into from
Nov 2, 2015
Merged

Add UCX support #718

merged 9 commits into from
Nov 2, 2015

Conversation

@yosefe
Copy link
Contributor

@yosefe yosefe commented Oct 26, 2015

@hjelmn
Copy link
Member

hjelmn commented Oct 26, 2015

Don't forget to set the milestone and labels. Also assign a reviewer.

@yosefe
Copy link
Contributor Author

yosefe commented Oct 26, 2015

@hjelmn looks like i'm not able to do that (no enough permissions)?

@mellanox-github
Copy link

mellanox-github commented Oct 26, 2015

Test PASSed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-v1.8-pr/927/ for details.

@hppritcha hppritcha added this to the v2.0.0 milestone Oct 26, 2015
@hjelmn
Copy link
Member

hjelmn commented Oct 26, 2015

Use the bot commands: https://github.com/open-mpi/ompi/wiki/OmpiReleaseBotCommands

@yosefe
Copy link
Contributor Author

yosefe commented Oct 26, 2015

bot:label:enhancement
bot:assign:@bosilca

@ompiteam-bot
Copy link

ompiteam-bot commented Oct 26, 2015

OMPIBot error: Label "feature" does not exist.

@yosefe
Copy link
Contributor Author

yosefe commented Oct 26, 2015

bot:label:enhancement

@bosilca
Copy link
Member

bosilca commented Oct 26, 2015

From a technical perspective I have nothing more (than open-mpi/ompi#1008) to add. I would appreciate few nights of MTT before moving forward with this.

@yosefe
Copy link
Contributor Author

yosefe commented Oct 28, 2015

adding @shamisp

@mellanox-github
Copy link

mellanox-github commented Oct 28, 2015

Test PASSed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-v1.8-pr/940/ for details.

@hppritcha
Copy link
Member

hppritcha commented Oct 28, 2015

@yosefe is there a corresponding sha on master for yosefe@b61bc0e ?

@yosefe
Copy link
Contributor Author

yosefe commented Oct 28, 2015

@hppritcha i've opened PR for this: open-mpi/ompi#1076

@mellanox-github
Copy link

mellanox-github commented Oct 28, 2015

Test PASSed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-v1.8-pr/942/ for details.

@hppritcha
Copy link
Member

hppritcha commented Oct 29, 2015

@yosefe the lanl build bot failure in 1076 is "spurious". For whatever reason, the mpirun launches on the Univ. Houston server that runs that test periodically hang. It can be ignored.

Is this pr ready to go?

@hppritcha
Copy link
Member

hppritcha commented Oct 30, 2015

Issue open-mpi/ompi#982 should be closed when this PR is merged.

@yosefe
Copy link
Contributor Author

yosefe commented Nov 1, 2015

@hppritcha Our MTT which ran IMB-datacheck and mpi_test_suite passed with this PR, so IMHO we can merge it.
We may find more bugs in UCX library, but this will most likely not require fixes in OpenMPI code.

hppritcha added a commit that referenced this pull request Nov 2, 2015
@hppritcha
Merge pull request #718 from yosefe/topic/ucx_support
7801098 
Add UCX support
@hppritcha hppritcha merged commit 7801098 into open-mpi:v2.x Nov 2, 2015
@yosefe yosefe deleted the topic/ucx_support branch November 3, 2015 08:18
alinask pushed a commit to alinask/ompi-release that referenced this pull request Dec 10, 2015
@ggouaillardet
Merge pull request open-mpi#718 from ggouaillardet/topic/configury_xrc_4
86c3fa8 
Topic/configury xrc 4
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

@bosilca bosilca

Labels

enhancement RM-Approved

Projects

None yet

Milestone

v2.0.0

Development

Successfully merging this pull request may close these issues.

7 participants

@yosefe @hjelmn @mellanox-github @ompiteam-bot @bosilca @hppritcha @alex-mikheev