-
Notifications
You must be signed in to change notification settings - Fork 342
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
EC2_instance: add support for controlling metadata options #414
EC2_instance: add support for controlling metadata options #414
Conversation
@mandar242 this PR contains the following merge commits: Please rebase your branch to remove these commits. |
f608eaa
to
d2ae4db
Compare
recheck |
changelogs/fragments/414-ec2_instance-support-controlling-metadata-options.yml
Outdated
Show resolved
Hide resolved
recheck |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution.
With tests it's good to do a four-step test for each 'change'
- Try change in check-mode (assert changed)
- Try change (assert changed and state)
- Repeat change in check-mode (assert not changed)
- Repeat change (assert state and not changed)
tests/integration/targets/ec2_instance/roles/ec2_instance/tasks/metadata_options.yml
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
rds_instance: map Tags parameters from dict to list on restore_db_instance_from_db_snapshot method Reviewed-by: https://github.com/apps/ansible-zuul
rds_instance: map Tags parameters from dict to list on restore_db_instance_from_db_snapshot method Reviewed-by: https://github.com/apps/ansible-zuul
SUMMARY
Adding support for controlling the metadata options, 'Metadata Accessible' and 'Metadata Version' in amazon.aws.ec2_instance.
Fixes #399
ISSUE TYPE
COMPONENT NAME
amazon.aws.ec2_instance