cmueller/ansible
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Use response.content instead of response object (#25772)

Browse source 
vca module utility uses response object instead of
response.content which raises exception in while fail_json
call. Use content attribute from response object instead which
is exact description of HTTP Response error.

Fixes #25378

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
This commit is contained in:
Abhijeet Kasurde 2017-08-14 22:10:06 +05:30 committed by jctanner
parent 8b7a58c961
commit 09cabaf702
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
lib/ansible/module_utils/vca.py
View file

@ -141,7 +141,7 @@ class VcaAnsibleModule(AnsibleModule):
login_org = self.params['org'] login_org = self.params['org']
if not self.vca.login(password=password, org=login_org): if not self.vca.login(password=password, org=login_org):
self.fail('Login to VCA failed', response=self.vca.response) self.fail('Login to VCA failed', response=self.vca.response.content)
try: try:
method_name = 'login_%s' % service_type method_name = 'login_%s' % service_type
@ -150,7 +150,7 @@ class VcaAnsibleModule(AnsibleModule):
except AttributeError: except AttributeError:
self.fail('no login method exists for service_type %s' % service_type) self.fail('no login method exists for service_type %s' % service_type)
except VcaError as e: except VcaError as e:
self.fail(e.message, response=self.vca.response, **e.kwargs) self.fail(e.message, response=self.vca.response.content, **e.kwargs)
def login_vca(self): def login_vca(self):
instance_id = self.params['instance_id'] instance_id = self.params['instance_id']