fix sanity test

Author: resmo

plugins/inventory/instance.py

@@ -4,16 +4,11 @@
 # GNU General Public License v3.0+ (see COPYING or https://www.gnu.org/licenses/gpl-3.0.txt)
 
 from __future__ import absolute_import, division, print_function
-__metaclass__ = type
 
-import sys
-import os
-import traceback
-import yaml
+__metaclass__ = type
 
 DOCUMENTATION = r'''
     name: instance
-    plugin_type: inventory
     short_description: Apache CloudStack instance inventory source
     author: Rafael del Valle (@rvalle)
     version_added: 2.1.0
@@ -148,11 +143,13 @@
   created: {{ instance.created }}
 '''
 
-from ansible.plugins.inventory import BaseInventoryPlugin, Constructable, AnsibleError
+import yaml
 from ansible.module_utils.basic import missing_required_lib
-from ..module_utils.cloudstack import HAS_LIB_CS
+from ansible.plugins.inventory import (AnsibleError, BaseInventoryPlugin,
+                                       Constructable)
 from jinja2 import Template
 
+from ..module_utils.cloudstack import HAS_LIB_CS
 
 try:
     from cs import CloudStack, CloudStackException

plugins/modules/cs_loadbalancer_rule.py

@@ -6,6 +6,7 @@
 # GNU General Public License v3.0+ (see COPYING or https://www.gnu.org/licenses/gpl-3.0.txt)
 
 from __future__ import absolute_import, division, print_function
+
 __metaclass__ = type
 
 
@@ -57,7 +58,7 @@
   open_firewall:
     description:
       - Whether the firewall rule for public port should be created, while creating the new rule.
-      - Use M(cs_firewall) for managing firewall rules.
+      - Use M(ngine_io.cloudstack.cs_firewall) for managing firewall rules.
     type: bool
     default: no
   cidr:
@@ -219,11 +220,9 @@
 '''
 
 from ansible.module_utils.basic import AnsibleModule
-from ..module_utils.cloudstack import (
-    AnsibleCloudStack,
-    cs_argument_spec,
-    cs_required_together,
-)
+
+from ..module_utils.cloudstack import (AnsibleCloudStack, cs_argument_spec,
+                                       cs_required_together)
 
 
 class AnsibleCloudStackLBRule(AnsibleCloudStack):

plugins/modules/cs_portforward.py

@@ -5,6 +5,7 @@
 # GNU General Public License v3.0+ (see COPYING or https://www.gnu.org/licenses/gpl-3.0.txt)
 
 from __future__ import absolute_import, division, print_function
+
 __metaclass__ = type
 
 
@@ -63,7 +64,7 @@
     description:
       - Whether the firewall rule for public port should be created, while creating the new rule.
       - Not supported when forwarding ports in a VPC.
-      - Use M(cs_firewall) for managing firewall rules.
+      - Use M(ngine_io.cloudstack.cs_firewall) for managing firewall rules.
     default: no
     type: bool
   vm_guest_ip:
@@ -227,7 +228,9 @@
 '''
 
 from ansible.module_utils.basic import AnsibleModule
-from ..module_utils.cloudstack import AnsibleCloudStack, cs_argument_spec, cs_required_together
+
+from ..module_utils.cloudstack import (AnsibleCloudStack, cs_argument_spec,
+                                       cs_required_together)
 
 
 class AnsibleCloudStackPortforwarding(AnsibleCloudStack):

plugins/modules/cs_role_permission.py

@@ -5,6 +5,7 @@
 # GNU General Public License v3.0+ (see COPYING or https://www.gnu.org/licenses/gpl-3.0.txt)
 
 from __future__ import absolute_import, division, print_function
+
 __metaclass__ = type
 
 
@@ -108,14 +109,11 @@
   sample: Deny createVPC for users
 '''
 
-from distutils.version import LooseVersion
-
 from ansible.module_utils.basic import AnsibleModule
-from ..module_utils.cloudstack import (
-    AnsibleCloudStack,
-    cs_argument_spec,
-    cs_required_together,
-)
+from ansible.module_utils.compat.version import LooseVersion
+
+from ..module_utils.cloudstack import (AnsibleCloudStack, cs_argument_spec,
+                                       cs_required_together)
 
 
 class AnsibleCloudStackRolePermission(AnsibleCloudStack):

plugins/modules/cs_router.py

@@ -5,6 +5,7 @@
 # GNU General Public License v3.0+ (see COPYING or https://www.gnu.org/licenses/gpl-3.0.txt)
 
 from __future__ import absolute_import, division, print_function
+
 __metaclass__ = type
 
 
@@ -14,7 +15,7 @@
 short_description: Manages routers on Apache CloudStack based clouds.
 description:
     - Start, restart, stop and destroy routers.
-    - I(state=present) is not able to create routers, use M(cs_network) instead.
+    - I(state=present) is not able to create routers, use M(ngine_io.cloudstack.cs_network) instead.
 author: René Moser (@resmo)
 version_added: 0.1.0
 options:
@@ -156,11 +157,9 @@
 '''
 
 from ansible.module_utils.basic import AnsibleModule
-from ..module_utils.cloudstack import (
-    AnsibleCloudStack,
-    cs_argument_spec,
-    cs_required_together,
-)
+
+from ..module_utils.cloudstack import (AnsibleCloudStack, cs_argument_spec,
+                                       cs_required_together)
 
 
 class AnsibleCloudStackRouter(AnsibleCloudStack):

plugins/modules/cs_vpn_connection.py

@@ -5,6 +5,7 @@
 # GNU General Public License v3.0+ (see COPYING or https://www.gnu.org/licenses/gpl-3.0.txt)
 
 from __future__ import absolute_import, division, print_function
+
 __metaclass__ = type
 
 
@@ -36,7 +37,7 @@
   force:
     description:
       - Activate the VPN gateway if not already activated on I(state=present).
-      - Also see M(cs_vpn_gateway).
+      - Also see M(ngine_io.cloudstack.cs_vpn_gateway).
     default: no
     type: bool
   state:
@@ -183,11 +184,9 @@
 '''
 
 from ansible.module_utils.basic import AnsibleModule
-from ..module_utils.cloudstack import (
-    AnsibleCloudStack,
-    cs_argument_spec,
-    cs_required_together
-)
+
+from ..module_utils.cloudstack import (AnsibleCloudStack, cs_argument_spec,
+                                       cs_required_together)
 
 
 class AnsibleCloudStackVpnConnection(AnsibleCloudStack):