Closed
Bug 983125
Opened 10 years ago
Closed 10 years ago
self-serve agents eat exceptions, causing stalls
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: dustin, Assigned: dustin)
Details
Attachments
(3 files)
689 bytes,
patch
|
bhearsum
:
review+
dustin
:
checked-in+
|
Details | Diff | Splinter Review |
1.10 KB,
patch
|
bhearsum
:
review+
nthomas
:
checked-in-
|
Details | Diff | Splinter Review |
1.12 KB,
patch
|
bhearsum
:
review+
dustin
:
checked-in+
|
Details | Diff | Splinter Review |
The errors get logged, but then eaten, and the ReliableConsumer doesn't know that an error occurred.
Comment 1•10 years ago
|
||
affects like retriggers don;t work at the moment
Assignee | ||
Comment 2•10 years ago
|
||
This also removes the system exit in response to a DB timeout - that doesn't seem like particularly good problem-solving!
Assignee: nobody → dustin
Attachment #8390467 -
Flags: review?(bhearsum)
Comment 3•10 years ago
|
||
For posterity, these can be worked around by restarting the self serve agents: https://wiki.mozilla.org/ReleaseEngineering/How_To/Restart_BuildAPI
Updated•10 years ago
|
Attachment #8390467 -
Flags: review?(bhearsum) → review+
Assignee | ||
Comment 4•10 years ago
|
||
Comment on attachment 8390467 [details] [diff] [review] bug983125.patch I bumped the version to 0.2.3 and put it on the puppet masters.
Attachment #8390467 -
Flags: checked-in+
Assignee | ||
Comment 5•10 years ago
|
||
I should have posted this with the other patch - sorry to interrupt you twice :/
Attachment #8390699 -
Flags: review?(bhearsum)
Updated•10 years ago
|
Attachment #8390699 -
Flags: review?(bhearsum) → review+
Assignee | ||
Updated•10 years ago
|
Attachment #8390699 -
Flags: checked-in+
Assignee | ||
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 6•10 years ago
|
||
Comment on attachment 8390699 [details] [diff] [review] bug983125-p1.patch Backed out: https://hg.mozilla.org/build/puppet/rev/43dc6c60f0cd No legit messages were being processed after we raised on the old messages from bug 980086.
Attachment #8390699 -
Flags: checked-in+ → checked-in-
Updated•10 years ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 7•10 years ago
|
||
I'll ship 0.2.4 for bug 980086 and then re-deploy here.
Assignee | ||
Comment 8•10 years ago
|
||
third patch is the charm..
Attachment #8392203 -
Flags: review?(bhearsum)
Updated•10 years ago
|
Attachment #8392203 -
Flags: review?(bhearsum) → review+
Assignee | ||
Comment 9•10 years ago
|
||
Comment on attachment 8392203 [details] [diff] [review] bug983125-p2.patch Checked in on 3/17/2014
Attachment #8392203 -
Flags: checked-in+
Assignee | ||
Updated•10 years ago
|
Status: REOPENED → RESOLVED
Closed: 10 years ago → 10 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
Component: Tools → General
You need to log in
before you can comment on or make changes to this bug.
Description
•