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

Conversation

@ggouaillardet
Copy link
Contributor

do not explicitly release ib verbs components since they will
be released in the object destructor

Thanks Durga for the report

(cherry picked from commit open-mpi/ompi@456b73d)

@ggouaillardet
Copy link
Contributor Author

:bot:assign: @miked-mellanox
:bot:milestone:v2.0.0
:bot🏷️bug

@ompiteam-bot ompiteam-bot added this to the v2.0.0 milestone May 13, 2016
ggouaillardet referenced this pull request in open-mpi/ompi May 13, 2016
do not explicitly release ib verbs components since they will
be released in the object destructor

Thanks Durga for the report
do not explicitly release ib verbs components since they will
be released in the object destructor

Thanks Durga for the report

(back-ported from commit open-mpi/ompi@456b73d)
@ggouaillardet ggouaillardet force-pushed the topic/v2.x/openib_error_path_cleanup branch from 0a55423 to 8a89d47 Compare May 13, 2016 00:24
@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@jsquyres
Copy link
Member

@ggouaillardet I think you meant to assign this to @jladd-mlnx -- Josh is the head of the MPI group now.

@jsquyres jsquyres assigned jladd-mlnx and unassigned mike-dubman May 16, 2016
@jsquyres
Copy link
Member

bot:retest

@jladd-mlnx
Copy link
Member

@jsquyres @ggouaillardet I'll review. Thanks.

@jladd-mlnx
Copy link
Member

👍
bot:label:reviewed

@ompiteam-bot
Copy link

OMPIBot error: Label "reviewed" is already set on issue 1160.

@mellanox-github
Copy link

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

@hppritcha hppritcha merged commit 2921f27 into open-mpi:v2.x May 16, 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.

7 participants