Skip to content
Snippets Groups Projects
Unverified Commit 563a03d9 authored by Sergio Oliveira Campos's avatar Sergio Oliveira Campos
Browse files

Fixed error msgs on FreeIPABaseModule subclasses

When a fail_json is called a SystemExit exeception is raised.
Since the FreeIPABaseModule has an internal context manager to deal
with exceptions this ContextManager captures the SystemExit. After
dealing destroying the kinit session the SystemExit must be raised again
to allow the fail_json to work properly.
parent da5dc0c4
No related branches found
No related tags found
No related merge requests found
...@@ -610,12 +610,15 @@ class FreeIPABaseModule(AnsibleModule): ...@@ -610,12 +610,15 @@ class FreeIPABaseModule(AnsibleModule):
exit the module with proper arguments. exit the module with proper arguments.
""" """
if exc_val:
self.fail_json(msg=str(exc_val))
# TODO: shouldn't we also disconnect from api backend? # TODO: shouldn't we also disconnect from api backend?
temp_kdestroy(self.ccache_dir, self.ccache_name) temp_kdestroy(self.ccache_dir, self.ccache_name)
if exc_type == SystemExit:
raise
if exc_val:
self.fail_json(msg=str(exc_val))
self.exit_json(changed=self.changed, user=self.exit_args) self.exit_json(changed=self.changed, user=self.exit_args)
def get_command_errors(self, command, result): def get_command_errors(self, command, result):
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment