self-serve agents eat exceptions, causing stalls

RESOLVED FIXED

Status

defect
RESOLVED FIXED
5 years ago
2 years ago

People

(Reporter: dustin, Assigned: dustin)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

The errors get logged, but then eaten, and the ReliableConsumer doesn't know that an error occurred.
affects like retriggers don;t work at the moment
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)
For posterity, these can be worked around by restarting the self serve agents: https://wiki.mozilla.org/ReleaseEngineering/How_To/Restart_BuildAPI
Attachment #8390467 - Flags: review?(bhearsum) → review+
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+
I should have posted this with the other patch - sorry to interrupt you twice :/
Attachment #8390699 - Flags: review?(bhearsum)
Attachment #8390699 - Flags: review?(bhearsum) → review+
Attachment #8390699 - Flags: checked-in+
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
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-
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
I'll ship 0.2.4 for bug 980086 and then re-deploy here.
third patch is the charm..
Attachment #8392203 - Flags: review?(bhearsum)
Attachment #8392203 - Flags: review?(bhearsum) → review+
Comment on attachment 8392203 [details] [diff] [review]
bug983125-p2.patch

Checked in on 3/17/2014
Attachment #8392203 - Flags: checked-in+
Status: REOPENED → RESOLVED
Closed: 5 years ago5 years ago
Resolution: --- → FIXED
Component: Tools → General
You need to log in before you can comment on or make changes to this bug.