Add AUS update URL to the Mozmill update report

RESOLVED FIXED

Status

Mozilla QA
Mozmill Tests
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: whimboo, Assigned: whimboo)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [mozmill-update])

Attachments

(4 attachments, 3 obsolete attachments)

As seen on bug 640361 we need the AUS update URL as part of the test report. Otherwise it will be hard to investigate issues related to the patch selected by the updater.

Looks pretty straight forward:
http://mxr.mozilla.org/mozilla-central/source/toolkit/mozapps/update/nsUpdateService.js#2046

Will hopefully have something soonish.
Can't you use serviceURL from active-update.xml ? You get the finalURL for the mar from there or updates.xml already don't you ?
I can't use the serviceURL because it is only available if updates are available. But we also want to catch cases when we don't get an update offered, and have to know the URL for update snippets.

With bug 640407 fixed it would have been easy. For now we need a workaround.
Depends on: 640407
Created attachment 518236 [details] [diff] [review]
Patch v1 [checked-in]

Patch which retrieves the URL of the update snippet and adds it to the pre/post build info. I will have to check on windows and linux before I will request review.
Comment on attachment 518236 [details] [diff] [review]
Patch v1 [checked-in]

Looks fine on all platforms.
Attachment #518236 - Flags: review?(anthony.s.hughes)
Attachment #518236 - Flags: review?(anthony.s.hughes) → review+
Landed as:
http://hg.mozilla.org/qa/mozmill-tests/rev/db1fdda460ae (default)
http://hg.mozilla.org/qa/mozmill-tests/rev/00b0aec74223 (2.0)

We need updated patches for the backports.
Attachment #518236 - Attachment description: Patch v1 → Patch v1 [checked-in]
Depends on: 640509
Created attachment 518325 [details] [diff] [review]
Follow-up (default)

We need that follow-up on default which moves some code around and also replaces some placeholders on its own instead of letting this do the URL formatter. Reason for that is that this code is kinda specific and varies from the general placeholders. To keep the softwareUpdate interface consistent across all branches those changes are necessary.
Attachment #518325 - Flags: review?(anthony.s.hughes)
Created attachment 518326 [details] [diff] [review]
Backport (1.9.2)
Attachment #518326 - Flags: review?(anthony.s.hughes)
Created attachment 518327 [details] [diff] [review]
Backport (1.9.1)
Attachment #518327 - Flags: review?(anthony.s.hughes)
Blocks: 640523
Blocks: 640524
Whiteboard: [mozmill-update]
Attachment #518325 - Flags: review?(anthony.s.hughes) → review+
Attachment #518326 - Flags: review?(anthony.s.hughes) → review+
Attachment #518327 - Flags: review?(anthony.s.hughes) → review+
Created attachment 518397 [details] [diff] [review]
Follow-up v2 (default)

With the update of the refactoring which happened today.
Attachment #518325 - Attachment is obsolete: true
Attachment #518397 - Flags: review+
Created attachment 518400 [details] [diff] [review]
Backport v2 (1.9.2)

Folder update for refactoring changes.
Attachment #518326 - Attachment is obsolete: true
Attachment #518400 - Flags: review+
Created attachment 518401 [details] [diff] [review]
Backport v2 (1.9.1)

Folder updates for refactoring work.
Attachment #518327 - Attachment is obsolete: true
Attachment #518401 - Flags: review+
Follow-up landed:
http://hg.mozilla.org/qa/mozmill-tests/rev/61867d9626a9 (default)
http://hg.mozilla.org/qa/mozmill-tests/rev/0d49e75b9a24 (2.0)

Backports landed:
http://hg.mozilla.org/qa/mozmill-tests/rev/1d4f376d14a1 (1.9.2)
http://hg.mozilla.org/qa/mozmill-tests/rev/0f2600fbe907 (1.9.1)
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Blocks: 640899
You need to log in before you can comment on or make changes to this bug.