Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1023)

Issue 1835943004: ensure_goma: use git retry for clone and fetch (Closed)

Created:
4 years, 8 months ago by ukai
Modified:
4 years, 8 months ago
CC:
chromium-reviews, infra-reviews+build_chromium.org, kjellander-cc_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/tools/build.git@master
Target Ref:
refs/heads/master
Project:
build
Visibility:
Public.

Description

ensure_goma: use git retry for clone and fetch sometimes, git clone failed like Cloning into '/b/build/slave/linux_chromium_gn_chromeos/build/src/build/goma/client'... error: RPC failed; curl 56 GnuTLS recv error (-9): A TLS packet with unexpected length was received. fatal: The remote end hung up unexpectedly fatal: early EOF fatal: index-pack failed Traceback (most recent call last): File "/b/build/scripts/slave/recipe_modules/goma/resources/ensure_goma.py", line 70, in <module> sys.exit(main(sys.argv[1:])) File "/b/build/scripts/slave/recipe_modules/goma/resources/ensure_goma.py", line 47, in main subprocess.check_call(['git', 'clone', config['repo'], client_dir]) File "/usr/lib/python2.7/subprocess.py", line 511, in check_call raise CalledProcessError(retcode, cmd) subprocess.CalledProcessError: Command '['git', 'clone', 'https://chrome-internal.googlesource.com/chrome/tools/goma/linux.git', '/b/build/slave/linux_chromium_gn_chromeos/build/src/build/goma/client']' returned non-zero exit status 128 step returned non-zero exit code: 1 use git-retry in depot_tools to retry for such failures. R=phajdan.jr@chromium.org, dnj@chromium.org BUG=597878 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=299523

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M scripts/slave/recipe_modules/goma/resources/ensure_goma.py View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 9 (5 generated)
ukai
4 years, 8 months ago (2016-03-29 01:23:39 UTC) #4
dnj
lgtm
4 years, 8 months ago (2016-03-29 01:26:00 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1835943004/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1835943004/1
4 years, 8 months ago (2016-03-29 01:45:15 UTC) #7
commit-bot: I haz the power
4 years, 8 months ago (2016-03-29 01:49:13 UTC) #9
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=299523

Powered by Google App Engine
This is Rietveld 408576698