ucx icon indicating copy to clipboard operation
ucx copied to clipboard

UCP/PROTO: Move non-rndv protocols to use probe() instead of init()

Open yosefe opened this issue 2 years ago • 1 comments

Why

Next phase of adding protocol variants, moving from init() to probe()

yosefe avatar Feb 18 '24 17:02 yosefe

@ivankochin @brminich can you pls review?

yosefe avatar Feb 20 '24 10:02 yosefe

Not directly connected to this patch, but anyway. As I understand, at the current moment, failing of probing or initialization doesn't entail any traces printing. Maybe it worth adding some logging which would print status of initialization or how many variants were added by probe? e.g. extend ucp_proto_select_add_proto

ivankochin avatar Feb 21 '24 09:02 ivankochin

/azp run UCX PR

yosefe avatar Mar 12 '24 10:03 yosefe

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Mar 12 '24 10:03 azure-pipelines[bot]