Skip to content
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 modifying instance metadata options #1918

Conversation

mandar242
Copy link
Contributor

SUMMARY

Fixes #1864
This PR add support for modifying instance metadata options.
Uses client. modify_instance_metadata_options()

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

ec2_instance

ADDITIONAL INFORMATION

@mandar242 mandar242 changed the title ec2_instance: add support for modifying instance metadata options [WIP] ec2_instance: add support for modifying instance metadata options Dec 11, 2023

This comment was marked as outdated.

@mandar242 mandar242 force-pushed the ec2_instance_metadata_options_update branch from 2f8446c to 58c5c67 Compare December 12, 2023 15:31

This comment was marked as outdated.

@mandar242 mandar242 force-pushed the ec2_instance_metadata_options_update branch from 58c5c67 to ce4a479 Compare December 13, 2023 05:06
@mandar242 mandar242 changed the title [WIP] ec2_instance: add support for modifying instance metadata options ec2_instance: add support for modifying instance metadata options Dec 13, 2023

This comment was marked as outdated.

This comment was marked as outdated.

@mandar242
Copy link
Contributor Author

recheck

This comment was marked as outdated.

@mandar242
Copy link
Contributor Author

recheck

This comment was marked as outdated.

@mandar242
Copy link
Contributor Author

recheck

Copy link
Contributor

Build failed.
https://ansible.softwarefactory-project.io/zuul/buildset/1e8741480b494369a480b00bf763820b

✔️ ansible-galaxy-importer SUCCESS in 4m 10s
✔️ build-ansible-collection SUCCESS in 15m 12s
✔️ ansible-test-splitter SUCCESS in 5m 40s
✔️ integration-amazon.aws-1 SUCCESS in 26m 59s
✔️ integration-amazon.aws-2 SUCCESS in 19m 59s
✔️ integration-amazon.aws-3 SUCCESS in 11m 20s
✔️ integration-amazon.aws-4 SUCCESS in 13m 54s
✔️ integration-amazon.aws-5 SUCCESS in 12m 21s
✔️ integration-amazon.aws-6 SUCCESS in 9m 30s
✔️ integration-amazon.aws-7 SUCCESS in 9m 52s
✔️ integration-amazon.aws-8 SUCCESS in 14m 44s
✔️ integration-amazon.aws-9 SUCCESS in 14m 22s
✔️ integration-amazon.aws-10 SUCCESS in 12m 36s
✔️ integration-amazon.aws-11 SUCCESS in 8m 12s
✔️ integration-amazon.aws-12 SUCCESS in 13m 02s
✔️ integration-amazon.aws-13 SUCCESS in 12m 06s
integration-amazon.aws-14 FAILURE in 17m 36s
✔️ integration-amazon.aws-15 SUCCESS in 11m 12s
✔️ integration-amazon.aws-16 SUCCESS in 19m 33s
✔️ integration-amazon.aws-17 SUCCESS in 8m 56s
✔️ integration-amazon.aws-18 SUCCESS in 11m 03s
integration-amazon.aws-19 RETRY_LIMIT in 1m 43s
Skipped 25 jobs

@mandar242 mandar242 force-pushed the ec2_instance_metadata_options_update branch from e8fdc3a to 3e10c43 Compare December 18, 2023 13:52
Copy link
Contributor

@tremble tremble left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please leave the inventory plugin and ignore files as they were.

The 2.17 ignore entry can be removed when the next milestone is released.

@mandar242 mandar242 force-pushed the ec2_instance_metadata_options_update branch from b751ea5 to 3e10c43 Compare December 18, 2023 14:28
@mandar242
Copy link
Contributor Author

Please leave the inventory plugin and ignore files as they were.

The 2.17 ignore entry can be removed when the next milestone is released.

Reverted.

@mandar242 mandar242 force-pushed the ec2_instance_metadata_options_update branch from 3e10c43 to 71a0b12 Compare December 18, 2023 14:32

This comment was marked as outdated.

Copy link
Contributor

Build failed.
https://ansible.softwarefactory-project.io/zuul/buildset/93ae95386151466faa233ca47f6484eb

✔️ ansible-galaxy-importer SUCCESS in 5m 07s
✔️ build-ansible-collection SUCCESS in 14m 28s
✔️ ansible-test-splitter SUCCESS in 5m 29s
✔️ integration-amazon.aws-1 SUCCESS in 23m 31s
✔️ integration-amazon.aws-2 SUCCESS in 13m 03s
✔️ integration-amazon.aws-3 SUCCESS in 12m 25s
✔️ integration-amazon.aws-4 SUCCESS in 7m 25s
✔️ integration-amazon.aws-5 SUCCESS in 12m 12s
✔️ integration-amazon.aws-6 SUCCESS in 8m 25s
✔️ integration-amazon.aws-7 SUCCESS in 7m 53s
✔️ integration-amazon.aws-8 SUCCESS in 13m 24s
✔️ integration-amazon.aws-9 SUCCESS in 11m 05s
✔️ integration-amazon.aws-10 SUCCESS in 10m 07s
integration-amazon.aws-11 RETRY_LIMIT in 1m 47s
✔️ integration-amazon.aws-12 SUCCESS in 11m 19s
✔️ integration-amazon.aws-13 SUCCESS in 13m 00s
integration-amazon.aws-14 FAILURE in 10m 00s
✔️ integration-amazon.aws-15 SUCCESS in 10m 44s
✔️ integration-amazon.aws-16 SUCCESS in 14m 13s
✔️ integration-amazon.aws-17 SUCCESS in 12m 16s
✔️ integration-amazon.aws-18 SUCCESS in 8m 39s
✔️ integration-amazon.aws-19 SUCCESS in 11m 01s
Skipped 25 jobs

@mandar242
Copy link
Contributor Author

recheck

This comment was marked as outdated.

plugins/modules/ec2_instance.py Outdated Show resolved Hide resolved

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

Copy link
Contributor

Build failed.
https://ansible.softwarefactory-project.io/zuul/buildset/516b47a77e4f481c977c4743823b3308

✔️ ansible-galaxy-importer SUCCESS in 4m 47s
✔️ build-ansible-collection SUCCESS in 13m 55s
✔️ ansible-test-splitter SUCCESS in 5m 39s
✔️ integration-amazon.aws-1 SUCCESS in 24m 10s
✔️ integration-amazon.aws-2 SUCCESS in 18m 46s
✔️ integration-amazon.aws-3 SUCCESS in 10m 52s
✔️ integration-amazon.aws-4 SUCCESS in 9m 51s
✔️ integration-amazon.aws-5 SUCCESS in 10m 54s
✔️ integration-amazon.aws-6 SUCCESS in 8m 53s
✔️ integration-amazon.aws-7 SUCCESS in 9m 43s
✔️ integration-amazon.aws-8 SUCCESS in 15m 13s
✔️ integration-amazon.aws-9 SUCCESS in 13m 54s
✔️ integration-amazon.aws-10 SUCCESS in 14m 19s
✔️ integration-amazon.aws-11 SUCCESS in 12m 13s
integration-amazon.aws-12 RETRY_LIMIT in 7m 47s
integration-amazon.aws-13 RETRY_LIMIT in 7m 48s
✔️ integration-amazon.aws-14 SUCCESS in 7m 28s
✔️ integration-amazon.aws-15 SUCCESS in 7m 20s
✔️ integration-amazon.aws-16 SUCCESS in 11m 56s
✔️ integration-amazon.aws-17 SUCCESS in 9m 29s
✔️ integration-amazon.aws-18 SUCCESS in 10m 34s
✔️ integration-amazon.aws-19 SUCCESS in 11m 49s
Skipped 25 jobs

@mandar242
Copy link
Contributor Author

recheck

Copy link
Contributor

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/514265d1bdae43fdaf9231cb323d987d

✔️ ansible-galaxy-importer SUCCESS in 4m 32s
✔️ build-ansible-collection SUCCESS in 14m 45s
✔️ ansible-test-splitter SUCCESS in 5m 09s
✔️ integration-amazon.aws-1 SUCCESS in 19m 21s
✔️ integration-amazon.aws-2 SUCCESS in 16m 41s
✔️ integration-amazon.aws-3 SUCCESS in 12m 22s
✔️ integration-amazon.aws-4 SUCCESS in 8m 49s
✔️ integration-amazon.aws-5 SUCCESS in 8m 29s
✔️ integration-amazon.aws-6 SUCCESS in 6m 30s
✔️ integration-amazon.aws-7 SUCCESS in 6m 40s
✔️ integration-amazon.aws-8 SUCCESS in 7m 52s
✔️ integration-amazon.aws-9 SUCCESS in 11m 18s
✔️ integration-amazon.aws-10 SUCCESS in 7m 53s
✔️ integration-amazon.aws-11 SUCCESS in 6m 50s
✔️ integration-amazon.aws-12 SUCCESS in 13m 25s
✔️ integration-amazon.aws-13 SUCCESS in 12m 55s
✔️ integration-amazon.aws-14 SUCCESS in 12m 35s
✔️ integration-amazon.aws-15 SUCCESS in 8m 56s
✔️ integration-amazon.aws-16 SUCCESS in 25m 35s
✔️ integration-amazon.aws-17 SUCCESS in 11m 40s
✔️ integration-amazon.aws-18 SUCCESS in 10m 43s
✔️ integration-amazon.aws-19 SUCCESS in 11m 48s
Skipped 25 jobs

@mandar242 mandar242 added the mergeit Merge the PR (SoftwareFactory) label Dec 20, 2023
Copy link
Contributor

Build succeeded (gate pipeline).
https://ansible.softwarefactory-project.io/zuul/buildset/8392612537f146c0afecb0667e441a17

✔️ ansible-galaxy-importer SUCCESS in 4m 55s
✔️ build-ansible-collection SUCCESS in 14m 27s
✔️ ansible-test-splitter SUCCESS in 5m 56s
✔️ integration-amazon.aws-1 SUCCESS in 20m 42s
✔️ integration-amazon.aws-2 SUCCESS in 19m 27s
✔️ integration-amazon.aws-3 SUCCESS in 12m 38s
✔️ integration-amazon.aws-4 SUCCESS in 11m 17s
✔️ integration-amazon.aws-5 SUCCESS in 15m 43s
✔️ integration-amazon.aws-6 SUCCESS in 7m 36s
✔️ integration-amazon.aws-7 SUCCESS in 10m 17s
✔️ integration-amazon.aws-8 SUCCESS in 11m 32s
✔️ integration-amazon.aws-9 SUCCESS in 10m 23s
✔️ integration-amazon.aws-10 SUCCESS in 10m 46s
✔️ integration-amazon.aws-11 SUCCESS in 14m 30s
✔️ integration-amazon.aws-12 SUCCESS in 13m 56s
✔️ integration-amazon.aws-13 SUCCESS in 17m 09s
✔️ integration-amazon.aws-14 SUCCESS in 15m 41s
✔️ integration-amazon.aws-15 SUCCESS in 7m 54s
✔️ integration-amazon.aws-16 SUCCESS in 12m 59s
✔️ integration-amazon.aws-17 SUCCESS in 13m 30s
✔️ integration-amazon.aws-18 SUCCESS in 8m 45s
✔️ integration-amazon.aws-19 SUCCESS in 13m 39s
Skipped 25 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit cef7268 into ansible-collections:main Dec 20, 2023
37 of 40 checks passed
@tremble tremble added the backport-7 PR should be backported to the stable-7 branch label Dec 20, 2023
Copy link

patchback bot commented Dec 20, 2023

Backport to stable-7: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-7/cef72686e56482ca373e1470d788e599f294af35/pr-1918

Backported as #1934

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Dec 20, 2023
)

ec2_instance: add support for modifying instance metadata options

SUMMARY

Fixes #1864
This PR add support for modifying instance metadata options.
Uses client. modify_instance_metadata_options()

ISSUE TYPE

Feature Pull Request

COMPONENT NAME

ec2_instance
ADDITIONAL INFORMATION

Reviewed-by: Helen Bailey <hebailey@redhat.com>
Reviewed-by: Mark Chappell
Reviewed-by: Bikouo Aubin
Reviewed-by: GomathiselviS
Reviewed-by: Mandar Kulkarni <mandar242@gmail.com>
(cherry picked from commit cef7268)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Dec 20, 2023
) (#1934)

[PR #1918/cef72686 backport][stable-7] ec2_instance: add support for modifying instance metadata options

This is a backport of PR #1918 as merged into main (cef7268).
SUMMARY

Fixes #1864
This PR add support for modifying instance metadata options.
Uses client. modify_instance_metadata_options()

ISSUE TYPE


Feature Pull Request

COMPONENT NAME

ec2_instance
ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-7 PR should be backported to the stable-7 branch mergeit Merge the PR (SoftwareFactory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ec2_instance metadata_options are not updated
5 participants