From 5036d1742c8ca73650bd41a006eafc78644182de Mon Sep 17 00:00:00 2001 From: David Wittman Date: Sun, 27 Mar 2016 15:30:27 -0500 Subject: [PATCH] Fix boolean choices in module docs fragments A few of the docs fragments have the available choices for some params defined as "BOOLEANS". Because choices accepts a list, it treats "BOOLEANS" as an iterable and then generates a list composed of each letter. Instead, define the available choices as a list of `['yes', 'no']`, as is common in most other modules. --- lib/ansible/utils/module_docs_fragments/eos.py | 8 ++++---- lib/ansible/utils/module_docs_fragments/ios.py | 4 ++-- lib/ansible/utils/module_docs_fragments/nxos.py | 4 ++-- lib/ansible/utils/module_docs_fragments/openswitch.py | 4 ++-- 4 files changed, 10 insertions(+), 10 deletions(-) diff --git a/lib/ansible/utils/module_docs_fragments/eos.py b/lib/ansible/utils/module_docs_fragments/eos.py index bd8d3f510e..6502710707 100644 --- a/lib/ansible/utils/module_docs_fragments/eos.py +++ b/lib/ansible/utils/module_docs_fragments/eos.py @@ -56,8 +56,8 @@ options: before sending any commands. If not specified, the device will attempt to excecute all commands in non-priviledged mode. required: false - default: false - choices: BOOLEANS + default: no + choices: ['yes', 'no'] auth_pass: description: - Specifies the password to use if required to enter privileged mode @@ -78,8 +78,8 @@ options: I(transport) argument is configured as eapi. If the transport argument is not eapi, this value is ignored required: false - default: true - choices: BOOLEANS + default: yes + choices: ['yes', 'no'] provider: description: - Convience method that allows all M(eos) arguments to be passed as diff --git a/lib/ansible/utils/module_docs_fragments/ios.py b/lib/ansible/utils/module_docs_fragments/ios.py index 66ba28ad02..4b6e53fc0c 100644 --- a/lib/ansible/utils/module_docs_fragments/ios.py +++ b/lib/ansible/utils/module_docs_fragments/ios.py @@ -54,8 +54,8 @@ options: before sending any commands. If not specified, the device will attempt to excecute all commands in non-priviledged mode. required: false - default: false - choices: BOOLEANS + default: no + choices: ['yes', 'no'] auth_pass: description: - Specifies the password to use if required to enter privileged mode diff --git a/lib/ansible/utils/module_docs_fragments/nxos.py b/lib/ansible/utils/module_docs_fragments/nxos.py index 26312155c4..d2d5343b39 100644 --- a/lib/ansible/utils/module_docs_fragments/nxos.py +++ b/lib/ansible/utils/module_docs_fragments/nxos.py @@ -63,8 +63,8 @@ options: I(transport) argument is configured as nxapi. If the transport argument is not nxapi, this value is ignored required: false - default: false - choices: BOOLEANS + default: no + choices: ['yes', 'no'] provider: description: - Convience method that allows all M(nxos) arguments to be passed as diff --git a/lib/ansible/utils/module_docs_fragments/openswitch.py b/lib/ansible/utils/module_docs_fragments/openswitch.py index 3b3dbcaecc..7a223ce761 100644 --- a/lib/ansible/utils/module_docs_fragments/openswitch.py +++ b/lib/ansible/utils/module_docs_fragments/openswitch.py @@ -68,8 +68,8 @@ options: I(transport) argument is configured as rest. If the transport argument is not rest, this value is ignored required: false - default: true - choices: BOOLEANS + default: yes + choices: ['yes', 'no'] provider: description: - Convience method that allows all M(openswitch) arguments to be passed as