service: hide output of sysv scripts if quiet is passed on the kernel cmdline

diff --git a/fixme b/fixme
index 5a5a486aced6cf92455b2284a6bd03c175778fc9..765d9d121d608d25f61e896dc2e6dabafcc8b364 100644 (file)
--- a/fixme
+++ b/fixme
@@ -1,7 +1,5 @@
 * dot output for --test for 'initial description'
 
-* 'quiet' sets log_level=null
-
 * conflicted-by: to have a defined winner for conflicts:
 
 * check 'disable'

diff --git a/src/dbus-manager.c b/src/dbus-manager.c
index c1238f006657e73e0f539ed82c165f1fdcbb8c54..549f175eb940e321114633767fee270ee9a45078 100644 (file)
--- a/src/dbus-manager.c
+++ b/src/dbus-manager.c
@@ -131,6 +131,7 @@
         "  <property name=\"Environment\" type=\"as\" access=\"read\"/>\n" \
         "  <property name=\"ConfirmSpawn\" type=\"b\" access=\"read\"/>\n" \
         "  <property name=\"ShowStatus\" type=\"b\" access=\"read\"/>\n" \
+        "  <property name=\"SysVConsole\" type=\"b\" access=\"read\"/>\n" \
         "  <property name=\"UnitPath\" type=\"as\" access=\"read\"/>\n" \
         "  <property name=\"SysVInitPath\" type=\"as\" access=\"read\"/>\n" \
         "  <property name=\"SysVRcndPath\" type=\"as\" access=\"read\"/>\n" \
@@ -240,6 +241,7 @@ static DBusHandlerResult bus_manager_message_handler(DBusConnection *connection,
                 { "org.freedesktop.systemd1.Manager", "Environment",   bus_property_append_strv,      "as", m->environment     },
                 { "org.freedesktop.systemd1.Manager", "ConfirmSpawn",  bus_property_append_bool,      "b",  &m->confirm_spawn  },
                 { "org.freedesktop.systemd1.Manager", "ShowStatus",    bus_property_append_bool,      "b",  &m->show_status    },
+                { "org.freedesktop.systemd1.Manager", "SysVConsole",   bus_property_append_bool,      "b",  &m->sysv_console   },
                 { "org.freedesktop.systemd1.Manager", "UnitPath",      bus_property_append_strv,      "as", m->lookup_paths.unit_path },
                 { "org.freedesktop.systemd1.Manager", "SysVInitPath",  bus_property_append_strv,      "as", m->lookup_paths.sysvinit_path },
                 { "org.freedesktop.systemd1.Manager", "SysVRcndPath",  bus_property_append_strv,      "as", m->lookup_paths.sysvrcnd_path },

diff --git a/src/main.c b/src/main.c
index cddfc88454f2dd14772195392efb2c60ca608f9f..79a46b8cd50324f24fdd8e76231677ae0f38b074 100644 (file)
--- a/src/main.c
+++ b/src/main.c
@@ -64,6 +64,7 @@ static int arg_crash_chvt = -1;
 static bool arg_confirm_spawn = false;
 static bool arg_nomodules = false;
 static bool arg_show_status = true;
+static bool arg_sysv_console = true;
 
 static FILE* serialization = NULL;
 
@@ -241,7 +242,6 @@ static int parse_proc_cmdline_word(const char *word) {
                 "4",      SPECIAL_RUNLEVEL4_TARGET,
                 "5",      SPECIAL_RUNLEVEL5_TARGET
         };
-        bool ignore_quiet = false;
 
         if (startswith(word, "systemd.unit="))
                 return set_default_unit(word + 13);
@@ -303,10 +303,16 @@ static int parse_proc_cmdline_word(const char *word) {
 
                 if ((r = parse_boolean(word + 20)) < 0)
                         log_warning("Failed to parse show status switch %s, Ignoring.", word + 20);
-                else {
+                else
                         arg_show_status = r;
-                        ignore_quiet = true;
-                }
+
+        } else if (startswith(word, "systemd.sysv_console=")) {
+                int r;
+
+                if ((r = parse_boolean(word + 21)) < 0)
+                        log_warning("Failed to parse SysV console switch %s, Ignoring.", word + 20);
+                else
+                        arg_sysv_console = r;
 
         } else if (startswith(word, "systemd.")) {
 
@@ -314,16 +320,17 @@ static int parse_proc_cmdline_word(const char *word) {
 
                 log_info("Supported kernel switches:\n"
                          "systemd.unit=UNIT                        Default unit to start\n"
-                         "systemd.log_target=console|kmsg|syslog|  Log target\n"
-                         "                   syslog-org-kmsg|null\n"
-                         "systemd.log_level=LEVEL                  Log level\n"
-                         "systemd.log_color=0|1                    Highlight important log messages\n"
-                         "systemd.log_location=0|1                 Include code location in log messages\n"
                          "systemd.dump_core=0|1                    Dump core on crash\n"
                          "systemd.crash_shell=0|1                  Run shell on crash\n"
                          "systemd.crash_chvt=N                     Change to VT #N on crash\n"
                          "systemd.confirm_spawn=0|1                Confirm every process spawn\n"
-                         "systemd.show_status=0|1                  Show status updates on the console during bootup\n");
+                         "systemd.show_status=0|1                  Show status updates on the console during bootup\n"
+                         "systemd.sysv_console=0|1                 Connect output of SysV scripts to console\n"
+                         "systemd.log_target=console|kmsg|syslog|syslog-org-kmsg|null\n"
+                         "                                         Log target\n"
+                         "systemd.log_level=LEVEL                  Log level\n"
+                         "systemd.log_color=0|1                    Highlight important log messages\n"
+                         "systemd.log_location=0|1                 Include code location in log messages\n");
 
         } else if (streq(word, "nomodules"))
                 arg_nomodules = true;
@@ -347,8 +354,8 @@ static int parse_proc_cmdline_word(const char *word) {
                 arg_console = w;
 
         } else if (streq(word, "quiet")) {
-                if (!ignore_quiet)
-                        arg_show_status = false;
+                arg_show_status = false;
+                arg_sysv_console = false;
         } else {
                 unsigned i;
 
@@ -492,6 +499,7 @@ static int parse_config_file(void) {
                 { "DumpCore",    config_parse_bool,         &arg_dump_core,   "Manager" },
                 { "CrashShell",  config_parse_bool,         &arg_crash_shell, "Manager" },
                 { "ShowStatus",  config_parse_bool,         &arg_show_status, "Manager" },
+                { "SysVConsole", config_parse_bool,         &arg_sysv_console,"Manager" },
                 { "CrashChVT",   config_parse_int,          &arg_crash_chvt,  "Manager" },
                 { "CPUAffinity", config_parse_cpu_affinity, NULL,             "Manager" },
                 { NULL, NULL, NULL, NULL }
@@ -574,6 +582,7 @@ static int parse_argv(int argc, char *argv[]) {
                 ARG_CRASH_SHELL,
                 ARG_CONFIRM_SPAWN,
                 ARG_SHOW_STATUS,
+                ARG_SYSV_CONSOLE,
                 ARG_DESERIALIZE,
                 ARG_INTROSPECT
         };
@@ -592,7 +601,8 @@ static int parse_argv(int argc, char *argv[]) {
                 { "dump-core",                no_argument,       NULL, ARG_DUMP_CORE                },
                 { "crash-shell",              no_argument,       NULL, ARG_CRASH_SHELL              },
                 { "confirm-spawn",            no_argument,       NULL, ARG_CONFIRM_SPAWN            },
-                { "show-status",              no_argument,       NULL, ARG_SHOW_STATUS              },
+                { "show-status",              optional_argument, NULL, ARG_SHOW_STATUS              },
+                { "sysv-console",             optional_argument, NULL, ARG_SYSV_CONSOLE             },
                 { "deserialize",              required_argument, NULL, ARG_DESERIALIZE              },
                 { "introspect",               optional_argument, NULL, ARG_INTROSPECT               },
                 { NULL,                       0,                 NULL, 0                            }
@@ -686,7 +696,27 @@ static int parse_argv(int argc, char *argv[]) {
                         break;
 
                 case ARG_SHOW_STATUS:
-                        arg_show_status = true;
+
+                        if (optarg) {
+                                if ((r = parse_boolean(optarg)) < 0) {
+                                        log_error("Failed to show status boolean %s.", optarg);
+                                        return r;
+                                }
+                                arg_show_status = r;
+                        } else
+                                arg_show_status = true;
+                        break;
+
+                case ARG_SYSV_CONSOLE:
+
+                        if (optarg) {
+                                if ((r = parse_boolean(optarg)) < 0) {
+                                        log_error("Failed to SysV console boolean %s.", optarg);
+                                        return r;
+                                }
+                                arg_sysv_console = r;
+                        } else
+                                arg_sysv_console = true;
                         break;
 
                 case ARG_DESERIALIZE: {
@@ -774,7 +804,8 @@ static int help(void) {
                "     --dump-core                 Dump core on crash\n"
                "     --crash-shell               Run shell on crash\n"
                "     --confirm-spawn             Ask for confirmation when spawning processes\n"
-               "     --show-status               Show status updates on the console during bootup\n"
+               "     --show-status[=0|1]         Show status updates on the console during bootup\n"
+               "     --sysv-console[=0|1]        Connect output of SysV scripts to console\n"
                "     --log-target=TARGET         Set log target (console, syslog, kmsg, syslog-or-kmsg, null)\n"
                "     --log-level=LEVEL           Set log level (debug, info, notice, warning, err, crit, alert, emerg)\n"
                "     --log-color[=0|1]           Highlight important log messages\n"
@@ -985,6 +1016,7 @@ int main(int argc, char *argv[]) {
 
         m->confirm_spawn = arg_confirm_spawn;
         m->show_status = arg_show_status;
+        m->sysv_console = arg_sysv_console;
 
         if ((r = manager_startup(m, serialization, fds)) < 0)
                 log_error("Failed to fully start up daemon: %s", strerror(-r));
@@ -1095,7 +1127,7 @@ finish:
         dbus_shutdown();
 
         if (reexecute) {
-                const char *args[14];
+                const char *args[15];
                 unsigned i = 0;
                 char sfd[16];
 
@@ -1125,7 +1157,14 @@ finish:
                         args[i++] = "--confirm-spawn";
 
                 if (arg_show_status)
-                        args[i++] = "--show-status";
+                        args[i++] = "--show-status=1";
+                else
+                        args[i++] = "--show-status=0";
+
+                if (arg_sysv_console)
+                        args[i++] = "--sysv-console=1";
+                else
+                        args[i++] = "--sysv-console=0";
 
                 snprintf(sfd, sizeof(sfd), "%i", fileno(serialization));
                 char_array_0(sfd);

diff --git a/src/manager.h b/src/manager.h
index a5bc9202ffe5a7b87b5a5dcdd295b28364b04e45..a762dbc7c33e0dfdd9e85b9983f8eb8656c6c90e 100644 (file)
--- a/src/manager.h
+++ b/src/manager.h
@@ -199,6 +199,7 @@ struct Manager {
 
         bool show_status;
         bool confirm_spawn;
+        bool sysv_console;
 };
 
 int manager_new(ManagerRunningAs running_as, Manager **m);

diff --git a/src/service.c b/src/service.c
index f081705604180cd01d56058a17b630ef23262bb1..c052d7cdcbc291363c38a405424d5221e0ebe96b 100644 (file)
--- a/src/service.c
+++ b/src/service.c
@@ -708,7 +708,7 @@ static int service_load_sysv_path(Service *s, const char *path) {
         s->type = SERVICE_FORKING;
         s->valid_no_process = true;
         s->restart = SERVICE_ONCE;
-        s->exec_context.std_output = EXEC_OUTPUT_TTY;
+        s->exec_context.std_output = s->meta.manager->sysv_console ? EXEC_OUTPUT_TTY : EXEC_OUTPUT_NULL;
         s->exec_context.kill_mode = KILL_PROCESS_GROUP;
 
         u->meta.load_state = UNIT_LOADED;