diff options
author | Kiyoung Kim <kiyoungkim@google.com> | 2023-04-18 02:04:54 +0000 |
---|---|---|
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2023-04-18 02:04:54 +0000 |
commit | c5d65b9f4249b797645a0a1780dc52e30bca6f47 (patch) | |
tree | 3db9be87e973a94975d18d459a2428a086cad898 | |
parent | 2938129eb43e151edc3df6a36ddd1faf500baa84 (diff) | |
parent | b8c38af9bcfb8cc4849019bfa75369b6957e04c3 (diff) | |
download | treble-c5d65b9f4249b797645a0a1780dc52e30bca6f47.tar.gz |
Let fetcher fails less from 503 am: 711f5e943f am: b8c38af9bc
Original change: https://android-review.googlesource.com/c/platform/tools/treble/+/2535284
Change-Id: Id6b37ce5e11b9959bb1f6b1caa084a607b0eded0
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
-rw-r--r-- | fetcher/Android.bp | 1 | ||||
-rw-r--r-- | fetcher/fetcher_lib.py | 12 |
2 files changed, 5 insertions, 8 deletions
diff --git a/fetcher/Android.bp b/fetcher/Android.bp index fd3ff06..654a750 100644 --- a/fetcher/Android.bp +++ b/fetcher/Android.bp @@ -11,7 +11,6 @@ python_library_host { libs: [ "py-google-api-python-client", "py-oauth2client", - "py-six", ], } diff --git a/fetcher/fetcher_lib.py b/fetcher/fetcher_lib.py index 9701494..3e6288f 100644 --- a/fetcher/fetcher_lib.py +++ b/fetcher/fetcher_lib.py @@ -15,7 +15,7 @@ if sys.version_info.major == 3: # pylint: disable=import-error,g-bad-import-order,g-import-not-at-top import apiclient from googleapiclient.discovery import build -from six.moves import http_client +from googleapiclient.errors import HttpError import httplib2 from oauth2client.service_account import ServiceAccountCredentials @@ -80,7 +80,7 @@ def _simple_execute(http_request, for _ in range(max_tries): try: return http_request.execute() - except http_client.errors.HttpError as e: + except HttpError as e: last_error = e if e.resp.status in masked_errors: return None @@ -103,7 +103,7 @@ def create_client(http): Returns: An authorized android build api client. """ - return build(serviceName='androidbuildinternal', version='v2beta1', http=http, + return build(serviceName='androidbuildinternal', version='v3', http=http, static_discovery=False) @@ -201,14 +201,12 @@ def list_artifacts(client, regex, **kwargs): """ matching_artifacts = [] kwargs.setdefault('attemptId', 'latest') - regex = re.compile(regex) - req = client.buildartifact().list(**kwargs) + req = client.buildartifact().list(nameRegexp=regex, **kwargs) while req: result = _simple_execute(req) if result and 'artifacts' in result: for a in result['artifacts']: - if regex.match(a['name']): - matching_artifacts.append(a['name']) + matching_artifacts.append(a['name']) req = client.buildartifact().list_next(req, result) return matching_artifacts |