From b82ebdde19ae5b721778456cadc5594d699ceb3a Mon Sep 17 00:00:00 2001 From: Matt Martz Date: Mon, 5 Oct 2015 13:22:32 -0500 Subject: [PATCH] sys.exit removal for cloud/xenserver_facts.py --- cloud/xenserver_facts.py | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/cloud/xenserver_facts.py b/cloud/xenserver_facts.py index 149030925f9..d679afce853 100644 --- a/cloud/xenserver_facts.py +++ b/cloud/xenserver_facts.py @@ -28,7 +28,6 @@ author: ''' import platform -import sys import XenAPI EXAMPLES = ''' @@ -75,12 +74,9 @@ class XenServerFacts: def get_xenapi_session(): - try: - session = XenAPI.xapi_local() - session.xenapi.login_with_password('', '') - return session - except XenAPI.Failure: - sys.exit(1) + session = XenAPI.xapi_local() + session.xenapi.login_with_password('', '') + return session def get_networks(session): @@ -163,8 +159,10 @@ def main(): module = AnsibleModule({}) obj = XenServerFacts() - session = get_xenapi_session() - + try: + session = get_xenapi_session() + except XenAPI.Failure, e: + module.fail_json(msg='%s' % e) data = { 'xenserver_version': obj.version,