From: Kefu Chai <kchai@redhat.com>
Date: Tue, 21 Aug 2018 08:18:24 +0000 (+0800)
Subject: test/crimson: exercise keepalive also
X-Git-Tag: v14.0.1~526^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F23642%2Fhead;p=ceph.git

test/crimson: exercise keepalive also

allow one to pass options to unittest_seastar_messenger for specifying
- verbose
- keepalive-ratio
- rounds

so we can have better coverage of crimson/messenger. and make it less
chatty unless it's asked to do so.

Signed-off-by: Kefu Chai <kchai@redhat.com>
---

diff --git a/src/test/crimson/test_messenger.cc b/src/test/crimson/test_messenger.cc
index 13f55822199b..166a9180d8d4 100644
--- a/src/test/crimson/test_messenger.cc
+++ b/src/test/crimson/test_messenger.cc
@@ -2,11 +2,21 @@
 #include "crimson/net/Connection.h"
 #include "crimson/net/Dispatcher.h"
 #include "crimson/net/SocketMessenger.h"
+
+#include <random>
+#include <boost/program_options.hpp>
 #include <seastar/core/app-template.hh>
 #include <seastar/core/future-util.hh>
 #include <seastar/core/reactor.hh>
 
-static seastar::future<> test_echo()
+namespace bpo = boost::program_options;
+
+static std::random_device rd;
+static std::default_random_engine rng{rd()};
+static bool verbose = false;
+
+static seastar::future<> test_echo(unsigned rounds,
+                                   double keepalive_ratio)
 {
   struct test_state {
     entity_addr_t addr;
@@ -16,7 +26,9 @@ static seastar::future<> test_echo()
       struct ServerDispatcher : ceph::net::Dispatcher {
         seastar::future<> ms_dispatch(ceph::net::ConnectionRef c,
                                       MessageRef m) override {
-          std::cout << "server got " << *m << std::endl;
+          if (verbose) {
+            std::cout << "server got " << *m << std::endl;
+          }
           // reply with a pong
           return c->send(MessageRef{new MPing(), false});
         }
@@ -24,42 +36,86 @@ static seastar::future<> test_echo()
     } server;
 
     struct {
+      unsigned rounds;
+      std::bernoulli_distribution keepalive_dist{};
       ceph::net::SocketMessenger messenger{entity_name_t::OSD(0)};
       struct ClientDispatcher : ceph::net::Dispatcher {
         seastar::promise<MessageRef> reply;
+        unsigned count = 0u;
         seastar::future<> ms_dispatch(ceph::net::ConnectionRef c,
                                       MessageRef m) override {
+          ++count;
+          if (verbose) {
+            std::cout << "client ms_dispatch " << count << std::endl;
+          }
           reply.set_value(std::move(m));
           return seastar::now();
         }
       } dispatcher;
+      seastar::future<> pingpong(ceph::net::ConnectionRef c) {
+        return seastar::repeat([conn=std::move(c), this] {
+          if (keepalive_dist(rng)) {
+            return conn->keepalive().then([] {
+              return seastar::make_ready_future<seastar::stop_iteration>(
+                seastar::stop_iteration::no);
+              });
+          } else {
+            return conn->send(MessageRef{new MPing(), false}).then([&] {
+              return dispatcher.reply.get_future();
+            }).then([&] (MessageRef msg) {
+              dispatcher.reply = seastar::promise<MessageRef>{};
+              if (verbose) {
+                std::cout << "client got reply " << *msg << std::endl;
+              }
+              return seastar::make_ready_future<seastar::stop_iteration>(
+                  seastar::stop_iteration::yes);
+            });
+          };
+        });
+      }
+      bool done() const {
+        return dispatcher.count >= rounds;
+      }
     } client;
   };
   return seastar::do_with(test_state{},
-    [] (test_state& t) {
+    [rounds, keepalive_ratio] (test_state& t) {
       // bind the server
       t.addr.set_family(AF_INET);
       t.addr.set_port(9010);
       t.server.messenger.bind(t.addr);
 
+      t.client.rounds = rounds;
+      t.client.keepalive_dist = std::bernoulli_distribution{keepalive_ratio};
+
       return t.server.messenger.start(&t.server.dispatcher)
         .then([&] {
           return t.client.messenger.start(&t.client.dispatcher)
             .then([&] {
-              return t.client.messenger.connect(t.addr, entity_name_t::TYPE_OSD);
-            }).then([] (ceph::net::ConnectionRef conn) {
-              std::cout << "client connected" << std::endl;
-              return conn->send(MessageRef{new MPing(), false});
-            }).then([&] {
-              return t.client.dispatcher.reply.get_future();
-            }).then([&] (MessageRef msg) {
-              std::cout << "client got reply " << *msg << std::endl;
+              return t.client.messenger.connect(t.addr,
+                                                entity_name_t::TYPE_OSD);
+            }).then([&client=t.client] (ceph::net::ConnectionRef conn) {
+              if (verbose) {
+                std::cout << "client connected" << std::endl;
+              }
+              return seastar::repeat([&client,conn=std::move(conn)] {
+                return client.pingpong(conn).then([&client] {
+                  return seastar::make_ready_future<seastar::stop_iteration>(
+                    client.done() ?
+                    seastar::stop_iteration::yes :
+                    seastar::stop_iteration::no);
+                });
+              });
             }).finally([&] {
-              std::cout << "client shutting down" << std::endl;
+              if (verbose) {
+                std::cout << "client shutting down" << std::endl;
+              }
               return t.client.messenger.shutdown();
             });
         }).finally([&] {
-          std::cout << "server shutting down" << std::endl;
+          if (verbose) {
+            std::cout << "server shutting down" << std::endl;
+          }
           return t.server.messenger.shutdown();
         });
     });
@@ -68,8 +124,19 @@ static seastar::future<> test_echo()
 int main(int argc, char** argv)
 {
   seastar::app_template app;
-  return app.run(argc, argv, [] {
-    return test_echo().then([] {
+  app.add_options()
+    ("verbose,v", bpo::value<bool>()->default_value(false),
+     "chatty if true")
+    ("rounds", bpo::value<unsigned>()->default_value(512),
+     "number of pingpong rounds")
+    ("keepalive-ratio", bpo::value<double>()->default_value(0.1),
+     "ratio of keepalive in ping messages");
+  return app.run(argc, argv, [&] {
+    auto&& config = app.configuration();
+    verbose = config["verbose"].as<bool>();
+    auto rounds = config["rounds"].as<unsigned>();
+    auto keepalive_ratio = config["keepalive-ratio"].as<double>();
+    return test_echo(rounds, keepalive_ratio).then([] {
       std::cout << "All tests succeeded" << std::endl;
     }).handle_exception([] (auto eptr) {
       std::cout << "Test failure" << std::endl;