Bug 2183 - [Hob2]Build failure scenario
Summary: [Hob2]Build failure scenario
Status: VERIFIED FIXED
Alias: None
Product: Hob
Classification: Build System, Metadata & Runtime
Component: hob (show other bugs)
Version: 1.2
Hardware: x86 Multiple
: Medium enhancement
Target Milestone: 1.3 M4
Assignee: anliming
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2012-03-28 10:23 UTC by Giulia
Modified: 2012-09-28 14:43 UTC (History)
11 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: ---


Attachments
build-failed.png (7.64 KB, image/png)
2012-05-28 11:34 UTC, Belen Barros Pena
no flags Details
build-failed.jpg (87.33 KB, image/jpeg)
2012-06-14 14:09 UTC, Belen Barros Pena
no flags Details
build-fail-spec.pdf (598.82 KB, application/pdf)
2012-07-12 14:25 UTC, Belen Barros Pena
no flags Details
build-fail-spec.pdf (598.31 KB, application/pdf)
2012-07-16 13:00 UTC, Belen Barros Pena
no flags Details
wrong-number-of-issues.png (37.88 KB, image/png)
2012-08-21 10:15 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 Giulia 2012-03-28 10:23:49 UTC
The build failure scenario hasn't been implemented as for the document https://wiki.yoctoproject.org/wiki/File:Build-fail-flow-v1.0.pdf (I filed this bug as an enhancement because this process is still to be reviewed by the development team)
Comment 1 anliming 2012-05-25 09:52:05 UTC
Hi Giulia, this function has be merged to poky/master, please to review.http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=374ad22550f0610f41e127352a3efb2c62689ff0
Comment 2 Belen Barros Pena 2012-05-28 11:34:25 UTC
Created attachment 546 [details]
build-failed.png

build-failed.png shows what I see when the build fails. Not good :(
Comment 3 anliming 2012-06-13 02:41:32 UTC
Hi Belen I can't see your attachment, please to attached again. and it your means the bug is not ok? what points about your concern
Comment 4 Belen Barros Pena 2012-06-14 14:09:00 UTC
Hi Liming,

It's a completely blank window. I've attached it again in a different format (build-failed.jpg). Let me know if you still can't see it.

(In reply to comment #3)
> Hi Belen I can't see your attachment, please to attached again. and it your
> means the bug is not ok? what points about your concern
Comment 5 Belen Barros Pena 2012-06-14 14:09:19 UTC
Created attachment 585 [details]
build-failed.jpg
Comment 6 anliming 2012-06-15 02:32:44 UTC
yes, it's a completely blank window, you want to changed to this window, if building failed? can you give me some description.
Comment 7 anliming 2012-06-18 01:30:28 UTC
Hi Benlen, can you describe it clearly to how to fixed it. Thanks!
Comment 8 Belen Barros Pena 2012-06-18 09:04:15 UTC
Will do.

(In reply to comment #7)
> Hi Benlen, can you describe it clearly to how to fixed it. Thanks!
Comment 9 Shane Wang 2012-06-26 05:28:26 UTC
Any update from Belen? It is also a blank window.
Comment 10 Belen Barros Pena 2012-06-26 10:49:18 UTC
You mean I am a blank window??? ;) I have some work to do on 2345, 2162 and finishing up 2165. Once those are done I'll be able to tackle this one. Sorry for the delay: I am just a bit over-stretched right now.

Cheers

Belen
Comment 11 anliming 2012-06-26 11:05:19 UTC
I know, Thanks!
Comment 12 Shane Wang 2012-06-26 11:49:24 UTC
(In reply to comment #10)
> You mean I am a blank window??? ;) I have some work to do on 2345, 2162 and
> finishing up 2165. Once those are done I'll be able to tackle this one.
> Sorry for the delay: I am just a bit over-stretched right now.

Cheers

Belen

I tried google translation to see any special meaning for "I am a blank window", but didn't have any findings:-) I didn't intend to offend you. Please let me know to avoid any misunderstanding in the future. Thanks.
Comment 13 Belen Barros Pena 2012-07-12 14:25:13 UTC
Created attachment 634 [details]
build-fail-spec.pdf

I've attached a design document showing what happens when a build fails. Please note that this design depends on the implementation of bugs 1991 and 2165.

Any questions, feedback, etc, let me know.

Belen
Comment 14 anliming 2012-07-13 01:13:49 UTC
Hi Belen, I know, I will refer to 1991 and 2165 to fixed this bug.Thanks!
Comment 15 Belen Barros Pena 2012-07-16 13:00:01 UTC
Created attachment 636 [details]
build-fail-spec.pdf
Comment 17 Belen Barros Pena 2012-08-15 12:25:52 UTC
Hi Liming,

I've checked your branch. I have a couple of comments:

1. The label of the primary button should always be 'Edit image'. Right now, if you build without customisation, the button says: "Edit image configuration". If you build with customisation and package building fails, it says: "Edit recipes"

2. When you build with customisation and package building fails, the heading of the fail message should say: "Hob cannot build the packages". Right now it says: "Hob cannot build your recipes". 

3. When the failure happens, the selected tab should be 'Issues'. Right now, the selected tab is 'Log'

Thanks!

Belen



(In reply to comment #16)
> Changed as new ui design, please to review
> http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/commit/?h=limx/hob-
> bug-fixes-continue&id=a218bf8a38dfdd61421893ee2ae36c0bb91b55b8
Comment 18 anliming 2012-08-17 02:40:09 UTC
Hi Belen, thanks your replay, I had changed it, please to reviewhttp://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/commit/?h=limx/hob-bug-fixes-continue
Comment 19 Belen Barros Pena 2012-08-21 08:50:54 UTC
Hi Liming,

This is looking good now. Thanks! 

Once your patch is merged to master, I'll check it there and mark it as verified.

Belen
Comment 20 Belen Barros Pena 2012-08-21 10:15:22 UTC
Created attachment 709 [details]
wrong-number-of-issues.png

Sorry, Liming, I forgot to mention that I am not sure the number of issues displayed is accurate. In the screenshot attached (wrong-number-of-issues.png) the tab announces 6 issues, but there are only 3 packages listed (zlib-native, gmp-native and pigz-native). How are we calculating the number of issues? 

Thanks!

Belen
Comment 21 anliming 2012-08-22 01:53:57 UTC
Hi Belen, the number of issues is same as the times of error occurs in the log page, so you can see there has two indicators or more in one recipe item which be built. because original design maybe mentioned the warning and error in issue category of 'issues', but now we only care of the error. Thanks!
Comment 22 Belen Barros Pena 2012-08-22 09:20:14 UTC
Thanks!

(In reply to comment #21)
> Hi Belen, the number of issues is same as the times of error occurs in the
> log page, so you can see there has two indicators or more in one recipe item
> which be built. because original design maybe mentioned the warning and
> error in issue category of 'issues', but now we only care of the error.
> Thanks!
Comment 23 Belen Barros Pena 2012-09-19 10:22:12 UTC
Hi Liming,

Do you know if your patch has been merged to master? Thanks!
Comment 24 Belen Barros Pena 2012-09-28 14:43:24 UTC
After some conversations with Cristiana when she was doing work on https://bugzilla.yoctoproject.org/show_bug.cgi?id=2537, we decided that button labels should change depending on where the build is stopped, and that they should be consistent in the case of the build failing. 

Therefore, for the case of a failed build:

1. If the build fails when building an image without customisation, we use 'edit image' as the button label. The button brings you back to the Image configuration screen (no matter when the build failed)

2. If when customising the base image the build fails while building the packages, we use 'edit recipes' as the button label. The button brings you back to the recipes selection screen

3. If when customising the base image the build fails while building the image, we use 'edit packages' as the button label. The button brings you back to the packages selection screen

This is how Hob is currently working, so marking as verified.