Sisyphus - add failure recovery to bugzilla api calls.

RESOLVED FIXED

Status

RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: bc, Assigned: bc)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

2.05 KB, patch
cmtalbert
: review+
Details | Diff | Splinter Review
(Assignee)

Description

9 years ago
Created attachment 430693 [details] [diff] [review]
patch

Currently, the sisyphus.bugzilla module does not handle http failures well. This patch adds an exception handler which will retry an http request once if the exception is related to the httplib2 library.
Attachment #430693 - Flags: review?(ctalbert)
(Assignee)

Updated

9 years ago
Blocks: 548019
(Assignee)

Comment 1

9 years ago
Created attachment 431113 [details] [diff] [review]
patch v2

missed import re
Attachment #430693 - Attachment is obsolete: true
Attachment #431113 - Flags: review?(ctalbert)
Attachment #430693 - Flags: review?(ctalbert)

Updated

9 years ago
Attachment #431113 - Flags: review?(ctalbert) → review+
(Assignee)

Comment 2

9 years ago
http://hg.mozilla.org/qa/sisyphus/rev/b95aafcf0160
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.