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

jewel: librados: modify Pipe::connect() to return the error code #11193

Merged
merged 1 commit into from Oct 14, 2016

Conversation

smithfarm
Copy link
Contributor

Fixes: http://tracker.ceph.com/issues/15308

Signed-off-by: Vikhyat Umrao <vumrao@redhat.com>
(cherry picked from commit cad38dc)
@smithfarm smithfarm self-assigned this Sep 22, 2016
@smithfarm smithfarm added this to the jewel milestone Sep 22, 2016
@ghost
Copy link

ghost commented Oct 10, 2016

jenkins test this please (logs are gone)

ghost pushed a commit that referenced this pull request Oct 10, 2016
…return the error code

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Oct 10, 2016
…return the error code

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Oct 13, 2016
…return the error code

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost ghost assigned liewegas Oct 14, 2016
@ghost
Copy link

ghost commented Oct 14, 2016

@liewegas does this backport look good to merge ? It passed the rados (http://tracker.ceph.com/issues/17487#note-12), upgrade/jewel-x and upgrade/hammer-x (http://tracker.ceph.com/issues/17487#note-15) suites (except for two bugs that are, I believe, unrelated)

@liewegas liewegas merged commit 952a81a into ceph:jewel Oct 14, 2016
@theanalyst theanalyst changed the title jewel: LIBRADOS modify Pipe::connect() to return the error code "jewel: librados: modify Pipe::connect() to return the error code" Nov 17, 2016
@theanalyst theanalyst changed the title "jewel: librados: modify Pipe::connect() to return the error code" jewel: librados: modify Pipe::connect() to return the error code Nov 17, 2016
@smithfarm smithfarm deleted the wip-17377-jewel branch December 8, 2016 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants