]> arthur.barton.de Git - netdata.git/commitdiff
allow STREAM receiver to unblock the single threaded web server
authorCosta Tsaousis <costa@tsaousis.gr>
Thu, 23 Feb 2017 15:44:07 +0000 (17:44 +0200)
committerCosta Tsaousis <costa@tsaousis.gr>
Thu, 23 Feb 2017 15:44:07 +0000 (17:44 +0200)
src/main.c
src/rrdpush.c
src/rrdpush.h
src/web_client.c

index 48bb6b90b2787daf1979b44cc763e0e80d6d4470..e0089dbbe0c45fe8865e0e5f2ddeffef2b8be939 100644 (file)
@@ -55,7 +55,7 @@ struct netdata_static_thread static_threads[] = {
     {"plugins.d",           NULL,       NULL,         1, NULL, NULL, pluginsd_main},
     {"web",                 NULL,       NULL,         1, NULL, NULL, socket_listen_main_multi_threaded},
     {"web-single-threaded", NULL,       NULL,         0, NULL, NULL, socket_listen_main_single_threaded},
-    {"central-netdata-push",NULL,       NULL,         0, NULL, NULL, central_netdata_push_thread},
+    {"central-netdata-push",NULL,       NULL,         0, NULL, NULL, rrdpush_sender_thread},
     {NULL,                  NULL,       NULL,         0, NULL, NULL, NULL}
 };
 
@@ -82,7 +82,7 @@ void web_server_threading_selection(void) {
         if(static_threads[i].start_routine == socket_listen_main_single_threaded)
             static_threads[i].enabled = single_threaded;
 
-        if(static_threads[i].start_routine == central_netdata_push_thread)
+        if(static_threads[i].start_routine == rrdpush_sender_thread)
             static_threads[i].enabled = rrdpush_thread;
 
         if(static_threads[i].start_routine == backends_main)
index e5c18ef124650fe0e95b4b6a95eef00ce7a09ad3..91f386f675ccfd2bb74011ac2e3895010bc29fd4 100644 (file)
@@ -179,7 +179,7 @@ int rrdpush_init() {
     return rrdpush_enabled;
 }
 
-void *central_netdata_push_thread(void *ptr) {
+void *rrdpush_sender_thread(void *ptr) {
     struct netdata_static_thread *static_thread = (struct netdata_static_thread *)ptr;
 
     info("STREAM: central netdata push thread created with task id %d", gettid());
@@ -394,3 +394,233 @@ cleanup:
     pthread_exit(NULL);
     return NULL;
 }
+
+
+// ----------------------------------------------------------------------------
+// STREAM receiver
+
+int rrdpush_receive(int fd, const char *key, const char *hostname, const char *machine_guid, const char *os, int update_every, char *client_ip, char *client_port) {
+    RRDHOST *host;
+    int history = default_rrd_history_entries;
+    RRD_MEMORY_MODE mode = default_rrd_memory_mode;
+    int health_enabled = default_health_enabled;
+
+    update_every = (int)appconfig_get_number(&stream_config, machine_guid, "update every", update_every);
+    if(update_every < 0) update_every = 1;
+
+    history = (int)appconfig_get_number(&stream_config, key, "default history", history);
+    history = (int)appconfig_get_number(&stream_config, machine_guid, "history", history);
+    if(history < 5) history = 5;
+
+    mode = rrd_memory_mode_id(appconfig_get(&stream_config, key, "default memory mode", rrd_memory_mode_name(mode)));
+    mode = rrd_memory_mode_id(appconfig_get(&stream_config, machine_guid, "memory mode", rrd_memory_mode_name(mode)));
+
+    health_enabled = appconfig_get_boolean_ondemand(&stream_config, key, "health enabled by default", health_enabled);
+    health_enabled = appconfig_get_boolean_ondemand(&stream_config, machine_guid, "health enabled", health_enabled);
+
+    if(!strcmp(machine_guid, "localhost"))
+        host = localhost;
+    else
+        host = rrdhost_find_or_create(hostname, machine_guid, os, update_every, history, mode, health_enabled?1:0);
+
+    info("STREAM request from client '%s:%s' for host '%s' with machine_guid '%s': update every = %d, history = %d, memory mode = %s, health %s",
+         client_ip, client_port,
+         hostname, machine_guid,
+         update_every,
+         history,
+         rrd_memory_mode_name(mode),
+         (health_enabled == CONFIG_BOOLEAN_NO)?"disabled":((health_enabled == CONFIG_BOOLEAN_YES)?"enabled":"auto")
+    );
+
+    struct plugind cd = {
+            .enabled = 1,
+            .update_every = default_rrd_update_every,
+            .pid = 0,
+            .serial_failures = 0,
+            .successful_collections = 0,
+            .obsolete = 0,
+            .started_t = now_realtime_sec(),
+            .next = NULL,
+    };
+
+    // put the client IP and port into the buffers used by plugins.d
+    snprintfz(cd.id,           CONFIG_MAX_NAME,  "%s:%s", client_ip, client_port);
+    snprintfz(cd.filename,     FILENAME_MAX,     "%s:%s", client_ip, client_port);
+    snprintfz(cd.fullfilename, FILENAME_MAX,     "%s:%s", client_ip, client_port);
+    snprintfz(cd.cmd,          PLUGINSD_CMD_MAX, "%s:%s", client_ip, client_port);
+
+    info("STREAM [%s]:%s: sending STREAM to initiate streaming...", client_ip, client_port);
+    if(send_timeout(fd, "STREAM", 6, 0, 60) != 6) {
+        error("STREAM [%s]:%s: cannot send STREAM.", client_ip, client_port);
+        return 0;
+    }
+
+    // remove the non-blocking flag from the socket
+    if(fcntl(fd, F_SETFL, fcntl(fd, F_GETFL, 0) & ~O_NONBLOCK) == -1)
+        error("STREAM [%s]:%s: cannot remove the non-blocking flag from socket %d", client_ip, client_port, fd);
+
+    // convert the socket to a FILE *
+    FILE *fp = fdopen(fd, "r");
+    if(!fp) {
+        error("STREAM [%s]:%s: failed to get a FILE for FD %d.", client_ip, client_port, fd);
+        return 0;
+    }
+
+    rrdhost_wrlock(host);
+    host->use_counter++;
+    rrdhost_unlock(host);
+
+    // call the plugins.d processor to receive the metrics
+    info("STREAM [%s]:%s: connecting client to plugins.d on host '%s' with machine GUID '%s'.", client_ip, client_port, host->hostname, host->machine_guid);
+    size_t count = pluginsd_process(host, &cd, fp, 1);
+    error("STREAM [%s]:%s: client disconnected (host '%s', machine GUID '%s').", client_ip, client_port, host->hostname, host->machine_guid);
+
+    rrdhost_wrlock(host);
+    host->use_counter--;
+    if(!host->use_counter && health_enabled == CONFIG_BOOLEAN_AUTO)
+        host->health_enabled = 0;
+    rrdhost_unlock(host);
+
+    // cleanup
+    fclose(fp);
+
+    return (int)count;
+}
+
+struct rrdpush_thread {
+    int fd;
+    char *key;
+    char *hostname;
+    char *machine_guid;
+    char *os;
+    char *client_ip;
+    char *client_port;
+    int update_every;
+};
+
+void *rrdpush_receiver_thread(void *ptr) {
+    struct rrdpush_thread *rpt = (struct rrdpush_thread *)ptr;
+
+    info("STREAM: central netdata receive thread created with task id %d, for client [%s]:%s", gettid(), rpt->client_ip, rpt->client_port);
+
+    if (pthread_setcanceltype(PTHREAD_CANCEL_DEFERRED, NULL) != 0)
+        error("STREAM: cannot set pthread cancel type to DEFERRED.");
+
+    if (pthread_setcancelstate(PTHREAD_CANCEL_ENABLE, NULL) != 0)
+        error("STREAM: cannot set pthread cancel state to ENABLE.");
+
+
+    rrdpush_receive(rpt->fd, rpt->key, rpt->hostname, rpt->machine_guid, rpt->os, rpt->update_every, rpt->client_ip, rpt->client_port);
+
+    close(rpt->fd);
+    freez(rpt->key);
+    freez(rpt->hostname);
+    freez(rpt->machine_guid);
+    freez(rpt->os);
+    freez(rpt->client_ip);
+    freez(rpt->client_port);
+    freez(rpt);
+
+    pthread_exit(NULL);
+    return NULL;
+}
+
+static inline int rrdpush_receive_validate_api_key(const char *key) {
+    return appconfig_get_boolean(&stream_config, key, "enabled", 0);
+}
+
+int rrdpush_receiver_thread_spawn(RRDHOST *host, struct web_client *w, char *url) {
+    (void)host;
+    
+    info("STREAM [%s]:%s: client connection.", w->client_ip, w->client_port);
+
+    char *key = NULL, *hostname = NULL, *machine_guid = NULL, *os = NULL;
+    int update_every = default_rrd_update_every;
+
+    while(url) {
+        char *value = mystrsep(&url, "?&");
+        if(!value || !*value) continue;
+
+        char *name = mystrsep(&value, "=");
+        if(!name || !*name) continue;
+        if(!value || !*value) continue;
+
+        if(!strcmp(name, "key"))
+            key = value;
+        else if(!strcmp(name, "hostname"))
+            hostname = value;
+        else if(!strcmp(name, "machine_guid"))
+            machine_guid = value;
+        else if(!strcmp(name, "update_every"))
+            update_every = (int)strtoul(value, NULL, 0);
+        else if(!strcmp(name, "os"))
+            os = value;
+    }
+
+    if(!key || !*key) {
+        error("STREAM [%s]:%s: request without an API key. Forbidding access.", w->client_ip, w->client_port);
+        buffer_flush(w->response.data);
+        buffer_sprintf(w->response.data, "You need an API key for this request.");
+        return 401;
+    }
+
+    if(!hostname || !*hostname) {
+        error("STREAM [%s]:%s: request without a hostname. Forbidding access.", w->client_ip, w->client_port);
+        buffer_flush(w->response.data);
+        buffer_sprintf(w->response.data, "You need to send a hostname too.");
+        return 400;
+    }
+
+    if(!machine_guid || !*machine_guid) {
+        error("STREAM [%s]:%s: request without a machine GUID. Forbidding access.", w->client_ip, w->client_port);
+        buffer_flush(w->response.data);
+        buffer_sprintf(w->response.data, "You need to send a machine GUID too.");
+        return 400;
+    }
+
+    if(!rrdpush_receive_validate_api_key(key)) {
+        error("STREAM [%s]:%s: API key '%s' is not allowed. Forbidding access.", w->client_ip, w->client_port, key);
+        buffer_flush(w->response.data);
+        buffer_sprintf(w->response.data, "Your API key is not permitted access.");
+        return 401;
+    }
+
+    if(!appconfig_get_boolean(&stream_config, machine_guid, "enabled", 1)) {
+        error("STREAM [%s]:%s: machine GUID '%s' is not allowed. Forbidding access.", w->client_ip, w->client_port, machine_guid);
+        buffer_flush(w->response.data);
+        buffer_sprintf(w->response.data, "Your machine guide is not permitted access.");
+        return 404;
+    }
+
+    struct rrdpush_thread *rpt = mallocz(sizeof(struct rrdpush_thread));
+    rpt->fd           = w->ifd;
+    rpt->key          = strdupz(key);
+    rpt->hostname     = strdupz(hostname);
+    rpt->machine_guid = strdupz(machine_guid);
+    rpt->os           = strdupz(os);
+    rpt->client_ip    = strdupz(w->client_ip);
+    rpt->client_port  = strdupz(w->client_port);
+    rpt->update_every = update_every;
+
+    pthread_t *thread = mallocz(sizeof(pthread_t));
+    pthread_attr_t attr;
+
+    debug(D_SYSTEM, "Starting STREAM thread for client [%s]:%s.", w->client_ip, w->client_port);
+
+    if(pthread_create(thread, &attr, rrdpush_receiver_thread, rpt))
+        error("failed to create new STREAM thread for client [%s]:%s.", w->client_ip, w->client_port);
+
+    else if(pthread_detach(*thread))
+        error("Cannot request detach newly created thread for client [%s]:%s.", w->client_ip, w->client_port);
+
+    rrdpush_receive(w->ifd, key, hostname, machine_guid, os, update_every, w->client_ip, w->client_port);
+
+    // prevent the caller from closing the streaming socket
+    if(w->ifd == w->ofd)
+        w->ifd = w->ofd = -1;
+    else
+        w->ifd = -1;
+
+    buffer_flush(w->response.data);
+    return 200;
+}
index d34e8c0332441aea93b8ceb5fd8c6aa9d7f7c1be..33eb3442649dc06eac8de4b8a48e8d642af98261 100644 (file)
@@ -6,6 +6,8 @@ extern int rrdpush_exclusive;
 
 extern int rrdpush_init();
 extern void rrdset_done_push(RRDSET *st);
-extern void *central_netdata_push_thread(void *ptr);
+extern void *rrdpush_sender_thread(void *ptr);
+
+extern int rrdpush_receiver_thread_spawn(RRDHOST *host, struct web_client *w, char *url);
 
 #endif //NETDATA_RRDPUSH_H
index 904bf7067dcf5007b207b18e766756aa1854e081..49aef9fba3f2b596e39eb3b09bd0579c39d91ed0 100644 (file)
@@ -1665,177 +1665,6 @@ int web_client_api_old_data_request(RRDHOST *host, struct web_client *w, char *u
     return 200;
 }
 
-int validate_stream_api_key(const char *key) {
-    if(appconfig_get_boolean(&stream_config, key, "enabled", 0))
-        return 1;
-
-    return 0;
-}
-
-int web_client_stream_request(RRDHOST *host, struct web_client *w, char *url) {
-    info("STREAM [%s]:%s: client connection.", w->client_ip, w->client_port);
-
-    char *key = NULL, *hostname = NULL, *machine_guid = NULL, *os = NULL;
-    int update_every = default_rrd_update_every;
-    int history = default_rrd_history_entries;
-    RRD_MEMORY_MODE mode = default_rrd_memory_mode;
-    int health_enabled = default_health_enabled;
-
-    while(url) {
-        char *value = mystrsep(&url, "?&");
-        if(!value || !*value) continue;
-
-        char *name = mystrsep(&value, "=");
-        if(!name || !*name) continue;
-        if(!value || !*value) continue;
-
-        if(!strcmp(name, "key"))
-            key = value;
-        else if(!strcmp(name, "hostname"))
-            hostname = value;
-        else if(!strcmp(name, "machine_guid"))
-            machine_guid = value;
-        else if(!strcmp(name, "update_every"))
-            update_every = (int)strtoul(value, NULL, 0);
-        else if(!strcmp(name, "os"))
-            os = value;
-    }
-
-    if(!key || !*key) {
-        error("STREAM [%s]:%s: request without an API key. Forbidding access.", w->client_ip, w->client_port);
-        buffer_flush(w->response.data);
-        buffer_sprintf(w->response.data, "You need an API key for this request.");
-        return 401;
-    }
-
-    if(!hostname || !*hostname) {
-        error("STREAM [%s]:%s: request without a hostname. Forbidding access.", w->client_ip, w->client_port);
-        buffer_flush(w->response.data);
-        buffer_sprintf(w->response.data, "You need to send a hostname too.");
-        return 400;
-    }
-
-    if(!machine_guid || !*machine_guid) {
-        error("STREAM [%s]:%s: request without a machine GUID. Forbidding access.", w->client_ip, w->client_port);
-        buffer_flush(w->response.data);
-        buffer_sprintf(w->response.data, "You need to send a machine GUID too.");
-        return 400;
-    }
-
-    if(!validate_stream_api_key(key)) {
-        error("STREAM [%s]:%s: API key '%s' is not allowed. Forbidding access.", w->client_ip, w->client_port, key);
-        buffer_flush(w->response.data);
-        buffer_sprintf(w->response.data, "Your API key is not permitted access.");
-        return 401;
-    }
-
-    if(!appconfig_get_boolean(&stream_config, machine_guid, "enabled", 1)) {
-        error("STREAM [%s]:%s: machine GUID '%s' is not allowed. Forbidding access.", w->client_ip, w->client_port, machine_guid);
-        buffer_flush(w->response.data);
-        buffer_sprintf(w->response.data, "Your machine guide is not permitted access.");
-        return 404;
-    }
-
-    update_every = (int)appconfig_get_number(&stream_config, machine_guid, "update every", update_every);
-    if(update_every < 0) update_every = 1;
-
-    history = (int)appconfig_get_number(&stream_config, key, "default history", history);
-    history = (int)appconfig_get_number(&stream_config, machine_guid, "history", history);
-    if(history < 5) history = 5;
-
-    mode = rrd_memory_mode_id(appconfig_get(&stream_config, key, "default memory mode", rrd_memory_mode_name(mode)));
-    mode = rrd_memory_mode_id(appconfig_get(&stream_config, machine_guid, "memory mode", rrd_memory_mode_name(mode)));
-
-    health_enabled = appconfig_get_boolean_ondemand(&stream_config, key, "health enabled by default", health_enabled);
-    health_enabled = appconfig_get_boolean_ondemand(&stream_config, machine_guid, "health enabled", health_enabled);
-
-    if(!strcmp(machine_guid, "localhost"))
-        host = localhost;
-    else
-        host = rrdhost_find_or_create(hostname, machine_guid, os, update_every, history, mode, health_enabled?1:0);
-
-    info("STREAM request from client '%s:%s' for host '%s' with machine_guid '%s': update every = %d, history = %d, memory mode = %s, health %s",
-            w->client_ip, w->client_port,
-            hostname, machine_guid,
-            update_every,
-            history,
-            rrd_memory_mode_name(mode),
-            (health_enabled == CONFIG_BOOLEAN_NO)?"disabled":((health_enabled == CONFIG_BOOLEAN_YES)?"enabled":"auto")
-    );
-
-    struct plugind cd = {
-            .enabled = 1,
-            .update_every = default_rrd_update_every,
-            .pid = 0,
-            .serial_failures = 0,
-            .successful_collections = 0,
-            .obsolete = 0,
-            .started_t = now_realtime_sec(),
-            .next = NULL,
-    };
-
-    // put the client IP and port into the buffers used by plugins.d
-    snprintfz(cd.id,           CONFIG_MAX_NAME,  "%s:%s", w->client_ip, w->client_port);
-    snprintfz(cd.filename,     FILENAME_MAX,     "%s:%s", w->client_ip, w->client_port);
-    snprintfz(cd.fullfilename, FILENAME_MAX,     "%s:%s", w->client_ip, w->client_port);
-    snprintfz(cd.cmd,          PLUGINSD_CMD_MAX, "%s:%s", w->client_ip, w->client_port);
-
-    info("STREAM [%s]:%s: sending STREAM to initiate streaming...", w->client_ip, w->client_port);
-    if(send_timeout(w->ifd, "STREAM", 6, 0, 60) != 6) {
-        error("STREAM [%s]:%s: cannot send STREAM.", w->client_ip, w->client_port);
-        buffer_flush(w->response.data);
-        buffer_sprintf(w->response.data, "Failed to reply back with STREAM");
-        return 400;
-    }
-
-    // remove the non-blocking flag from the socket
-    if(fcntl(w->ifd, F_SETFL, fcntl(w->ifd, F_GETFL, 0) & ~O_NONBLOCK) == -1)
-        error("STREAM [%s]:%s: cannot remove the non-blocking flag from socket %d", w->client_ip, w->client_port, w->ifd);
-
-    /*
-    char buffer[1000 + 1];
-    ssize_t len;
-    while((len = read(w->ifd, buffer, 1000)) != -1) {
-        buffer[len] = '\0';
-        fprintf(stderr, "BEGIN READ %zu bytes\n%s\nEND READ\n", (size_t)len, buffer);
-    }
-    */
-
-    // convert the socket to a FILE *
-    FILE *fp = fdopen(w->ifd, "r");
-    if(!fp) {
-        error("STREAM [%s]:%s: failed to get a FILE for FD %d.", w->client_ip, w->client_port, w->ifd);
-        buffer_flush(w->response.data);
-        buffer_sprintf(w->response.data, "Failed to get a FILE for an FD.");
-        return 500;
-    }
-
-    rrdhost_wrlock(host);
-    host->use_counter++;
-    rrdhost_unlock(host);
-
-    // call the plugins.d processor to receive the metrics
-    info("STREAM [%s]:%s: connecting client to plugins.d on host '%s' with machine GUID '%s'.", w->client_ip, w->client_port, host->hostname, host->machine_guid);
-    size_t count = pluginsd_process(host, &cd, fp, 1);
-    error("STREAM [%s]:%s: client disconnected (host '%s', machine GUID '%s').", w->client_ip, w->client_port, host->hostname, host->machine_guid);
-
-    rrdhost_wrlock(host);
-    host->use_counter--;
-    if(!host->use_counter && health_enabled == CONFIG_BOOLEAN_AUTO)
-        host->health_enabled = 0;
-    rrdhost_unlock(host);
-
-    // cleanup
-    fclose(fp);
-    w->ifd = -1;
-
-    // this will not send anything
-    // the socket is closed
-    buffer_flush(w->response.data);
-    if(count) return 200;
-    return 400;
-}
-
 const char *web_content_type_to_string(uint8_t contenttype) {
     switch(contenttype) {
         case CT_TEXT_HTML:
@@ -2485,7 +2314,7 @@ void web_client_process_request(struct web_client *w) {
         case HTTP_VALIDATION_OK:
             switch(w->mode) {
                 case WEB_CLIENT_MODE_STREAM:
-                    w->response.code = web_client_stream_request(localhost, w, w->decoded_url);
+                    w->response.code = rrdpush_receiver_thread_spawn(localhost, w, w->decoded_url);
                     return;
 
                 case WEB_CLIENT_MODE_OPTIONS: