Project

General

Profile

Actions

Feature #16310

open

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 almost 8 years ago. Updated about 5 years 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

Actions #1

Updated by Brad Hubbard almost 8 years ago

  • Assignee set to Brad Hubbard
Actions #2

Updated by Vikhyat Umrao over 7 years ago

  • Parent task set to #15308
Actions #3

Updated by Greg Farnum about 5 years ago

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

Actions #4

Updated by Brad Hubbard about 5 years 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.

Actions #5

Updated by Brad Hubbard about 5 years ago

Vikhyat,

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

Actions #6

Updated by Vikhyat Umrao about 5 years 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

Actions

Also available in: Atom PDF