Enabling validation-modules for system modules (#1212) (#1225)

* fixed validation-modules for aix_devices.py

* fixed validation-modules for aix_filesystem.py

* fixed validation-modules for aix_inittab.py

* fixed validation-modules for aix_lvg.py

* fixed validation-modules for aix_lvol.py

* fixed validation-modules for awall.py

* fixed validation-modules for dconf.py

* fixed validation-modules for gconftool2.py

* fixed validation-modules for interfaces_file.py

* fixed validation-modules for java_keystore.py

* fixed validation-modules for kernel_blacklist.py

* fixed validation-modules for plugins/modules/system/lbu.py

* fixed validation-modules for plugins/modules/system/locale_gen.py

* fixed validation-modules for plugins/modules/system/lvg.py

* fixed validation-modules for plugins/modules/system/lvol.py

* fixed validation-modules for plugins/modules/system/mksysb.py

* fixed validation-modules for plugins/modules/system/modprobe.py

* fixed validation-modules for plugins/modules/system/nosh.py

* fixed validation-modules for plugins/modules/system/open_iscsi.py

* fixed validation-modules for plugins/modules/system/openwrt_init.py

* fixed validation-modules for plugins/modules/system/osx_defaults.py

* fixed validation-modules for plugins/modules/system/pamd.py

* fixed validation-modules for plugins/modules/system/pam_limits.py

* fixed validation-modules for plugins/modules/system/parted.py

* fixed validation-modules for plugins/modules/system/puppet.py

* fixed validation-modules for plugins/modules/system/python_requirements_info.py

* fixed validation-modules for plugins/modules/system/runit.py

the parameter "dist" is not used anywhere in the module

* fixed validation-modules for plugins/modules/system/sefcontext.py

* fixed validation-modules for plugins/modules/system/selogin.py

* fixed validation-modules for plugins/modules/system/seport.py

* fixed validation-modules for plugins/modules/system/solaris_zone.py

* fixed validation-modules for plugins/modules/system/syspatch.py

* fixed validation-modules for plugins/modules/system/vdo.py

* fixed validation-modules for plugins/modules/system/xfconf.py

* removed ignore almost all validate-modules lines in system

* removed unnecessary validations, per shippable test

* kernel_blacklist: keeping blacklist_file as str instead of path

* mksysb: keeping storage_path as str instead of path

* pam_limits: keeping dest as str instead of path

* rollback on adding doc for puppet.py legacy param

* rolledback param seuser required in selogin module

* rolledback changes in runit

* rolledback changes in osx_defaults

* rolledback changes in aix_defaults

(cherry picked from commit 1202d034b3)

Co-authored-by: Alexei Znamensky <103110+russoz@users.noreply.github.com>

plugins/modules/system/aix_filesystem.py

@@ -28,6 +28,7 @@ options:
     description:
       - Specifies attributes for files system separated by comma.
     type: list
+    elements: str
     default: agblksize='4096',isnapshot='no'
   auto_mount:
     description:
@@ -90,7 +91,6 @@ options:
       - C(mounted) checks if the file system is mounted or mount the file system.
       - C(unmounted) check if the file system is unmounted or unmount the file system.
     type: str
-    required: true
     choices: [ absent, mounted, present, unmounted ]
     default: present
   vg:
@@ -455,7 +455,7 @@ def main():
     module = AnsibleModule(
         argument_spec=dict(
             account_subsystem=dict(type='bool', default=False),
-            attributes=dict(type='list', default=["agblksize='4096'", "isnapshot='no'"]),
+            attributes=dict(type='list', elements='str', default=["agblksize='4096'", "isnapshot='no'"]),
             auto_mount=dict(type='bool', default=True),
             device=dict(type='str'),
             filesystem=dict(type='str', required=True),

plugins/modules/system/aix_inittab.py

@@ -31,7 +31,6 @@ options:
     description:
     - Action what the init has to do with this entry.
     type: str
-    required: yes
     choices:
     - boot
     - bootwait
@@ -237,7 +236,7 @@ def main():
                     [rmitab, module.params['name']])
                 if rc != 0:
                     module.fail_json(
-                        msg="could not remove entry grom inittab)", rc=rc, err=err)
+                        msg="could not remove entry from inittab)", rc=rc, err=err)
             result['msg'] = "removed inittab entry" + " " + current_entry['name']
             result['changed'] = True
 

plugins/modules/system/aix_lvg.py

@@ -31,6 +31,7 @@ options:
     - Required when creating or extending (C(present) state) the volume group.
     - If not informed reducing (C(absent) state) the volume group will be removed.
     type: list
+    elements: str
   state:
     description:
     - Control if the volume group exists and volume group AIX state varyonvg C(varyon) or varyoffvg C(varyoff).
@@ -315,7 +316,7 @@ def main():
         argument_spec=dict(
             force=dict(type='bool', default=False),
             pp_size=dict(type='int'),
-            pvs=dict(type='list'),
+            pvs=dict(type='list', elements='str'),
             state=dict(type='str', default='present', choices=['absent', 'present', 'varyoff', 'varyon']),
             vg=dict(type='str', required=True),
             vg_type=dict(type='str', default='normal', choices=['big', 'normal', 'scalable'])

plugins/modules/system/aix_lvol.py

@@ -65,6 +65,7 @@ options:
     description:
     - A list of physical volumes e.g. C(hdisk1,hdisk2).
     type: list
+    elements: str
 '''
 
 EXAMPLES = r'''
@@ -214,7 +215,7 @@ def main():
             copies=dict(type='int', default=1),
             state=dict(type='str', default='present', choices=['absent', 'present']),
             policy=dict(type='str', default='maximum', choices=['maximum', 'minimum']),
-            pvs=dict(type='list', default=list())
+            pvs=dict(type='list', elements='str', default=list())
         ),
         supports_check_mode=True,
     )

plugins/modules/system/awall.py

@@ -22,6 +22,7 @@ options:
     description:
       - One or more policy names.
     type: list
+    elements: str
   state:
     description:
       - Whether the policies should be enabled or disabled.
@@ -122,7 +123,7 @@ def main():
     module = AnsibleModule(
         argument_spec=dict(
             state=dict(type='str', default='enabled', choices=['disabled', 'enabled']),
-            name=dict(type='list'),
+            name=dict(type='list', elements='str'),
             activate=dict(type='bool', default=False),
         ),
         required_one_of=[['name', 'activate']],

plugins/modules/system/dconf.py

@@ -45,10 +45,12 @@ notes:
     /path/to/dir/) or C(dconf read /path/to/key).
 options:
   key:
+    type: str
     required: true
     description:
       - A dconf key to modify or read from the dconf database.
   value:
+    type: str
     required: false
     description:
       - Value to set for the specified dconf key. Value should be specified in
@@ -56,6 +58,7 @@ options:
         look at existing values in the dconf database. Required for
         C(state=present).
   state:
+    type: str
     required: false
     default: present
     choices:

plugins/modules/system/gconftool2.py

@@ -18,25 +18,30 @@ description:
     gconftool-2.  Please see the gconftool-2(1) man pages for more details.
 options:
   key:
+    type: str
     description:
     - A GConf preference key is an element in the GConf repository
       that corresponds to an application preference. See man gconftool-2(1)
     required: yes
   value:
+    type: str
     description:
     - Preference keys typically have simple values such as strings,
       integers, or lists of strings and integers. This is ignored if the state
       is "get". See man gconftool-2(1)
   value_type:
+    type: str
     description:
     - The type of value being set. This is ignored if the state is "get".
     choices: [ bool, float, int, string ]
   state:
+    type: str
     description:
     - The action to take upon the key/value.
     required: yes
     choices: [ absent, get, present ]
   config_source:
+    type: str
     description:
     - Specify a configuration source to use rather than the default path.
       See man gconftool-2(1)

plugins/modules/system/interfaces_file.py

@@ -19,19 +19,24 @@ description:
      - Read information about interfaces from interfaces-styled files
 options:
   dest:
+    type: path
     description:
       - Path to the interfaces file
     default: /etc/network/interfaces
   iface:
+    type: str
     description:
       - Name of the interface, required for value changes or option remove
   address_family:
+    type: str
     description:
       - Address family of the interface, useful if same interface name is used for both inet and inet6
   option:
+    type: str
     description:
       - Name of the option, required for value changes or option remove
   value:
+    type: str
     description:
       - If I(option) is not presented for the I(interface) and I(state) is C(present) option will be added.
         If I(option) already exists and is not C(pre-up), C(up), C(post-up) or C(down), it's value will be updated.
@@ -44,6 +49,7 @@ options:
     type: bool
     default: 'no'
   state:
+    type: str
     description:
       - If set to C(absent) the option or section will be removed if present instead of created.
     default: "present"

plugins/modules/system/java_keystore.py

@@ -16,14 +16,17 @@ description:
      - Create or delete a Java keystore in JKS format for a given certificate.
 options:
     name:
+        type: str
         description:
           - Name of the certificate.
         required: true
     certificate:
+        type: str
         description:
           - Certificate that should be used to create the key store.
         required: true
     private_key:
+        type: str
         description:
           - Private key that should be used to create the key store.
         required: true
@@ -34,10 +37,12 @@ options:
         required: false
         version_added: '0.2.0'
     password:
+        type: str
         description:
           - Password that should be used to secure the key store.
         required: true
     dest:
+        type: path
         description:
           - Absolute path where the jks should be generated.
         required: true
@@ -61,6 +66,9 @@ options:
         default: 'no'
 requirements: [openssl, keytool]
 author: Guillaume Grossetie (@Mogztter)
+extends_documentation_fragment:
+- files
+
 '''
 
 EXAMPLES = '''
@@ -286,7 +294,7 @@ class ArgumentSpec(object):
             certificate=dict(required=True, no_log=True),
             private_key=dict(required=True, no_log=True),
             password=dict(required=True, no_log=True),
-            dest=dict(required=True),
+            dest=dict(required=True, type='path'),
             force=dict(required=False, default=False, type='bool'),
             private_key_passphrase=dict(required=False, no_log=True, type='str')
         )

plugins/modules/system/kernel_blacklist.py

@@ -17,15 +17,18 @@ description:
     - Add or remove kernel modules from blacklist.
 options:
     name:
+        type: str
         description:
             - Name of kernel module to black- or whitelist.
         required: true
     state:
+        type: str
         description:
             - Whether the module should be present in the blacklist or absent.
         choices: [ absent, present ]
         default: present
     blacklist_file:
+        type: str
         description:
             - If specified, use this blacklist file instead of
               C(/etc/modprobe.d/blacklist-ansible.conf).

plugins/modules/system/lbu.py

@@ -26,10 +26,12 @@ options:
     description:
     - List of paths to exclude.
     type: list
+    elements: str
   include:
     description:
     - List of paths to include.
     type: list
+    elements: str
 
 author:
 - Kaarle Ritvanen (@kunkku)

plugins/modules/system/locale_gen.py

@@ -16,10 +16,12 @@ author:
 - Augustus Kling (@AugustusKling)
 options:
     name:
+        type: str
         description:
              - Name and encoding of the locale, such as "en_GB.UTF-8".
         required: true
     state:
+      type: str
       description:
            - Whether the locale shall be present.
       choices: [ absent, present ]

plugins/modules/system/lvg.py

@@ -28,6 +28,7 @@ options:
     - Required when creating or resizing volume group.
     - The module will take care of running pvcreate if needed.
     type: list
+    elements: str
   pesize:
     description:
     - "The size of the physical extent. I(pesize) must be a power of 2 of at least 1 sector
@@ -149,7 +150,7 @@ def main():
     module = AnsibleModule(
         argument_spec=dict(
             vg=dict(type='str', required=True),
-            pvs=dict(type='list'),
+            pvs=dict(type='list', elements='str'),
             pesize=dict(type='str', default='4'),
             pv_options=dict(type='str', default=''),
             pvresize=dict(type='bool', default=False),

plugins/modules/system/lvol.py

@@ -18,12 +18,16 @@ description:
   - This module creates, removes or resizes logical volumes.
 options:
   vg:
+    type: str
+    required: true
     description:
     - The volume group this logical volume is part of.
   lv:
+    type: str
     description:
     - The name of the logical volume.
   size:
+    type: str
     description:
     - The size of the logical volume, according to lvcreate(8) --size, by
       default in megabytes or optionally with one of [bBsSkKmMgGtTpPeE] units; or
@@ -31,6 +35,7 @@ options:
       Float values must begin with a digit.
       Resizing using percentage values was not supported prior to 2.1.
   state:
+    type: str
     description:
     - Control if the logical volume exists. If C(present) and the
       volume does not already exist then the C(size) option is required.
@@ -48,15 +53,19 @@ options:
     type: bool
     default: 'no'
   opts:
+    type: str
     description:
     - Free-form options to be passed to the lvcreate command.
   snapshot:
+    type: str
     description:
     - The name of the snapshot volume
   pvs:
+    type: str
     description:
     - Comma separated list of physical volumes (e.g. /dev/sda,/dev/sdb).
   thinpool:
+    type: str
     description:
     - The thin pool volume name. When you want to create a thin provisioned volume, specify a thin pool volume name.
   shrink:

plugins/modules/system/mksysb.py

@@ -48,6 +48,7 @@ options:
     type: bool
     default: "yes"
   name:
+    type: str
     description:
       - Backup name
     required: true
@@ -63,6 +64,7 @@ options:
     type: bool
     default: "no"
   storage_path:
+    type: str
     description:
       - Storage path where the mksysb will stored.
     required: true
@@ -107,10 +109,10 @@ def main():
             exclude_files=dict(type='bool', default=False),
             exclude_wpar_files=dict(type='bool', default=False),
             extended_attrs=dict(type='bool', default=True),
-            name=dict(required=True),
+            name=dict(type='str', required=True),
             new_image_data=dict(type='bool', default=True),
             software_packing=dict(type='bool', default=False),
-            storage_path=dict(required=True),
+            storage_path=dict(type='str', required=True),
             use_snapshot=dict(type='bool', default=False)
         ),
         supports_check_mode=True,

plugins/modules/system/modprobe.py

@@ -19,15 +19,18 @@ description:
     - Load or unload kernel modules.
 options:
     name:
+        type: str
         required: true
         description:
             - Name of kernel module to manage.
     state:
+        type: str
         description:
             - Whether the module should be present or absent.
         choices: [ absent, present ]
         default: present
     params:
+        type: str
         description:
             - Modules parameters.
         default: ''

plugins/modules/system/nosh.py

@@ -19,10 +19,12 @@ description:
     - BSD and Linux systems are supported.
 options:
     name:
+        type: str
         required: true
         description:
             - Name of the service to manage.
     state:
+        type: str
         required: false
         choices: [ started, stopped, reset, restarted, reloaded ]
         description:
@@ -488,8 +490,8 @@ def handle_state(module, result, service_path):
 def main():
     module = AnsibleModule(
         argument_spec=dict(
-            name=dict(required=True),
+            name=dict(type='str', required=True),
-            state=dict(choices=['started', 'stopped', 'reset', 'restarted', 'reloaded'], type='str'),
+            state=dict(type='str', choices=['started', 'stopped', 'reset', 'restarted', 'reloaded']),
             enabled=dict(type='bool'),
             preset=dict(type='bool'),
             user=dict(type='bool', default=False),

plugins/modules/system/open_iscsi.py

@@ -267,8 +267,7 @@ def main():
             show_nodes=dict(type='bool', default=False),
         ),
 
-        required_together=[['discover_user', 'discover_pass'],
+        required_together=[['node_user', 'node_pass']],
-                           ['node_user', 'node_pass']],
         supports_check_mode=True,
     )
 

plugins/modules/system/openwrt_init.py

@@ -16,11 +16,13 @@ description:
     - Controls OpenWrt services on remote hosts.
 options:
     name:
+        type: str
         description:
             - Name of the service.
         required: true
         aliases: ['service']
     state:
+        type: str
         description:
             - C(started)/C(stopped) are idempotent actions that will not run commands unless necessary.
               C(restarted) will always bounce the service. C(reloaded) will always reload.
@@ -30,6 +32,7 @@ options:
             - Whether the service should start on boot. B(At least one of state and enabled are required.)
         type: bool
     pattern:
+        type: str
         description:
         - If the service does not respond to the 'running' command, name a
           substring to look for as would be found in the output of the I(ps)
@@ -93,9 +96,9 @@ def main():
     module = AnsibleModule(
         argument_spec=dict(
             name=dict(required=True, type='str', aliases=['service']),
-            state=dict(choices=['started', 'stopped', 'restarted', 'reloaded'], type='str'),
+            state=dict(type='str', choices=['started', 'stopped', 'restarted', 'reloaded']),
             enabled=dict(type='bool'),
-            pattern=dict(required=False, default=None),
+            pattern=dict(type='str', required=False, default=None),
         ),
         supports_check_mode=True,
         required_one_of=[['state', 'enabled']],

plugins/modules/system/pam_limits.py

@@ -20,15 +20,18 @@ description:
        limits.conf).
 options:
   domain:
+    type: str
     description:
       - A username, @groupname, wildcard, uid/gid range.
     required: true
   limit_type:
+    type: str
     description:
       - Limit type, see C(man 5 limits.conf) for an explanation
     required: true
     choices: [ "hard", "soft", "-" ]
   limit_item:
+    type: str
     description:
       - The limit to be set
     required: true
@@ -53,6 +56,7 @@ options:
         - "rtprio"
         - "chroot"
   value:
+    type: str
     description:
       - The value of the limit.
     required: true
@@ -80,11 +84,13 @@ options:
     type: bool
     default: "no"
   dest:
+    type: str
     description:
       - Modify the limits.conf path.
     required: false
     default: "/etc/security/limits.conf"
   comment:
+    type: str
     description:
       - Comment associated with the limit.
     required: false

plugins/modules/system/pamd.py

@@ -67,6 +67,7 @@ options:
     - Furthermore, if the module argument takes a value denoted by C(=),
       the value will be changed to that specified in module_arguments.
     type: list
+    elements: str
   state:
     description:
     - The default of C(updated) will modify an existing rule if type,
@@ -770,7 +771,7 @@ def main():
             new_type=dict(type='str', choices=VALID_TYPES),
             new_control=dict(type='str'),
             new_module_path=dict(type='str'),
-            module_arguments=dict(type='list'),
+            module_arguments=dict(type='list', elements='str'),
             state=dict(type='str', default='updated', choices=['absent', 'after', 'args_absent', 'args_present', 'before', 'updated']),
             path=dict(type='path', default='/etc/pam.d'),
             backup=dict(type='bool', default=False),

plugins/modules/system/parted.py

@@ -89,6 +89,7 @@ options:
   flags:
     description: A list of the flags that has to be set on the partition.
     type: list
+    elements: str
   state:
     description:
     - Whether to create or delete a partition.
@@ -581,7 +582,7 @@ def main():
             name=dict(type='str'),
 
             # set <partition> <flag> <state> command
-            flags=dict(type='list'),
+            flags=dict(type='list', elements='str'),
 
             # rm/mkpart command
             state=dict(type='str', default='info', choices=['absent', 'info', 'present']),

plugins/modules/system/puppet.py

@@ -66,6 +66,7 @@ options:
     description:
       - A list of puppet tags to be used.
     type: list
+    elements: str
   execute:
     description:
       - Execute a specific piece of Puppet code.
@@ -177,7 +178,7 @@ def main():
             facter_basename=dict(type='str', default='ansible'),
             environment=dict(type='str'),
             certname=dict(type='str'),
-            tags=dict(type='list'),
+            tags=dict(type='list', elements='str'),
             execute=dict(type='str'),
             summarize=dict(type='bool', default=False),
             debug=dict(type='bool', default=False),

plugins/modules/system/python_requirements_info.py

@@ -13,6 +13,8 @@ description:
     - This module was called C(python_requirements_facts) before Ansible 2.9. The usage did not change.
 options:
   dependencies:
+    type: list
+    elements: str
     description: >
       A list of version-likes or module names to check for installation.
       Supported operators: <, >, <=, >=, or ==. The bare module name like
@@ -107,7 +109,7 @@ operations = {
 def main():
     module = AnsibleModule(
         argument_spec=dict(
-            dependencies=dict(type='list')
+            dependencies=dict(type='list', elements='str')
         ),
         supports_check_mode=True,
     )

plugins/modules/system/sefcontext.py

@@ -250,7 +250,7 @@ def main():
         argument_spec=dict(
             ignore_selinux_state=dict(type='bool', default=False),
             target=dict(type='str', required=True, aliases=['path']),
-            ftype=dict(type='str', default='a', choices=option_to_file_type_str.keys()),
+            ftype=dict(type='str', default='a', choices=list(option_to_file_type_str.keys())),
             setype=dict(type='str', required=True),
             seuser=dict(type='str'),
             selevel=dict(type='str', aliases=['serange']),

plugins/modules/system/selogin.py

@@ -27,27 +27,30 @@ description:
      - Manages linux user to SELinux user mapping
 options:
   login:
+    type: str
     description:
       - a Linux user
     required: true
   seuser:
+    type: str
     description:
       - SELinux user name
-    required: true
   selevel:
+    type: str
     aliases: [ serange ]
     description:
       - MLS/MCS Security Range (MLS/MCS Systems only) SELinux Range for SELinux login mapping defaults to the SELinux user record range.
     default: s0
   state:
+    type: str
     description:
       - Desired mapping value.
-    required: true
     default: present
     choices: [ 'present', 'absent' ]
   reload:
     description:
       - Reload SELinux policy after commit.
+    type: bool
     default: yes
   ignore_selinux_state:
     description:

plugins/modules/system/seport.py

@@ -19,6 +19,7 @@ options:
       - Ports or port ranges.
       - Can be a list (since 2.6) or comma separated string.
     type: list
+    elements: str
     required: true
   proto:
     description:
@@ -258,7 +259,7 @@ def main():
     module = AnsibleModule(
         argument_spec=dict(
             ignore_selinux_state=dict(type='bool', default=False),
-            ports=dict(type='list', required=True),
+            ports=dict(type='list', elements='str', required=True),
             proto=dict(type='str', required=True, choices=['tcp', 'udp']),
             setype=dict(type='str', required=True),
             state=dict(type='str', default='present', choices=['absent', 'present']),

plugins/modules/system/solaris_zone.py

@@ -34,7 +34,6 @@ options:
     type: str
     choices: [ absent, attached, configured, detached, installed, present, running, started, stopped ]
     default: present
-    required: true
   name:
     description:
       - Zone name.

plugins/modules/system/syspatch.py

@@ -18,6 +18,7 @@ description:
 
 options:
     apply:
+        type: bool
         description:
             - Apply all available system patches.
             - By default, apply all patches.

plugins/modules/system/vdo.py

@@ -41,7 +41,6 @@ options:
               "absent" VDO volume does not exist, it will not be
               removed.
         type: str
-        required: true
         choices: [ absent, present ]
         default: present
     activated:

plugins/modules/system/xfconf.py

@@ -46,6 +46,7 @@ options:
     elements: str
     choices: [ int, uint, bool, float, double, string ]
   state:
+    type: str
     description:
     - The action to take upon the property/value.
     choices: [ get, present, absent ]

tests/sanity/ignore-2.10.txt

@@ -1155,72 +1155,32 @@ plugins/modules/storage/zfs/zfs_facts.py validate-modules:parameter-type-not-in-
 plugins/modules/storage/zfs/zpool_facts.py validate-modules:nonexistent-parameter-documented
 plugins/modules/storage/zfs/zpool_facts.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/aix_devices.py validate-modules:doc-required-mismatch
-plugins/modules/system/aix_filesystem.py validate-modules:doc-required-mismatch
-plugins/modules/system/aix_filesystem.py validate-modules:parameter-list-no-elements
-plugins/modules/system/aix_inittab.py validate-modules:doc-required-mismatch
-plugins/modules/system/aix_lvg.py validate-modules:parameter-list-no-elements
-plugins/modules/system/aix_lvol.py validate-modules:parameter-list-no-elements
 plugins/modules/system/alternatives.py pylint:blacklisted-name
-plugins/modules/system/awall.py validate-modules:parameter-list-no-elements
 plugins/modules/system/beadm.py pylint:blacklisted-name
 plugins/modules/system/cronvar.py pylint:blacklisted-name
 plugins/modules/system/dconf.py pylint:blacklisted-name
-plugins/modules/system/dconf.py validate-modules:doc-missing-type
-plugins/modules/system/dconf.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/filesystem.py pylint:blacklisted-name
 plugins/modules/system/gconftool2.py pylint:blacklisted-name
 plugins/modules/system/gconftool2.py validate-modules:parameter-state-invalid-choice
-plugins/modules/system/gconftool2.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/interfaces_file.py pylint:blacklisted-name
-plugins/modules/system/interfaces_file.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/iptables_state.py validate-modules:undocumented-parameter
 plugins/modules/system/java_cert.py pylint:blacklisted-name
-plugins/modules/system/java_keystore.py validate-modules:doc-missing-type
-plugins/modules/system/java_keystore.py validate-modules:parameter-type-not-in-doc
-plugins/modules/system/java_keystore.py validate-modules:undocumented-parameter
-plugins/modules/system/kernel_blacklist.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/launchd.py use-argspec-type-path  # False positive
-plugins/modules/system/lbu.py validate-modules:doc-elements-mismatch
-plugins/modules/system/locale_gen.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/lvg.py pylint:blacklisted-name
-plugins/modules/system/lvg.py validate-modules:parameter-list-no-elements
 plugins/modules/system/lvol.py pylint:blacklisted-name
-plugins/modules/system/lvol.py validate-modules:doc-required-mismatch
-plugins/modules/system/lvol.py validate-modules:parameter-type-not-in-doc
-plugins/modules/system/mksysb.py validate-modules:doc-missing-type
-plugins/modules/system/modprobe.py validate-modules:parameter-type-not-in-doc
-plugins/modules/system/nosh.py validate-modules:doc-missing-type
-plugins/modules/system/nosh.py validate-modules:parameter-type-not-in-doc
-plugins/modules/system/open_iscsi.py validate-modules:required_together-unknown
-plugins/modules/system/openwrt_init.py validate-modules:doc-missing-type
-plugins/modules/system/openwrt_init.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/osx_defaults.py validate-modules:doc-required-mismatch
 plugins/modules/system/osx_defaults.py validate-modules:parameter-state-invalid-choice
-plugins/modules/system/pam_limits.py validate-modules:parameter-type-not-in-doc
-plugins/modules/system/pamd.py validate-modules:parameter-list-no-elements
 plugins/modules/system/parted.py pylint:blacklisted-name
-plugins/modules/system/parted.py validate-modules:parameter-list-no-elements
 plugins/modules/system/parted.py validate-modules:parameter-state-invalid-choice
 plugins/modules/system/puppet.py use-argspec-type-path
 plugins/modules/system/puppet.py validate-modules:parameter-invalid
-plugins/modules/system/puppet.py validate-modules:parameter-list-no-elements
 plugins/modules/system/puppet.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/puppet.py validate-modules:undocumented-parameter
-plugins/modules/system/python_requirements_info.py validate-modules:parameter-list-no-elements
-plugins/modules/system/python_requirements_info.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/runit.py validate-modules:doc-default-does-not-match-spec
 plugins/modules/system/runit.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/runit.py validate-modules:undocumented-parameter
-plugins/modules/system/sefcontext.py validate-modules:invalid-ansiblemodule-schema
-plugins/modules/system/selogin.py validate-modules:doc-required-mismatch
-plugins/modules/system/selogin.py validate-modules:parameter-type-not-in-doc
-plugins/modules/system/seport.py validate-modules:parameter-list-no-elements
-plugins/modules/system/solaris_zone.py validate-modules:doc-required-mismatch
-plugins/modules/system/syspatch.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/timezone.py pylint:blacklisted-name
-plugins/modules/system/vdo.py validate-modules:doc-required-mismatch
 plugins/modules/system/xfconf.py validate-modules:parameter-state-invalid-choice
-plugins/modules/system/xfconf.py validate-modules:parameter-type-not-in-doc
 plugins/modules/web_infrastructure/jenkins_plugin.py use-argspec-type-path
 plugins/modules/web_infrastructure/rundeck_acl_policy.py pylint:blacklisted-name
 plugins/modules/web_infrastructure/sophos_utm/utm_aaa_group.py validate-modules:doc-elements-mismatch

tests/sanity/ignore-2.11.txt

@@ -1155,72 +1155,32 @@ plugins/modules/storage/zfs/zfs_facts.py validate-modules:parameter-type-not-in-
 plugins/modules/storage/zfs/zpool_facts.py validate-modules:nonexistent-parameter-documented
 plugins/modules/storage/zfs/zpool_facts.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/aix_devices.py validate-modules:doc-required-mismatch
-plugins/modules/system/aix_filesystem.py validate-modules:doc-required-mismatch
-plugins/modules/system/aix_filesystem.py validate-modules:parameter-list-no-elements
-plugins/modules/system/aix_inittab.py validate-modules:doc-required-mismatch
-plugins/modules/system/aix_lvg.py validate-modules:parameter-list-no-elements
-plugins/modules/system/aix_lvol.py validate-modules:parameter-list-no-elements
 plugins/modules/system/alternatives.py pylint:blacklisted-name
-plugins/modules/system/awall.py validate-modules:parameter-list-no-elements
 plugins/modules/system/beadm.py pylint:blacklisted-name
 plugins/modules/system/cronvar.py pylint:blacklisted-name
 plugins/modules/system/dconf.py pylint:blacklisted-name
-plugins/modules/system/dconf.py validate-modules:doc-missing-type
-plugins/modules/system/dconf.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/filesystem.py pylint:blacklisted-name
 plugins/modules/system/gconftool2.py pylint:blacklisted-name
 plugins/modules/system/gconftool2.py validate-modules:parameter-state-invalid-choice
-plugins/modules/system/gconftool2.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/interfaces_file.py pylint:blacklisted-name
-plugins/modules/system/interfaces_file.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/iptables_state.py validate-modules:undocumented-parameter
 plugins/modules/system/java_cert.py pylint:blacklisted-name
-plugins/modules/system/java_keystore.py validate-modules:doc-missing-type
-plugins/modules/system/java_keystore.py validate-modules:parameter-type-not-in-doc
-plugins/modules/system/java_keystore.py validate-modules:undocumented-parameter
-plugins/modules/system/kernel_blacklist.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/launchd.py use-argspec-type-path  # False positive
-plugins/modules/system/lbu.py validate-modules:doc-elements-mismatch
-plugins/modules/system/locale_gen.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/lvg.py pylint:blacklisted-name
-plugins/modules/system/lvg.py validate-modules:parameter-list-no-elements
 plugins/modules/system/lvol.py pylint:blacklisted-name
-plugins/modules/system/lvol.py validate-modules:doc-required-mismatch
-plugins/modules/system/lvol.py validate-modules:parameter-type-not-in-doc
-plugins/modules/system/mksysb.py validate-modules:doc-missing-type
-plugins/modules/system/modprobe.py validate-modules:parameter-type-not-in-doc
-plugins/modules/system/nosh.py validate-modules:doc-missing-type
-plugins/modules/system/nosh.py validate-modules:parameter-type-not-in-doc
-plugins/modules/system/open_iscsi.py validate-modules:required_together-unknown
-plugins/modules/system/openwrt_init.py validate-modules:doc-missing-type
-plugins/modules/system/openwrt_init.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/osx_defaults.py validate-modules:doc-required-mismatch
 plugins/modules/system/osx_defaults.py validate-modules:parameter-state-invalid-choice
-plugins/modules/system/pam_limits.py validate-modules:parameter-type-not-in-doc
-plugins/modules/system/pamd.py validate-modules:parameter-list-no-elements
 plugins/modules/system/parted.py pylint:blacklisted-name
-plugins/modules/system/parted.py validate-modules:parameter-list-no-elements
 plugins/modules/system/parted.py validate-modules:parameter-state-invalid-choice
 plugins/modules/system/puppet.py use-argspec-type-path
 plugins/modules/system/puppet.py validate-modules:parameter-invalid
-plugins/modules/system/puppet.py validate-modules:parameter-list-no-elements
 plugins/modules/system/puppet.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/puppet.py validate-modules:undocumented-parameter
-plugins/modules/system/python_requirements_info.py validate-modules:parameter-list-no-elements
-plugins/modules/system/python_requirements_info.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/runit.py validate-modules:doc-default-does-not-match-spec
 plugins/modules/system/runit.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/runit.py validate-modules:undocumented-parameter
-plugins/modules/system/sefcontext.py validate-modules:invalid-ansiblemodule-schema
-plugins/modules/system/selogin.py validate-modules:doc-required-mismatch
-plugins/modules/system/selogin.py validate-modules:parameter-type-not-in-doc
-plugins/modules/system/seport.py validate-modules:parameter-list-no-elements
-plugins/modules/system/solaris_zone.py validate-modules:doc-required-mismatch
-plugins/modules/system/syspatch.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/timezone.py pylint:blacklisted-name
-plugins/modules/system/vdo.py validate-modules:doc-required-mismatch
 plugins/modules/system/xfconf.py validate-modules:parameter-state-invalid-choice
-plugins/modules/system/xfconf.py validate-modules:parameter-type-not-in-doc
 plugins/modules/web_infrastructure/jenkins_plugin.py use-argspec-type-path
 plugins/modules/web_infrastructure/rundeck_acl_policy.py pylint:blacklisted-name
 plugins/modules/web_infrastructure/sophos_utm/utm_aaa_group.py validate-modules:doc-elements-mismatch

tests/sanity/ignore-2.9.txt

@@ -920,42 +920,23 @@ plugins/modules/system/alternatives.py pylint:blacklisted-name
 plugins/modules/system/beadm.py pylint:blacklisted-name
 plugins/modules/system/cronvar.py pylint:blacklisted-name
 plugins/modules/system/dconf.py pylint:blacklisted-name
-plugins/modules/system/dconf.py validate-modules:doc-missing-type
-plugins/modules/system/dconf.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/filesystem.py pylint:blacklisted-name
 plugins/modules/system/gconftool2.py pylint:blacklisted-name
-plugins/modules/system/gconftool2.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/interfaces_file.py pylint:blacklisted-name
-plugins/modules/system/interfaces_file.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/iptables_state.py validate-modules:undocumented-parameter
 plugins/modules/system/java_cert.py pylint:blacklisted-name
-plugins/modules/system/java_keystore.py validate-modules:doc-missing-type
-plugins/modules/system/kernel_blacklist.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/launchd.py use-argspec-type-path  # False positive
-plugins/modules/system/locale_gen.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/lvg.py pylint:blacklisted-name
 plugins/modules/system/lvol.py pylint:blacklisted-name
-plugins/modules/system/lvol.py validate-modules:parameter-type-not-in-doc
-plugins/modules/system/mksysb.py validate-modules:doc-missing-type
-plugins/modules/system/modprobe.py validate-modules:parameter-type-not-in-doc
-plugins/modules/system/nosh.py validate-modules:doc-missing-type
-plugins/modules/system/nosh.py validate-modules:parameter-type-not-in-doc
-plugins/modules/system/openwrt_init.py validate-modules:doc-missing-type
-plugins/modules/system/openwrt_init.py validate-modules:parameter-type-not-in-doc
-plugins/modules/system/pam_limits.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/parted.py pylint:blacklisted-name
 plugins/modules/system/puppet.py use-argspec-type-path
 plugins/modules/system/puppet.py validate-modules:parameter-invalid
 plugins/modules/system/puppet.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/puppet.py validate-modules:undocumented-parameter
-plugins/modules/system/python_requirements_info.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/runit.py validate-modules:doc-default-does-not-match-spec
 plugins/modules/system/runit.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/runit.py validate-modules:undocumented-parameter
-plugins/modules/system/selogin.py validate-modules:parameter-type-not-in-doc
-plugins/modules/system/syspatch.py validate-modules:parameter-type-not-in-doc
 plugins/modules/system/timezone.py pylint:blacklisted-name
-plugins/modules/system/xfconf.py validate-modules:parameter-type-not-in-doc
 plugins/modules/web_infrastructure/jenkins_plugin.py use-argspec-type-path
 plugins/modules/web_infrastructure/nginx_status_facts.py validate-modules:deprecation-mismatch
 plugins/modules/web_infrastructure/nginx_status_facts.py validate-modules:invalid-documentation