diff --git a/man/systemd-nspawn.xml b/man/systemd-nspawn.xml index 88def4b2ad1..2645a6b217c 100644 --- a/man/systemd-nspawn.xml +++ b/man/systemd-nspawn.xml @@ -566,7 +566,7 @@ specified - An init program is automatically searched for (unless the is used) and run as PID 1 in the container. The passed parameters are used as invocation parameters for this process. + An init program is automatically searched for and run as PID 1 in the container. The passed parameters are used as invocation parameters for this process. @@ -578,14 +578,6 @@ - - - - Invoke the specified path as the init program in the container when is used. - - - - diff --git a/man/systemd.nspawn.xml b/man/systemd.nspawn.xml index 14f4b9bc4f0..591933a10cc 100644 --- a/man/systemd.nspawn.xml +++ b/man/systemd.nspawn.xml @@ -101,16 +101,6 @@ - - Init= - - Takes an absolute path specifying the init program to invoke in the container when - Boot= is enabled. This setting corresponds to the option - on the systemd-nspawn command line. - - - - Ephemeral= diff --git a/shell-completion/bash/systemd-nspawn b/shell-completion/bash/systemd-nspawn index fc0286c8e13..0a1761d110e 100644 --- a/shell-completion/bash/systemd-nspawn +++ b/shell-completion/bash/systemd-nspawn @@ -74,7 +74,7 @@ _systemd_nspawn() { --pivot-root --property --private-users --private-users-ownership --network-namespace-path --network-ipvlan --network-veth-extra --network-zone -p --port --system-call-filter --overlay --overlay-ro --settings --rlimit --hostname --no-new-privileges --oom-score-adjust --cpu-affinity - --resolv-conf --timezone --root-hash-sig --background --oci-bundle --verity-data --init' + --resolv-conf --timezone --root-hash-sig --background --oci-bundle --verity-data' ) _init_completion || return diff --git a/shell-completion/zsh/_systemd-nspawn b/shell-completion/zsh/_systemd-nspawn index 1f510ccb74c..a9856b50083 100644 --- a/shell-completion/zsh/_systemd-nspawn +++ b/shell-completion/zsh/_systemd-nspawn @@ -21,7 +21,6 @@ _arguments \ '(--ephemeral -x)'{--ephemeral,-x}'[Run container with snapshot of root directory, and remove it after exit.]' \ '(--image -i)'{--image=,-i+}'[Disk image to mount the root directory for the container from.]:disk image: _files' \ '(--boot -b)'{--boot,-b}'[Automatically search for an init binary and invoke it instead of a shell or a user supplied program.]' \ - '--init=[Invoke the specified program as init in the container.]: : _message "path to init"' \ '(--user -u)'{--user=,-u+}'[Run the command under specified user, create home directory and cd into it.]:user:_users' \ '(--machine -M)'{--machine=,-M+}'[Sets the machine name for this container.]: : _message "container name"' \ '--uuid=[Set the specified uuid for the container.]: : _message "container UUID"' \ diff --git a/src/nspawn/nspawn-gperf.gperf b/src/nspawn/nspawn-gperf.gperf index b112f5d2613..123ef0c6c87 100644 --- a/src/nspawn/nspawn-gperf.gperf +++ b/src/nspawn/nspawn-gperf.gperf @@ -20,7 +20,6 @@ struct ConfigPerfItem; %includes %% Exec.Boot, config_parse_boot, 0, 0 -Exec.Init, config_parse_path, 0, offsetof(Settings, init) Exec.Ephemeral, config_parse_tristate, 0, offsetof(Settings, ephemeral) Exec.ProcessTwo, config_parse_pid2, 0, 0 Exec.Parameters, config_parse_strv, 0, offsetof(Settings, parameters) diff --git a/src/nspawn/nspawn-settings.h b/src/nspawn/nspawn-settings.h index e48db0fdc34..135b3dbb0a6 100644 --- a/src/nspawn/nspawn-settings.h +++ b/src/nspawn/nspawn-settings.h @@ -125,10 +125,9 @@ typedef enum SettingsMask { SETTING_CREDENTIALS = UINT64_C(1) << 30, SETTING_BIND_USER = UINT64_C(1) << 31, SETTING_SUPPRESS_SYNC = UINT64_C(1) << 32, - SETTING_INIT = UINT64_C(1) << 33, - SETTING_RLIMIT_FIRST = UINT64_C(1) << 34, /* we define one bit per resource limit here */ - SETTING_RLIMIT_LAST = UINT64_C(1) << (34 + _RLIMIT_MAX - 1), - _SETTINGS_MASK_ALL = (UINT64_C(1) << (34 + _RLIMIT_MAX)) - 1, + SETTING_RLIMIT_FIRST = UINT64_C(1) << 33, /* we define one bit per resource limit here */ + SETTING_RLIMIT_LAST = UINT64_C(1) << (33 + _RLIMIT_MAX - 1), + _SETTINGS_MASK_ALL = (UINT64_C(1) << (33 + _RLIMIT_MAX)) -1, _SETTING_FORCE_ENUM_WIDTH = UINT64_MAX } SettingsMask; @@ -160,7 +159,6 @@ typedef struct OciHook { typedef struct Settings { /* [Exec] */ StartMode start_mode; - char *init; int ephemeral; char **parameters; char **environment; diff --git a/src/nspawn/nspawn.c b/src/nspawn/nspawn.c index 8c68a2058c5..9cd52e06c08 100644 --- a/src/nspawn/nspawn.c +++ b/src/nspawn/nspawn.c @@ -139,7 +139,6 @@ static char *arg_slice = NULL; static bool arg_private_network = false; static bool arg_read_only = false; static StartMode arg_start_mode = START_PID1; -static char *arg_init = NULL; static bool arg_ephemeral = false; static LinkJournal arg_link_journal = LINK_AUTO; static bool arg_link_journal_try = false; @@ -245,7 +244,6 @@ STATIC_DESTRUCTOR_REGISTER(arg_supplementary_gids, freep); STATIC_DESTRUCTOR_REGISTER(arg_machine, freep); STATIC_DESTRUCTOR_REGISTER(arg_hostname, freep); STATIC_DESTRUCTOR_REGISTER(arg_slice, freep); -STATIC_DESTRUCTOR_REGISTER(arg_init, freep); STATIC_DESTRUCTOR_REGISTER(arg_setenv, strv_freep); STATIC_DESTRUCTOR_REGISTER(arg_network_interfaces, strv_freep); STATIC_DESTRUCTOR_REGISTER(arg_network_macvlan, strv_freep); @@ -349,7 +347,6 @@ static int help(void) { "\n%3$sExecution:%4$s\n" " -a --as-pid2 Maintain a stub init as PID1, invoke binary as PID2\n" " -b --boot Boot up full system (i.e. invoke init)\n" - " --init=PATH Path to init to invoke\n" " --chdir=PATH Set working directory in the container\n" " -E --setenv=NAME[=VALUE] Pass an environment variable to PID 1\n" " -u --user=USER Run the command under specified user or UID\n" @@ -700,7 +697,6 @@ static int parse_argv(int argc, char *argv[]) { enum { ARG_VERSION = 0x100, ARG_PRIVATE_NETWORK, - ARG_INIT, ARG_UUID, ARG_READ_ONLY, ARG_CAPABILITY, @@ -768,7 +764,6 @@ static int parse_argv(int argc, char *argv[]) { { "private-network", no_argument, NULL, ARG_PRIVATE_NETWORK }, { "as-pid2", no_argument, NULL, 'a' }, { "boot", no_argument, NULL, 'b' }, - { "init", required_argument, NULL, ARG_INIT }, { "uuid", required_argument, NULL, ARG_UUID }, { "read-only", no_argument, NULL, ARG_READ_ONLY }, { "capability", required_argument, NULL, ARG_CAPABILITY }, @@ -989,14 +984,6 @@ static int parse_argv(int argc, char *argv[]) { arg_settings_mask |= SETTING_START_MODE; break; - case ARG_INIT: - r = parse_path_argument(optarg, /* suppress_root= */ false, &arg_init); - if (r < 0) - return r; - - arg_settings_mask |= SETTING_INIT; - break; - case 'a': if (arg_start_mode == START_BOOT) return log_error_errno(SYNTHETIC_ERRNO(EINVAL), @@ -1791,9 +1778,6 @@ static int verify_arguments(void) { if (arg_userns_mode == USER_NAMESPACE_NO && !strv_isempty(arg_bind_user)) return log_error_errno(SYNTHETIC_ERRNO(EINVAL), "--bind-user= requires --private-users"); - if (arg_start_mode != START_BOOT && arg_init) - return log_error_errno(SYNTHETIC_ERRNO(EINVAL), "Cannot use --init= without --boot"); - /* Drop duplicate --bind-user= entries */ strv_uniq(arg_bind_user); @@ -3606,21 +3590,15 @@ static int inner_child( memcpy_safe(a + 1, arg_parameters, m * sizeof(char*)); a[1 + m] = NULL; - if (arg_init) { - a[0] = arg_init; + FOREACH_STRING(init, + "/usr/lib/systemd/systemd", + "/lib/systemd/systemd", + "/sbin/init") { + a[0] = (char*) init; execve(a[0], a, env_use); - exec_target = arg_init; - } else { - FOREACH_STRING(init, - "/usr/lib/systemd/systemd", - "/lib/systemd/systemd", - "/sbin/init") { - a[0] = (char*) init; - execve(a[0], a, env_use); - } - - exec_target = "/usr/lib/systemd/systemd, /lib/systemd/systemd, /sbin/init"; } + + exec_target = "/usr/lib/systemd/systemd, /lib/systemd/systemd, /sbin/init"; } else if (!strv_isempty(arg_parameters)) { const char *dollar_path; @@ -4607,9 +4585,6 @@ static int merge_settings(Settings *settings, const char *path) { strv_free_and_replace(arg_parameters, settings->parameters); } - if ((arg_settings_mask & SETTING_INIT) == 0 && settings->init) - free_and_replace(arg_init, settings->init); - if ((arg_settings_mask & SETTING_EPHEMERAL) == 0 && settings->ephemeral >= 0) arg_ephemeral = settings->ephemeral; diff --git a/test/units/TEST-13-NSPAWN.nspawn.sh b/test/units/TEST-13-NSPAWN.nspawn.sh index 2de9f9079fa..ad11468886f 100755 --- a/test/units/TEST-13-NSPAWN.nspawn.sh +++ b/test/units/TEST-13-NSPAWN.nspawn.sh @@ -984,36 +984,6 @@ testcase_check_os_release() { rm -fr "$root" "$base" } -testcase_init() { - local root common_opts - - root="$(mktemp -d /var/lib/machines/TEST-13-NSPAWN.init.XXX)" - create_dummy_container "$root" - - cat >"$root/sbin/custom-init" </run/systemd/nspawn/foo-bar.nspawn - systemd-nspawn "${common_opts[@]}" --settings=yes |& grep "Hello from custom init, beautiful day, innit?" - - rm -fr "$root" -} - testcase_ip_masquerade() { local root