From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH OSSTEST] Arrange for core dumps to be placed in /var/core and collect them Date: Tue, 24 Mar 2015 12:46:46 +0000 Message-ID: <1427201206.21742.389.camel@citrix.com> References: <1425915623-14878-1-git-send-email-ian.campbell@citrix.com> <21777.23376.332679.268338@mariner.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <21777.23376.332679.268338@mariner.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Jackson Cc: xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On Tue, 2015-03-24 at 12:40 +0000, Ian Jackson wrote: > Ian Campbell writes ("[PATCH OSSTEST] Arrange for core dumps to be placed in /var/core and collect them"): > > Refactor the $kvp_replace helper in ts-xen-install into a generic > > helper (which requires using ::EO and ::EI for namespacing) for use > > with target_editfile [etc.] > > Acked-by: Ian Jackson > > > I did debate making the presence of cores in /var/core fail the test > > (somehow), but I decided that would be annoying for standalone mode or > > shared host scenarios where the core files might be stale or related > > to another job. > > I think you should add /var/core to the leak check step. Which in practice just means listing it in the arugment to inventory_files?