Update wizard fallback page fails the update when advancing to the downloading page after staging is done

RESOLVED FIXED in Firefox 54

Status

()

Toolkit
Application Update
RESOLVED FIXED
8 months ago
7 months ago

People

(Reporter: mhowell, Assigned: mhowell)

Tracking

unspecified
mozilla55
Points:
---
Dependency tree / graph
Bug Flags:
qe-verify -

Firefox Tracking Flags

(firefox54 fixed, firefox55 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Assignee)

Description

8 months ago
Copied from bug 1316564 comment 55:

1) Starting from 53.0b8, update via the about window and close Firefox instead of clicking the restart button. This results in a partial update being successfully staged.
2) Overwrite the update.status file with "failed: 6".
3) Start the browser. Observe the "Update Failed" dialog, but don't click OK yet. The complete update downloads and stages while this window is showing.
4) Click OK, but only _after_ the update is downloaded and staged.
5) The progress window that appears next is broken; it hangs on "Connecting to the update server..."
6) Open the About window and click the restart button that is already showing there.
7) Observe that the update has not been applied.

The reason the update hasn't been applied is that step 4 causes the update.status file to be overwritten with "null". This is clearly a bug.
Comment hidden (mozreview-request)
(Assignee)

Comment 2

8 months ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=e4f1ef758e0f010fb06b46fb2b938e9aa85a7968
Comment on attachment 8855076 [details]
Bug 1353917 - Skip the downloading page in the update wizard if the update is already staged.

https://reviewboard.mozilla.org/r/126982/#review129670
Attachment #8855076 - Flags: review?(robert.strong.bugs) → review+

Comment 4

8 months ago
Pushed by mhowell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/165a34ba15d9
Skip the downloading page in the update wizard if the update is already staged. r=rstrong

Comment 5

8 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/165a34ba15d9
Status: ASSIGNED → RESOLVED
Last Resolved: 8 months ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
(Assignee)

Comment 6

8 months ago
Comment on attachment 8855076 [details]
Bug 1353917 - Skip the downloading page in the update wizard if the update is already staged.

Approval Request Comment

[Feature/Bug causing the regression]:
Unknown; regression range doesn't show any apparently relevant changes.

[User impact if declined]:
This patch is a potential fix for bug 1316564, which is a high-ranking test failure on beta and ESR52. Per bug 1316564 comment 62, we would like to verify the fix (to the extent possible) on aurora and get it uplifted from there as soon as possible if it does work.

[Is this code covered by automated tests?]:
Yes

[Has the fix been verified in Nightly?]:
Not yet, just landed today

[Needs manual test from QE? If yes, steps to reproduce]: 
No

[List of other uplifts needed for the feature/fix]:
None

[Is the change risky?]:
No

[Why is the change risky/not risky?]:
It's a minor and straightforward fix, and it's in UI that's not on any normal update path.

[String changes made/needed]:
None
Attachment #8855076 - Flags: approval-mozilla-aurora?

Updated

8 months ago
status-firefox54: --- → affected

Comment 7

8 months ago
Comment on attachment 8855076 [details]
Bug 1353917 - Skip the downloading page in the update wizard if the update is already staged.

Fix an update issue. Aurora54+.
Attachment #8855076 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Sadly the regression in bug 1355009 prevents us from verifying this fix. I will try to get this fixed ASAP.

Comment 9

8 months ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-aurora/rev/49b1ef6307a9
status-firefox54: affected → fixed
(In reply to Matt Howell [:mhowell] from comment #6)
> [Is this code covered by automated tests?]:
> Yes
> [Needs manual test from QE? If yes, steps to reproduce]: 
> No

Setting qe-verify- based on Matt's assessment on manual testing needs and the fact that this fix has automated coverage.
Flags: qe-verify-
Blocks: 1355818
You need to log in before you can comment on or make changes to this bug.