Closed
Bug 1434166
Opened 7 years ago
Closed 7 years ago
autoland's pingbacks to reviewboard are stuck in a 404 loop
Categories
(Conduit :: Transplant, defect)
Conduit
Transplant
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: glob, Assigned: glob)
Details
Attachments
(1 file)
> trying to post mozreview update to: https://reviewboard.mozilla.org/api/extensions/mozreview.extension.MozReviewExtension/autoland-request-updates/ for request: 45441
> Starting new HTTPS connection (1): reviewboard.mozilla.org
> "POST /api/extensions/mozreview.extension.MozReviewExtension/autoland-request-updates/ HTTP/1.1" 404 70
> failed: 404 - {"stat": "fail", "err": {"msg": "Object does not exist", "code": 100}}
i will investigate why this pingback is 404'ing
currently autoland stops processing the list of pending pingbacks at the first failure, and it treats 404 as a transient failure. the end result is this 404 is being retried infinitely. autoland should treat a 404 as a fatal error for that pingback, and move on to the next request.
here's the request:
> {"landed": true, "trysyntax": "try: -b do -p all -u all -t none", "result": "af6a04127bfb", "request_id": 45441, "rev": "5105ffc915ea5dfc7ec55479e9d99e34b9217ff7", "destination": "try", "tree": "gecko", "error_msg": ""}
it's missing from rb's mozreview_autolandrequest table, triggering the 404. i don't see anything in the logs that indicate why this row is missing.
i've removed the row from autoland's MozreviewUpdate table, and the large pile of pingbacks stuck behind it were all successfully sent. patch to rework 404 handling soon.
Comment hidden (mozreview-request) |
Comment 4•7 years ago
|
||
mozreview-review |
Comment on attachment 8946544 [details]
autoland: treat a 404 from a pingback as a fatal error (bug 1434166)
https://reviewboard.mozilla.org/r/216562/#review222248
Attachment #8946544 -
Flags: review?(gps) → review+
Pushed by bjones@mozilla.com:
https://hg.mozilla.org/hgcustom/version-control-tools/rev/6f75d61627ad
autoland: treat a 404 from a pingback as a fatal error r=gps
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Comment 7•7 years ago
|
||
This same thing came up 4 months ago in Bug 1405126. No idea how these are failing to be saved in MozReview's DB...
Updated•6 years ago
|
Product: MozReview → Conduit
You need to log in
before you can comment on or make changes to this bug.
Description
•