-
Notifications
You must be signed in to change notification settings - Fork 343
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
[PR #2227/f97c56b1 backport][stable-8] aws_ec2: fix populating SSM inventory for multiple hosts #2435
base: stable-8
Are you sure you want to change the base?
Conversation
Build failed.
|
recheck |
Build failed.
|
recheck |
Build failed. ✔️ ansible-galaxy-importer SUCCESS in 3m 44s |
…I info (#2448) PR #2397 added some integration test cleanup that is needed in the stable-8 branch to fix the failing integration tests in PR #2435. However the entire PR can't be backported to stable-8 because it also includes some updates that are only relevant to the stable-9 branch. This backport PR includes only the commit from #2397 with the necessary integration test updates. Reviewed-by: Mark Chappell
SUMMARY The function _get_ssm_information() returns dicts with one object. Multiple results cannot be concatenated/merged using d.update() since it will keep overwriting one object. Concatenate the nested lists instead. Fixes #2226 ISSUE TYPE Bugfix Pull Request COMPONENT NAME aws_ec2 Reviewed-by: Bikouo Aubin Reviewed-by: GomathiselviS (cherry picked from commit f97c56b)
48c9d23
to
f29e059
Compare
Build succeeded. ✔️ ansible-galaxy-importer SUCCESS in 8m 14s |
This is a backport of PR #2227 as merged into main (f97c56b).
SUMMARY
The function
_get_ssm_information()
returns dicts with one object. Multiple results cannot be concatenated/merged usingd.update()
since it will keep overwriting one object.Concatenate the nested lists instead.
Fixes #2226
ISSUE TYPE
COMPONENT NAME
aws_ec2