Re: [lxc-devel] [PATCH v2] Create log file in lxcpath for non-system containers

2013-04-30 Thread Dwight Engen
On Tue, 30 Apr 2013 08:37:44 -0500 Serge Hallyn wrote: > Quoting Dwight Engen (dwight.en...@oracle.com): > > +/* > > + * Build the path to the log file > > + * @name : the name of the container > > + * @lxcpath : the lxcpath to use as a basename or NULL to use > > LOGPATH > > + * Returns mal

Re: [lxc-devel] [PATCH v2] Create log file in lxcpath for non-system containers

2013-04-30 Thread Serge Hallyn
Quoting Dwight Engen (dwight.en...@oracle.com): > +/* > + * Build the path to the log file > + * @name : the name of the container > + * @lxcpath : the lxcpath to use as a basename or NULL to use LOGPATH > + * Returns malloced path on sucess, or NULL on failure > + */ > +static char *build_log

Re: [lxc-devel] [PATCH v2] Create log file in lxcpath for non-system containers

2013-04-29 Thread Serge Hallyn
Quoting Dwight Engen (dwight.en...@oracle.com): > So I did this, only to realize that lxc-init is passing "none" for the > file anyway, so it currently doesn't intend to log. This makes me > think that passing NULL for lxcpath is the right thing to do in > this patch. If you want me to make it so l

Re: [lxc-devel] [PATCH v2] Create log file in lxcpath for non-system containers

2013-04-29 Thread Dwight Engen
On Fri, 26 Apr 2013 10:18:12 -0500 Serge Hallyn wrote: > Quoting Dwight Engen (dwight.en...@oracle.com): > > On Fri, 26 Apr 2013 09:37:49 -0500 > > Serge Hallyn wrote: > > > > > Quoting Dwight Engen (dwight.en...@oracle.com): > > > > Using lxc configured with --enable-configpath-log, and > > >