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 #1674/8237ebb7 backport][stable-5] aws_ssm - split S3 region/endpoint discovery into dedicated function #1677

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Jan 23, 2023

This is a backport of PR #1674 as merged into main (8237ebb).

Depends-On: #1670

SUMMARY

fixes: #1616

Newer AWS regions don't generate valid presigned URLs unless you explicitly pass the endpoint_url for the region (see also boto/boto3#3015)

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

aws_ssm

ADDITIONAL INFORMATION

…1674)

aws_ssm - split S3 region/endpoint discovery into dedicated function

Depends-On: #1670
SUMMARY
fixes: #1616
Newer AWS regions don't generate valid presigned URLs unless you explicitly pass the endpoint_url for the region (see also boto/boto3#3015)
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
aws_ssm
ADDITIONAL INFORMATION

Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Alina Buzachis <None>
(cherry picked from commit 8237ebb)
@markuman markuman added the mergeit Merge the PR (SoftwareFactory) label Jan 23, 2023
@ansibullbot ansibullbot added bug This issue/PR relates to a bug community_review connection connection plugin integration tests/integration new_contributor Help guide this first time contributor plugins plugin (any type) tests tests labels Jan 23, 2023
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 30s
✔️ build-ansible-collection SUCCESS in 5m 51s
✔️ ansible-test-sanity-docker-devel SUCCESS in 8m 46s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 02s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 03s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 39s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 15s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 5m 58s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 37s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 5m 52s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 6m 55s
✔️ ansible-test-changelog SUCCESS in 2m 25s
✔️ ansible-test-splitter SUCCESS in 2m 38s
✔️ integration-community.aws-1 SUCCESS in 8m 54s
✔️ integration-community.aws-2 SUCCESS in 7m 28s
✔️ integration-community.aws-3 SUCCESS in 9m 29s
✔️ integration-community.aws-4 SUCCESS in 9m 19s
✔️ integration-community.aws-5 SUCCESS in 9m 21s
✔️ integration-community.aws-6 SUCCESS in 8m 30s
✔️ integration-community.aws-7 SUCCESS in 7m 36s
✔️ integration-community.aws-8 SUCCESS in 11m 54s
⚠️ 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 a2f6dbd into stable-5 Jan 23, 2023
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-5/8237ebb75303c5e91b9151bb6157f8250f957054/pr-1674 branch January 23, 2023 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug community_review connection connection plugin integration tests/integration mergeit Merge the PR (SoftwareFactory) new_contributor Help guide this first time contributor plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants