Merge ~cjwatson/launchpad:mlist-sync-retry into launchpad:master
Proposed by
Colin Watson
Status: | Merged |
---|---|
Approved by: | Colin Watson |
Approved revision: | bb97df2493723ab639812e2d20153d4ef9ac7e1e |
Merge reported by: | Otto Co-Pilot |
Merged at revision: | not available |
Proposed branch: | ~cjwatson/launchpad:mlist-sync-retry |
Merge into: | launchpad:master |
Diff against target: |
37 lines (+16/-2) 1 file modified
lib/lp/services/mailman/scripts/mlist_sync.py (+16/-2) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ioana Lasc (community) | Approve | ||
Review via email:
|
Commit message
Retry mlist-sync's call to updateTeamAddresses
Description of the change
The new updateTeamAddresses API used by mlist-sync is a bit liable to time out when the database is cold. Testing the relevant query by hand on staging and dogfood suggests that it warms up quite quickly on subsequent calls; so, rather than doing anything more elaborate for a staging-only API, just retry the call a few times if we get an OOPS.
To post a comment you must log in.
looks good.