On Fri, 2014-05-02 at 21:03 +0000, Serge Hallyn wrote: > Quoting Michael H. Warfield ([email protected]): > > On Thu, 2014-05-01 at 17:26 -0400, Michael H. Warfield wrote: > > > Looks good to me with the provisio that I feel that autoboot on bootup > > > should be for all groups and not just the null group. Dwight told me in > > > another message that he did it that way because the Upstart script did > > > it that way. Was it an oversight or was it a decision? That can be a > > > separate patch to fix. > > > > I haven't seen Dwight's patch go in and I haven't seen any other > > commentary about it or my suggested change. His patch fixes a reported > > problem and my change supports allowing containers which are members of > > designated groups to boot (which I require). Therefore, I'm submitting > > this patch. Both "boot" and "autoboot" were suggested for names of the > > autoboot group. I choose to use "onboot" just to make it a little > > clearer and disambiguate it from some of the other options. > > > > This patch is dependent on Dwight's patch Subject: > > [PATCH v2] have systemd service call lxc-autostart via script > > > > Regards, > > Mike > > > > -- > > Add a non-null group for autobooting > > > > This adds a non-null group (onboot) to the sysvinit startup script > > for autobooting containers. This allows for containers which are > > in other groups to be included in the autoboot process. > > > > This script is used by both the sysvinit systems and the systemd > > systems. > > > > Signed-off-by: Michael H. Warfield <[email protected]>
> While this is the right place to do it, I don't think we should encourage > differences among init systems... So we should either do it in upstart > as well, or not do it at all imo. Otherwise admins will become very > confused. I absolutely agree but I haven't been involved in that one. I would strongly recommend that someone update the Upstart stuff. Do you want me to look at it or do you want someone more familiar with Upstart to look at it? I'll take a look. Regards, Mike > > --- > > config/init/sysvinit/lxc.in | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/config/init/sysvinit/lxc.in b/config/init/sysvinit/lxc.in > > index b1a725c..0b802af 100644 > > --- a/config/init/sysvinit/lxc.in > > +++ b/config/init/sysvinit/lxc.in > > @@ -64,6 +64,9 @@ case "$1" in > > > > # Start containers > > wait_for_bridge > > + # Start autoboot containers first. > > + action $"Starting LXC onboot containers: " /usr/bin/lxc-autostart -g > > onboot > > + # Start NULL group containers last. > > action $"Starting LXC containers: " "$bindir"/lxc-autostart > > touch "$localstatedir"/lock/subsys/lxc > > ;; > > -- > > 1.9.0 > > > > > > > > -- > > Michael H. Warfield (AI4NB) | (770) 978-7061 | [email protected] > > /\/\|=mhw=|\/\/ | (678) 463-0932 | http://www.wittsend.com/mhw/ > > NIC whois: MHW9 | An optimist believes we live in the best of > > all > > PGP Key: 0x674627FF | possible worlds. A pessimist is sure of it! > > > > > > > _______________________________________________ > > lxc-devel mailing list > > [email protected] > > http://lists.linuxcontainers.org/listinfo/lxc-devel > > _______________________________________________ > lxc-devel mailing list > [email protected] > http://lists.linuxcontainers.org/listinfo/lxc-devel > -- Michael H. Warfield (AI4NB) | (770) 978-7061 | [email protected] /\/\|=mhw=|\/\/ | (678) 463-0932 | http://www.wittsend.com/mhw/ NIC whois: MHW9 | An optimist believes we live in the best of all PGP Key: 0x674627FF | possible worlds. A pessimist is sure of it!
signature.asc
Description: This is a digitally signed message part
_______________________________________________ lxc-devel mailing list [email protected] http://lists.linuxcontainers.org/listinfo/lxc-devel
