[update verification] verify.sh: Print out what locale failed

RESOLVED FIXED

Status

P2
normal
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: jlorenzo, Assigned: jlorenzo)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [releaseduty])

Attachments

(1 attachment)

(Assignee)

Description

a year ago
While investigating bug 1389466, I stumbled upon an easy modification that will help debugging update verification tests (like bug 1388431). This won't be as efficient as splitting update verification per locale (bug 1389478), but that will concatenate the failed locales at the end of the log file.
Comment hidden (mozreview-request)
Priority: -- → P2
Whiteboard: [releaseduty]

Comment 2

a year ago
mozreview-review
Comment on attachment 8899456 [details]
Bug 1392262 - [update verification] verify.sh: Print out what locale failed

https://reviewboard.mozilla.org/r/170738/#review176610

Good idea, but shall we go a step further and use [$release $locale $patch_type] to cover other types of bustages ?
Attachment #8899456 - Flags: review?(nthomas) → review+
Comment hidden (mozreview-request)
(Assignee)

Comment 4

a year ago
Good point! I initially though $release wasn't worth it because the value is already present in $source_file. However, not every error message has $source_file in it. Thus, I fully implemented your suggestion.

Landed on default at: https://hg.mozilla.org/build/tools/rev/366a7b278252cf7497e34fd79e3294da610b2db5
(Assignee)

Updated

a year ago
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.