Joe McDonnell has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/21421 )

Change subject: IMPALA-13072: Add retries for s3 uploads to combat flakiness
......................................................................

IMPALA-13072: Add retries for s3 uploads to combat flakiness

On ARM toolchain builds, we have seen some uploads to s3 fail
with a segementation fault. Given the number of artifacts that
the toolchain uploads, even a relatively low error rate can
make it hard to get a passing build. This modifies the s3
upload code to retry up to 10 times to avoid this flakiness.

Testing:
 - Ran an ARM toolchain build and saw the retry happen
   successfully
 - Ran a toolchain build with an invalid s3 bucket and verified
   it failed after 10 retries

Change-Id: I95d858c99e965730303c2bfd90478ac5f68acf83
Reviewed-on: http://gerrit.cloudera.org:8080/21421
Reviewed-by: Michael Smith <michael.sm...@cloudera.com>
Reviewed-by: Laszlo Gaal <laszlo.g...@cloudera.com>
Tested-by: Joe McDonnell <joemcdonn...@cloudera.com>
---
M functions.sh
1 file changed, 20 insertions(+), 6 deletions(-)

Approvals:
  Michael Smith: Looks good to me, but someone else must approve
  Laszlo Gaal: Looks good to me, approved
  Joe McDonnell: Verified

--
To view, visit http://gerrit.cloudera.org:8080/21421
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: native-toolchain
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I95d858c99e965730303c2bfd90478ac5f68acf83
Gerrit-Change-Number: 21421
Gerrit-PatchSet: 2
Gerrit-Owner: Joe McDonnell <joemcdonn...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>
Gerrit-Reviewer: Laszlo Gaal <laszlo.g...@cloudera.com>
Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com>

Reply via email to