diff --git a/changelogs/fragments/57750-option-is-marked-as-required-but-specifies-a-default.yml b/changelogs/fragments/57750-option-is-marked-as-required-but-specifies-a-default.yml new file mode 100644 index 00000000000..0601fada99c --- /dev/null +++ b/changelogs/fragments/57750-option-is-marked-as-required-but-specifies-a-default.yml @@ -0,0 +1,2 @@ +bugfixes: + - option is marked as required but specifies a default.(https://github.com/ansible/ansible/pull/57257) diff --git a/lib/ansible/modules/network/cloudengine/ce_reboot.py b/lib/ansible/modules/network/cloudengine/ce_reboot.py index e8b8cb0f839..040f103b4be 100644 --- a/lib/ansible/modules/network/cloudengine/ce_reboot.py +++ b/lib/ansible/modules/network/cloudengine/ce_reboot.py @@ -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) diff --git a/test/sanity/validate-modules/ignore.txt b/test/sanity/validate-modules/ignore.txt index 66c794a9885..a565b7683e9 100644 --- a/test/sanity/validate-modules/ignore.txt +++ b/test/sanity/validate-modules/ignore.txt @@ -464,7 +464,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