Tidy up validate-modules ignores for modules: cloud/centurylink (rebased) (#1365)

* fixed validation-modules for plugins/modules/cloud/centurylink/clc_aa_policy.py

fixed validation-modules for plugins/modules/cloud/centurylink/clc_alert_policy.py

fixed validation-modules for plugins/modules/cloud/centurylink/clc_firewall_policy.py

fixed validation-modules for plugins/modules/cloud/centurylink/clc_blueprint_package.py

fixed validation-modules for plugins/modules/cloud/centurylink/clc_group.py

fixed validation-modules for plugins/modules/cloud/centurylink/clc_loadbalancer.py

fixed validation-modules for plugins/modules/cloud/centurylink/clc_modify_server.py

fixed validation-modules for plugins/modules/cloud/centurylink/clc_publicip.py

fixed validation-modules for plugins/modules/cloud/centurylink/clc_server.py

fixed validation-modules for plugins/modules/cloud/centurylink/clc_server_snapshot.py

* Tidy up validate-modules ignores for cloud/centurylink modules

* Update plugins/modules/cloud/centurylink/clc_firewall_policy.py

Co-authored-by: Felix Fontein <felix@fontein.de>

Co-authored-by: Felix Fontein <felix@fontein.de>

plugins/modules/cloud/centurylink/clc_firewall_policy.py

@@ -16,45 +16,54 @@ options:
   location:
     description:
       - Target datacenter for the firewall policy
+    type: str
     required: True
   state:
     description:
       - Whether to create or delete the firewall policy
+    type: str
     default: present
     choices: ['present', 'absent']
   source:
     description:
       - The list  of source addresses for traffic on the originating firewall.
         This is required when state is 'present'
+    type: list
   destination:
     description:
       - The list of destination addresses for traffic on the terminating firewall.
         This is required when state is 'present'
+    type: list
   ports:
     description:
       - The list of ports associated with the policy.
         TCP and UDP can take in single ports or port ranges.
-    choices: ['any', 'icmp', 'TCP/123', 'UDP/123', 'TCP/123-456', 'UDP/123-456']
+      - "Example: C(['any', 'icmp', 'TCP/123', 'UDP/123', 'TCP/123-456', 'UDP/123-456'])."
+    type: list
   firewall_policy_id:
     description:
       - Id of the firewall policy. This is required to update or delete an existing firewall policy
+    type: str
   source_account_alias:
     description:
       - CLC alias for the source account
+    type: str
     required: True
   destination_account_alias:
     description:
       - CLC alias for the destination account
+    type: str
   wait:
     description:
       - Whether to wait for the provisioning tasks to finish before returning.
-    type: bool
-    default: 'yes'
+    type: str
+    default: 'True'
   enabled:
     description:
       - Whether the firewall policy is enabled or disabled
+    type: str
     choices: [True, False]
-    default: 'yes'
+    default: True
 requirements:
     - python = 2.7
     - requests >= 2.5.0
@@ -89,7 +98,6 @@ EXAMPLES = '''
         ports: Any
         destination_account_alias: WFAD
 
----
 - name: Delete Firewall Policy
   hosts: localhost
   gather_facts: False
@@ -206,13 +214,13 @@ class ClcFirewallPolicy:
         """
         argument_spec = dict(
             location=dict(required=True),
-            source_account_alias=dict(required=True, default=None),
+            source_account_alias=dict(required=True, default=None),   # @FIXME remove default=None
             destination_account_alias=dict(default=None),
             firewall_policy_id=dict(default=None),
             ports=dict(default=None, type='list'),
             source=dict(default=None, type='list'),
             destination=dict(default=None, type='list'),
-            wait=dict(default=True),
+            wait=dict(default=True),   # @FIXME type=bool
             state=dict(default='present', choices=['present', 'absent']),
             enabled=dict(default=True, choices=[True, False])
         )