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 - split S3 region/endpoint discovery into dedicated function #1674

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Jan 23, 2023

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

@softwarefactory-project-zuul

This comment was marked as resolved.

@tremble tremble force-pushed the aws_ssm/cross_region branch from 437cd01 to ae0f5ac Compare January 23, 2023 14:16
@ansibullbot ansibullbot added bug This issue/PR relates to a bug community_review connection connection plugin integration tests/integration needs_triage plugins plugin (any type) tests tests labels Jan 23, 2023
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 16s
✔️ build-ansible-collection SUCCESS in 7m 01s
ansible-test-sanity-docker-devel FAILURE in 9m 38s (non-voting)
ansible-test-sanity-docker-milestone FAILURE in 9m 46s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 01s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 48s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 42s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 14s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 38s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 21s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 6m 28s
✔️ ansible-test-changelog SUCCESS in 2m 28s
✔️ ansible-test-splitter SUCCESS in 2m 36s
✔️ integration-community.aws-1 SUCCESS in 9m 31s
✔️ integration-community.aws-2 SUCCESS in 7m 44s
✔️ integration-community.aws-3 SUCCESS in 8m 19s
✔️ integration-community.aws-4 SUCCESS in 9m 42s
✔️ integration-community.aws-5 SUCCESS in 9m 19s
✔️ integration-community.aws-6 SUCCESS in 9m 00s
✔️ integration-community.aws-7 SUCCESS in 7m 54s
✔️ integration-community.aws-8 SUCCESS in 10m 42s
⚠️ 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

@tremble tremble marked this pull request as ready for review January 23, 2023 15:00
@tremble tremble added mergeit Merge the PR (SoftwareFactory) backport-5 PR should be backported to the stable-5 branch labels Jan 23, 2023
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 21s
✔️ build-ansible-collection SUCCESS in 5m 59s
ansible-test-sanity-docker-devel FAILURE in 9m 24s (non-voting)
ansible-test-sanity-docker-milestone FAILURE in 9m 47s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 03s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 27s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 30s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 12s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 49s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 29s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 6m 19s
✔️ ansible-test-changelog SUCCESS in 2m 30s
✔️ ansible-test-splitter SUCCESS in 2m 44s
✔️ integration-community.aws-1 SUCCESS in 8m 29s
✔️ integration-community.aws-2 SUCCESS in 7m 59s
✔️ integration-community.aws-3 SUCCESS in 10m 22s
✔️ integration-community.aws-4 SUCCESS in 8m 28s
✔️ integration-community.aws-5 SUCCESS in 8m 24s
✔️ integration-community.aws-6 SUCCESS in 11m 09s
✔️ integration-community.aws-7 SUCCESS in 7m 42s
✔️ integration-community.aws-8 SUCCESS in 10m 14s
⚠️ 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 8237ebb into ansible-collections:main Jan 23, 2023
@patchback
Copy link

patchback bot commented Jan 23, 2023

Backport to stable-5: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-5/8237ebb75303c5e91b9151bb6157f8250f957054/pr-1674

Backported as #1677

🤖 @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 23, 2023
…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)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Jan 23, 2023
…1674) (#1677)

[PR #1674/8237ebb7 backport][stable-5] aws_ssm - split S3 region/endpoint discovery into dedicated function

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

Reviewed-by: Markus Bergholz <[email protected]>
@tremble tremble deleted the aws_ssm/cross_region branch February 15, 2023 09:24
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 bug This issue/PR relates to a bug community_review connection connection plugin integration tests/integration mergeit Merge the PR (SoftwareFactory) needs_triage plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

aws_ssm: Wrong pre-signed URL when curl-ing AnsiballZ_systemd.py on newer AWS regions
4 participants