From c5ff49db56e1785693231272e181916388585f0c Mon Sep 17 00:00:00 2001 From: "patchback[bot]" <45432694+patchback[bot]@users.noreply.github.com> Date: Wed, 24 Aug 2022 20:29:08 +0200 Subject: [PATCH] Adjust booleans in misc modules. (#5160) (#5174) (cherry picked from commit 403c4f7477f6f85563af7beb5f45283cfaba59ab) Co-authored-by: Felix Fontein --- .../database/influxdb/influxdb_database.py | 4 +-- .../influxdb/influxdb_retention_policy.py | 12 ++++---- .../database/influxdb/influxdb_user.py | 4 +-- .../database/misc/elasticsearch_plugin.py | 2 +- plugins/modules/database/misc/odbc.py | 8 ++--- plugins/modules/database/misc/riak.py | 6 ++-- plugins/modules/database/mssql/mssql_db.py | 2 +- plugins/modules/files/archive.py | 4 +-- plugins/modules/files/ini_file.py | 30 +++++++++---------- plugins/modules/files/iso_create.py | 4 +-- plugins/modules/files/iso_extract.py | 14 ++++----- plugins/modules/files/read_csv.py | 4 +-- plugins/modules/files/xml.py | 20 ++++++------- plugins/modules/identity/ipa/ipa_group.py | 16 +++++----- .../identity/keycloak/keycloak_client.py | 2 +- .../identity/keycloak/keycloak_realm_info.py | 2 +- .../remote_management/cobbler/cobbler_sync.py | 12 ++++---- .../cobbler/cobbler_system.py | 14 ++++----- .../remote_management/hpilo/hpilo_boot.py | 2 +- .../remote_management/hpilo/hponcfg.py | 2 +- .../modules/remote_management/imc/imc_rest.py | 18 +++++------ .../remote_management/ipmi/ipmi_boot.py | 4 +-- .../manageiq/manageiq_provider.py | 8 ++--- .../remote_management/stacki/stacki_host.py | 2 +- .../modules/storage/hpe3par/ss_3par_cpg.py | 6 ++-- .../modules/storage/zfs/zfs_delegate_admin.py | 10 +++---- plugins/modules/storage/zfs/zfs_facts.py | 8 ++--- 27 files changed, 110 insertions(+), 110 deletions(-) diff --git a/plugins/modules/database/influxdb/influxdb_database.py b/plugins/modules/database/influxdb/influxdb_database.py index 1c245ad6da..8ffbece606 100644 --- a/plugins/modules/database/influxdb/influxdb_database.py +++ b/plugins/modules/database/influxdb/influxdb_database.py @@ -56,8 +56,8 @@ EXAMPLES = r''' username: "{{influxdb_username}}" password: "{{influxdb_password}}" database_name: "{{influxdb_database_name}}" - ssl: yes - validate_certs: yes + ssl: true + validate_certs: true ''' RETURN = r''' diff --git a/plugins/modules/database/influxdb/influxdb_retention_policy.py b/plugins/modules/database/influxdb/influxdb_retention_policy.py index 80f0513678..1b7d7eec9a 100644 --- a/plugins/modules/database/influxdb/influxdb_retention_policy.py +++ b/plugins/modules/database/influxdb/influxdb_retention_policy.py @@ -77,8 +77,8 @@ EXAMPLES = r''' policy_name: test duration: 1h replication: 1 - ssl: yes - validate_certs: yes + ssl: true + validate_certs: true state: present - name: Create 1 day retention policy with 1 hour shard group duration @@ -108,8 +108,8 @@ EXAMPLES = r''' policy_name: test duration: INF replication: 1 - ssl: no - validate_certs: no + ssl: false + validate_certs: false shard_group_duration: 1w state: present @@ -120,8 +120,8 @@ EXAMPLES = r''' policy_name: test duration: 5d1h30m replication: 1 - ssl: no - validate_certs: no + ssl: false + validate_certs: false shard_group_duration: 1d10h30m state: present diff --git a/plugins/modules/database/influxdb/influxdb_user.py b/plugins/modules/database/influxdb/influxdb_user.py index 30b92d9bfe..06ef13e2b3 100644 --- a/plugins/modules/database/influxdb/influxdb_user.py +++ b/plugins/modules/database/influxdb/influxdb_user.py @@ -35,7 +35,7 @@ options: description: - Whether the user should be in the admin role or not. - Since version 2.8, the role will also be updated. - default: no + default: false type: bool state: description: @@ -73,7 +73,7 @@ EXAMPLES = r''' community.general.influxdb_user: user_name: john user_password: s3cr3t - admin: yes + admin: true hostname: "{{ influxdb_hostname }}" login_username: "{{ influxdb_username }}" login_password: "{{ influxdb_password }}" diff --git a/plugins/modules/database/misc/elasticsearch_plugin.py b/plugins/modules/database/misc/elasticsearch_plugin.py index 3bf259f584..12aedc182f 100644 --- a/plugins/modules/database/misc/elasticsearch_plugin.py +++ b/plugins/modules/database/misc/elasticsearch_plugin.py @@ -109,7 +109,7 @@ EXAMPLES = ''' community.general.elasticsearch_plugin: name: ingest-geoip state: present - force: yes + force: true ''' import os diff --git a/plugins/modules/database/misc/odbc.py b/plugins/modules/database/misc/odbc.py index e220543d6f..3b6e8ad226 100644 --- a/plugins/modules/database/misc/odbc.py +++ b/plugins/modules/database/misc/odbc.py @@ -20,12 +20,12 @@ options: dsn: description: - The connection string passed into ODBC. - required: yes + required: true type: str query: description: - The SQL query to perform. - required: yes + required: true type: str params: description: @@ -38,7 +38,7 @@ options: - Some databases allow a commit after a select whereas others raise an exception. - Default is C(true) to support legacy module behavior. type: bool - default: yes + default: true version_added: 1.3.0 requirements: - "python >= 2.6" @@ -58,7 +58,7 @@ EXAMPLES = ''' params: - "value1" commit: false - changed_when: no + changed_when: false ''' RETURN = ''' diff --git a/plugins/modules/database/misc/riak.py b/plugins/modules/database/misc/riak.py index d8679bef2f..38f1c33c78 100644 --- a/plugins/modules/database/misc/riak.py +++ b/plugins/modules/database/misc/riak.py @@ -57,10 +57,10 @@ options: type: str validate_certs: description: - - If C(no), SSL certificates will not be validated. This should only be used + - If C(false), SSL certificates will not be validated. This should only be used on personally controlled sites using self-signed certificates. type: bool - default: 'yes' + default: true ''' EXAMPLES = ''' @@ -71,7 +71,7 @@ EXAMPLES = ''' - name: Wait for handoffs to finish. Use with async and poll. community.general.riak: - wait_for_handoffs: yes + wait_for_handoffs: true - name: Wait for riak_kv service to startup community.general.riak: diff --git a/plugins/modules/database/mssql/mssql_db.py b/plugins/modules/database/mssql/mssql_db.py index ad66fc921c..7e64dcc2cc 100644 --- a/plugins/modules/database/mssql/mssql_db.py +++ b/plugins/modules/database/mssql/mssql_db.py @@ -57,7 +57,7 @@ options: - Automatically commit the change only if the import succeed. Sometimes it is necessary to use autocommit=true, since some content can't be changed within a transaction. type: bool - default: 'no' + default: false notes: - Requires the pymssql Python package on the remote host. For Ubuntu, this is as easy as pip install pymssql (See M(ansible.builtin.pip).) diff --git a/plugins/modules/files/archive.py b/plugins/modules/files/archive.py index c1086d9e9d..9d82017b3c 100644 --- a/plugins/modules/files/archive.py +++ b/plugins/modules/files/archive.py @@ -64,7 +64,7 @@ options: description: - Remove any added source files and trees after adding to archive. type: bool - default: no + default: false notes: - Requires tarfile, zipfile, gzip and bzip2 packages on target host. - Requires lzma or backports.lzma if using xz format. @@ -84,7 +84,7 @@ EXAMPLES = r''' - name: Compress regular file /path/to/foo into /path/to/foo.gz and remove it community.general.archive: path: /path/to/foo - remove: yes + remove: true - name: Create a zip archive of /path/to/foo community.general.archive: diff --git a/plugins/modules/files/ini_file.py b/plugins/modules/files/ini_file.py index f3040632c0..1ad82a5180 100644 --- a/plugins/modules/files/ini_file.py +++ b/plugins/modules/files/ini_file.py @@ -65,44 +65,44 @@ options: - Create a backup file including the timestamp information so you can get the original file back if you somehow clobbered it incorrectly. type: bool - default: no + default: false state: description: - - If set to C(absent) and I(exclusive) set to C(yes) all matching I(option) lines are removed. - - If set to C(absent) and I(exclusive) set to C(no) the specified C(option=value) lines are removed, + - If set to C(absent) and I(exclusive) set to C(true) all matching I(option) lines are removed. + - If set to C(absent) and I(exclusive) set to C(false) the specified C(option=value) lines are removed, but the other I(option)s with the same name are not touched. - - If set to C(present) and I(exclusive) set to C(no) the specified C(option=values) lines are added, + - If set to C(present) and I(exclusive) set to C(false) the specified C(option=values) lines are added, but the other I(option)s with the same name are not touched. - - If set to C(present) and I(exclusive) set to C(yes) all given C(option=values) lines will be + - If set to C(present) and I(exclusive) set to C(true) all given C(option=values) lines will be added and the other I(option)s with the same name are removed. type: str choices: [ absent, present ] default: present exclusive: description: - - If set to C(yes) (default), all matching I(option) lines are removed when I(state=absent), + - If set to C(true) (default), all matching I(option) lines are removed when I(state=absent), or replaced when I(state=present). - - If set to C(no), only the specified I(value(s)) are added when I(state=present), + - If set to C(false), only the specified I(value(s)) are added when I(state=present), or removed when I(state=absent), and existing ones are not modified. type: bool - default: yes + default: true version_added: 3.6.0 no_extra_spaces: description: - Do not insert spaces before and after '=' symbol. type: bool - default: no + default: false create: description: - - If set to C(no), the module will fail if the file does not already exist. + - If set to C(false), the module will fail if the file does not already exist. - By default it will create the file if it is missing. type: bool - default: yes + default: true allow_no_value: description: - Allow option without value and without '=' symbol. type: bool - default: no + default: false notes: - While it is possible to add an I(option) without specifying a I(value), this makes no sense. - As of Ansible 2.3, the I(dest) option has been changed to I(path) as default, but I(dest) still works as well. @@ -121,7 +121,7 @@ EXAMPLES = r''' option: fav value: lemonade mode: '0600' - backup: yes + backup: true - name: Ensure "temperature=cold is in section "[drinks]" in specified file community.general.ini_file: @@ -129,7 +129,7 @@ EXAMPLES = r''' section: drinks option: temperature value: cold - backup: yes + backup: true - name: Add "beverage=lemon juice" is in section "[drinks]" in specified file community.general.ini_file: @@ -139,7 +139,7 @@ EXAMPLES = r''' value: lemon juice mode: '0600' state: present - exclusive: no + exclusive: false - name: Ensure multiple values "beverage=coke" and "beverage=pepsi" are in section "[drinks]" in specified file community.general.ini_file: diff --git a/plugins/modules/files/iso_create.py b/plugins/modules/files/iso_create.py index 275b62a66b..1340ab9ce4 100644 --- a/plugins/modules/files/iso_create.py +++ b/plugins/modules/files/iso_create.py @@ -31,14 +31,14 @@ options: underscores (_). File names are limited to 31 characters, directory nesting is limited to 8 levels, and path names are limited to 255 characters.' type: list - required: yes + required: true elements: path dest_iso: description: - The absolute path with file name of the new generated ISO file on local machine. - Will create intermediate folders when they does not exist. type: path - required: yes + required: true interchange_level: description: - The ISO9660 interchange level to use, it dictates the rules on the names of files. diff --git a/plugins/modules/files/iso_extract.py b/plugins/modules/files/iso_extract.py index 904997a752..fb88f09882 100644 --- a/plugins/modules/files/iso_extract.py +++ b/plugins/modules/files/iso_extract.py @@ -35,26 +35,26 @@ options: description: - The ISO image to extract files from. type: path - required: yes + required: true aliases: [ path, src ] dest: description: - The destination directory to extract files to. type: path - required: yes + required: true files: description: - A list of files to extract from the image. - Extracting directories does not work. type: list elements: str - required: yes + required: true force: description: - - If C(yes), which will replace the remote file when contents are different than the source. - - If C(no), the file will only be extracted and copied if the destination does not already exist. + - If C(true), which will replace the remote file when contents are different than the source. + - If C(false), the file will only be extracted and copied if the destination does not already exist. type: bool - default: yes + default: true executable: description: - The path to the C(7z) executable to use for extracting files from the ISO. @@ -62,7 +62,7 @@ options: type: path notes: - Only the file checksum (content) is taken into account when extracting files - from the ISO image. If C(force=no), only checks the presence of the file. + from the ISO image. If C(force=false), only checks the presence of the file. - In Ansible 2.3 this module was using C(mount) and C(umount) commands only, requiring root access. This is no longer needed with the introduction of 7zip for extraction. diff --git a/plugins/modules/files/read_csv.py b/plugins/modules/files/read_csv.py index c6b8b9fe90..38c91ded07 100644 --- a/plugins/modules/files/read_csv.py +++ b/plugins/modules/files/read_csv.py @@ -21,7 +21,7 @@ options: description: - The CSV filename to read data from. type: path - required: yes + required: true aliases: [ filename ] key: description: @@ -45,7 +45,7 @@ options: description: - Whether the C(key) used is expected to be unique. type: bool - default: yes + default: true delimiter: description: - A one-character string used to separate fields. diff --git a/plugins/modules/files/xml.py b/plugins/modules/files/xml.py index d5a1fbfb78..1f8f50c4a4 100644 --- a/plugins/modules/files/xml.py +++ b/plugins/modules/files/xml.py @@ -81,18 +81,18 @@ options: - Search for a given C(xpath) and provide the count of any matches. - This parameter requires C(xpath) to be set. type: bool - default: no + default: false print_match: description: - Search for a given C(xpath) and print out any matches. - This parameter requires C(xpath) to be set. type: bool - default: no + default: false pretty_print: description: - Pretty print XML output. type: bool - default: no + default: false content: description: - Search for a given C(xpath) and get content. @@ -110,13 +110,13 @@ options: - Create a backup file including the timestamp information so you can get the original file back if you somehow clobbered it incorrectly. type: bool - default: no + default: false strip_cdata_tags: description: - Remove CDATA tags surrounding text values. - Note that this might break your XML file if text values contain characters that could be interpreted as XML. type: bool - default: no + default: false insertbefore: description: - Add additional child-element(s) before the first selected element for a given C(xpath). @@ -125,7 +125,7 @@ options: or a hash where the key is an element name and the value is the element value. - This parameter requires C(xpath) to be set. type: bool - default: no + default: false insertafter: description: - Add additional child-element(s) after the last selected element for a given C(xpath). @@ -134,7 +134,7 @@ options: or a hash where the key is an element name and the value is the element value. - This parameter requires C(xpath) to be set. type: bool - default: no + default: false requirements: - lxml >= 2.3.0 notes: @@ -193,7 +193,7 @@ EXAMPLES = r''' community.general.xml: path: /foo/bar.xml xpath: /business/beers/beer - count: yes + count: true register: hits - ansible.builtin.debug: @@ -219,7 +219,7 @@ EXAMPLES = r''' community.general.xml: path: /foo/bar.xml xpath: '/business/beers/beer[text()="Rochefort 10"]' - insertbefore: yes + insertbefore: true add_children: - beer: Old Rasputin - beer: Old Motor Oil @@ -330,7 +330,7 @@ actions: backup_file: description: The name of the backup file that was created type: str - returned: when backup=yes + returned: when I(backup=true) sample: /path/to/file.xml.1942.2017-08-24@14:16:01~ count: description: The count of xpath matches. diff --git a/plugins/modules/identity/ipa/ipa_group.py b/plugins/modules/identity/ipa/ipa_group.py index 9919a601f4..fa3c610d90 100644 --- a/plugins/modules/identity/ipa/ipa_group.py +++ b/plugins/modules/identity/ipa/ipa_group.py @@ -17,9 +17,9 @@ description: options: append: description: - - If C(yes), add the listed I(user) and I(group) to the group members. - - If C(no), only the listed I(user) and I(group) will be group members, removing any other members. - default: no + - If C(true), add the listed I(user) and I(group) to the group members. + - If C(false), only the listed I(user) and I(group) will be group members, removing any other members. + default: false type: bool version_added: 4.0.0 cn: @@ -45,9 +45,9 @@ options: group: description: - List of group names assigned to this group. - - If I(append=no) and an empty list is passed all groups will be removed from this group. + - If I(append=false) and an empty list is passed all groups will be removed from this group. - Groups that are already assigned but not passed will be removed. - - If I(append=yes) the listed groups will be assigned without removing other groups. + - If I(append=true) the listed groups will be assigned without removing other groups. - If option is omitted assigned groups will not be checked or changed. type: list elements: str @@ -58,9 +58,9 @@ options: user: description: - List of user names assigned to this group. - - If I(append=no) and an empty list is passed all users will be removed from this group. + - If I(append=false) and an empty list is passed all users will be removed from this group. - Users that are already assigned but not passed will be removed. - - If I(append=yes) the listed users will be assigned without removing other users. + - If I(append=true) the listed users will be assigned without removing other users. - If option is omitted assigned users will not be checked or changed. type: list elements: str @@ -110,7 +110,7 @@ EXAMPLES = r''' name: developers user: - john - append: yes + append: true state: present ipa_host: ipa.example.com ipa_user: admin diff --git a/plugins/modules/identity/keycloak/keycloak_client.py b/plugins/modules/identity/keycloak/keycloak_client.py index 547f6fd5fa..4fffeb2955 100644 --- a/plugins/modules/identity/keycloak/keycloak_client.py +++ b/plugins/modules/identity/keycloak/keycloak_client.py @@ -620,7 +620,7 @@ EXAMPLES = ''' client_template: test use_template_config: False use_template_scope: false - use_template_mappers: no + use_template_mappers: false always_display_in_console: true registered_nodes: node01.example.com: 1507828202 diff --git a/plugins/modules/identity/keycloak/keycloak_realm_info.py b/plugins/modules/identity/keycloak/keycloak_realm_info.py index 4214ba06be..0956f1b525 100644 --- a/plugins/modules/identity/keycloak/keycloak_realm_info.py +++ b/plugins/modules/identity/keycloak/keycloak_realm_info.py @@ -38,7 +38,7 @@ options: description: - Verify TLS certificates (do not disable this in production). type: bool - default: yes + default: true realm: type: str diff --git a/plugins/modules/remote_management/cobbler/cobbler_sync.py b/plugins/modules/remote_management/cobbler/cobbler_sync.py index fd1f4010ad..5e7082ddf5 100644 --- a/plugins/modules/remote_management/cobbler/cobbler_sync.py +++ b/plugins/modules/remote_management/cobbler/cobbler_sync.py @@ -36,15 +36,15 @@ options: type: str use_ssl: description: - - If C(no), an HTTP connection will be used instead of the default HTTPS connection. + - If C(false), an HTTP connection will be used instead of the default HTTPS connection. type: bool - default: 'yes' + default: true validate_certs: description: - - If C(no), SSL certificates will not be validated. - - This should only set to C(no) when used on personally controlled sites using self-signed certificates. + - If C(false), SSL certificates will not be validated. + - This should only set to C(false) when used on personally controlled sites using self-signed certificates. type: bool - default: 'yes' + default: true author: - Dag Wieers (@dagwieers) todo: @@ -60,7 +60,7 @@ EXAMPLES = r''' host: cobbler01 username: cobbler password: MySuperSecureP4sswOrd - run_once: yes + run_once: true delegate_to: localhost ''' diff --git a/plugins/modules/remote_management/cobbler/cobbler_system.py b/plugins/modules/remote_management/cobbler/cobbler_system.py index b824f5f4a4..7b70ccac34 100644 --- a/plugins/modules/remote_management/cobbler/cobbler_system.py +++ b/plugins/modules/remote_management/cobbler/cobbler_system.py @@ -36,15 +36,15 @@ options: type: str use_ssl: description: - - If C(no), an HTTP connection will be used instead of the default HTTPS connection. + - If C(false), an HTTP connection will be used instead of the default HTTPS connection. type: bool - default: 'yes' + default: true validate_certs: description: - - If C(no), SSL certificates will not be validated. - - This should only set to C(no) when used on personally controlled sites using self-signed certificates. + - If C(false), SSL certificates will not be validated. + - This should only set to C(false) when used on personally controlled sites using self-signed certificates. type: bool - default: 'yes' + default: true name: description: - The system name to manage. @@ -62,7 +62,7 @@ options: - Sync on changes. - Concurrently syncing Cobbler is bound to fail. type: bool - default: no + default: false state: description: - Whether the system should be present, absent or a query is made. @@ -101,7 +101,7 @@ EXAMPLES = r''' password: ins3965! name: bdsol-aci51-apic1.cisco.com properties: - netboot_enabled: yes + netboot_enabled: true state: present delegate_to: localhost diff --git a/plugins/modules/remote_management/hpilo/hpilo_boot.py b/plugins/modules/remote_management/hpilo/hpilo_boot.py index 483b82554c..f663a7b5f5 100644 --- a/plugins/modules/remote_management/hpilo/hpilo_boot.py +++ b/plugins/modules/remote_management/hpilo/hpilo_boot.py @@ -63,7 +63,7 @@ options: description: - Whether to force a reboot (even when the system is already booted). - As a safeguard, without force, hpilo_boot will refuse to reboot a server that is already running. - default: no + default: false type: bool ssl_version: description: diff --git a/plugins/modules/remote_management/hpilo/hponcfg.py b/plugins/modules/remote_management/hpilo/hponcfg.py index eeecf9a35b..9d2afc0ab1 100644 --- a/plugins/modules/remote_management/hpilo/hponcfg.py +++ b/plugins/modules/remote_management/hpilo/hponcfg.py @@ -36,7 +36,7 @@ options: verbose: description: - Run hponcfg in verbose mode (-v). - default: no + default: false type: bool requirements: - hponcfg tool diff --git a/plugins/modules/remote_management/imc/imc_rest.py b/plugins/modules/remote_management/imc/imc_rest.py index c4ffce3002..4259558053 100644 --- a/plugins/modules/remote_management/imc/imc_rest.py +++ b/plugins/modules/remote_management/imc/imc_rest.py @@ -71,10 +71,10 @@ options: type: int validate_certs: description: - - If C(no), SSL certificates will not be validated. - - This should only set to C(no) used on personally controlled sites using self-signed certificates. + - If C(false), SSL certificates will not be validated. + - This should only set to C(false) used on personally controlled sites using self-signed certificates. type: bool - default: 'yes' + default: true notes: - The XML fragments don't need an authentication cookie, this is injected by the module automatically. - The Cisco IMC XML output is being translated to JSON using the Cobra convention. @@ -93,7 +93,7 @@ EXAMPLES = r''' hostname: '{{ imc_hostname }}' username: '{{ imc_username }}' password: '{{ imc_password }}' - validate_certs: no + validate_certs: false content: | @@ -105,7 +105,7 @@ EXAMPLES = r''' hostname: '{{ imc_hostname }}' username: '{{ imc_username }}' password: '{{ imc_password }}' - validate_certs: no + validate_certs: false timeout: 120 content: | @@ -130,7 +130,7 @@ EXAMPLES = r''' hostname: '{{ imc_hostname }}' username: '{{ imc_username }}' password: '{{ imc_password }}' - validate_certs: no + validate_certs: false content: | @@ -148,7 +148,7 @@ EXAMPLES = r''' hostname: '{{ imc_host }}' username: '{{ imc_username }}' password: '{{ imc_password }}' - validate_certs: no + validate_certs: false content: | @@ -160,7 +160,7 @@ EXAMPLES = r''' hostname: '{{ imc_host }}' username: '{{ imc_username }}' password: '{{ imc_password }}' - validate_certs: no + validate_certs: false content: | @@ -172,7 +172,7 @@ EXAMPLES = r''' hostname: '{{ imc_host }}' username: '{{ imc_username }}' password: '{{ imc_password }}' - validate_certs: no + validate_certs: false timeout: 120 content: | diff --git a/plugins/modules/remote_management/ipmi/ipmi_boot.py b/plugins/modules/remote_management/ipmi/ipmi_boot.py index 48f9116181..3cfa60d18e 100644 --- a/plugins/modules/remote_management/ipmi/ipmi_boot.py +++ b/plugins/modules/remote_management/ipmi/ipmi_boot.py @@ -77,14 +77,14 @@ options: - If set, ask that system firmware uses this device beyond next boot. Be aware many systems do not honor this. type: bool - default: 'no' + default: false uefiboot: description: - If set, request UEFI boot explicitly. Strictly speaking, the spec suggests that if not set, the system should BIOS boot and offers no "don't care" option. In practice, this flag not being set does not preclude UEFI boot on any system I've encountered. type: bool - default: 'no' + default: false requirements: - "python >= 2.6" - pyghmi diff --git a/plugins/modules/remote_management/manageiq/manageiq_provider.py b/plugins/modules/remote_management/manageiq/manageiq_provider.py index f33176e15a..50565936f4 100644 --- a/plugins/modules/remote_management/manageiq/manageiq_provider.py +++ b/plugins/modules/remote_management/manageiq/manageiq_provider.py @@ -59,7 +59,7 @@ options: aliases: [ keystone_v3_domain_id ] tenant_mapping_enabled: type: bool - default: 'no' + default: false description: Whether to enable mapping of existing tenants. defaults to False. api_version: type: str @@ -88,7 +88,7 @@ options: validate_certs: description: Whether SSL certificates should be verified for HTTPS requests (deprecated). defaults to True. type: bool - default: 'yes' + default: true security_protocol: type: str description: How SSL certificates should be used for HTTPS requests. defaults to None. @@ -119,7 +119,7 @@ options: validate_certs: description: Whether SSL certificates should be verified for HTTPS requests (deprecated). defaults to True. type: bool - default: 'yes' + default: true security_protocol: type: str choices: ['ssl-with-validation','ssl-with-validation-custom-ca','ssl-without-validation','non-ssl'] @@ -179,7 +179,7 @@ options: description: - Whether certificates should be verified for connections. type: bool - default: yes + default: true aliases: [ verify_ssl ] ''' diff --git a/plugins/modules/remote_management/stacki/stacki_host.py b/plugins/modules/remote_management/stacki/stacki_host.py index 9a13066865..9b46698a02 100644 --- a/plugins/modules/remote_management/stacki/stacki_host.py +++ b/plugins/modules/remote_management/stacki/stacki_host.py @@ -56,7 +56,7 @@ options: description: - Set value to C(true) to force node into install state if it already exists in stacki. type: bool - default: no + default: false state: description: - Set value to the desired state for the specified host. diff --git a/plugins/modules/storage/hpe3par/ss_3par_cpg.py b/plugins/modules/storage/hpe3par/ss_3par_cpg.py index e8a88fc4f4..f692ff3ecb 100644 --- a/plugins/modules/storage/hpe3par/ss_3par_cpg.py +++ b/plugins/modules/storage/hpe3par/ss_3par_cpg.py @@ -85,7 +85,7 @@ options: description: - Specifies whether the certificate needs to be validated while communicating. type: bool - default: no + default: false extends_documentation_fragment: - community.general.hpe3par @@ -108,7 +108,7 @@ EXAMPLES = r''' set_size: 8 high_availability: MAG disk_type: FC - secure: no + secure: false - name: Delete CPG sample_cpg community.general.ss_3par_cpg: @@ -117,7 +117,7 @@ EXAMPLES = r''' storage_system_password: password state: absent cpg_name: sample_cpg - secure: no + secure: false ''' RETURN = r''' diff --git a/plugins/modules/storage/zfs/zfs_delegate_admin.py b/plugins/modules/storage/zfs/zfs_delegate_admin.py index df95d0db3d..36a213a0fe 100644 --- a/plugins/modules/storage/zfs/zfs_delegate_admin.py +++ b/plugins/modules/storage/zfs/zfs_delegate_admin.py @@ -48,7 +48,7 @@ options: description: - Apply permissions to everyone. type: bool - default: no + default: false permissions: description: - The list of permission(s) to delegate (required if C(state) is C(present)). @@ -68,7 +68,7 @@ options: description: - Unallow permissions recursively (ignored when C(state) is C(present)). type: bool - default: no + default: false author: - Nate Coraor (@natefoo) ''' @@ -84,7 +84,7 @@ EXAMPLES = r''' community.general.zfs_delegate_admin: name: rpool/myvol groups: backup - everyone: yes + everyone: true permissions: send - name: Grant `zfs send,receive` to users `foo` and `bar` with local scope only @@ -92,12 +92,12 @@ EXAMPLES = r''' name: rpool/myfs users: foo,bar permissions: send,receive - local: yes + local: true - name: Revoke all permissions from everyone (permissions specifically assigned to users and groups remain) community.general.zfs_delegate_admin: name: rpool/myfs - everyone: yes + everyone: true state: absent ''' diff --git a/plugins/modules/storage/zfs/zfs_facts.py b/plugins/modules/storage/zfs/zfs_facts.py index 41342472c0..2b11f8a090 100644 --- a/plugins/modules/storage/zfs/zfs_facts.py +++ b/plugins/modules/storage/zfs/zfs_facts.py @@ -20,7 +20,7 @@ options: name: description: - ZFS dataset name. - required: yes + required: true aliases: [ "ds", "dataset" ] type: str recurse: @@ -28,13 +28,13 @@ options: - Specifies if properties for any children should be recursively displayed. type: bool - default: 'no' + default: false parsable: description: - Specifies if property values should be displayed in machine friendly format. type: bool - default: 'no' + default: false properties: description: - Specifies which dataset properties should be queried in comma-separated format. @@ -62,7 +62,7 @@ EXAMPLES = ''' - name: Report space usage on ZFS filesystems under data/home community.general.zfs_facts: name: data/home - recurse: yes + recurse: true type: filesystem - ansible.builtin.debug: