Closed Bug 821523 Opened 12 years ago Closed 12 years ago

During a system update (stalled or not) forcing an update check from the settings app the status gets stuck at "Checking for updates..."

Categories

(Firefox OS Graveyard :: General, defect, P4)

x86_64
Linux
defect

Tracking

(blocking-b2g:-, blocking-basecamp:-, b2g18+)

RESOLVED DUPLICATE of bug 831701
blocking-b2g -
blocking-basecamp -
Tracking Status
b2g18 + ---

People

(Reporter: dholbert, Assigned: dhylands)

References

Details

(Keywords: b2g-testdriver, unagi, Whiteboard: [UX-P?])

Attachments

(1 file)

This morning, I was on the build ID 20121206062128. I received an update notification for a ~52 meg update, and tried to update a few times, but the update stalled at some point during the download. I rebooted my phone and clicked "Check for updates" in Settings, and it just stayed stuck at "Checking for updates..." Meanwhile, from watching logcat, I could see that the update actually _had resumed on its own_ (after the reboot), and it ultimately downloaded & extracted itself in the background (so that when I rebooted _again_, I was updated). That's cool, but it didn't update the UI at all to indicate this -- there was nothing in the notification screen the whole time, and the Settings app stayed at "Checking for updates..." Filing this bug on the UI inconsistency there.
Summary: After update download issues & phone-reboot, update downloads but doesn't update UI. (stuck at "Checking for updates...") → After phone-reboot midway through an update download, the update resumes but doesn't update UI. (stuck at "Checking for updates...")
Here's the logcat output, from when I rebooted the phone & tried to check for updates (and the update resumed in the background, with no UI indication at all).
Component: Gaia → Gaia::System
Triage: BB+, P3, C3 - confuse users if the update is done or not or are they waiting for new updates
blocking-basecamp: ? → +
Priority: -- → P3
Target Milestone: --- → B2G C3 (12dec-1jan)
Assignee: nobody → etienne
Assignee: etienne → felash
This looks like a platform bug but I'll dig further just to be sure. I'll also check what happens for app updates. However there is a UX decision to be made here. I think this is wrong that we automatically resume the update. We should rather not resume and show to the user that there is an update available and propose him to update, just as if nothing was done before. Asking for UX decision here.
Flags: needinfo?(jcarpenter)
Whiteboard: [UX-P?]
I guess platform should be re-sending the update-apply-prompt here for a resume?
(In reply to Marshall Culpepper [:marshall_law] from comment #4) > I guess platform should be re-sending the update-apply-prompt here for a > resume? Gaia will be listening :)
I think we should even restart at update-available. We don't want a download to happen without the user consent. Gaia is already listening to update-available anyway so this would work without a change in gaia. Étienne, what do you think ?
(In reply to Julien Wajsberg [:julienw] from comment #6) > I think we should even restart at update-available. > > We don't want a download to happen without the user consent. > > Gaia is already listening to update-available anyway so this would work > without a change in gaia. > > Étienne, what do you think ? Makes sense.
I don't know if this could be fixed in the same bug, but I noticed if I don't start the update download, I still don't get the system update notification when I reboot the phone. However I get it when I force "check for update" from Settings.
Cleaning up this bug to focus on the Settings app part, the rest is tracked in bug 819548.
Summary: After phone-reboot midway through an update download, the update resumes but doesn't update UI. (stuck at "Checking for updates...") → During a system update (stalled or not) forcing an update check from the settings app the status gets stuck at "Checking for updates..."
(In reply to Julien Wajsberg [:julienw] from comment #6) > I think we should even restart at update-available. > > We don't want a download to happen without the user consent. I agree. If the device restarts we must prompt the user to resume, _never_ automatically resume, given the cost of bandwidth in the target markets.
Flags: needinfo?(jcarpenter)
Oops, I see the scope of this bug has changed. Am copying my comments over to #819548.
I've seen when working on Bug 819548 that if we have an activeUpdate (as in UpdateManager.activeUpdate), we never get the message back from the platform. Worth investigating, but I'm not sure this should be blocking basecamp, so renominating.
blocking-basecamp: + → ?
Note that in Bug 819548's resolution we should clear UpdateManager.activeUpdate if present at startup. So when Bug 819548 is fixed, then this should only happen if we force an update when there is already an update happening. That's why I'm renominating.
Triage: BB-. tracking-b2g18+, p4 - with stricter criteria for traige now, move it to BB-
blocking-basecamp: ? → -
tracking-b2g18: --- → +
Priority: P3 → P4
Target Milestone: B2G C3 (12dec-1jan) → ---
This is still an issue w/ 20130104070203 (the most recent build, up until this afternoon) -- I reproduced this while updating from that build to today's beta release. (i.e. while today's update was downloading, I hit the "Check now" button again, just for fun, and the text below it said "Checking for updates..." and it stayed stuck there until I rebooted my phone.)
this makes sense, as we didn't fix this bug yet :-)
Still seeing this. Requesting blocking-b2g:tef+ because update-related failure, and that flow *must* be bulletproof. Today I checked, and got the notification, but the Settings UI was still stuck on "Checking for updates...".
blocking-b2g: --- → tef?
Dietrich, we currently have Bug 829522 producing what I think you're seeing. I plan to work on this tomorrow. This is not related to the system update so this is not this bug.
To be clear, this bug is about the following STR: * check for update (from settings) * there is a system update * begin the system update download * check for update again => it never returns To me, this occurs because of [1] : 2248 _selectAndInstallUpdate: function AUS__selectAndInstallUpdate(updates) { 2249 // Return early if there's an active update. The user is already aware and 2250 // is downloading or performed some user action to prevent notification. 2251 var um = Cc["@mozilla.org/updates/update-manager;1"]. 2252 getService(Ci.nsIUpdateManager); 2253 if (um.activeUpdate) 2254 return; and/or [2] : 55 onCheckComplete: function UCL_onCheckComplete(request, updates, updateCount) { 56 if (Services.um.activeUpdate) { 57 return; 58 } [1] https://mxr.mozilla.org/mozilla-central/source/toolkit/mozapps/update/nsUpdateService.js#2253 [2] https://mxr.mozilla.org/mozilla-central/source/b2g/components/UpdatePrompt.js#55
Dietrich, also, Bug 813770 just got fixed yesterday and maybe is not arrived in a build near you yet.
blocking-b2g: tef? → -
Assigning this to Dave, he may infirm or confirm what I said in Comment 20, and maybe give us an indication of the complexity for fixing this. Then we could maybe renom depending on that.
Assignee: felash → dhylands
Component: Gaia::System → General
I'll take a look at this. I haven't yet really looked at this aspect of the update. I can say that with bug 813770 landed you should never see a download resume on its own (as mentioned in comment 1).
Yep Dave, the history of this bug is complicated as it was really several bugs :) But the last bug remaining (imho) is the one with STR in comment 20.
Depends on: 831701
Dave said that the fix in Bug 831701 will fix these last STR. Houra !
Yes - all paths through onCheckComplete now call setUpdateStatus, so you shouldn't see this any more. Marking as a dup of bug 831701.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: