Closed Bug 1217068 Opened 9 years ago Closed 9 years ago

If Autoland hits an error reporting status to MozReview it should wait longer before trying again

Categories

(MozReview Graveyard :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: dminor, Assigned: dminor)

Details

Attachments

(1 file)

At the moment, this causes the error messages to pile up.
autoland: wait between retries if we hit an error posting to mozreview (bug 1217068) r?mdoglio

This adds a retry timeout if we hit an error posting to MozReview. The errors
we've seen so far are due to coding mistakes or misconfiguration so we don't
gain anything by retrying 4 times a second.

We also remove the requirement that the pingback_url be non-empty because it's
useful to leave the pingback_url empty when testing.
Attachment #8679080 - Flags: review?(mdoglio)
Attachment #8679080 - Flags: review?(mdoglio) → review+
Comment on attachment 8679080 [details]
MozReview Request: autoland: wait between retries if we hit an error posting to mozreview (bug 1217068) r=mdoglio

https://reviewboard.mozilla.org/r/23369/#review20895
Comment on attachment 8679080 [details]
MozReview Request: autoland: wait between retries if we hit an error posting to mozreview (bug 1217068) r=mdoglio

autoland: wait between retries if we hit an error posting to mozreview (bug 1217068) r=mdoglio

This adds a retry timeout if we hit an error posting to MozReview. The errors
we've seen so far are due to coding mistakes or misconfiguration so we don't
gain anything by retrying 4 times a second.

We also remove the requirement that the pingback_url be non-empty because it's
useful to leave the pingback_url empty when testing.
Attachment #8679080 - Attachment description: MozReview Request: autoland: wait between retries if we hit an error posting to mozreview (bug 1217068) r?mdoglio → MozReview Request: autoland: wait between retries if we hit an error posting to mozreview (bug 1217068) r=mdoglio
Thanks!

https://hg.mozilla.org/hgcustom/version-control-tools/rev/ea1760f17be1
Assignee: nobody → dminor
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Product: Developer Services → MozReview
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: