summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPrasanna Kumar Kalever <prasanna.kalever@redhat.com>2016-08-30 13:40:41 +0530
committerRaghavendra G <rgowdapp@redhat.com>2016-09-06 02:30:05 -0700
commitdb3286184a1ab89d3edb9cf4d645a2db724fada0 (patch)
treed21e510517962e23c099fa249f632e12a54b1dc0
parentc67386e1e8edcbdf2f8dca06fc7984df2adb1f06 (diff)
protocol-server: generate events
This patch add/generate the events seen from/in the protocol server side, which will be consumed by the new eventing framework. ------------------------------------------------------------------------- | event | description | |-------------------------------|---------------------------------------| | EVENT_CLIENT_CONNECT | new client connected | |-------------------------------|---------------------------------------| | EVENT_CLIENT_AUTH_REJECT | client cannot be authenticated | |-------------------------------|---------------------------------------| | EVENT_CLIENT_DISCONNECT | client had disconnected | ------------------------------------------------------------------------- Thanks to "Raghavendra Talur" <rtalur@redhat.com> for all the help Change-Id: I4fda83fae7747507f64d81a7645cc3a8a2fbaeae BUG: 1369403 Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com> Reviewed-on: http://review.gluster.org/15294 Tested-by: Prasanna Kumar Kalever <pkalever@redhat.com> Smoke: Gluster Build System <jenkins@build.gluster.org> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> CentOS-regression: Gluster Build System <jenkins@build.gluster.org> Reviewed-by: Raghavendra G <rgowdapp@redhat.com>
-rw-r--r--xlators/protocol/server/src/server-handshake.c17
-rw-r--r--xlators/protocol/server/src/server.c55
2 files changed, 70 insertions, 2 deletions
diff --git a/xlators/protocol/server/src/server-handshake.c b/xlators/protocol/server/src/server-handshake.c
index 771595228eb..1230cdf7ab5 100644
--- a/xlators/protocol/server/src/server-handshake.c
+++ b/xlators/protocol/server/src/server-handshake.c
@@ -18,6 +18,7 @@
#include "authenticate.h"
#include "server-messages.h"
#include "syscall.h"
+#include "events.h"
struct __get_xl_struct {
const char *name;
@@ -690,6 +691,15 @@ server_setvolume (rpcsvc_request_t *req)
"accepted client from %s (version: %s)",
client->client_uid,
(clnt_version) ? clnt_version : "old");
+
+ gf_event (EVENT_CLIENT_CONNECT, "client_uid=%s;"
+ "client_identifier=%s;server_identifier=%s;"
+ "brick_path=%s",
+ client->client_uid,
+ req->trans->peerinfo.identifier,
+ req->trans->myinfo.identifier,
+ name);
+
op_ret = 0;
client->bound_xl = xl;
ret = dict_set_str (reply, "ERROR", "Success");
@@ -697,6 +707,13 @@ server_setvolume (rpcsvc_request_t *req)
gf_msg_debug (this->name, 0, "failed to set error "
"msg");
} else {
+ gf_event (EVENT_CLIENT_AUTH_REJECT, "client_uid=%s;"
+ "client_identifier=%s;server_identifier=%s;"
+ "brick_path=%s",
+ client->client_uid,
+ req->trans->peerinfo.identifier,
+ req->trans->myinfo.identifier,
+ name);
gf_msg (this->name, GF_LOG_ERROR, EACCES,
PS_MSG_AUTHENTICATE_ERROR, "Cannot authenticate client"
" from %s %s", client->client_uid,
diff --git a/xlators/protocol/server/src/server.c b/xlators/protocol/server/src/server.c
index f04dd67f61d..a99c1470276 100644
--- a/xlators/protocol/server/src/server.c
+++ b/xlators/protocol/server/src/server.c
@@ -21,6 +21,7 @@
#include "defaults.h"
#include "authenticate.h"
#include "event.h"
+#include "events.h"
#include "server-messages.h"
rpcsvc_cbk_program_t server_cbk_prog = {
@@ -76,7 +77,8 @@ grace_time_handler (void *data)
*/
gf_client_ref (client);
gf_client_put (client, &detached);
- if (detached)//reconnection did not happen :-(
+
+ if (detached) /* reconnection did not happen :-( */
server_connection_cleanup (this, client,
INTERNAL_LOCKS | POSIX_LOCKS);
gf_client_unref (client);
@@ -487,6 +489,8 @@ server_rpc_notify (rpcsvc_t *rpc, void *xl, rpcsvc_event_t event,
client_t *client = NULL;
server_ctx_t *serv_ctx = NULL;
struct timespec grace_ts = {0, };
+ char *auth_path = NULL;
+ int ret = -1;
if (!xl || !data) {
gf_msg_callingfn ("server", GF_LOG_WARNING, 0,
@@ -546,19 +550,45 @@ server_rpc_notify (rpcsvc_t *rpc, void *xl, rpcsvc_event_t event,
PS_MSG_CLIENT_DISCONNECTING, "disconnecting connection"
" from %s", client->client_uid);
+ ret = dict_get_str (this->options, "auth-path", &auth_path);
+ if (ret) {
+ gf_msg (this->name, GF_LOG_WARNING, 0,
+ PS_MSG_DICT_GET_FAILED,
+ "failed to get auth-path");
+ auth_path = NULL;
+ }
+
/* If lock self heal is off, then destroy the
conn object, else register a grace timer event */
if (!conf->lk_heal) {
gf_client_ref (client);
gf_client_put (client, &detached);
- if (detached)
+ if (detached) {
server_connection_cleanup (this, client,
INTERNAL_LOCKS | POSIX_LOCKS);
+
+ gf_event (EVENT_CLIENT_DISCONNECT,
+ "client_uid=%s;"
+ "client_identifier=%s;"
+ "server_identifier=%s;"
+ "brick_path=%s",
+ client->client_uid,
+ trans->peerinfo.identifier,
+ trans->myinfo.identifier,
+ auth_path);
+ }
gf_client_unref (client);
break;
}
trans->xl_private = NULL;
server_connection_cleanup (this, client, INTERNAL_LOCKS);
+ gf_event (EVENT_CLIENT_DISCONNECT, "client_uid=%s;"
+ "client_identifier=%s;server_identifier=%s;"
+ "brick_path=%s",
+ client->client_uid,
+ trans->peerinfo.identifier,
+ trans->myinfo.identifier,
+ auth_path);
serv_ctx = server_ctx_get (client, this);
@@ -716,6 +746,7 @@ reconfigure (xlator_t *this, dict_t *options)
int ret = 0;
char *statedump_path = NULL;
int32_t new_nthread = 0;
+ char *auth_path = NULL;
conf = this->private;
@@ -840,6 +871,26 @@ reconfigure (xlator_t *this, dict_t *options)
"authorized client, hence we "
"continue with this connection");
} else {
+ ret = dict_get_str (this->options,
+ "auth-path",
+ &auth_path);
+ if (ret) {
+ gf_msg (this->name,
+ GF_LOG_WARNING, 0,
+ PS_MSG_DICT_GET_FAILED,
+ "failed to get "
+ "auth-path");
+ auth_path = NULL;
+ }
+ gf_event (EVENT_CLIENT_AUTH_REJECT,
+ "client_uid=%s;"
+ "client_identifier=%s;"
+ "server_identifier=%s;"
+ "brick_path=%s",
+ xprt->xl_private->client_uid,
+ xprt->peerinfo.identifier,
+ xprt->myinfo.identifier,
+ auth_path);
gf_msg (this->name, GF_LOG_INFO,
EACCES,
PS_MSG_AUTHENTICATE_ERROR,