Merge lp://qastaging/~clint-fewbar/ubuntu/oneiric/upstart/add-plymouth-messages into lp://qastaging/ubuntu/oneiric/upstart

Proposed by Clint Byrum
Status: Merged
Merged at revision: 1326
Proposed branch: lp://qastaging/~clint-fewbar/ubuntu/oneiric/upstart/add-plymouth-messages
Merge into: lp://qastaging/ubuntu/oneiric/upstart
Diff against target: 52 lines (+36/-1)
2 files modified
debian/changelog (+8/-0)
debian/conf/failsafe.conf (+28/-1)
To merge this branch: bzr merge lp://qastaging/~clint-fewbar/ubuntu/oneiric/upstart/add-plymouth-messages
Reviewer Review Type Date Requested Status
Ubuntu branches Pending
Review via email: mp+75278@code.qastaging.launchpad.net

Description of the change

This attempts to make it as clear as possible to users what is blocking the system boot when they have interfaces defined in /etc/network/interfaces that cannot be brought up.

To post a comment you must log in.
Revision history for this message
Steve Langasek (vorlon) wrote :

+ $PLYMOUTH pause-progress || :

Why does the progress need to be paused? This probably makes no difference with the standard Ubuntu plymouth theme, since we simply cycle the dots on the screen, but for themes that do care about progress bars I'm not sure pausing is the right thing. Have you tried it without this change?

+ $PLYMOUTH message --text="Waiting for network configuration..." || :
+ sleep 40
+ $PLYMOUTH message --text="Waiting up to 60 more seconds for network configuration..." || :
+ sleep 59

What's the value in having two different messages here instead of just one? Is this to keep the user from thinking the machine has hung? (If so, see above about not pausing progress...?)

+ continue_boot () {
+ $PLYMOUTH unpause-progress || :
+ }

Does this work in practice, or does upstart kill the job with too much force?

Also, I think this leaves the last message on the screen - do we want to clear it instead?

+ sleep 1

Not sure what this buys us - is this to try to ensure there's time to see the message before plymouth exits?

Revision history for this message
Clint Byrum (clint-fewbar) wrote :

Re pause/unpause, I hadn't thought about it much, but I suppose it makes sense that its still "Making progress".

I feel like this is a situation that we're hoping nobody runs into who didn't hack on their system in a weird way. So its best to inform the user of whats going on so they don't get a frustrating experience. Another option would be to watch for a keystroke and let the user skip the wait.. but that seems more complicated than just updating the message a few times while we wait.

The EXIT trap works fine as long as the plymouth command finishes within 5 seconds, but I think I'll take out the pause/unpause anyway, as I have no good reason to do that.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: