From 34ce174d55cb1cab2e9afcca69f89653f448a5c6 Mon Sep 17 00:00:00 2001 From: Brant Evans <bevans@redhat.com> Date: Wed, 8 May 2019 16:08:24 -0600 Subject: [PATCH] Fix errors when ipaservers variable is not set --- roles/ipaclient/library/ipaclient_test.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/roles/ipaclient/library/ipaclient_test.py b/roles/ipaclient/library/ipaclient_test.py index dda1c4cb..c75f5f5c 100644 --- a/roles/ipaclient/library/ipaclient_test.py +++ b/roles/ipaclient/library/ipaclient_test.py @@ -306,8 +306,9 @@ def main(): # Get domain from first server if domain is not set, but if there are # servers - if options.domain_name is None and len(options.servers) > 0: - options.domain_name = options.servers[0][options.servers[0].find(".")+1:] + if options.domain_name is None and options.servers is not None: + if len(options.servers) > 0: + options.domain_name = options.servers[0][options.servers[0].find(".")+1:] try: self = options @@ -324,7 +325,8 @@ def main(): ### ServiceInstallInterface ### - validate_domain_name(options.domain_name) + if options.domain_name: + validate_domain_name(options.domain_name) if options.realm_name: argspec = inspect.getargspec(validate_domain_name) -- GitLab