Add HTTP status test when discovering CM endpoint (#253)

Signed-off-by: Webster Mudge <[email protected]>

plugins/module_utils/cm_utils.py

@@ -29,7 +29,7 @@
 from time import sleep
 from ansible.module_utils.basic import AnsibleModule
 from ansible.module_utils.common.dict_transformations import recursive_diff
-from ansible.module_utils.common.text.converters import to_text
+from ansible.module_utils.common.text.converters import to_native, to_text
 from time import sleep
 from cm_client import (
     ApiClient,
@@ -434,6 +434,12 @@ def discover_endpoint(self, config):
         if not self.version:
             pre_versioned = urljoin(rendered_url, "/api/version")
             versioned = rest.pool_manager.request("GET", pre_versioned, headers=headers)
+            if versioned.status != 200:
+                self.module.fail_json(
+                    "Unable to retrieve API version",
+                    status=to_native(versioned.status),
+                    reason=to_native(versioned.reason),
+                )
             self.version = versioned.data.decode("utf-8")
 
         # Construct the discovered API endpoint

plugins/modules/cm_endpoint_info.py

@@ -23,7 +23,6 @@
 }
 
 DOCUMENTATION = r"""
----
 module: cm_endpoint_info
 short_description: Discover the Cloudera Manager API endpoint
 description:
@@ -38,7 +37,6 @@
 """
 
 EXAMPLES = r"""
----
 # This will first try 'http://example.cloudera.com:7180' and will
 # follow any redirects
 - name: Gather details using auto-discovery
@@ -50,9 +48,8 @@
 """
 
 RETURN = r"""
----
 endpoint:
-    description: The discovered Cloudera Manager API endpoint
+    description: The discovered Cloudera Manager API endpoint, i.e. the base URL.
     type: str
     returned: always
 """