Skip to content

Commit

Permalink
catch AdminError for update_dispatch because it could be a false nega…
Browse files Browse the repository at this point in the history
…tive
  • Loading branch information
whoarethebritons committed Aug 21, 2019
1 parent 671d9ee commit a5ef9af
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions appscale/tools/appscale.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
from appscale.tools.parse_args import ParseArgs
from appscale.tools.registration_helper import RegistrationHelper
from appscale.tools.remote_helper import RemoteHelper

from appscale.tools.admin_api.client import AdminError

class AppScale():
""" AppScale provides a configuration-file-based alternative to the
Expand Down Expand Up @@ -580,7 +580,7 @@ def deploy(self, app, project_id=None):
try:
AppScaleTools.update_dispatch(options.file, options.keyname,
options.project, options.verbose)
except AppScaleException as e:
except (AdminError, AppScaleException) as e:
AppScaleLogger.warn('Request to update dispatch failed, if your '
'dispatch references undeployed services, ignore '
'this exception: {}'.format(e))
Expand Down

0 comments on commit a5ef9af

Please sign in to comment.