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

[PR #1663/5759afa1 backport][stable-5] aws_ssm - Refactor boto3 client initialization #1668

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Jan 20, 2023

This is a backport of PR #1663 as merged into main (5759afa).

Depends-On: ansible/ansible-zuul-jobs#1748

SUMMARY

Move the boto3 client initialization into a consistent point (rather than initializing them on the fly)

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

aws_ssm

ADDITIONAL INFORMATION

In preparation for further refactoring and using ansible-collections/amazon.aws#1340

aws_ssm - Refactor boto3 client initialization

Depends-On: ansible/ansible-zuul-jobs#1748
SUMMARY
Move the boto3 client initialization into a consistent point (rather than initializing them on the fly)
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
aws_ssm
ADDITIONAL INFORMATION
In preparation for further refactoring and using ansible-collections/amazon.aws#1340

Reviewed-by: Markus Bergholz <git@osuv.de>
(cherry picked from commit 5759afa)
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 19s
✔️ build-ansible-collection SUCCESS in 5m 41s
✔️ ansible-test-sanity-docker-devel SUCCESS in 9m 40s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 50s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 58s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 13s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 10m 00s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 7m 48s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 54s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 12s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 6m 17s
✔️ ansible-test-changelog SUCCESS in 2m 32s
✔️ ansible-test-splitter SUCCESS in 2m 50s
✔️ integration-community.aws-1 SUCCESS in 11m 16s
✔️ integration-community.aws-2 SUCCESS in 12m 37s
✔️ integration-community.aws-3 SUCCESS in 8m 10s
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

@ansibullbot ansibullbot added community_review connection connection plugin feature This issue/PR relates to a feature request needs_triage new_contributor Help guide this first time contributor plugins plugin (any type) labels Jan 20, 2023
@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Jan 20, 2023
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 03s
✔️ build-ansible-collection SUCCESS in 6m 24s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 11s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 08s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 09s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 57s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 8m 58s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 05s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 8m 17s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 50s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 6m 18s
✔️ ansible-test-changelog SUCCESS in 2m 23s
✔️ ansible-test-splitter SUCCESS in 3m 02s
✔️ integration-community.aws-1 SUCCESS in 12m 37s
✔️ integration-community.aws-2 SUCCESS in 12m 46s
✔️ integration-community.aws-3 SUCCESS in 7m 31s
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 67a62e7 into stable-5 Jan 20, 2023
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-5/5759afa17c97a26609ec14eafb5531e499efa5c2/pr-1663 branch January 20, 2023 10:42
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Jan 20, 2023
[PR #1664/7b68f119 backport][stable-5] aws_ssm - Refactor _file_transport_command

Depends-On: #1668
This is a backport of PR #1664 as merged into main (7b68f11).
SUMMARY
Yet more cleanup of aws_ssm
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
aws_ssm
ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell <None>
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
Default filters to empty dictionary

SUMMARY

Changes the default for the filters option from None to an empty dictionary.

Fixes: ansible-collections#1668
ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME

ec2_vpc_route_table_info
ADDITIONAL INFORMATION


This matches what other *info modules (e.g. ec2_vpc_endpoint_info or ec2_vpc_subnet_info) do for the filters option default.

Reviewed-by: Alina Buzachis
Reviewed-by: Brant Evans <bevans@redhat.com>
Reviewed-by: Mark Chappell
Reviewed-by: Mike Graves <mgraves@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community_review connection connection plugin feature This issue/PR relates to a feature request mergeit Merge the PR (SoftwareFactory) needs_triage new_contributor Help guide this first time contributor plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants