The following pull request was submitted through Github. It can be accessed and reviewed at: https://github.com/lxc/lxc/pull/2941
This e-mail was sent by the LXC bot, direct replies will not reach the author unless they happen to be subscribed to this list. === Description (from pull-request) === These configuration options use the same syntax and therefore it seems more intuitive to have the same behavior for both of them, which is not the case today since mount hooks and autodev mounts are called between the two. See: https://github.com/lxc/lxc/issues/2932 Signed-off-by: Felix Abecassis <fabecas...@nvidia.com>
From c631115d059b05bf88bf5fcfa716105bede95760 Mon Sep 17 00:00:00 2001 From: Felix Abecassis <fabecas...@nvidia.com> Date: Tue, 16 Apr 2019 14:40:03 -0700 Subject: [PATCH] conf: do lxc.mount.entry mounts right after lxc.mount.fstab These configuration options use the same syntax and therefore it seems more intuitive to have the same behavior for both of them, which is not the case today since mount hooks and autodev mounts are called between the two. See: https://github.com/lxc/lxc/issues/2932 Signed-off-by: Felix Abecassis <fabecas...@nvidia.com> --- src/lxc/conf.c | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/src/lxc/conf.c b/src/lxc/conf.c index 5d381b7137..986cb89b02 100644 --- a/src/lxc/conf.c +++ b/src/lxc/conf.c @@ -3546,6 +3546,15 @@ int lxc_setup(struct lxc_handler *handler) return -1; } + if (!lxc_list_empty(&lxc_conf->mount_list)) { + ret = setup_mount_entries(lxc_conf, &lxc_conf->rootfs, + &lxc_conf->mount_list, name, lxcpath); + if (ret < 0) { + ERROR("Failed to setup mount entries"); + return -1; + } + } + if (lxc_conf->is_execute) { if (execveat_supported()) { int fd; @@ -3604,15 +3613,6 @@ int lxc_setup(struct lxc_handler *handler) } } - if (!lxc_list_empty(&lxc_conf->mount_list)) { - ret = setup_mount_entries(lxc_conf, &lxc_conf->rootfs, - &lxc_conf->mount_list, name, lxcpath); - if (ret < 0) { - ERROR("Failed to setup mount entries"); - return -1; - } - } - /* Make sure any start hooks are in the container */ if (!verify_start_hooks(lxc_conf)) { ERROR("Failed to verify start hooks");
_______________________________________________ lxc-devel mailing list lxc-devel@lists.linuxcontainers.org http://lists.linuxcontainers.org/listinfo/lxc-devel