640369 - Add AUS update URL to the Mozmill update report

Status: RESOLVED FIXED

(Mozilla QA Graveyard :: Mozmill Tests, defect)

Description

[Henrik Skupin [:whimboo][⌚️UTC+2]]

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.

Comment 1

[Nick Thomas [:nthomas] (UTC+12)]

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 ?

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+2]]

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.

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: 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 4

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Comment on attachment 518236 [details] [diff] [review]
Patch v1 [checked-in]

Looks fine on all platforms.

Comment 5

[Henrik Skupin [:whimboo][⌚️UTC+2]]

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.

Comment 6

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: 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.

Comment 7

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Backport (1.9.2)

Comment 8

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Backport (1.9.1)

Comment 9

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Follow-up v2 (default)

With the update of the refactoring which happened today.

Comment 10

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Backport v2 (1.9.2)

Folder update for refactoring changes.

Comment 11

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Backport v2 (1.9.1)

Folder updates for refactoring work.

Comment 12

[Henrik Skupin [:whimboo][⌚️UTC+2]]

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)