Merge pull request ceph#43858 from rzarzynski/wip-msgr-no-cpp_error-on-read

idryomov · web-flow · commit 25cf30204989 · 2022-03-07T18:30:03.000+01:00
msg/async: remove the misleading error description on read failures

Reviewed-by: Casey Bodley &lt;cbodley@redhat.com&gt;
Reviewed-by: Ilya Dryomov &lt;idryomov@gmail.com&gt;
diff --git a/src/msg/async/ProtocolV2.cc b/src/msg/async/ProtocolV2.cc
@@ -1090,7 +1090,7 @@ CtPtr ProtocolV2::handle_read_frame_preamble_main(rx_buffer_t &&buffer, int r) {
 
   if (r < 0) {
     ldout(cct, 1) << __func__ << " read frame preamble failed r=" << r
-                  << " (" << cpp_strerror(r) << ")" << dendl;
+                  << dendl;
     return _fault();
   }
 

Original file line number	Diff line number	Diff line change
`@@ -1090,7 +1090,7 @@ CtPtr ProtocolV2::handle_read_frame_preamble_main(rx_buffer_t &&buffer, int r) {`
`1090`	`1090`
`1091`	`1091`	`if (r < 0) {`
`1092`	`1092`	`ldout(cct, 1) << __func__ << " read frame preamble failed r=" << r`
`1093`		`- << " (" << cpp_strerror(r) << ")" << dendl;`
	`1093`	`+ << dendl;`
`1094`	`1094`	`return _fault();`
`1095`	`1095`	`}`
`1096`	`1096`