add a debug message for showing exit of instances

[project/procd.git] / instance.c

diff --git a/instance.c b/instance.c
index df1ea5b..cc839d7 100644 (file)
--- a/instance.c
+++ b/instance.c
@@ -1,3 +1,5 @@
+#include <unistd.h>
+
 #include "procd.h"
 #include "service.h"
 #include "instance.h"
@@ -15,12 +17,56 @@ static const struct blobmsg_policy instance_attr[__INSTANCE_ATTR_MAX] = {
        [INSTANCE_ATTR_DATA] = { "data", BLOBMSG_TYPE_TABLE },
 };
 
+static void
+instance_run(struct service_instance *in)
+{
+       struct blobmsg_list_node *var;
+       struct blob_attr *cur;
+       char **argv;
+       int argc = 1; /* NULL terminated */
+       int rem;
+
+       blobmsg_for_each_attr(cur, in->command, rem)
+               argc++;
+
+       blobmsg_list_for_each(&in->env, var)
+               putenv(blobmsg_data(var->data));
+
+       argv = alloca(sizeof(char *) * argc);
+       argc = 0;
+
+       blobmsg_for_each_attr(cur, in->command, rem)
+               argv[argc++] = blobmsg_data(cur);
+
+       argv[argc] = NULL;
+       execvp(argv[0], argv);
+       exit(127);
+}
+
 void
 instance_start(struct service_instance *in)
 {
+       int pid;
+
+       if (in->proc.pending)
+               return;
+
        in->restart = false;
        if (!in->valid)
                return;
+
+       pid = fork();
+       if (pid < 0)
+               return;
+
+       if (!pid) {
+               instance_run(in);
+               return;
+       }
+
+       DPRINTF("Started instance %s::%s\n", in->srv->name, in->name);
+       in->proc.pid = pid;
+       uloop_process_add(&in->proc);
 }
 
 static void
@@ -40,6 +86,7 @@ instance_exit(struct uloop_process *p, int ret)
        struct service_instance *in;
 
        in = container_of(p, struct service_instance, proc);
+       DPRINTF("Instance %s::%s exit with error code %d\n", in->srv->name, in->name, ret);
        uloop_timeout_cancel(&in->timeout);
        if (in->restart)
                instance_start(in);
@@ -76,19 +123,40 @@ static bool
 instance_config_parse(struct service_instance *in)
 {
        struct blob_attr *tb[__INSTANCE_ATTR_MAX];
-       struct blob_attr *cur;
+       struct blob_attr *cur, *cur2;
+       int argc = 0;
+       int rem;
 
        blobmsg_parse(instance_attr, __INSTANCE_ATTR_MAX, tb,
                blobmsg_data(in->config), blobmsg_data_len(in->config));
 
-       if (!tb[INSTANCE_ATTR_COMMAND])
+       cur = tb[INSTANCE_ATTR_COMMAND];
+       if (!cur)
+               return false;
+
+       if (!blobmsg_check_attr_list(cur, BLOBMSG_TYPE_STRING))
+               return false;
+
+       blobmsg_for_each_attr(cur2, cur, rem) {
+               argc++;
+               break;
+       }
+       if (!argc)
                return false;
 
-       if ((cur = tb[INSTANCE_ATTR_ENV]))
+       in->command = cur;
+
+       if ((cur = tb[INSTANCE_ATTR_ENV])) {
+               if (!blobmsg_check_attr_list(cur, BLOBMSG_TYPE_STRING))
+                       return false;
                blobmsg_list_fill(&in->env, blobmsg_data(cur), blobmsg_data_len(cur));
+       }
 
-       if ((cur = tb[INSTANCE_ATTR_DATA]))
+       if ((cur = tb[INSTANCE_ATTR_DATA])) {
+               if (!blobmsg_check_attr_list(cur, BLOBMSG_TYPE_STRING))
+                       return false;
                blobmsg_list_fill(&in->data, blobmsg_data(cur), blobmsg_data_len(cur));
+       }
 
        return true;
 }
@@ -107,6 +175,7 @@ instance_config_move(struct service_instance *in, struct service_instance *in_sr
        blobmsg_list_move(&in->env, &in_src->env);
        blobmsg_list_move(&in->data, &in_src->data);
        in->command = in_src->command;
+       in->name = in_src->name;
 }
 
 bool
@@ -134,8 +203,10 @@ instance_free(struct service_instance *in)
 }
 
 void
-instance_init(struct service_instance *in, struct blob_attr *config)
+instance_init(struct service_instance *in, struct service *s, struct blob_attr *config)
 {
+       in->srv = s;
+       in->name = blobmsg_name(config);
        in->config = config;
        in->timeout.cb = instance_timeout;
        in->proc.cb = instance_exit;
@@ -144,3 +215,15 @@ instance_init(struct service_instance *in, struct blob_attr *config)
        blobmsg_list_simple_init(&in->data);
        in->valid = instance_config_parse(in);
 }
+
+void instance_dump(struct blob_buf *b, struct service_instance *in)
+{
+       void *i;
+
+       i = blobmsg_open_table(b, in->name);
+       blobmsg_add_u8(b, "running", in->proc.pending);
+       if (in->proc.pending)
+               blobmsg_add_u32(b, "pid", in->proc.pid);
+       blobmsg_add_blob(b, in->command);
+       blobmsg_close_table(b, i);
+}