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

RESOLVED FIXED

Status

Release Engineering
Release Automation
P2
normal
RESOLVED FIXED
5 months ago
5 months ago

People

(Reporter: jlorenzo, Assigned: jlorenzo)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [releaseduty])

MozReview Requests

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

Attachments

(1 attachment)

(Assignee)

Description

5 months 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

5 months 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

5 months 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

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