From: Yingxin Cheng <yingxin.cheng@intel.com>
Date: Sat, 12 Oct 2019 08:18:00 +0000 (+0800)
Subject: test/crimson: send message only after getting the connection
X-Git-Tag: v15.1.0~1210^2~4
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=7c5d55bd383dc63f708486b94dd494f78adc73c2;p=ceph-ci.git

test/crimson: send message only after getting the connection

It is possible for stateless/stateful server to lose the pending message
due to the remote markdown, causing unexpected test failure.

Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
---

diff --git a/src/test/crimson/test_messenger.cc b/src/test/crimson/test_messenger.cc
index 3046c8090ff..3d6aeb6744d 100644
--- a/src/test/crimson/test_messenger.cc
+++ b/src/test/crimson/test_messenger.cc
@@ -2945,8 +2945,8 @@ test_v2_lossy_client(FailoverTest& test) {
       return suite.markdown();
     }).then([&suite] {
       return suite.connect_peer();
-    }).then([&test] {
-      return test.send_bidirectional();
+    }).then([&suite] {
+      return suite.send_peer();
     }).then([&suite] {
       return suite.wait_results(2);
     }).then([] (ConnResults& results) {
@@ -2995,7 +2995,7 @@ test_v2_stateless_server(FailoverTest& test) {
     }).then([&test] {
       return test.peer_connect_me();
     }).then([&test] {
-      return test.send_bidirectional();
+      return test.peer_send_me();
     }).then([&suite] {
       return suite.wait_results(2);
     }).then([] (ConnResults& results) {