From 83dd22c7157e3f41b49ede5bc81a7f165d3e67a9 Mon Sep 17 00:00:00 2001
From: Felipe Monteiro <felipe.monteiro@att.com>
Date: Wed, 20 Dec 2017 16:35:34 +0000
Subject: [PATCH] [client] Fix 503 exception raising attribute error instead

Change-Id: I87e1e7ebea196cb35351695753e9744f5a3865b2
---
 deckhand/client/exceptions.py | 31 ++++++++++++++++---------------
 1 file changed, 16 insertions(+), 15 deletions(-)

diff --git a/deckhand/client/exceptions.py b/deckhand/client/exceptions.py
index db7c7af1..db1be100 100644
--- a/deckhand/client/exceptions.py
+++ b/deckhand/client/exceptions.py
@@ -100,29 +100,30 @@ _code_map = dict((c.http_status, c)
 
 
 def from_response(response, body, url, method=None):
-    """Return an instance of a ``ClientException`` or subclass  based on a
+    """Return an instance of a ``ClientException`` or subclass based on a
     request's response.
-
-    Usage::
-
-        resp, body = requests.request(...)
-        if resp.status_code != 200:
-            raise exception.from_response(resp, rest.text)
     """
     cls = _code_map.get(response.status_code, ClientException)
 
     try:
-        body = yaml.safe_load(body)
+        kwargs = yaml.safe_load(body)
     except yaml.YAMLError as e:
-        body = {}
+        kwargs = None
         LOG.debug('Could not convert error from server into dict: %s',
                   six.text_type(e))
 
-    kwargs = body
-    kwargs.update({
-        'code': response.status_code,
-        'method': method,
-        'url': url,
-    })
+    if isinstance(kwargs, dict):
+        kwargs.update({
+            'code': response.status_code,
+            'method': method,
+            'url': url
+        })
+    else:
+        kwargs = {
+            'code': response.status_code,
+            'method': method,
+            'url': url,
+            'message': response.text
+        }
 
     return cls(**kwargs)