Skip to content

Commit

Permalink
(from AES) Merge pull request #1611 from datawire/lukeshu/503
Browse files Browse the repository at this point in the history
[rel/v1.6] diagd: Change the "Can't do overview before XXX" errors from HTTP 400 to HTTP 503
  • Loading branch information
LukeShu authored Jul 11, 2020
2 parents d8b06cc + 2800f34 commit 4939ce6
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions python/ambassador_diag/diagd.py
Original file line number Diff line number Diff line change
Expand Up @@ -602,7 +602,7 @@ def show_overview(reqid=None):

if not diag:
app.logger.debug("OV %s - can't do overview before configuration" % reqid)
return "Can't do overview before configuration", 400
return "Can't do overview before configuration", 503

app.logger.debug("OV %s - showing overview" % reqid)

Expand Down Expand Up @@ -728,7 +728,7 @@ def show_intermediate(source=None, reqid=None):

if not diag:
app.logger.debug("SRC %s - can't do intermediate before configuration" % reqid)
return "Can't do overview before configuration", 400
return "Can't do overview before configuration", 503

app.logger.debug("SRC %s - getting intermediate for '%s'" % (reqid, source))

Expand Down

0 comments on commit 4939ce6

Please sign in to comment.