Bug 2338 - [Hob2] Hob should report that it's running sanity checks when it is
Summary: [Hob2] Hob should report that it's running sanity checks when it is
Status: VERIFIED FIXED
Alias: None
Product: Hob
Classification: Build System, Metadata & Runtime
Component: hob (show other bugs)
Version: 1.2
Hardware: x86 Multiple
: Low enhancement
Target Milestone: 1.3
Assignee: Cristian Iorga
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2012-04-18 18:48 UTC by Dave Stewart
Modified: 2012-10-12 14:24 UTC (History)
11 users (show)

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


Attachments
message and progress bar within primary application window (157.52 KB, image/png)
2012-05-25 16:45 UTC, Belen Barros Pena
no flags Details
sanity-checks.png (17.59 KB, image/png)
2012-10-12 14:24 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 Dave Stewart 2012-04-18 18:48:14 UTC
Running the near-final Hob for v1.2 in the Build Appliance.  When you start a build for the first time, there are sanity checks which are run. Would be really nice for hob to tell the user that it's running sanity checks.
Comment 1 Dexuan Cui 2012-04-19 06:49:50 UTC
(In reply to comment #0)
> Running the near-final Hob for v1.2 in the Build Appliance.  When you start a
> build for the first time, there are sanity checks which are run. Would be
> really nice for hob to tell the user that it's running sanity checks.

I agree.
Comment 2 Dongxiao Xu 2012-05-07 06:28:16 UTC
This feature needs GUI design. Belen, could you help on that?

Our first idea is that we can have a dialog to indicate the sanity check is running.
Comment 3 Belen Barros Pena 2012-05-08 09:00:53 UTC
Yep, maybe I can help. I would need to see some screenshots of what's going on exactly while Hob is running the sanity checks, if possible by running the build appliance myself. Is there anybody who could get me a link to an already built build appliance?

(In reply to comment #2)
> This feature needs GUI design. Belen, could you help on that?
> 
> Our first idea is that we can have a dialog to indicate the sanity check is
> running.
Comment 4 Saul Wold 2012-05-17 01:43:29 UTC
Belen, the Build appliance is available on the www.yoctoproject.org website under tools.

I just posted a blog entry.
Comment 5 Dongxiao Xu 2012-05-17 01:44:24 UTC
There is also sanity check when running Hob in normal machine. It happens in the stage when hob startup. You can see the mouse is spinning when startup, and sanity check is part of that.
Comment 6 Belen Barros Pena 2012-05-21 13:37:05 UTC
Thanks Saul and Dongxiao.

I've been trying to download the build appliance for the past week, but the download keeps on failing. It might a network issue on my side. I'll keep on trying...

Cheers

Belen
Comment 7 Belen Barros Pena 2012-05-25 16:45:19 UTC
Created attachment 538 [details]
message and progress bar within primary application window

I am struggling to run the build appliance, so I'll ask some questions instead. When do this sanity checks happen: when I launch Hob for the first time in the build appliance, or when you start building an image for the first time?

If the checks happen when I launch Hob, we should display a message with a progress bar in the primary window of the application. I've attached a mockup that I created for a different bug (2492) to give you an idea of how this could be handled. This if the delay justifies it, i.e. it is enough for users to think the application is not working and it is enough time to read the explanatory message. The hard work will be writing a nice message that explains what's happening in simple and understandable terms (Scott might be able to help with that).

If the checks happen when I start building an image for the first time, we should use the 'Build in progress' screen to provide this information. We would simply write 'Sanity checks' as the running task at the top of the screen, with the 'recipe' / 'package' details empty.

Let me know if this makes sense.

Belen
Comment 8 Saul Wold 2012-06-05 02:51:37 UTC
(In reply to comment #7)
> Created attachment 538 [details]
> message and progress bar within primary application window
> 
> I am struggling to run the build appliance, so I'll ask some questions
> instead. When do this sanity checks happen: when I launch Hob for the first
> time in the build appliance, or when you start building an image for the
> first time?
> 
Belen,

As I said before, this is not a  build appliance issue, but a generic Hob issue
that can occur when you do not have any proxies set, and the first time you
run hob, it tries to do the network activity.

> If the checks happen when I launch Hob, we should display a message with a
> progress bar in the primary window of the application. I've attached a
> mockup that I created for a different bug (2492) to give you an idea of how
> this could be handled. This if the delay justifies it, i.e. it is enough for
> users to think the application is not working and it is enough time to read
> the explanatory message. The hard work will be writing a nice message that
> explains what's happening in simple and understandable terms (Scott might be
> able to help with that).
> 
I think this needs to be re-assigned to someone who is implementing this now.

I am not sure if this should still be a 1.2.1 bug, while it is usablitily, this
is more of an enhancement.


> If the checks happen when I start building an image for the first time, we
> should use the 'Build in progress' screen to provide this information. We
> would simply write 'Sanity checks' as the running task at the top of the
> screen, with the 'recipe' / 'package' details empty.
> 
> Let me know if this makes sense.
> 
> Belen
Comment 9 Shane Wang 2012-06-07 02:36:12 UTC
Put it to 1.3 M3, since that is a new feature enhancement.
Comment 10 Belen Barros Pena 2012-06-12 13:05:50 UTC
As part of this bug, we need to implement 2 different messages:

- the attached one (see bug 2492)
- a message for when Hob is running sanity checks. Message content could be:

Hob is checking that your build system is correctly set up. It shouldn't take long. 

Scott: is the above message ok? Would you like to suggest something different?

It is important that we set rules regarding when the sanity check message should appear and how long it should stay. This is what I propose:

1. If sanity checks take longer than 4 seconds, display the sanity check message
2. Sanity check message should display for at least 5 seconds to give users time to read and understand what's happening (even if the sanity checks finish before then). We don't want a fleeting message that disappears so quickly people don't have time to see what's about.

If you'd like to suggest different time frames, please do so. 

Belen
Comment 11 Scott Rifenbark 2012-06-12 13:50:46 UTC
Belen,

This would be better if room allows:


"Hob is checking for correct build system setup. The check shouldn't take long."


Scott
Comment 12 Belen Barros Pena 2012-06-12 13:55:21 UTC
Works for me: space should not be an issue in this case. 

Thanks!

Belen

(In reply to comment #11)
> Belen,
> 
> This would be better if room allows:
> 
> 
> "Hob is checking for correct build system setup. The check shouldn't take
> long."
> 
> 
> Scott
Comment 13 Cristian Iorga 2012-09-25 10:25:20 UTC
This will get re-tested once bug 3025 will be resolved.
It seems that some parts of this bug will get fixed than.
Comment 14 Belen Barros Pena 2012-10-12 14:23:50 UTC
3025 fixes this problem: Hob now tells you when it's running sanity checks (see attached sanity-checks.png)
Comment 15 Belen Barros Pena 2012-10-12 14:24:31 UTC
Created attachment 886 [details]
sanity-checks.png
Comment 16 Belen Barros Pena 2012-10-12 14:24:42 UTC
verified