Bug 8443 - Builds that failed at the build request stage need a custom dashboard
Summary: Builds that failed at the build request stage need a custom dashboard
Status: RESOLVED FIXED
Alias: None
Product: Toaster
Classification: Build System, Metadata & Runtime
Component: toaster (show other bugs)
Version: 1.8
Hardware: x86 Multiple
: Medium+ normal
Target Milestone: 2.2 M3
Assignee: Elliot Smith
QA Contact: Libertad
URL:
Whiteboard: Look at 8440
Depends on: 8440
Blocks:
  Show dependency tree
 
Reported: 2015-10-05 18:20 UTC by Belen Barros Pena
Modified: 2016-08-22 10:59 UTC (History)
3 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)


Attachments
Design - Displaying failed build requests (858.34 KB, application/pdf)
2015-10-05 18:20 UTC, Belen Barros Pena
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Belen Barros Pena 2015-10-05 18:20:43 UTC
Created attachment 2782 [details]
Design - Displaying failed build requests

Those builds have no logs, no time on task associated with them, and no data whatsoever. So their build dashboard and build status item across Toaster requires special treatment. See attached design document for details.
Comment 1 Elliot Smith 2016-02-01 14:05:25 UTC
I've applied this design to *all* failed builds in the work I've done so far.

If this is how we want this to work, the work I've done is ready for review; if not, could you please supply additional information about when we should use the failed build request dashboard vs. the dashboard we usually show?
Comment 2 Elliot Smith 2016-02-01 14:43:09 UTC
The branch is:
contrib, elliot/toaster/failed_builds_display-8443
Comment 3 Elliot Smith 2016-02-02 10:54:51 UTC
After discussion with Belen, we've decided to only show the custom dashboard for builds which fail before the BuildStarted event. (The BuildStarted event is the one which fires just after bitbake shows the banner with the layer versions, bitbake version etc.)

However, to do this, we need the patches for bug 8440. These create the build object as soon as the buildTargets() command goes to bitbake and will let us catch failed "build requests".

At the point when buildTargets() is called, the build can be considered "not started", as bitbake is still figuring out whether the targets are valid; so if the build fails here, it should display with the custom failed build dashboard.

But when BuildStarted fires, the build proper is started, so we use the standard dashboard from this point on, even if the build fails.
Comment 4 Elliot Smith 2016-02-26 14:48:15 UTC
Submitted RFC for branch (rebased on 8440 branch):
https://lists.yoctoproject.org/pipermail/toaster/2016-February/003979.html
Comment 5 Elliot Smith 2016-04-07 13:27:59 UTC
I did submit this for review, but the steps which make this work visible won't work until bug 8440 is fixed.
Comment 6 Elliot Smith 2016-04-14 07:17:05 UTC
This needs to wait until we have a fix for bug 8440, which is under discussion.

Please ignore the review request for now.
Comment 7 Elliot Smith 2016-07-15 12:52:09 UTC
Submitted patches to mailing list:
https://lists.yoctoproject.org/pipermail/toaster/2016-July/004997.html