Closed
Bug 1127566
Opened 11 years ago
Closed 11 years ago
shipit-notifier should sleep before failing
Categories
(Release Engineering :: Release Automation, defect)
Release Engineering
Release Automation
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: rail, Assigned: rail)
Details
Attachments
(1 file)
|
654 bytes,
patch
|
bhearsum
:
review+
rail
:
checked-in+
|
Details | Diff | Splinter Review |
Today I received an email from pulse.m.o that the queue has 2k+ messages.
I checked shipit-notifier and it was disabled in supervisor because it was failing too fast (there were some connection issues).
This should work around this problem.
Attachment #8556731 -
Flags: review?(bhearsum)
Comment 1•11 years ago
|
||
Comment on attachment 8556731 [details] [diff] [review]
shipit_notifier_sleep-tools.diff
Review of attachment 8556731 [details] [diff] [review]:
-----------------------------------------------------------------
Should you adjust the supervisor config to not be so aggressive, too?
Attachment #8556731 -
Flags: review?(bhearsum) → review+
| Assignee | ||
Comment 2•11 years ago
|
||
(In reply to Ben Hearsum [:bhearsum] from comment #1)
> Comment on attachment 8556731 [details] [diff] [review]
> shipit_notifier_sleep-tools.diff
>
> Review of attachment 8556731 [details] [diff] [review]:
> -----------------------------------------------------------------
>
> Should you adjust the supervisor config to not be so aggressive, too?
supervisor is not that flexible :(
| Assignee | ||
Comment 3•11 years ago
|
||
Comment on attachment 8556731 [details] [diff] [review]
shipit_notifier_sleep-tools.diff
https://hg.mozilla.org/build/tools/rev/083a62d0147e
Attachment #8556731 -
Flags: checked-in+
| Assignee | ||
Comment 4•11 years ago
|
||
Deployed
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•