Revert "gs_fetch_binpkg: Add retries to the Copy."
This reverts commit c0bec2e80cfee0ca0dc5976916227566a7206b36.
Reason for revert: The change had no measurable effect on the download failure rates, reverting to simplify the code.
Original change's description:
> gs_fetch_binpkg: Add retries to the Copy.
>
> Increase retries to 10 in the script itself, up from the default 3.
>
> BUG=b:197574857
> TEST=cq
>
> Change-Id: Ie26e2147074ca05852c4648ccb1110c45708bd9a
> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/3200193
> Tested-by: Alex Klein <saklein@chromium.org>
> Reviewed-by: Allen Webb <allenwebb@google.com>
> Commit-Queue: Alex Klein <saklein@chromium.org>
Bug: b:197574857
Change-Id: I3c66f48bc2cfcaa34b428bb721709dcadafc2742
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/3230061
Auto-Submit: Alex Klein <saklein@chromium.org>
Tested-by: Alex Klein <saklein@chromium.org>
Reviewed-by: Allen Webb <allenwebb@google.com>
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Commit-Queue: Alex Klein <saklein@chromium.org>
diff --git a/scripts/gs_fetch_binpkg.py b/scripts/gs_fetch_binpkg.py
index 8b6fcbb..ed3266f 100644
--- a/scripts/gs_fetch_binpkg.py
+++ b/scripts/gs_fetch_binpkg.py
@@ -37,7 +37,7 @@
temp_path = '%s.tmp' % filename
osutils.SafeUnlink(temp_path)
try:
- ctx.Copy(uri, temp_path, retries=10)
+ ctx.Copy(uri, temp_path)
shutil.move(temp_path, filename)
finally:
osutils.SafeUnlink(temp_path)