X-Git-Url: https://git.m6w6.name/?a=blobdiff_plain;ds=sidebyside;f=libtest%2Fmemcached.cc;h=3b63c0fc485e08f63c077879ff844e0ae203dae8;hb=a6253e0a9ee914db2e55680a83cd03fdd5a2bc40;hp=04e6986069e463123a1142b7e87ec32efc603c97;hpb=46ce3491f10a4cdd069e17c2b1cf598ba367a9f0;p=awesomized%2Flibmemcached

diff --git a/libtest/memcached.cc b/libtest/memcached.cc
index 04e69860..3b63c0fc 100644
--- a/libtest/memcached.cc
+++ b/libtest/memcached.cc
@@ -1,9 +1,8 @@
 /*  vim:expandtab:shiftwidth=2:tabstop=2:smarttab:
  * 
- *  Libmemcached library
+ *  libtest
  *
  *  Copyright (C) 2011 Data Differential, http://datadifferential.com/
- *  Copyright (C) 2006-2009 Brian Aker All rights reserved.
  *
  *  Redistribution and use in source and binary forms, with or without
  *  modification, are permitted provided that the following conditions are
@@ -35,230 +34,189 @@
  *
  */
 
+#include <libtest/common.h>
 
-/*
-  Startup, and shutdown the memcached servers.
-*/
-
-#define TEST_PORT_BASE MEMCACHED_DEFAULT_PORT+10
+#include <libmemcached/memcached.h>
+#include <libmemcached/util.h>
 
-#include <libtest/common.h>
+using namespace libtest;
 
 #include <cassert>
 #include <cerrno>
 #include <cstdio>
 #include <cstdlib>
 #include <cstring>
-#include <ctime>
-#include <limits.h>
+#include <iostream>
 #include <signal.h>
-#include <sys/time.h>
+#include <sys/types.h>
+#include <sys/wait.h>
 #include <unistd.h>
-#include <iostream>
-
-#include <libmemcached/memcached.h>
-#include <libmemcached/util.h>
 
 #include <libtest/server.h>
-#include <libtest/killpid.h>
 #include <libtest/wait.h>
 
-#define CERR_PREFIX std::endl << __FILE__ << ":" << __LINE__ << " "
+#include <libtest/memcached.h>
 
-#define SOCKET_FILE "/tmp/memcached.socket"
+#ifndef __INTEL_COMPILER
+#pragma GCC diagnostic ignored "-Wold-style-cast"
+#endif
 
-static pid_t __getpid(server_st& server)
-{
-  memcached_return_t rc;
-  pid_t pid= libmemcached_util_getpid(server.hostname(), server.port(), &rc);
-  return pid;
-}
+using namespace libtest;
 
-static bool __ping(server_st& server)
+class Memcached : public Server
 {
-  memcached_return_t rc;
-  bool ret= libmemcached_util_ping(server.hostname(), server.port(), &rc);
-  return ret;
-}
+public:
+  Memcached(const std::string& host_arg, const in_port_t port_arg, const bool is_socket_arg) :
+    Server(host_arg, port_arg, is_socket_arg)
+  { }
 
-static bool cycle_server(server_st  *server)
-{
-  while (1)
+  pid_t get_pid(bool error_is_ok)
   {
-    if (libmemcached_util_ping(server->hostname(), server->port(), NULL))
+    // Memcached is slow to start, so we need to do this
+    if (not pid_file().empty())
     {
-      // First we try to kill it, and on fail of that we flush it.
-      pid_t pid= libmemcached_util_getpid(server->hostname(), server->port(), NULL);
+      Wait wait(pid_file(), 0);
 
-      if (pid > 0 and kill_pid(pid))
+      if (error_is_ok and not wait.successful())
       {
-        std::cerr << CERR_PREFIX << "Killed existing server," << *server << " with pid:" << pid << std::endl;
-        continue;
-      }
-      else if (libmemcached_util_flush(server->hostname(), server->port(), NULL)) // If we can flush it, we will just use it
-      { 
-        std::cerr << CERR_PREFIX << "Found server on port " << int(server->port()) << ", flushed it!" << std::endl;
-        server->set_used();
-        return true;
-      } // No idea what is wrong here, so we need to find a different port
-      else
-      {
-        return false;
+        Error << "Pidfile was not found:" << pid_file();
+        return -1;
       }
     }
 
-    break;
-  }
+    pid_t local_pid;
+    memcached_return_t rc;
+    if (has_socket())
+    {
+      local_pid= libmemcached_util_getpid(socket().c_str(), port(), &rc);
+    }
+    else
+    {
+      local_pid= libmemcached_util_getpid(hostname().c_str(), port(), &rc);
+    }
 
-  return true;
-}
+    if (error_is_ok and ((memcached_failed(rc) or local_pid < 1)))
+    {
+      Error << "libmemcached_util_getpid(" << memcached_strerror(NULL, rc) << ") pid: " << local_pid << " for:" << *this;
+    }
 
-bool server_startup(server_startup_st *construct)
-{
-  if (getenv(((char *)"MEMCACHED_SERVERS")))
-  {
-    construct->server_list= getenv(((char *)"MEMCACHED_SERVERS"));
-    printf("servers %s\n", construct->server_list.c_str());
-    construct->count= 0;
+    return local_pid;
   }
-  else
-  {
-    std::string server_config_string;
 
-    uint32_t port_base= 0;
-    for (uint32_t x= 0; x < (construct->count -1); x++)
+  bool ping()
+  {
+    // Memcached is slow to start, so we need to do this
+    if (not pid_file().empty())
     {
-      server_st *server= NULL;
+      Wait wait(pid_file(), 0);
 
+      if (not wait.successful())
       {
-        char *var;
-        char variable_buffer[1024];
-
-        snprintf(variable_buffer, sizeof(variable_buffer), "LIBMEMCACHED_PORT_%u", x);
-
-        if ((var= getenv(variable_buffer)))
-        {
-          server= new server_st((in_port_t)atoi(var), __getpid, __ping);
-        }
-        else
-        {
-          server= new server_st(in_port_t(x +TEST_PORT_BASE +port_base), __getpid, __ping);
-
-          while (not cycle_server(server))
-          {
-            std::cerr << CERR_PREFIX << "Found server " << *server << ", could not flush it, so trying next port." << std::endl;
-            port_base++;
-            server->set_port(in_port_t(x +TEST_PORT_BASE +port_base));
-          }
-        }
+        Error << "Pidfile was not found:" << pid_file();
+        return -1;
       }
+    }
 
-      if (server->is_used())
-      {
-        std::cerr << std::endl << "Using server at : " << server << std::endl;
-      }
-      else
-      {
-        char buffer[FILENAME_MAX];
-        if (x == 0)
-        {
-          snprintf(buffer, sizeof(buffer), "%s -d -t 1 -p %u -U %u -m 128",
-                   MEMCACHED_BINARY, server->port(), server->port());
-        }
-        else
-        {
-          snprintf(buffer, sizeof(buffer), "%s -d -t 1 -p %u -U %u",
-                   MEMCACHED_BINARY, server->port(), server->port());
-        }
-        server->set_command(buffer);
-
-        if (not server->start())
-        {
-          std::cerr << CERR_PREFIX << "Failed system(" << buffer << ")" << std::endl;
-          delete server;
-          return false;
-        }
-        std::cerr << "STARTING SERVER: " << buffer << " pid:" << server->pid() << std::endl;
-      }
-      construct->push_server(server);
+    memcached_return_t rc;
+    bool ret;
+    if (has_socket())
+    {
+      ret= libmemcached_util_ping(socket().c_str(), 0, &rc);
+    }
+    else
+    {
+      ret= libmemcached_util_ping(hostname().c_str(), port(), &rc);
+    }
 
-      if (x == 0)
-      {
-        assert(server->has_port());
-        set_default_port(server->port());
-      }
+    if (memcached_failed(rc) or not ret)
+    {
+      Error << "libmemcached_util_ping(" << memcached_strerror(NULL, rc) << ")";
+    }
+    return ret;
+  }
+
+  const char *name()
+  {
+    return "memcached";
+  };
 
-      char port_str[NI_MAXSERV];
-      snprintf(port_str, sizeof(port_str), "%u", int(server->port()));
+  const char *executable()
+  {
+    return MEMCACHED_BINARY;
+  }
 
-      server_config_string+= "--server=";
-      server_config_string+= server->hostname();
-      server_config_string+= ":";
-      server_config_string+= port_str;
-      server_config_string+= " ";
-    }
+  const char *pid_file_option()
+  {
+    return "-P ";
+  }
 
-    // Socket
-    {
+  const char *socket_file_option() const
+  {
+    return "-s ";
+  }
 
-      std::string socket_file(SOCKET_FILE);
-      char *var;
+  const char *daemon_file_option()
+  {
+    return "-d";
+  }
 
-      if ((var= getenv("LIBMEMCACHED_SOCKET")))
-      {
-        socket_file= var;
-      }
+  const char *log_file_option()
+  {
+    return NULL;
+  }
 
-      server_st *server= new server_st(SOCKET_FILE, __getpid, __ping);
+  const char *port_option()
+  {
+    return "-p ";
+  }
 
-      if (not cycle_server(server))
-      {
-        std::cerr << CERR_PREFIX << "Found server " << server << ", could not flush it, failing since socket file is not available." << std::endl;
-        return false;
-      }
+  bool is_libtool()
+  {
+    return false;
+  }
 
-      if (server->is_used())
-      {
-        std::cerr << std::endl << "Using server at : " << *server << std::endl;
-      }
-      else
-      {
-        char buffer[FILENAME_MAX];
-        snprintf(buffer, sizeof(buffer), "%s -d -t 1 -s %s", MEMCACHED_BINARY, SOCKET_FILE);
-        server->set_command(buffer);
-
-        if (not server->start())
-        {
-          std::cerr << CERR_PREFIX << "Failed system(" << buffer << ")" << std::endl;
-          delete server;
-          return false;
-        }
-        std::cerr << "STARTING SERVER: " << buffer << " pid:" << server->pid() << std::endl;
-      }
-      set_default_socket(server->hostname());
-      construct->push_server(server);
+  // Memcached's pidfile is broken
+  bool broken_pid_file()
+  {
+    return true;
+  }
 
-      {
-        server_config_string+= "--socket=\"";
-        server_config_string+= server->hostname();
-        server_config_string+= "\" ";
-      }
-    }
+  bool build(int argc, const char *argv[]);
+};
+
+
+#include <sstream>
+
+bool Memcached::build(int argc, const char *argv[])
+{
+  std::stringstream arg_buffer;
+
+  if (getuid() == 0 or geteuid() == 0)
+  {
+    arg_buffer << " -u root ";
+  }
 
-    server_config_string.resize(server_config_string.size() -1); // Remove final space
-    construct->server_list= server_config_string;
+  for (int x= 1 ; x < argc ; x++)
+  {
+    arg_buffer << " " << argv[x] << " ";
   }
 
-  srandom((unsigned int)time(NULL));
+  set_extra_args(arg_buffer.str());
 
-  std::cerr << std::endl;
   return true;
 }
 
-void server_shutdown(server_startup_st *construct)
+namespace libtest {
+
+Server *build_memcached(const std::string& hostname, const in_port_t try_port)
 {
-  if (not construct)
-    return;
+  return new Memcached(hostname, try_port, false);
+}
 
-  construct->shutdown();
+Server *build_memcached_socket(const std::string& hostname, const in_port_t try_port)
+{
+  return new Memcached(hostname, try_port, true);
 }
+
+}
+