From: Tom Gundersen Date: Thu, 24 Apr 2014 19:28:46 +0000 (+0200) Subject: networkd-wait-online: drop config file and add commandline options instead X-Git-Tag: v213~305 X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?p=elogind.git;a=commitdiff_plain;h=cef8b0735831b2bfeaf21d8a245d8024c3de816a networkd-wait-online: drop config file and add commandline options instead --- diff --git a/Makefile-man.am b/Makefile-man.am index db66e020c..60ee9917c 100644 --- a/Makefile-man.am +++ b/Makefile-man.am @@ -1040,7 +1040,6 @@ if ENABLE_NETWORKD MANPAGES += \ man/systemd-networkd.service.8 \ man/systemd-networkd-wait-online.service.8 \ - man/networkd-wait-online.conf.5 \ man/systemd.netdev.5 \ man/systemd.network.5 MANPAGES_ALIAS += \ diff --git a/Makefile.am b/Makefile.am index 1d37357a3..7f3924ff5 100644 --- a/Makefile.am +++ b/Makefile.am @@ -4170,9 +4170,6 @@ systemd_networkd_wait_online_SOURCES = \ src/network/networkd-wait-online.c \ src/network/networkd-wait-online.h -nodist_systemd_networkd_wait_online_SOURCES = \ - src/network/networkd-wait-online-gperf.c - systemd_networkd_wait_online_LDADD = \ libsystemd-internal.la \ libsystemd-network.la @@ -4186,19 +4183,14 @@ test_network_LDADD = \ tests += \ test-network -dist_pkgsysconf_DATA += \ - src/network/networkd-wait-online.conf - EXTRA_DIST += \ src/network/networkd-network-gperf.gperf \ src/network/networkd-netdev-gperf.gperf \ - src/network/networkd-wait-online-gperf.gperf \ units/systemd-networkd.service.in CLEANFILES += \ src/network/networkd-network-gperf.c \ - src/network/networkd-netdev-gperf.c \ - src/network/networkd-wait-online-gperf.c + src/network/networkd-netdev-gperf.c endif # ------------------------------------------------------------------------------ diff --git a/man/networkd-wait-online.conf.xml b/man/networkd-wait-online.conf.xml deleted file mode 100644 index 821b0efed..000000000 --- a/man/networkd-wait-online.conf.xml +++ /dev/null @@ -1,95 +0,0 @@ - - - - - - - - - networkd-wait-online.conf - systemd - - - - Developer - Tom - Gundersen - teg@jklm.no - - - - - - networkd-wait-online.conf - 5 - - - - networkd-wait-online.conf - Network manager configuration file - - - - /etc/systemd/networkd-wait-online.conf - - - - Description - - This file configures various parameters of the systemd network manager's wait-online service, systemd-networkd-wait-online.service8. - - - - - Options - - All options are configured in the - [WaitOnline] section: - - - - - After= - - Takes a white-space - separated list of network interface - names. Configures which network interfaces - to wait for before deciding if the machine - is considered to be online. The default is - to wait for at least one that is managed by - systemd-networkd.service - 8, - or one other which is brought up by someone else. - - - - - - - - See Also - - systemd1, - systemd-networkd.service8, - - - - diff --git a/man/systemd-networkd-wait-online.service.xml b/man/systemd-networkd-wait-online.service.xml index d8f222655..c8190e1ab 100644 --- a/man/systemd-networkd-wait-online.service.xml +++ b/man/systemd-networkd-wait-online.service.xml @@ -45,7 +45,7 @@ systemd-networkd-wait-online.service systemd-networkd-wait-online - Wait for network to become online + Wait for network to come online @@ -58,22 +58,37 @@ systemd-networkd-wait-online is a one-shot system service that waits for the network to be configured. - It will wait for all links managed by + By default it will wait for all links it is aware of that are managed by systemd-networkd.service8, - to be fully configured. It will wait for at least one link managed by - systemd-networkd to appear, or for a link to be brought up by someone else. + to be fully configured or failed, and for at least one link to gain a + carrier. + - To wait for a specific set of links, see - systemd-networkd.conf5. - This may be necessary on systems with more than one active link. - + + Options + + The following options are understood: + + + + + + + Network interface to wait for + before deciding if the system is online. This is + useful when a system has several interfaces which + will be configured, but a particular one is necessary + to access some network resources. This option may be + used more than once to wait for multiple network + interfaces. + + See Also systemd1, - systemd-networkd.conf5, systemd-networkd.service8, diff --git a/src/network/.gitignore b/src/network/.gitignore index e3b86e2d4..885859648 100644 --- a/src/network/.gitignore +++ b/src/network/.gitignore @@ -1,3 +1,2 @@ /networkd-network-gperf.c /networkd-netdev-gperf.c -/networkd-wait-online-gperf.c diff --git a/src/network/networkd-wait-online-gperf.gperf b/src/network/networkd-wait-online-gperf.gperf deleted file mode 100644 index 0a5e5eee5..000000000 --- a/src/network/networkd-wait-online-gperf.gperf +++ /dev/null @@ -1,17 +0,0 @@ -%{ -#include -#include "conf-parser.h" -#include "networkd-wait-online.h" -%} -struct ConfigPerfItem; -%null_strings -%language=ANSI-C -%define slot-name section_and_lvalue -%define hash-function-name wait_online_gperf_hash -%define lookup-function-name wait_online_gperf_lookup -%readonly-tables -%omit-struct-type -%struct-type -%includes -%% -WaitOnline.ExpectedLinks, config_parse_strv, 0, offsetof(Manager, expected_links) diff --git a/src/network/networkd-wait-online.c b/src/network/networkd-wait-online.c index 6a7c62fea..6530659c1 100644 --- a/src/network/networkd-wait-online.c +++ b/src/network/networkd-wait-online.c @@ -21,6 +21,7 @@ #include #include +#include #include "sd-event.h" #include "event-util.h" @@ -35,6 +36,75 @@ #include "conf-parser.h" #include "strv.h" #include "util.h" +#include "build.h" + +static bool arg_quiet = false; +static char **arg_interfaces = NULL; + +static int help(void) { + + printf("%s [OPTIONS...]\n\n" + "Block until network is configured.\n\n" + " -h --help Show this help\n" + " --version Print version string\n" + " -q --quiet Do not show status information\n" + " -i --interface=INTERFACE Block until at least these interfaces have appeared\n", + program_invocation_short_name); + + return 0; +} + +static int parse_argv(int argc, char *argv[]) { + + enum { + ARG_VERSION = 0x100, + }; + + static const struct option options[] = { + { "help", no_argument, NULL, 'h' }, + { "version", no_argument, NULL, ARG_VERSION }, + { "quiet", no_argument, NULL, 'q' }, + { "interface", required_argument, NULL, 'i' }, + {} + }; + + int c; + + assert(argc >= 0); + assert(argv); + + while ((c = getopt_long(argc, argv, "+hq", options, NULL)) >= 0) { + + switch (c) { + + case 'h': + return help(); + + case 'q': + arg_quiet = true; + break; + + case ARG_VERSION: + puts(PACKAGE_STRING); + puts(SYSTEMD_FEATURES); + return 0; + + case 'i': + if (strv_extend(&arg_interfaces, optarg) < 0) + return log_oom(); + + break; + + case '?': + return -EINVAL; + + default: + assert_not_reached("Unhandled option"); + } + } + + return 1; +} static bool all_configured(Manager *m) { _cleanup_free_ unsigned *indices = NULL; @@ -46,7 +116,7 @@ static bool all_configured(Manager *m) { if (n <= 0) return false; - STRV_FOREACH(ifname, m->expected_links) { + STRV_FOREACH(ifname, arg_interfaces) { _cleanup_rtnl_message_unref_ sd_rtnl_message *message = NULL, *reply = NULL; bool found = false; int index; @@ -91,7 +161,7 @@ static bool all_configured(Manager *m) { } } - if (!found) + if (!found) { /* link exists, but networkd is not yet aware of it */ return false; } @@ -139,6 +209,7 @@ static bool all_configured(Manager *m) { continue; } else if (r < 0 || !streq(state, "configured")) + /* managed by networkd, but not yet configured */ return false; /* we wait for at least one link to appear */ @@ -173,35 +244,12 @@ static int newlink_event_handler(sd_rtnl *rtnl, sd_rtnl_message *message, void * return 1; } -static int parse_config_file(Manager *m) { - static const char fn[] = "/etc/systemd/networkd-wait-online.conf"; - _cleanup_fclose_ FILE *f = NULL; - int r; - - f = fopen(fn, "re"); - if (!f) { - if (errno == ENOENT) - return 0; - - log_warning("Failed to open configuration file %s: %m", fn); - return -errno; - } - - r = config_parse(NULL, fn, f, "WaitOnline\0", config_item_perf_lookup, - (void*) wait_online_gperf_lookup, false, false, m); - if (r < 0) - log_warning("Failed to parse configuration file: %s", strerror(-r)); - - return r; -} - void manager_free(Manager *m) { if (!m) return; sd_event_unref(m->event); sd_rtnl_unref(m->rtnl); - strv_free(m->expected_links); free(m); } @@ -212,26 +260,22 @@ int main(int argc, char *argv[]) { _cleanup_network_monitor_unref_ sd_network_monitor *monitor = NULL; int r, fd, events; - log_set_target(LOG_TARGET_AUTO); + umask(0022); + log_parse_environment(); log_open(); - umask(0022); + r = parse_argv(argc, argv); + if (r <= 0) + return r; - if (argc != 1) { - log_error("This program takes no arguments."); - r = -EINVAL; - goto out; - } + if (arg_quiet) + log_set_max_level(LOG_WARNING); m = new0(Manager, 1); if (!m) return log_oom(); - r = parse_config_file(m); - if (r < 0) - goto out; - r = sd_network_monitor_new(NULL, &monitor); if (r < 0) { log_error("Could not create monitor: %s", strerror(-r)); diff --git a/src/network/networkd-wait-online.conf b/src/network/networkd-wait-online.conf deleted file mode 100644 index fa3d92fe6..000000000 --- a/src/network/networkd-wait-online.conf +++ /dev/null @@ -1,11 +0,0 @@ -# This file is part of systemd. -# -# systemd is free software; you can redistribute it and/or modify it -# under the terms of the GNU Lesser General Public License as published by -# the Free Software Foundation; either version 2.1 of the License, or -# (at your option) any later version. -# -# See networkd-wait-online.conf(5) for details - -[WaitOnline] -#ExpectedLinks= diff --git a/src/network/networkd-wait-online.h b/src/network/networkd-wait-online.h index 463df1616..91edce1f6 100644 --- a/src/network/networkd-wait-online.h +++ b/src/network/networkd-wait-online.h @@ -29,13 +29,9 @@ typedef struct Manager { sd_event *event; sd_rtnl *rtnl; - char **expected_links; } Manager; void manager_free(Manager *m); DEFINE_TRIVIAL_CLEANUP_FUNC(Manager*, manager_free); #define _cleanup_manager_free_ _cleanup_(manager_freep) - -/* gperf lookup function */ -const struct ConfigPerfItem* wait_online_gperf_lookup(const char *key, unsigned length);