Project

General

Profile

Feature #16310

Feature #15308: LIBRADOS modify Pipe::connect() to return the error code

Take pipe::connect() returned errno to rados_connect() Part2 of http://tracker.ceph.com/issues/15308

Added by Vikhyat Umrao about 4 years ago. Updated over 1 year ago.

Status:
New
Priority:
Normal
Assignee:
Category:
-
Target version:
-
% Done:

0%

Source:
Support
Tags:
Backport:
Reviewed:
Affected Versions:
Pull request ID:

Description

- Take pipe::connect() returned errno to rados_connect() Part2 of http://tracker.ceph.com/issues/15308
- Part1 tracker http://tracker.ceph.com/issues/15308 PR : https://github.com/ceph/ceph/pull/8416

History

#1 Updated by Brad Hubbard about 4 years ago

  • Assignee set to Brad Hubbard

#2 Updated by Vikhyat Umrao about 4 years ago

  • Parent task set to #15308

#3 Updated by Greg Farnum over 1 year ago

Brad, do you know if there's something still missing here?

#4 Updated by Brad Hubbard over 1 year ago

Yes I think it still needs work Greg. I'm hoping to get to it next week. I'll bump it up my list.

#5 Updated by Brad Hubbard over 1 year ago

Vikhyat,

I don't think this is worth fixing in SimpleMessenger is it? I will concentrate on a fix for AsyncMessenger.

#6 Updated by Vikhyat Umrao over 1 year ago

Brad Hubbard wrote:

Vikhyat,

I don't think this is worth fixing in SimpleMessenger is it? I will concentrate on a fix for AsyncMessenger.

Brad - I agree I think AsyncMessanger should be the one as that is being used now. I think we also have Messenger V2
coming do we need there I mean hope they taking care of this or we need to check?

Thank you,
Vikhyat

Also available in: Atom PDF