cmueller/ansible
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

option is marked as required but specifies a default (#57257)

Browse source 
* required and default are conflicting
This commit is contained in:
xuxiaowei0512 2019-06-05 22:43:46 +08:00 committed by Nathaniel Case
parent 229d20b6d9
commit 05e6339c49
2 changed files with 3 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
lib/ansible/modules/network/cloudengine/ce_reboot.py
View file

@ -34,7 +34,7 @@ options:
description:
- Safeguard boolean. Set to true if you're sure you want to reboot.
type: bool
default: false
required: true
save_config:
description:
- Flag indicating whether to save the configuration.
@ -134,8 +134,8 @@ def main():
""" main """
argument_spec = dict(
confirm=dict(required=True, type='bool', default='false'),
save_config=dict(required=False, type='bool', default='false')
confirm=dict(required=True, type='bool'),
save_config=dict(default=False, type='bool')
)
argument_spec.update(ce_argument_spec)

1
test/sanity/validate-modules/ignore.txt
View file

@ -450,7 +450,6 @@ lib/ansible/modules/network/cloudengine/ce_ntp.py E322
lib/ansible/modules/network/cloudengine/ce_ntp_auth.py E322
lib/ansible/modules/network/cloudengine/ce_ospf.py E322
lib/ansible/modules/network/cloudengine/ce_ospf_vrf.py E322
lib/ansible/modules/network/cloudengine/ce_reboot.py E317
lib/ansible/modules/network/cloudengine/ce_reboot.py E322
lib/ansible/modules/network/cloudengine/ce_rollback.py E322
lib/ansible/modules/network/cloudengine/ce_sflow.py E322