diff options
| author | Ralph Nathan <ralphnathan@chromium.org> | 2015-06-08 17:06:13 -0700 |
|---|---|---|
| committer | ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> | 2015-06-09 22:48:00 +0000 |
| commit | 0d7dbf3119650045ac62c2060e5b9068189accb2 (patch) | |
| tree | cff480fbaa4934561773379ec6934de6637cfbc9 /scripts | |
| parent | 943bfc3014f81df20355fd144afa916ad546f5f2 (diff) | |
| download | chromite-0d7dbf3119650045ac62c2060e5b9068189accb2.tar.gz | |
Fixup: Display curl output when downloading chroot tarball.
Currently, curl is used to download the chroot tarball. However,
we do not get any progress information from it. This CL displays
the output of curl so users know how long it will take to
download the tarball.
BUG=none
TEST=manual test
Change-Id: I0b1e5180054521e30e62aad31ab509a24d7329e0
Reviewed-on: https://chromium-review.googlesource.com/276244
Reviewed-by: Bertrand Simonnet <bsimonnet@chromium.org>
Tested-by: Ralph Nathan <ralphnathan@chromium.org>
Commit-Queue: Ralph Nathan <ralphnathan@chromium.org>
Trybot-Ready: Ralph Nathan <ralphnathan@chromium.org>
Diffstat (limited to 'scripts')
| -rw-r--r-- | scripts/cros_sdk.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/scripts/cros_sdk.py b/scripts/cros_sdk.py index d6613b0b2..d4442fbab 100644 --- a/scripts/cros_sdk.py +++ b/scripts/cros_sdk.py @@ -132,8 +132,8 @@ def FetchRemoteTarballs(storage_dir, urls, desc, allow_none=False): content_length = 0 logging.debug('Attempting download from %s', url) result = retry_util.RunCurl( - ['-I', url], fail=False, redirect_stdout=True, redirect_stderr=True, - print_cmd=False) + ['-I', url], fail=False, capture_output=False, redirect_stdout=True, + redirect_stderr=True, print_cmd=False, debug_level=logging.NOTICE) successful = False for header in result.output.splitlines(): # We must walk the output to find the string '200 OK' for use cases where @@ -162,7 +162,7 @@ def FetchRemoteTarballs(storage_dir, urls, desc, allow_none=False): if current_size < content_length: retry_util.RunCurl( ['-L', '-y', '30', '-C', '-', '--output', tarball_dest, url], - print_cmd=False) + print_cmd=False, capture_output=False, debug_level=logging.NOTICE) # Cleanup old tarballs now since we've successfull fetched; only cleanup # the tarballs for our prefix, or unknown ones. This gets a bit tricky |