Quoting Wolfgang Bumiller (w.bumil...@proxmox.com):
> > For ease of future git history reviewers, please point out specifically
> > that this means setting lxc.network.veth.pair.
> 
> will do
> 
> > Tested that this does fix the issue, so once you handle the error
> > cases in __lxc_start (easily done with a new label), I'll happily
> > ack - thanks much.
> 
> I can do that, but wanted to run another idea by you first:
> Since conf->reboot is currently an int instead of a bool, it might be even
> easier to make it a three-state value, that way no current or future error
> caseswould have to deal with it specially.
> Essentially doing:
> 
> if (conf->reboot == 1) { // note the ==1
>     conf->reboot = 2; // avoid infinite looping
>     goto reboot;
> }
> 
> It would retain its boolean truth value and deal with the error cases.
> The reset to 0 after spawning would still be there to allow multiple
> reboots in a row (obviously).
> 

Sounds good, thanks.
_______________________________________________
lxc-devel mailing list
lxc-devel@lists.linuxcontainers.org
http://lists.linuxcontainers.org/listinfo/lxc-devel

Reply via email to