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

aws_ssm - Refactor boto3 client initialization #1663

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Jan 19, 2023

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

@ansibullbot ansibullbot added community_review connection connection plugin feature This issue/PR relates to a feature request needs_triage plugins plugin (any type) labels Jan 19, 2023
@softwarefactory-project-zuul

This comment was marked as resolved.

@tremble
Copy link
Contributor Author

tremble commented Jan 19, 2023

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 26s
✔️ build-ansible-collection SUCCESS in 6m 12s
✔️ ansible-test-sanity-docker-devel SUCCESS in 8m 55s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 27s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 12m 39s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 45s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 10m 13s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 12s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 45s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 59s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 10m 02s
✔️ ansible-test-changelog SUCCESS in 2m 37s
✔️ ansible-test-splitter SUCCESS in 2m 55s
✔️ integration-community.aws-1 SUCCESS in 10m 53s
✔️ integration-community.aws-2 SUCCESS in 12m 19s
✔️ integration-community.aws-3 SUCCESS in 7m 50s
⚠️ 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

@markuman markuman added backport-5 PR should be backported to the stable-5 branch backport-4 PR should be backported to the stable-4 branch labels Jan 19, 2023
@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Jan 19, 2023
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 05s
✔️ build-ansible-collection SUCCESS in 5m 40s
✔️ ansible-test-sanity-docker-devel SUCCESS in 9m 35s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 15s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 26s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 8m 46s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 8m 50s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 5m 45s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 7m 21s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 5m 37s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 5m 49s
✔️ ansible-test-changelog SUCCESS in 2m 25s
✔️ ansible-test-splitter SUCCESS in 2m 58s
✔️ integration-community.aws-1 SUCCESS in 10m 41s
✔️ integration-community.aws-2 SUCCESS in 12m 26s
✔️ integration-community.aws-3 SUCCESS in 7m 50s
⚠️ 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 5759afa into ansible-collections:main Jan 19, 2023
@patchback

This comment was marked as outdated.

@patchback

This comment was marked as resolved.

softwarefactory-project-zuul bot pushed a commit that referenced this pull request Jan 19, 2023
aws_ssm - Refactor _file_transport_command

Depends-On: ansible/ansible-zuul-jobs#1748
Depends-On: #1663
SUMMARY
Yet more cleanup of aws_ssm
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
aws_ssm
ADDITIONAL INFORMATION

Reviewed-by: Markus Bergholz <[email protected]>
patchback bot pushed a commit that referenced this pull request Jan 19, 2023
aws_ssm - Refactor _file_transport_command

Depends-On: ansible/ansible-zuul-jobs#1748
Depends-On: #1663
SUMMARY
Yet more cleanup of aws_ssm
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
aws_ssm
ADDITIONAL INFORMATION

Reviewed-by: Markus Bergholz <[email protected]>
(cherry picked from commit 7b68f11)
patchback bot pushed a commit that referenced this pull request Jan 19, 2023
aws_ssm - Refactor _file_transport_command

Depends-On: ansible/ansible-zuul-jobs#1748
Depends-On: #1663
SUMMARY
Yet more cleanup of aws_ssm
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
aws_ssm
ADDITIONAL INFORMATION

Reviewed-by: Markus Bergholz <[email protected]>
(cherry picked from commit 7b68f11)
@tremble tremble removed the backport-4 PR should be backported to the stable-4 branch label Jan 20, 2023
@tremble tremble added backport-5 PR should be backported to the stable-5 branch and removed backport-5 PR should be backported to the stable-5 branch labels Jan 20, 2023
@patchback
Copy link

patchback bot commented Jan 20, 2023

Backport to stable-5: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-5/5759afa17c97a26609ec14eafb5531e499efa5c2/pr-1663

Backported as #1668

🤖 @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 Jan 20, 2023
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 <[email protected]>
(cherry picked from commit 5759afa)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Jan 20, 2023
[PR #1663/5759afa1 backport][stable-5] aws_ssm - Refactor boto3 client initialization

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

Reviewed-by: Mark Chappell <None>
@tremble tremble deleted the cleanup/aws_ssm/client branch February 15, 2023 09:24
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…ing GlobalClusterIdentifier (ansible-collections#1663)

Allow user to associate rds cluster with a global rds cluster

SUMMARY

Allow user to associate rds cluster with a global rds cluster

ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME

rds_cluster
ADDITIONAL INFORMATION

Reviewed-by: Jill R
Reviewed-by: Mandar Kulkarni <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-5 PR should be backported to the stable-5 branch community_review connection connection plugin feature This issue/PR relates to a feature request mergeit Merge the PR (SoftwareFactory) needs_triage plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants