Closed
Bug 842992
Opened 12 years ago
Closed 12 years ago
l10n changesets error messages in release sanity suck
Categories
(Release Engineering :: Release Automation, defect)
Release Engineering
Release Automation
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bhearsum, Assigned: bhearsum)
Details
Attachments
(1 file)
|
1.20 KB,
patch
|
rail
:
review+
bhearsum
:
checked-in+
|
Details | Diff | Splinter Review |
When we check the revisions of l10n repos in release sanity we do a simple http get to URLs such as: https://hg.mozilla.org/releases/l10n/mozilla-beta/zh-CN/file/1e65623aaf2e
When we get ANY failure from them, we throw a message like:
ERROR : cannot find l10n locale zh-CN in repo https://hg.mozilla.org/releases/l10n/mozilla-beta/zh-CN/file/1e65623aaf2e
There's a couple of problems with this:
1) The error message is ambiguous (it should say something like "couldn't find revision X for locale Y at URL Z)
2) Not all errors should be treated equally. 5xx errors should throw a different (after retrying, if possible).
|
Assignee | |
Comment 1•12 years ago
|
||
Bonus: checks locales in alphabetical order.
|
||
Updated•12 years ago
|
Attachment #717297 -
Flags: review?(rail) → review+
|
|
Assignee | |
Updated•12 years ago
|
Attachment #717297 -
Flags: checked-in+
|
|
Assignee | |
Updated•12 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
|
||
Updated•12 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•