[Toaster] [review-request] adamian/20150107-allbuilds-6671

[Barros Pena, Belen]

On 12/01/2015 16:32, "Damian, Alexandru" <alexandru.damian at intel.com>
wrote:

>I guess we need to find reasonable values for ttruncation -
>
>
>- Not all the error messages are displayed - just the first 3, but there
>may be lots of errors not being displayed.
>
>
>- In the table, the error messages are truncated after the first word
>ending after 32 characters, because the errors can be quite long and will
>mess up the table.

We kind of have design for this. As a general rule, these builds behave
like any other builds, i.e., they show the number of errors thrown and
clicking the number brings you to the build dashboard page where the full
error(s) text is shown.

The only exception are 4 types of errors that hopefully are easy to
identify and that we can help users recover from because we know what's
causing the issue:

1. Layers that fail to clone
2. Nothing provides <target>
3. Invalid machine
4. Invalid distro

For these builds, we still show the number of errors link to the build
dashboard page, but we also show some custom text to explain the problem
in a bit more detail to users. Details in pages 44 to 56 of the design
document 'Configuring projects' attached to 6587:

https://bugzilla.yoctoproject.org/attachment.cgi?id=2230
 


We might need a separate Bugzilla entry for this work.

Cheers

Belen

>
>
>
>
> 
>
>
>On Mon, Jan 12, 2015 at 3:51 PM, Barros Pena, Belen
><belen.barros.pena at intel.com> wrote:
>
>Seems to be working for me: failed build requests now show in both project
>page and all builds page. Although we'll need to change the way we are
>displaying them (we don't want all the errors listed in the build item,
>which is just a summary)
>
>Thanks!
>
>Belén
>
>On 12/01/2015 14:50, "Damian, Alexandru" <alexandru.damian at intel.com>
>wrote:
>
>>Hello,
>>
>>
>>Can you please review the implementation for showing failed and pending
>>build requests in the "all builds" page, for bug 6671 ?
>>
>>
>>The patch is pushed here:
>>poky-contrib:adamian/20150107-allbuilds-6671
>>
>>
>>
>>
>>
>>​Cheers,
>>Alex​
>>
>>
>>--
>>Alex Damian
>>Yocto Project
>>
>>SSG / OTC
>>
>>
>>
>
>
>
>
>
>
>
>
>
>
>-- 
>Alex Damian
>Yocto Project
>
>SSG / OTC 
>
>
>