Closed Bug 1353917 Opened 3 years ago Closed 3 years ago

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

Categories

(Toolkit :: Application Update, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox54 --- fixed
firefox55 --- fixed

People

(Reporter: mhowell, Assigned: mhowell)

References

Details

Attachments

(1 file)

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 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+
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
https://hg.mozilla.org/mozilla-central/rev/165a34ba15d9
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
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?
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.
(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-
You need to log in before you can comment on or make changes to this bug.