Change in jenkins[master]: Add retries to mirror_client on JSON errors

Code Review gerrit at ovirt.org
Mon May 29 11:34:10 UTC 2017


>From Barak Korren <bkorren at redhat.com>:

Barak Korren has submitted this change and it was merged.

Change subject: Add retries to mirror_client on JSON errors
......................................................................


Add retries to mirror_client on JSON errors

We've seen some reoccurring failures where JSON decoding for information
from the mirrors server fails. This is probably because we're seeing a
race condition with the data update in the server.

This path adds retries so that when this happens we will re-download the
information from the server and hopefully get a full file.

Change-Id: I2f53126d0e2bd9421edaf57f019db9d5e520adc6
Signed-off-by: Barak Korren <bkorren at redhat.com>
Jira-Ticket: https://ovirt-jira.atlassian.net/browse/OVIRT-1086
---
M scripts/mirror_client.py
1 file changed, 15 insertions(+), 5 deletions(-)

Approvals:
  Barak Korren: Verified; Looks good to me, approved; Ready for merge
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/77450
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I2f53126d0e2bd9421edaf57f019db9d5e520adc6
Gerrit-PatchSet: 4
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Barak Korren <bkorren at redhat.com>
Gerrit-Reviewer: Anton Marchukov <amarchuk at redhat.com>
Gerrit-Reviewer: Barak Korren <bkorren at redhat.com>
Gerrit-Reviewer: Daniel Belenky <dbelenky at redhat.com>
Gerrit-Reviewer: Evgheni Dereveanchin <ederevea at redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri at redhat.com>
Gerrit-Reviewer: Gil Shinar <gshinar at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nadav Goldin <ngoldin at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Shlomo Ben David <sbendavi at redhat.com>


More information about the Engine-commits mailing list