From fdf244d488729fe356f2892308ebc60f07f15424 Mon Sep 17 00:00:00 2001
From: "patchback[bot]" <45432694+patchback[bot]@users.noreply.github.com>
Date: Thu, 17 Sep 2020 20:59:12 +0200
Subject: [PATCH] Reduce ignored sanity tests in cloud/misc modules (#845)
 (#920)

* Reduce ignored sanity tests in cloud/misc modules

* Reduce ignored sanity tests in cloud/misc modules for proxmox_kvm

* Fix

* Remove in ignore-2.9.txt

* Fix

* Remove unneeded alias

(cherry picked from commit d046dc34bfd5a3ba3febfb3eeecc9480e597f897)

Co-authored-by: Amin Vakil <info@aminvakil.com>
---
 .../cloud/misc/cloud_init_data_facts.py       |  1 +
 plugins/modules/cloud/misc/helm.py            | 11 +++-
 plugins/modules/cloud/misc/ovirt.py           | 34 ++++++++++-
 plugins/modules/cloud/misc/proxmox.py         | 24 ++++++++
 plugins/modules/cloud/misc/proxmox_kvm.py     | 56 +++++++++++++++++++
 .../modules/cloud/misc/proxmox_template.py    | 11 +++-
 plugins/modules/cloud/misc/rhevm.py           | 10 +++-
 plugins/modules/cloud/misc/serverless.py      |  3 +-
 plugins/modules/cloud/misc/terraform.py       | 14 ++++-
 tests/sanity/ignore-2.10.txt                  | 19 -------
 tests/sanity/ignore-2.11.txt                  | 19 -------
 tests/sanity/ignore-2.9.txt                   | 15 -----
 12 files changed, 155 insertions(+), 62 deletions(-)

diff --git a/plugins/modules/cloud/misc/cloud_init_data_facts.py b/plugins/modules/cloud/misc/cloud_init_data_facts.py
index ad8eea86c2..2efb90cfeb 100644
--- a/plugins/modules/cloud/misc/cloud_init_data_facts.py
+++ b/plugins/modules/cloud/misc/cloud_init_data_facts.py
@@ -19,6 +19,7 @@ options:
   filter:
     description:
       - Filter facts
+    type: str
     choices: [ status, result ]
 notes:
   - See http://cloudinit.readthedocs.io/ for more information about cloud-init.
diff --git a/plugins/modules/cloud/misc/helm.py b/plugins/modules/cloud/misc/helm.py
index afa6da810b..dd592d6e92 100644
--- a/plugins/modules/cloud/misc/helm.py
+++ b/plugins/modules/cloud/misc/helm.py
@@ -25,30 +25,37 @@ options:
   host:
     description:
       - Tiller's server host.
+    type: str
     default: "localhost"
   port:
     description:
       - Tiller's server port.
+    type: int
     default: 44134
   namespace:
     description:
       - Kubernetes namespace where the chart should be installed.
+    type: str
     default: "default"
   name:
     description:
       - Release name to manage.
+    type: str
   state:
     description:
       - Whether to install C(present), remove C(absent), or purge C(purged) a package.
     choices: ['absent', 'purged', 'present']
+    type: str
     default: "present"
   chart:
-    description: |
-      A map describing the chart to install. See examples for available options.
+    description:
+      - A map describing the chart to install. See examples for available options.
+    type: dict
     default: {}
   values:
     description:
       - A map of value options for the chart.
+    type: dict
     default: {}
   disable_hooks:
     description:
diff --git a/plugins/modules/cloud/misc/ovirt.py b/plugins/modules/cloud/misc/ovirt.py
index 1692935fcf..e037261b16 100644
--- a/plugins/modules/cloud/misc/ovirt.py
+++ b/plugins/modules/cloud/misc/ovirt.py
@@ -23,116 +23,148 @@ options:
   user:
     description:
      - The user to authenticate with.
+    type: str
     required: true
   url:
     description:
      - The url of the oVirt instance.
+    type: str
     required: true
   instance_name:
     description:
      - The name of the instance to use.
+    type: str
     required: true
     aliases: [ vmname ]
   password:
     description:
      - Password of the user to authenticate with.
+    type: str
     required: true
   image:
     description:
      - The template to use for the instance.
+    type: str
   resource_type:
     description:
      - Whether you want to deploy an image or create an instance from scratch.
+    type: str
     choices: [ new, template ]
   zone:
     description:
      - Deploy the image to this oVirt cluster.
+    type: str
   instance_disksize:
     description:
      - Size of the instance's disk in GB.
+    type: str
     aliases: [ vm_disksize]
   instance_cpus:
     description:
      - The instance's number of CPUs.
+    type: str
     default: 1
     aliases: [ vmcpus ]
   instance_nic:
     description:
      - The name of the network interface in oVirt/RHEV.
+    type: str
     aliases: [ vmnic  ]
   instance_network:
     description:
      - The logical network the machine should belong to.
+    type: str
     default: rhevm
     aliases: [ vmnetwork ]
   instance_mem:
     description:
      - The instance's amount of memory in MB.
+    type: str
     aliases: [ vmmem ]
   instance_type:
     description:
      - Define whether the instance is a server, desktop or high_performance.
      - I(high_performance) is supported since Ansible 2.5 and oVirt/RHV 4.2.
+    type: str
     choices: [ desktop, server, high_performance ]
     default: server
     aliases: [ vmtype ]
   disk_alloc:
     description:
      - Define whether disk is thin or preallocated.
+    type: str
     choices: [ preallocated, thin ]
     default: thin
   disk_int:
     description:
      - Interface type of the disk.
+    type: str
     choices: [ ide, virtio ]
     default: virtio
   instance_os:
     description:
      - Type of Operating System.
+    type: str
     aliases: [ vmos ]
   instance_cores:
     description:
      - Define the instance's number of cores.
+    type: str
     default: 1
     aliases: [ vmcores ]
   sdomain:
     description:
      - The Storage Domain where you want to create the instance's disk on.
+    type: str
   region:
     description:
      - The oVirt/RHEV datacenter where you want to deploy to.
+    type: str
   instance_dns:
     description:
      - Define the instance's Primary DNS server.
+    type: str
     aliases: [ dns ]
   instance_domain:
     description:
      - Define the instance's Domain.
+    type: str
     aliases: [ domain ]
   instance_hostname:
     description:
      - Define the instance's Hostname.
+    type: str
     aliases: [ hostname ]
   instance_ip:
     description:
      - Define the instance's IP.
+    type: str
     aliases: [ ip ]
   instance_netmask:
     description:
      - Define the instance's Netmask.
+    type: str
     aliases: [ netmask ]
+  instance_gateway:
+    description:
+     - Define the instance's Gateway.
+    type: str
+    aliases: [ gateway ]
   instance_rootpw:
     description:
      - Define the instance's Root password.
+    type: str
     aliases: [ rootpw ]
   instance_key:
     description:
      - Define the instance's Authorized key.
+    type: str
     aliases: [ key ]
   state:
     description:
      - Create, terminate or remove instances.
-    choices: [ absent, present, restarted, shutdown, started ]
+    type: str
+    choices: [ absent, present, restart, shutdown, started ]
     default: present
 requirements:
   - ovirt-engine-sdk-python
diff --git a/plugins/modules/cloud/misc/proxmox.py b/plugins/modules/cloud/misc/proxmox.py
index 2642db1978..71e43edcdb 100644
--- a/plugins/modules/cloud/misc/proxmox.py
+++ b/plugins/modules/cloud/misc/proxmox.py
@@ -16,20 +16,24 @@ options:
   api_host:
     description:
       - the host of the Proxmox VE cluster
+    type: str
     required: true
   api_user:
     description:
       - the user to authenticate with
+    type: str
     required: true
   api_password:
     description:
       - the password to authenticate with
       - you can use PROXMOX_PASSWORD environment variable
+    type: str
   vmid:
     description:
       - the instance id
       - if not set, the next available VM ID will be fetched from ProxmoxAPI.
       - if not set, will be fetched from PromoxAPI based on the hostname
+    type: str
   validate_certs:
     description:
       - enable / disable https certificate verification
@@ -40,51 +44,64 @@ options:
       - Proxmox VE node, when new VM will be created
       - required only for C(state=present)
       - for another states will be autodiscovered
+    type: str
   pool:
     description:
       - Proxmox VE resource pool
+    type: str
   password:
     description:
       - the instance root password
       - required only for C(state=present)
+    type: str
   hostname:
     description:
       - the instance hostname
       - required only for C(state=present)
       - must be unique if vmid is not passed
+    type: str
   ostemplate:
     description:
       - the template for VM creating
       - required only for C(state=present)
+    type: str
   disk:
     description:
       - hard disk size in GB for instance
+    type: str
     default: 3
   cores:
     description:
       - Specify number of cores per socket.
+    type: int
     default: 1
   cpus:
     description:
       - numbers of allocated cpus for instance
+    type: int
     default: 1
   memory:
     description:
       - memory size in MB for instance
+    type: int
     default: 512
   swap:
     description:
       - swap memory size in MB for instance
+    type: int
     default: 0
   netif:
     description:
       - specifies network interfaces for the container. As a hash/dictionary defining interfaces.
+    type: dict
   mounts:
     description:
       - specifies additional mounts (separate disks) for the container. As a hash/dictionary defining mount points
+    type: dict
   ip_address:
     description:
       - specifies the address the container will be assigned
+    type: str
   onboot:
     description:
       - specifies whether a VM will be started during system bootup
@@ -93,20 +110,25 @@ options:
   storage:
     description:
       - target storage
+    type: str
     default: 'local'
   cpuunits:
     description:
       - CPU weight for a VM
+    type: int
     default: 1000
   nameserver:
     description:
       - sets DNS server IP address for a container
+    type: str
   searchdomain:
     description:
       - sets DNS search domain for a container
+    type: str
   timeout:
     description:
       - timeout for operations
+    type: int
     default: 30
   force:
     description:
@@ -119,11 +141,13 @@ options:
   state:
     description:
      - Indicate desired state of the instance
+    type: str
     choices: ['present', 'started', 'absent', 'stopped', 'restarted']
     default: present
   pubkey:
     description:
       - Public key to add to /root/.ssh/authorized_keys. This was added on Proxmox 4.2, it is ignored for earlier versions
+    type: str
   unprivileged:
     description:
       - Indicate if the container should be unprivileged
diff --git a/plugins/modules/cloud/misc/proxmox_kvm.py b/plugins/modules/cloud/misc/proxmox_kvm.py
index 9f29fda4d3..cbc77c20b7 100644
--- a/plugins/modules/cloud/misc/proxmox_kvm.py
+++ b/plugins/modules/cloud/misc/proxmox_kvm.py
@@ -28,19 +28,23 @@ options:
     description:
       - Pass arbitrary arguments to kvm.
       - This option is for experts only!
+    type: str
     default: "-serial unix:/var/run/qemu-server/VMID.serial,server,nowait"
   api_host:
     description:
       - Specify the target host of the Proxmox VE cluster.
+    type: str
     required: true
   api_user:
     description:
       - Specify the user to authenticate with.
+    type: str
     required: true
   api_password:
     description:
       - Specify the password to authenticate with.
       - You can use C(PROXMOX_PASSWORD) environment variable.
+    type: str
   autostart:
     description:
       - Specify if the VM should be automatically restarted after crash (currently ignored in PVE API).
@@ -50,50 +54,62 @@ options:
     description:
       - Specify the amount of RAM for the VM in MB.
       - Using zero disables the balloon driver.
+    type: int
     default: 0
   bios:
     description:
       - Specify the BIOS implementation.
+    type: str
     choices: ['seabios', 'ovmf']
   boot:
     description:
       - Specify the boot order -> boot on floppy C(a), hard disk C(c), CD-ROM C(d), or network C(n).
       - You can combine to set order.
+    type: str
     default: cnd
   bootdisk:
     description:
       - Enable booting from specified disk. C((ide|sata|scsi|virtio)\d+)
+    type: str
   clone:
     description:
       - Name of VM to be cloned. If C(vmid) is setted, C(clone) can take arbitrary value but required for initiating the clone.
+    type: str
   cores:
     description:
       - Specify number of cores per socket.
+    type: int
     default: 1
   cpu:
     description:
       - Specify emulated CPU type.
+    type: str
     default: kvm64
   cpulimit:
     description:
       - Specify if CPU usage will be limited. Value 0 indicates no CPU limit.
       - If the computer has 2 CPUs, it has total of '2' CPU time
+    type: int
   cpuunits:
     description:
       - Specify CPU weight for a VM.
       - You can disable fair-scheduler configuration by setting this to 0
+    type: int
     default: 1000
   delete:
     description:
       - Specify a list of settings you want to delete.
+    type: str
   description:
     description:
       - Specify the description for the VM. Only used on the configuration web interface.
       - This is saved as comment inside the configuration file.
+    type: str
   digest:
     description:
       - Specify if to prevent changes if current configuration file has different SHA1 digest.
       - This can be used to prevent concurrent modifications.
+    type: str
   force:
     description:
       - Allow to force stop VM.
@@ -104,6 +120,7 @@ options:
     description:
       - Target drive's backing file's data format.
       - Used only with clone
+    type: str
     default: qcow2
     choices: [ "cloop", "cow", "qcow", "qcow2", "qed", "raw", "vmdk" ]
   freeze:
@@ -127,14 +144,17 @@ options:
       - C(rombar=boolean) I(default=1) Specify whether or not the device's ROM will be visible in the guest's memory map.
       - C(x-vga=boolean) I(default=0) Enable vfio-vga device support.
       - /!\ This option allows direct access to host hardware. So it is no longer possible to migrate such machines - use with special care.
+    type: dict
   hotplug:
     description:
       - Selectively enable hotplug features.
       - This is a comma separated list of hotplug features C('network', 'disk', 'cpu', 'memory' and 'usb').
       - Value 0 disables hotplug completely and value 1 is an alias for the default C('network,disk,usb').
+    type: str
   hugepages:
     description:
       - Enable/disable hugepages memory.
+    type: str
     choices: ['any', '2', '1024']
   ide:
     description:
@@ -144,9 +164,11 @@ options:
       - C(storage) is the storage identifier where to create the disk.
       - C(size) is the size of the disk in GB.
       - C(format) is the drive's backing file's data format. C(qcow2|raw|subvol).
+    type: dict
   keyboard:
     description:
       - Sets the keyboard layout for VNC server.
+    type: str
   kvm:
     description:
       - Enable/disable KVM hardware virtualization.
@@ -160,26 +182,32 @@ options:
   lock:
     description:
       - Lock/unlock the VM.
+    type: str
     choices: ['migrate', 'backup', 'snapshot', 'rollback']
   machine:
     description:
       - Specifies the Qemu machine type.
       - type => C((pc|pc(-i440fx)?-\d+\.\d+(\.pxe)?|q35|pc-q35-\d+\.\d+(\.pxe)?))
+    type: str
   memory:
     description:
       - Memory size in MB for instance.
+    type: int
     default: 512
   migrate_downtime:
     description:
       - Sets maximum tolerated downtime (in seconds) for migrations.
+    type: int
   migrate_speed:
     description:
       - Sets maximum speed (in MB/s) for migrations.
       - A value of 0 is no limit.
+    type: int
   name:
     description:
       - Specifies the VM name. Only used on the configuration web interface.
       - Required only for C(state=present).
+    type: str
   net:
     description:
       - A hash/dictionary of network interfaces for the VM. C(net='{"key":"value", "key":"value"}').
@@ -190,15 +218,18 @@ options:
       - The C(bridge) parameter can be used to automatically add the interface to a bridge device. The Proxmox VE standard bridge is called 'vmbr0'.
       - Option C(rate) is used to limit traffic bandwidth from and to this interface. It is specified as floating point number, unit is 'Megabytes per second'.
       - If you specify no bridge, we create a kvm 'user' (NATed) network device, which provides DHCP and DNS services.
+    type: dict
   newid:
     description:
       - VMID for the clone. Used only with clone.
       - If newid is not set, the next available VM ID will be fetched from ProxmoxAPI.
+    type: int
   node:
     description:
       - Proxmox VE node, where the new VM will be created.
       - Only required for C(state=present).
       - For other states, it will be autodiscovered.
+    type: str
   numa:
     description:
       - A hash/dictionaries of NUMA topology. C(numa='{"key":"value", "key":"value"}').
@@ -208,6 +239,7 @@ options:
       - C(hostnodes) Host NUMA nodes to use.
       - C(memory) Amount of memory this NUMA node provides.
       - C(policy) NUMA allocation policy.
+    type: dict
   onboot:
     description:
       - Specifies whether a VM will be started during system bootup.
@@ -217,6 +249,7 @@ options:
     description:
       - Specifies guest operating system. This is used to enable special optimization/features for specific operating systems.
       - The l26 is Linux 2.6/3.X Kernel.
+    type: str
     choices: ['other', 'wxp', 'w2k', 'w2k3', 'w2k8', 'wvista', 'win7', 'win8', 'win10', 'l24', 'l26', 'solaris']
     default: l26
   parallel:
@@ -224,9 +257,11 @@ options:
       - A hash/dictionary of map host parallel devices. C(parallel='{"key":"value", "key":"value"}').
       - Keys allowed are - (parallel[n]) where 0 ≤ n ≤ 2.
       - Values allowed are - C("/dev/parport\d+|/dev/usb/lp\d+").
+    type: dict
   pool:
     description:
       - Add the new VM to the specified pool.
+    type: str
   protection:
     description:
       - Enable/disable the protection flag of the VM. This will enable/disable the remove VM and remove disk operations.
@@ -238,6 +273,7 @@ options:
   revert:
     description:
       - Revert a pending change.
+    type: str
   sata:
     description:
       - A hash/dictionary of volume used as sata hard disk or CD-ROM. C(sata='{"key":"value", "key":"value"}').
@@ -246,6 +282,7 @@ options:
       - C(storage) is the storage identifier where to create the disk.
       - C(size) is the size of the disk in GB.
       - C(format) is the drive's backing file's data format. C(qcow2|raw|subvol).
+    type: dict
   scsi:
     description:
       - A hash/dictionary of volume used as SCSI hard disk or CD-ROM. C(scsi='{"key":"value", "key":"value"}').
@@ -254,9 +291,11 @@ options:
       - C(storage) is the storage identifier where to create the disk.
       - C(size) is the size of the disk in GB.
       - C(format) is the drive's backing file's data format. C(qcow2|raw|subvol).
+    type: dict
   scsihw:
     description:
       - Specifies the SCSI controller model.
+    type: str
     choices: ['lsi', 'lsi53c810', 'virtio-scsi-pci', 'virtio-scsi-single', 'megasas', 'pvscsi']
   serial:
     description:
@@ -264,44 +303,54 @@ options:
       - Keys allowed are - serial[n](str; required) where 0 ≤ n ≤ 3.
       - Values allowed are - C((/dev/.+|socket)).
       - /!\ If you pass through a host serial device, it is no longer possible to migrate such machines - use with special care.
+    type: dict
   shares:
     description:
       - Rets amount of memory shares for auto-ballooning. (0 - 50000).
       - The larger the number is, the more memory this VM gets.
       - The number is relative to weights of all other running VMs.
       - Using 0 disables auto-ballooning, this means no limit.
+    type: int
   skiplock:
     description:
       - Ignore locks
       - Only root is allowed to use this option.
+    type: bool
   smbios:
     description:
       - Specifies SMBIOS type 1 fields.
+    type: str
   snapname:
     description:
       - The name of the snapshot. Used only with clone.
+    type: str
   sockets:
     description:
       - Sets the number of CPU sockets. (1 - N).
+    type: int
     default: 1
   startdate:
     description:
       - Sets the initial date of the real time clock.
       - Valid format for date are C('now') or C('2016-09-25T16:01:21') or C('2016-09-25').
+    type: str
   startup:
     description:
       - Startup and shutdown behavior. C([[order=]\d+] [,up=\d+] [,down=\d+]).
       - Order is a non-negative number defining the general startup order.
       - Shutdown in done with reverse ordering.
+    type: str
   state:
     description:
       - Indicates desired state of the instance.
       - If C(current), the current state of the VM will be fetched. You can access it with C(results.status)
+    type: str
     choices: ['present', 'started', 'absent', 'stopped', 'restarted','current']
     default: present
   storage:
     description:
       - Target storage for full clone.
+    type: str
   tablet:
     description:
       - Enables/disables the USB tablet device.
@@ -311,6 +360,7 @@ options:
     description:
       - Target node. Only allowed if the original VM is on shared storage.
       - Used only with clone
+    type: str
   tdf:
     description:
       - Enables/disables time drift fix.
@@ -323,6 +373,7 @@ options:
   timeout:
     description:
       - Timeout for operations.
+    type: int
     default: 30
   update:
     description:
@@ -339,9 +390,11 @@ options:
   vcpus:
     description:
       - Sets number of hotplugged vcpus.
+    type: int
   vga:
     description:
       - Select VGA type. If you want to use high resolution modes (>= 1280x1024x16) then you should use option 'std' or 'vmware'.
+    type: str
     choices: ['std', 'cirrus', 'vmware', 'qxl', 'serial0', 'serial1', 'serial2', 'serial3', 'qxl2', 'qxl3', 'qxl4']
     default: std
   virtio:
@@ -352,13 +405,16 @@ options:
       - C(storage) is the storage identifier where to create the disk.
       - C(size) is the size of the disk in GB.
       - C(format) is the drive's backing file's data format. C(qcow2|raw|subvol).
+    type: dict
   vmid:
     description:
       - Specifies the VM ID. Instead use I(name) parameter.
       - If vmid is not set, the next available VM ID will be fetched from ProxmoxAPI.
+    type: int
   watchdog:
     description:
       - Creates a virtual hardware watchdog device.
+    type: str
 requirements: [ "proxmoxer", "requests" ]
 '''
 
diff --git a/plugins/modules/cloud/misc/proxmox_template.py b/plugins/modules/cloud/misc/proxmox_template.py
index 7ffda09b64..5b0af9dfbe 100644
--- a/plugins/modules/cloud/misc/proxmox_template.py
+++ b/plugins/modules/cloud/misc/proxmox_template.py
@@ -18,15 +18,18 @@ options:
   api_host:
     description:
       - the host of the Proxmox VE cluster
+    type: str
     required: true
   api_user:
     description:
       - the user to authenticate with
+    type: str
     required: true
   api_password:
     description:
       - the password to authenticate with
       - you can use PROXMOX_PASSWORD environment variable
+    type: str
   validate_certs:
     description:
       - enable / disable https certificate verification
@@ -35,29 +38,34 @@ options:
   node:
     description:
       - Proxmox VE node, when you will operate with template
+    type: str
     required: true
   src:
     description:
       - path to uploaded file
       - required only for C(state=present)
-    aliases: ['path']
+    type: path
   template:
     description:
       - the template name
       - required only for states C(absent), C(info)
+    type: str
   content_type:
     description:
       - content type
       - required only for C(state=present)
+    type: str
     default: 'vztmpl'
     choices: ['vztmpl', 'iso']
   storage:
     description:
       - target storage
+    type: str
     default: 'local'
   timeout:
     description:
       - timeout for operations
+    type: int
     default: 30
   force:
     description:
@@ -67,6 +75,7 @@ options:
   state:
     description:
      - Indicate desired state of the template
+    type: str
     choices: ['present', 'absent']
     default: present
 notes:
diff --git a/plugins/modules/cloud/misc/rhevm.py b/plugins/modules/cloud/misc/rhevm.py
index 5afed5baf7..2aebc3464d 100644
--- a/plugins/modules/cloud/misc/rhevm.py
+++ b/plugins/modules/cloud/misc/rhevm.py
@@ -29,6 +29,7 @@ options:
         description:
             - The password for user authentication.
         type: str
+        required: true
     server:
         description:
             - The name/IP of your RHEV-m/oVirt instance.
@@ -111,15 +112,18 @@ options:
         description:
             - This option uses complex arguments and is a list of disks with the options name, size and domain.
         type: list
+        elements: str
     ifaces:
         description:
             - This option uses complex arguments and is a list of interfaces with the options name and vlan.
         type: list
+        elements: str
         aliases: [ interfaces, nics ]
     boot_order:
         description:
             - This option uses complex arguments and is a list of items that specify the bootorder.
         type: list
+        elements: str
         default: [ hd, network ]
     del_prot:
         description:
@@ -1480,14 +1484,14 @@ def main():
             vmhost=dict(type='str'),
             vmcpu=dict(type='int', default=2),
             vmmem=dict(type='int', default=1),
-            disks=dict(type='list'),
+            disks=dict(type='list', elements='str'),
             osver=dict(type='str', default="rhel_6x64"),
-            ifaces=dict(type='list', aliases=['interfaces', 'nics']),
+            ifaces=dict(type='list', elements='str', aliases=['interfaces', 'nics']),
             timeout=dict(type='int'),
             mempol=dict(type='int', default=1),
             vm_ha=dict(type='bool', default=True),
             cpu_share=dict(type='int', default=0),
-            boot_order=dict(type='list', default=['hd', 'network']),
+            boot_order=dict(type='list', elements='str', default=['hd', 'network']),
             del_prot=dict(type='bool', default=True),
             cd_drive=dict(type='str'),
         ),
diff --git a/plugins/modules/cloud/misc/serverless.py b/plugins/modules/cloud/misc/serverless.py
index 07257561de..912d4226a8 100644
--- a/plugins/modules/cloud/misc/serverless.py
+++ b/plugins/modules/cloud/misc/serverless.py
@@ -39,6 +39,7 @@ options:
       - A list of specific functions to deploy.
       - If this is not provided, all functions in the service will be deployed.
     type: list
+    elements: str
     default: []
   region:
     description:
@@ -166,7 +167,7 @@ def main():
         argument_spec=dict(
             service_path=dict(type='path', required=True),
             state=dict(type='str', default='present', choices=['absent', 'present']),
-            functions=dict(type='list'),
+            functions=dict(type='list', elements='str'),
             region=dict(type='str', default=''),
             stage=dict(type='str', default=''),
             deploy=dict(type='bool', default=True),
diff --git a/plugins/modules/cloud/misc/terraform.py b/plugins/modules/cloud/misc/terraform.py
index 697517a44a..c0fad285b1 100644
--- a/plugins/modules/cloud/misc/terraform.py
+++ b/plugins/modules/cloud/misc/terraform.py
@@ -20,19 +20,23 @@ options:
     choices: ['planned', 'present', 'absent']
     description:
       - Goal state of given stage/project
+    type: str
     default: present
   binary_path:
     description:
       - The path of a terraform binary to use, relative to the 'service_path'
         unless you supply an absolute path.
+    type: path
   project_path:
     description:
       - The path to the root of the Terraform directory with the
         vars.tf/main.tf/etc to use.
+    type: path
     required: true
   workspace:
     description:
       - The terraform workspace to work with.
+    type: str
     default: default
   purge_workspace:
     description:
@@ -46,11 +50,13 @@ options:
       - The path to an existing Terraform plan file to apply. If this is not
         specified, Ansible will build a new TF plan and execute it.
         Note that this option is required if 'state' has the 'planned' value.
+    type: path
   state_file:
     description:
       - The path to an existing Terraform state file to use when building plan.
         If this is not specified, the default `terraform.tfstate` will be used.
       - This option is ignored when plan is specified.
+    type: path
   variables_files:
     description:
       - The path to a variables file for Terraform to fill into the TF
@@ -63,19 +69,24 @@ options:
     description:
       - A group of key-values to override template variables or those in
         variables files.
+    type: dict
   targets:
     description:
       - A list of specific resources to target in this plan/application. The
         resources selected here will also auto-include any dependencies.
+    type: list
+    elements: str
   lock:
     description:
       - Enable statefile locking, if you use a service that accepts locks (such
         as S3+DynamoDB) to store your statefile.
     type: bool
+    default: true
   lock_timeout:
     description:
       - How long to maintain the lock on the statefile, if you use a service
         that accepts locks (such as S3+DynamoDB).
+    type: int
   force_init:
     description:
       - To avoid duplicating infra, if a state file can't be found this will
@@ -86,6 +97,7 @@ options:
   backend_config:
     description:
       - A group of key-values to provide at init stage to the -backend-config parameter.
+    type: dict
   backend_config_files:
     description:
       - The path to a configuration file to provide at init state to the -backend-config parameter.
@@ -281,7 +293,7 @@ def main():
             variables_files=dict(aliases=['variables_file'], type='list', elements='path', default=None),
             plan_file=dict(type='path'),
             state_file=dict(type='path'),
-            targets=dict(type='list', default=[]),
+            targets=dict(type='list', elements='str', default=[]),
             lock=dict(type='bool', default=True),
             lock_timeout=dict(type='int',),
             force_init=dict(type='bool', default=False),
diff --git a/tests/sanity/ignore-2.10.txt b/tests/sanity/ignore-2.10.txt
index 02e1abfe74..dce5536093 100644
--- a/tests/sanity/ignore-2.10.txt
+++ b/tests/sanity/ignore-2.10.txt
@@ -216,30 +216,11 @@ plugins/modules/cloud/memset/memset_server_info.py validate-modules:parameter-ty
 plugins/modules/cloud/memset/memset_zone.py validate-modules:parameter-type-not-in-doc
 plugins/modules/cloud/memset/memset_zone_domain.py validate-modules:parameter-type-not-in-doc
 plugins/modules/cloud/memset/memset_zone_record.py validate-modules:parameter-type-not-in-doc
-plugins/modules/cloud/misc/cloud_init_data_facts.py validate-modules:doc-missing-type
-plugins/modules/cloud/misc/helm.py validate-modules:doc-missing-type
-plugins/modules/cloud/misc/helm.py validate-modules:parameter-type-not-in-doc
-plugins/modules/cloud/misc/ovirt.py validate-modules:doc-choices-do-not-match-spec
-plugins/modules/cloud/misc/ovirt.py validate-modules:parameter-type-not-in-doc
-plugins/modules/cloud/misc/ovirt.py validate-modules:undocumented-parameter
-plugins/modules/cloud/misc/proxmox.py validate-modules:doc-missing-type
-plugins/modules/cloud/misc/proxmox.py validate-modules:parameter-type-not-in-doc
 plugins/modules/cloud/misc/proxmox_kvm.py validate-modules:doc-default-does-not-match-spec
-plugins/modules/cloud/misc/proxmox_kvm.py validate-modules:doc-missing-type
 plugins/modules/cloud/misc/proxmox_kvm.py validate-modules:parameter-type-not-in-doc
 plugins/modules/cloud/misc/proxmox_kvm.py validate-modules:undocumented-parameter
-plugins/modules/cloud/misc/proxmox_template.py validate-modules:doc-missing-type
 plugins/modules/cloud/misc/proxmox_template.py validate-modules:doc-required-mismatch
-plugins/modules/cloud/misc/proxmox_template.py validate-modules:nonexistent-parameter-documented
-plugins/modules/cloud/misc/proxmox_template.py validate-modules:parameter-type-not-in-doc
-plugins/modules/cloud/misc/rhevm.py validate-modules:doc-required-mismatch
-plugins/modules/cloud/misc/rhevm.py validate-modules:parameter-list-no-elements
 plugins/modules/cloud/misc/rhevm.py validate-modules:parameter-state-invalid-choice
-plugins/modules/cloud/misc/serverless.py validate-modules:parameter-list-no-elements
-plugins/modules/cloud/misc/terraform.py validate-modules:doc-default-does-not-match-spec
-plugins/modules/cloud/misc/terraform.py validate-modules:doc-missing-type
-plugins/modules/cloud/misc/terraform.py validate-modules:parameter-list-no-elements
-plugins/modules/cloud/misc/terraform.py validate-modules:parameter-type-not-in-doc
 plugins/modules/cloud/misc/terraform.py validate-modules:return-syntax-error
 plugins/modules/cloud/oneandone/oneandone_firewall_policy.py validate-modules:doc-required-mismatch
 plugins/modules/cloud/oneandone/oneandone_firewall_policy.py validate-modules:parameter-list-no-elements
diff --git a/tests/sanity/ignore-2.11.txt b/tests/sanity/ignore-2.11.txt
index 02e1abfe74..dce5536093 100644
--- a/tests/sanity/ignore-2.11.txt
+++ b/tests/sanity/ignore-2.11.txt
@@ -216,30 +216,11 @@ plugins/modules/cloud/memset/memset_server_info.py validate-modules:parameter-ty
 plugins/modules/cloud/memset/memset_zone.py validate-modules:parameter-type-not-in-doc
 plugins/modules/cloud/memset/memset_zone_domain.py validate-modules:parameter-type-not-in-doc
 plugins/modules/cloud/memset/memset_zone_record.py validate-modules:parameter-type-not-in-doc
-plugins/modules/cloud/misc/cloud_init_data_facts.py validate-modules:doc-missing-type
-plugins/modules/cloud/misc/helm.py validate-modules:doc-missing-type
-plugins/modules/cloud/misc/helm.py validate-modules:parameter-type-not-in-doc
-plugins/modules/cloud/misc/ovirt.py validate-modules:doc-choices-do-not-match-spec
-plugins/modules/cloud/misc/ovirt.py validate-modules:parameter-type-not-in-doc
-plugins/modules/cloud/misc/ovirt.py validate-modules:undocumented-parameter
-plugins/modules/cloud/misc/proxmox.py validate-modules:doc-missing-type
-plugins/modules/cloud/misc/proxmox.py validate-modules:parameter-type-not-in-doc
 plugins/modules/cloud/misc/proxmox_kvm.py validate-modules:doc-default-does-not-match-spec
-plugins/modules/cloud/misc/proxmox_kvm.py validate-modules:doc-missing-type
 plugins/modules/cloud/misc/proxmox_kvm.py validate-modules:parameter-type-not-in-doc
 plugins/modules/cloud/misc/proxmox_kvm.py validate-modules:undocumented-parameter
-plugins/modules/cloud/misc/proxmox_template.py validate-modules:doc-missing-type
 plugins/modules/cloud/misc/proxmox_template.py validate-modules:doc-required-mismatch
-plugins/modules/cloud/misc/proxmox_template.py validate-modules:nonexistent-parameter-documented
-plugins/modules/cloud/misc/proxmox_template.py validate-modules:parameter-type-not-in-doc
-plugins/modules/cloud/misc/rhevm.py validate-modules:doc-required-mismatch
-plugins/modules/cloud/misc/rhevm.py validate-modules:parameter-list-no-elements
 plugins/modules/cloud/misc/rhevm.py validate-modules:parameter-state-invalid-choice
-plugins/modules/cloud/misc/serverless.py validate-modules:parameter-list-no-elements
-plugins/modules/cloud/misc/terraform.py validate-modules:doc-default-does-not-match-spec
-plugins/modules/cloud/misc/terraform.py validate-modules:doc-missing-type
-plugins/modules/cloud/misc/terraform.py validate-modules:parameter-list-no-elements
-plugins/modules/cloud/misc/terraform.py validate-modules:parameter-type-not-in-doc
 plugins/modules/cloud/misc/terraform.py validate-modules:return-syntax-error
 plugins/modules/cloud/oneandone/oneandone_firewall_policy.py validate-modules:doc-required-mismatch
 plugins/modules/cloud/oneandone/oneandone_firewall_policy.py validate-modules:parameter-list-no-elements
diff --git a/tests/sanity/ignore-2.9.txt b/tests/sanity/ignore-2.9.txt
index 969a64e4c1..2df37b73ca 100644
--- a/tests/sanity/ignore-2.9.txt
+++ b/tests/sanity/ignore-2.9.txt
@@ -149,28 +149,13 @@ plugins/modules/cloud/memset/memset_server_info.py validate-modules:parameter-ty
 plugins/modules/cloud/memset/memset_zone.py validate-modules:parameter-type-not-in-doc
 plugins/modules/cloud/memset/memset_zone_domain.py validate-modules:parameter-type-not-in-doc
 plugins/modules/cloud/memset/memset_zone_record.py validate-modules:parameter-type-not-in-doc
-plugins/modules/cloud/misc/cloud_init_data_facts.py validate-modules:doc-missing-type
 plugins/modules/cloud/misc/helm.py validate-modules:deprecation-mismatch
-plugins/modules/cloud/misc/helm.py validate-modules:doc-missing-type
 plugins/modules/cloud/misc/helm.py validate-modules:invalid-documentation
-plugins/modules/cloud/misc/helm.py validate-modules:parameter-type-not-in-doc
 plugins/modules/cloud/misc/ovirt.py validate-modules:deprecation-mismatch
-plugins/modules/cloud/misc/ovirt.py validate-modules:doc-choices-do-not-match-spec
 plugins/modules/cloud/misc/ovirt.py validate-modules:invalid-documentation
-plugins/modules/cloud/misc/ovirt.py validate-modules:parameter-type-not-in-doc
-plugins/modules/cloud/misc/ovirt.py validate-modules:undocumented-parameter
-plugins/modules/cloud/misc/proxmox.py validate-modules:doc-missing-type
-plugins/modules/cloud/misc/proxmox.py validate-modules:parameter-type-not-in-doc
 plugins/modules/cloud/misc/proxmox_kvm.py validate-modules:doc-default-does-not-match-spec
-plugins/modules/cloud/misc/proxmox_kvm.py validate-modules:doc-missing-type
 plugins/modules/cloud/misc/proxmox_kvm.py validate-modules:parameter-type-not-in-doc
 plugins/modules/cloud/misc/proxmox_kvm.py validate-modules:undocumented-parameter
-plugins/modules/cloud/misc/proxmox_template.py validate-modules:doc-missing-type
-plugins/modules/cloud/misc/proxmox_template.py validate-modules:nonexistent-parameter-documented
-plugins/modules/cloud/misc/proxmox_template.py validate-modules:parameter-type-not-in-doc
-plugins/modules/cloud/misc/terraform.py validate-modules:doc-default-does-not-match-spec
-plugins/modules/cloud/misc/terraform.py validate-modules:doc-missing-type
-plugins/modules/cloud/misc/terraform.py validate-modules:parameter-type-not-in-doc
 plugins/modules/cloud/oneandone/oneandone_firewall_policy.py validate-modules:parameter-type-not-in-doc
 plugins/modules/cloud/oneandone/oneandone_load_balancer.py validate-modules:doc-default-does-not-match-spec
 plugins/modules/cloud/oneandone/oneandone_load_balancer.py validate-modules:doc-missing-type