[lxc-devel] [PATCH] Add a non-null group for autobooting

[Serge Hallyn]

Quoting Michael H. Warfield (mhw at WittsEnd.com):
> On Fri, 2014-05-02 at 21:03 +0000, Serge Hallyn wrote:
> > Quoting Michael H. Warfield (mhw at WittsEnd.com):
> > > 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 <mhw at WittsEnd.com>

Acked-by: Serge E. Hallyn <serge.hallyn at ubuntu.com>

> 
> > 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

Agreed.  Anyway your patch looks ok to me, but let's just wait and see
what Stéphane thinks...  if he's happy with it then he or I can do the
usptart bit.

thanks,
-serge