Quoting Stéphane Graber (stgra...@ubuntu.com): > On 04/24/2013 10:16 PM, Serge Hallyn wrote: > > Otherwise we leave bad containers sitting around and further confuse > > things on retries. > > > > Reported-by: Mukanyiligira Didacienne <siyana...@gmail.com> > > Signed-off-by: Serge Hallyn <serge.hal...@ubuntu.com> > > --- > > src/lxc/lxc-create.in | 10 +++++----- > > 1 file changed, 5 insertions(+), 5 deletions(-) > > > > diff --git a/src/lxc/lxc-create.in b/src/lxc/lxc-create.in > > index fbb566f..553ffd2 100644 > > --- a/src/lxc/lxc-create.in > > +++ b/src/lxc/lxc-create.in > > @@ -286,7 +286,7 @@ fi > > > > if [ ! -r "$lxc_config" ]; then > > echo "$(basename $0): '$lxc_config' configuration file not found" >&2 > > - exit 1 > > + cleanup > > fi > > Isn't this test before the one for already existing containers and so > would lead to a lxc-destroy of the existing container?
No. The check for existing container is at line 208. The check further down is for the rootfs - which could be an LV. -serge ------------------------------------------------------------------------------ Try New Relic Now & We'll Send You this Cool Shirt New Relic is the only SaaS-based application performance monitoring service that delivers powerful full stack analytics. Optimize and monitor your browser, app, & servers with just a few lines of code. Try New Relic and get this awesome Nerd Life shirt! http://p.sf.net/sfu/newrelic_d2d_apr _______________________________________________ Lxc-devel mailing list Lxc-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/lxc-devel
