Retry the following unexpected errors in gam print users.

ERROR: 400: failedPrecondition - Precondition check failed. ERROR: 500: unknownError - Unknown Error.
taers232c · Oct 8, 2024 · c6ae245 · c6ae245
1 parent 097d236
commit c6ae245
Show file tree

Hide file tree

Showing 4 changed files with 32 additions and 12 deletions.
diff --git a/.github/workflows/winget.yml b/.github/workflows/winget.yml
@@ -10,7 +10,7 @@ jobs:
       - uses: vedantmgoyal2009/winget-releaser@v2
         with:
           identifier: taers232c.GAMADV-XTD3
-          release-tag: v7.00.13
+          release-tag: v7.00.14
           max-versions-to-keep: 1 # keep only latest versions
           installers-regex: '\.msi$'
           token: ${{ secrets.WINGET_TOKEN }}
diff --git a/.travis.yml b/.travis.yml
@@ -95,10 +95,10 @@ script:
 
 before_deploy:
 # Comment out for Linux Xenial and Trusty
-#- yes | gem update --system --force
-#- gem install bundler
-#- gem install uri
-#- gem install logger
+- yes | gem update --system --force
+- gem install bundler
+- gem install uri
+- gem install logger
 - export TRAVIS_TAG="preview"
 - unset LD_LIBRARY_PATH
 
@@ -111,10 +111,10 @@ deploy:
   skip_cleanup: true
   draft: true
 # Linux 64-Bit Bionic and Linux ARM64 Focal and Linux ARM64 Bionic and Linux ARM64 Xenial 
-#  edge: true
+  edge: true
 # Linux Xenial and Trusty
-  edge:
-    branch: v2.0.3-beta.4
+#  edge:
+#    branch: v2.0.3-beta.4
 #    branch: v2.0.5-beta.1
   on:
     repo: taers232c/GAMADV-XTD3

diff --git a/src/GamUpdate.txt b/src/GamUpdate.txt
@@ -1,3 +1,11 @@
+7.00.14
+
+Retry the following unexpected errors in `gam print users`.
+```
+ERROR: 400: failedPrecondition - Precondition check failed.
+ERROR: 500: unknownError - Unknown Error.
+```
+
 7.00.13
 
 Align version number with GAM7.

diff --git a/src/gam/__init__.py b/src/gam/__init__.py
@@ -25,7 +25,7 @@
 """
 
 __author__ = 'Ross Scroggs <[email protected]>'
-__version__ = '7.00.13'
+__version__ = '7.00.14'
 __license__ = 'Apache License 2.0 (http://www.apache.org/licenses/LICENSE-2.0)'
 
 #pylint: disable=wrong-import-position
@@ -18536,7 +18536,9 @@ def writeAliases(target, targetEmail, targetType):
         entityList = callGAPIpages(cd.users(), 'list', 'users',
                                    pageMessage=getPageMessage(showFirstLastItems=True), messageAttribute='primaryEmail',
                                    throwReasons=[GAPI.INVALID_ORGUNIT, GAPI.INVALID_INPUT, GAPI.DOMAIN_NOT_FOUND,
-                                                 GAPI.RESOURCE_NOT_FOUND, GAPI.FORBIDDEN, GAPI.BAD_REQUEST],
+                                                 GAPI.RESOURCE_NOT_FOUND, GAPI.FORBIDDEN, GAPI.BAD_REQUEST,
+                                                 GAPI.UNKNOWN_ERROR, GAPI.FAILED_PRECONDITION],
+                                   retryReasons=[GAPI.UNKNOWN_ERROR, GAPI.FAILED_PRECONDITION],
                                    query=query, orderBy='email',
                                    fields=f'nextPageToken,users({",".join(userFields)})',
                                    maxResults=GC.Values[GC.USER_MAX_RESULTS], **kwargs)
@@ -18549,6 +18551,8 @@ def writeAliases(target, targetEmail, targetType):
       except GAPI.domainNotFound as e:
         entityActionFailedWarning([Ent.ALIAS, None, Ent.DOMAIN, kwargs['domain']], str(e))
         continue
+      except (GAPI.unknownError, GAPI.failedPrecondition) as e:
+        entityActionFailedExit([Ent.USER, None], str(e))
       except (GAPI.resourceNotFound, GAPI.forbidden, GAPI.badRequest):
         accessErrorExit(cd)
   count = len(users)
@@ -18631,9 +18635,13 @@ def doPrintAddresses():
   try:
     entityList = callGAPIpages(cd.users(), 'list', 'users',
                                pageMessage=getPageMessage(showFirstLastItems=True), messageAttribute='primaryEmail',
-                               throwReasons=[GAPI.RESOURCE_NOT_FOUND, GAPI.FORBIDDEN, GAPI.BAD_REQUEST],
+                               throwReasons=[GAPI.RESOURCE_NOT_FOUND, GAPI.FORBIDDEN, GAPI.BAD_REQUEST,
+                                             GAPI.UNKNOWN_ERROR, GAPI.FAILED_PRECONDITION],
+                               retryReasons=[GAPI.UNKNOWN_ERROR, GAPI.FAILED_PRECONDITION],
                                orderBy='email', fields=f'nextPageToken,users({",".join(userFields)})',
                                maxResults=GC.Values[GC.USER_MAX_RESULTS], **kwargs)
+  except (GAPI.unknownError, GAPI.failedPrecondition) as e:
+    entityActionFailedExit([Ent.USER, None], str(e))
   except (GAPI.resourceNotFound, GAPI.forbidden, GAPI.badRequest):
     accessErrorExit(cd)
   for user in entityList:
@@ -44598,7 +44606,9 @@ def _callbackPrintUser(request_id, response, exception):
         feed = yieldGAPIpages(cd.users(), 'list', 'users',
                               pageMessage=pageMessage, messageAttribute='primaryEmail',
                               throwReasons=[GAPI.DOMAIN_NOT_FOUND, GAPI.INVALID_ORGUNIT, GAPI.INVALID_INPUT,
-                                            GAPI.BAD_REQUEST, GAPI.RESOURCE_NOT_FOUND, GAPI.FORBIDDEN],
+                                            GAPI.BAD_REQUEST, GAPI.RESOURCE_NOT_FOUND, GAPI.FORBIDDEN,
+                                            GAPI.UNKNOWN_ERROR, GAPI.FAILED_PRECONDITION],
+                              retryReasons=[GAPI.UNKNOWN_ERROR, GAPI.FAILED_PRECONDITION],
                               query=query, fields=fields,
                               showDeleted=showDeleted, orderBy=orderBy, sortOrder=sortOrder, viewType=viewType,
                               projection=schemaParms['projection'], customFieldMask=schemaParms['customFieldMask'],
@@ -44639,6 +44649,8 @@ def _callbackPrintUser(request_id, response, exception):
         else:
           entityActionFailedWarning([Ent.USER, None], str(e))
         continue
+      except (GAPI.unknownError, GAPI.failedPrecondition) as e:
+        entityActionFailedExit([Ent.USER, None], str(e))
       except (GAPI.badRequest, GAPI.resourceNotFound, GAPI.forbidden):
         accessErrorExit(cd)
     if showItemCountOnly: