iam: Make 'name' required to match the documentation (#65557)
With the exception of deleting Users and Roles (where we'd do nothing and exit) not passing a name would trigger a boto exception.
This commit is contained in:
parent
eac7d2a958
commit
cf3f1b9d3f
3 changed files with 4 additions and 2 deletions
2
changelogs/fragments/65557-iam-make-name-required.yml
Normal file
2
changelogs/fragments/65557-iam-make-name-required.yml
Normal file
|
@ -0,0 +1,2 @@
|
|||
minor_changes:
|
||||
- iam - make `name` a required field.
|
|
@ -24,6 +24,7 @@ options:
|
|||
- Type of IAM resource.
|
||||
choices: ["user", "group", "role"]
|
||||
type: str
|
||||
required: true
|
||||
name:
|
||||
description:
|
||||
- Name of IAM resource to create or identify.
|
||||
|
@ -632,7 +633,7 @@ def main():
|
|||
'Active', 'Inactive', 'Create', 'Remove']),
|
||||
access_key_ids=dict(type='list', default=None, required=False),
|
||||
key_count=dict(type='int', default=1, required=False),
|
||||
name=dict(default=None, required=False),
|
||||
name=dict(required=True),
|
||||
trust_policy_filepath=dict(default=None, required=False),
|
||||
trust_policy=dict(type='dict', default=None, required=False),
|
||||
new_name=dict(default=None, required=False),
|
||||
|
|
|
@ -492,7 +492,6 @@ lib/ansible/modules/cloud/alicloud/ali_instance.py validate-modules:parameter-ty
|
|||
lib/ansible/modules/cloud/alicloud/ali_instance_info.py validate-modules:doc-missing-type
|
||||
lib/ansible/modules/cloud/alicloud/ali_instance_info.py validate-modules:doc-required-mismatch
|
||||
lib/ansible/modules/cloud/alicloud/ali_instance_info.py validate-modules:parameter-type-not-in-doc
|
||||
lib/ansible/modules/cloud/amazon/iam.py validate-modules:doc-required-mismatch
|
||||
lib/ansible/modules/cloud/amazon/iam_cert.py validate-modules:doc-required-mismatch
|
||||
lib/ansible/modules/cloud/amazon/iam_policy.py validate-modules:doc-required-mismatch
|
||||
lib/ansible/modules/cloud/atomic/atomic_container.py validate-modules:doc-missing-type
|
||||
|
|
Loading…
Reference in a new issue