From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [OSSTEST PATCH 3/8] emails: honour OSSTEST_EMAIL_SUBJECT_PREFIX Date: Fri, 27 Feb 2015 09:31:13 +0000 Message-ID: <1425029473.14641.111.camel@citrix.com> References: <1424869277-16764-1-git-send-email-ian.jackson@eu.citrix.com> <1424869277-16764-4-git-send-email-ian.jackson@eu.citrix.com> <1424870672.20243.98.camel@citrix.com> <21743.23437.578256.663822@mariner.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta3.messagelabs.com ([195.245.230.39]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1YRHIF-0008WH-9F for xen-devel@lists.xenproject.org; Fri, 27 Feb 2015 09:33:11 +0000 In-Reply-To: <21743.23437.578256.663822@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.xenproject.org List-Id: xen-devel@lists.xenproject.org On Thu, 2015-02-26 at 17:44 +0000, Ian Jackson wrote: > Ian Campbell writes ("Re: [OSSTEST PATCH 3/8] emails: honour OSSTEST_EMAIL_SUBJECT_PREFIX"): > > On Wed, 2015-02-25 at 13:01 +0000, Ian Jackson wrote: > > > This is prefixed before the other computed prefixes. It makes it > > > easier to distinguish an adhoc cr-daily-branch test runs for a real > > > branch. > > > > Do they not already get "adhoc" in the $subject? i.e. my commissioning > > runs for the new arm create (following README.dev procedure) resulted in > > mails with: > > > > [adhoc test] 34418: trouble: blocked/broken/fail/pass > > > > (IOW it seems $branch is replaced by adhoc somewhere along the say) > > That happens if you use mg-execute-flight. If you let cr-daily-branch > run the flight for you, it uses the standard email stuff. Ah, OK, I didn't realise there was a difference. So Ack to this and the next patch which I didn't ack for similar reasons. (I think that makes the whole series acked, FWIW) > > Ian.