Bug 2732

Summary:	Better build failure email.
Product:	[Infrastructure] AutoBuilder	Reporter:	Beth Flanagan <elizabeth.flanagan>
Component:	autobuilder	Assignee:	Beth Flanagan <elizabeth.flanagan>
Status:	VERIFIED OBSOLETE	QA Contact:
Severity:	normal
Priority:	Medium	CC:	alexandru.c.georgescu, alexandrux.palalau, corneliux.stoicescu, elizabeth.flanagan, infras.ab.watcher, Infras.watcher, sgw, song.liu
Version:	1.3
Target Milestone:	1.3 M4
Hardware:	x86
OS:	Multiple
Whiteboard:
OS type for building Yocto:	---	Type of Regression:	---
Verified:		Documentation change:	---

Description Beth Flanagan 2012-07-10 17:00:00 UTC

The build failure email is good but needs better failure emails. Todo:

grep for "| ERROR:" and "| WARN:"
if the buildstep is a copy, ignore.
send out the first 20 lines not the last 20
send out the consolidated error from bitbake (end of build)

Comment 1 Beth Flanagan 2012-08-27 21:21:35 UTC

Fixed in commit 77d297b489b315bbb3e358c3a4be02b778f72d5e in the autobuilder production repo.

Comment 2 Alexandru Palalau 2012-10-04 10:02:34 UTC

At this moment I do not see all of the mentioned changes within the autobuilder email messages.
-> The last 20 lines are sent (not the first 20)
-> Not all the ERROR and WARN lines are presented

Consolidated error is present but this happens due to the fact that it is written last (it gets included within the 20 last lines).

Comment 3 Beth Flanagan 2012-10-04 17:06:33 UTC

I'm marking this as Resolved:

We currently have this implemented in the production autobuilder although the initial specifications are different than what I implemented. It does what is needed without spamming the list with lots of bad emails.

Comment 4 Stoicescu Cornel 2012-10-08 09:06:00 UTC

Hello

How can I verify this? Or should we put it on Obsolete as the initial specifications are no longer valid?

Thank you

Comment 5 Beth Flanagan 2012-10-08 16:27:45 UTC

The original specs are no longer valid. This should be listed as resolved. "better" is kind of subjective here.

Comment 6 Alexandru Georgescu 2012-10-09 13:04:46 UTC

(In reply to comment #5)
> The original specs are no longer valid. This should be listed as resolved.
> "better" is kind of subjective here.

Marking this as Verified as per the last comment.