Closed
Bug 613970
Opened 15 years ago
Closed 14 years ago
l10n repack scripts should retry anything that touches the network
Categories
(Release Engineering :: General, defect, P3)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bhearsum, Assigned: bhearsum)
References
Details
(Whiteboard: [automation][l10n])
Attachments
(1 file, 1 obsolete file)
|
5.52 KB,
patch
|
rail
:
review+
bhearsum
:
checked-in+
|
Details | Diff | Splinter Review |
Just like we retry tagging when it fails to push we should retry any operations that touch the network when they fail -- hg out/push/pull/clone. Similar to bug 613953 but more general.
|
Assignee | |
Updated•15 years ago
|
Priority: -- → P5
|
|
Assignee | |
Comment 1•15 years ago
|
||
Planning to fix this this quarter.
Assignee: nobody → bhearsum
Status: NEW → ASSIGNED
Priority: P5 → P3
|
|
Assignee | |
Comment 3•14 years ago
|
||
This patch sprinkles around retry across l10n scripts. I originally tried to avoid putting them in the lib/ parts, but that had the unfortunate side effect of eg, redoing the entire repack if the upload fails.
While I was poking around I also changed the error output in such a way that it should fix bug 628392.
Attachment #525082 -
Flags: review?(catlee)
Attachment #525082 -
Flags: review?(armenzg)
|
||
Comment 4•14 years ago
|
||
Comment on attachment 525082 [details] [diff] [review]
retry l10n stuff, fix error output
It looks straight forward.
Thanks for fixing the error reporting at the end.
Attachment #525082 -
Flags: review?(armenzg) → review+
|
|
Assignee | |
Comment 5•14 years ago
|
||
Comment on attachment 525082 [details] [diff] [review]
retry l10n stuff, fix error output
Moving this to Rail to expedite it.
Attachment #525082 -
Flags: review?(catlee) → review?(rail)
|
|
Assignee | |
Comment 6•14 years ago
|
||
Changed the format of the per-locale printouts to be prettier, and added an explicit list of locales that failed at the end.
Attachment #525082 -
Attachment is obsolete: true
Attachment #525427 -
Flags: review?(rail)
Attachment #525082 -
Flags: review?(rail)
|
||
Updated•14 years ago
|
Attachment #525427 -
Flags: review?(rail) → review+
|
|
Assignee | |
Comment 7•14 years ago
|
||
Comment on attachment 525427 [details] [diff] [review]
output tweaked slightly
Landed this
Attachment #525427 -
Flags: checked-in+
|
|
Assignee | |
Updated•14 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 14 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
•