Tidy up validate-modules:doc-required-mismatch (#1415)

* Tidy up validate-modules ignores doc-required-mismatch

* Tidy up validate-modules ignores doc-required-mismatch - update on 2.11

* Fixed chengelog frag

* rolledback removal of parameter from cloud/smartos/vmadm.py

* removed changelog frag for the rollback

* Update plugins/modules/cloud/smartos/vmadm.py

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

* Revert "removed changelog frag for the rollback"

This reverts commit 56a02ead3b.

* suggestion from PR

* yet another PR suggestion

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

plugins/modules/remote_management/cobbler/cobbler_sync.py

@@ -18,18 +18,21 @@ options:
     description:
     - The name or IP address of the Cobbler system.
     default: 127.0.0.1
+    type: str
   port:
     description:
     - Port number to be used for REST connection.
     - The default value depends on parameter C(use_ssl).
+    type: int
   username:
     description:
     - The username to log in to Cobbler.
     default: cobbler
+    type: str
   password:
     description:
     - The password to log in to Cobbler.
-    required: yes
+    type: str
   use_ssl:
     description:
     - If C(no), an HTTP connection will be used instead of the default HTTPS connection.

plugins/modules/remote_management/cobbler/cobbler_system.py

@@ -18,18 +18,21 @@ options:
     description:
     - The name or IP address of the Cobbler system.
     default: 127.0.0.1
+    type: str
   port:
     description:
     - Port number to be used for REST connection.
     - The default value depends on parameter C(use_ssl).
+    type: int
   username:
     description:
     - The username to log in to Cobbler.
     default: cobbler
+    type: str
   password:
     description:
     - The password to log in to Cobbler.
-    required: yes
+    type: str
   use_ssl:
     description:
     - If C(no), an HTTP connection will be used instead of the default HTTPS connection.
@@ -44,12 +47,15 @@ options:
   name:
     description:
     - The system name to manage.
+    type: str
   properties:
     description:
     - A dictionary with system properties.
+    type: dict
   interfaces:
     description:
     - A list of dictionaries containing interface options.
+    type: dict
   sync:
     description:
     - Sync on changes.
@@ -61,6 +67,7 @@ options:
     - Whether the system should be present, absent or a query is made.
     choices: [ absent, present, query ]
     default: present
+    type: str
 author:
 - Dag Wieers (@dagwieers)
 notes:

plugins/modules/remote_management/manageiq/manageiq_provider.py

@@ -32,7 +32,6 @@ options:
   type:
     type: str
     description: The provider's type.
-    required: true
     choices: ['Openshift', 'Amazon', 'oVirt', 'VMware', 'Azure', 'Director', 'OpenStack', 'GCE']
   zone:
     type: str

plugins/modules/remote_management/oneview/oneview_fc_network.py

@@ -21,6 +21,7 @@ options:
               C(present) will ensure data properties are compliant with OneView.
               C(absent) will remove the resource from OneView, if it exists.
         choices: ['present', 'absent']
+        required: true
     data:
         description:
             - List with the Fibre Channel Network properties.