From 3ef37e88fe83fe7619b2da29e92ce24130778c22 Mon Sep 17 00:00:00 2001
From: Rene Moser <mail@renemoser.net>
Date: Sun, 28 May 2017 13:32:34 +0200
Subject: [PATCH] cloudstack: fix pep8 cs_facts

---
 .../modules/cloud/cloudstack/cs_facts.py      | 44 +++++++++----------
 test/sanity/pep8/legacy-files.txt             |  1 -
 2 files changed, 21 insertions(+), 24 deletions(-)

diff --git a/lib/ansible/modules/cloud/cloudstack/cs_facts.py b/lib/ansible/modules/cloud/cloudstack/cs_facts.py
index cf67f584690..b6056e724ef 100644
--- a/lib/ansible/modules/cloud/cloudstack/cs_facts.py
+++ b/lib/ansible/modules/cloud/cloudstack/cs_facts.py
@@ -104,36 +104,40 @@ cloudstack_user_data:
 '''
 
 import os
+from ansible.module_utils.basic import AnsibleModule
+from ansible.module_utils.urls import fetch_url
+from ansible.module_utils.facts import ansible_facts, module
 
 try:
     import yaml
-    has_lib_yaml = True
+    HAS_LIB_YAML = True
 except ImportError:
-    has_lib_yaml = False
+    HAS_LIB_YAML = False
 
 CS_METADATA_BASE_URL = "http://%s/latest/meta-data"
 CS_USERDATA_BASE_URL = "http://%s/latest/user-data"
 
+
 class CloudStackFacts(object):
 
     def __init__(self):
         self.facts = ansible_facts(module)
         self.api_ip = None
         self.fact_paths = {
-            'cloudstack_service_offering':  'service-offering',
+            'cloudstack_service_offering': 'service-offering',
             'cloudstack_availability_zone': 'availability-zone',
-            'cloudstack_public_hostname':   'public-hostname',
-            'cloudstack_public_ipv4':       'public-ipv4',
-            'cloudstack_local_hostname':    'local-hostname',
-            'cloudstack_local_ipv4':        'local-ipv4',
-            'cloudstack_instance_id':       'instance-id'
+            'cloudstack_public_hostname': 'public-hostname',
+            'cloudstack_public_ipv4': 'public-ipv4',
+            'cloudstack_local_hostname': 'local-hostname',
+            'cloudstack_local_ipv4': 'local-ipv4',
+            'cloudstack_instance_id': 'instance-id'
         }
 
     def run(self):
         result = {}
         filter = module.params.get('filter')
         if not filter:
-            for key,path in self.fact_paths.items():
+            for key, path in self.fact_paths.items():
                 result[key] = self._fetch(CS_METADATA_BASE_URL + "/" + path)
             result['cloudstack_user_data'] = self._get_user_data_json()
         else:
@@ -143,7 +147,6 @@ class CloudStackFacts(object):
                 result[filter] = self._fetch(CS_METADATA_BASE_URL + "/" + self.fact_paths[filter])
         return result
 
-
     def _get_user_data_json(self):
         try:
             # this data come form users, we try what we can to parse it...
@@ -151,7 +154,6 @@ class CloudStackFacts(object):
         except:
             return None
 
-
     def _fetch(self, path):
         api_ip = self._get_api_ip()
         if not api_ip:
@@ -164,22 +166,20 @@ class CloudStackFacts(object):
             data = None
         return data
 
-
     def _get_dhcp_lease_file(self):
         """Return the path of the lease file."""
         default_iface = self.facts['default_ipv4']['interface']
         dhcp_lease_file_locations = [
-            '/var/lib/dhcp/dhclient.%s.leases' % default_iface, # debian / ubuntu
-            '/var/lib/dhclient/dhclient-%s.leases' % default_iface, # centos 6
-            '/var/lib/dhclient/dhclient--%s.lease' % default_iface, # centos 7
-            '/var/db/dhclient.leases.%s' % default_iface, # openbsd
+            '/var/lib/dhcp/dhclient.%s.leases' % default_iface,  # debian / ubuntu
+            '/var/lib/dhclient/dhclient-%s.leases' % default_iface,  # centos 6
+            '/var/lib/dhclient/dhclient--%s.lease' % default_iface,  # centos 7
+            '/var/db/dhclient.leases.%s' % default_iface,  # openbsd
         ]
         for file_path in dhcp_lease_file_locations:
             if os.path.exists(file_path):
                 return file_path
         module.fail_json(msg="Could not find dhclient leases file.")
 
-
     def _get_api_ip(self):
         """Return the IP of the DHCP server."""
         if not self.api_ip:
@@ -198,8 +198,8 @@ class CloudStackFacts(object):
 def main():
     global module
     module = AnsibleModule(
-        argument_spec = dict(
-            filter = dict(default=None, choices=[
+        argument_spec=dict(
+            filter=dict(default=None, choices=[
                 'cloudstack_service_offering',
                 'cloudstack_availability_zone',
                 'cloudstack_public_hostname',
@@ -213,15 +213,13 @@ def main():
         supports_check_mode=False
     )
 
-    if not has_lib_yaml:
+    if not HAS_LIB_YAML:
         module.fail_json(msg="missing python library: yaml")
 
     cs_facts = CloudStackFacts().run()
     cs_facts_result = dict(changed=False, ansible_facts=cs_facts)
     module.exit_json(**cs_facts_result)
 
-from ansible.module_utils.basic import *
-from ansible.module_utils.urls import *
-from ansible.module_utils.facts import *
+
 if __name__ == '__main__':
     main()
diff --git a/test/sanity/pep8/legacy-files.txt b/test/sanity/pep8/legacy-files.txt
index 3024a562462..3456d7a3c03 100644
--- a/test/sanity/pep8/legacy-files.txt
+++ b/test/sanity/pep8/legacy-files.txt
@@ -165,7 +165,6 @@ lib/ansible/modules/cloud/azure/azure_rm_virtualnetwork_facts.py
 lib/ansible/modules/cloud/centurylink/clc_loadbalancer.py
 lib/ansible/modules/cloud/cloudscale/cloudscale_server.py
 lib/ansible/modules/cloud/cloudstack/cs_configuration.py
-lib/ansible/modules/cloud/cloudstack/cs_facts.py
 lib/ansible/modules/cloud/cloudstack/cs_instance.py
 lib/ansible/modules/cloud/cloudstack/cs_instance_facts.py
 lib/ansible/modules/cloud/cloudstack/cs_instancegroup.py