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 #2267/db552aa1 backport][stable-8] s3_bucket - fix MethodNotAllowed in regions that don't support transfer acceleration #2269

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Aug 29, 2024

This is a backport of PR #2267 as merged into main (db552aa).

fixes: #2266

SUMMARY

While #2202 (combined with #2222) fixes the issue for the aws-gov partition, the aws partition throws a different Error (MethodNotAllowed).

I suspect the two different exceptions are a side effect of how AWS implement the Gov-Cloud partition vs the normal partition.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

s3_bucket

ADDITIONAL INFORMATION

See also: #2266

…er acceleration (#2267)

fixes: #2266
SUMMARY
While #2202 (combined with #2222) fixes the issue for the aws-gov partition, the aws partition throws a different Error (MethodNotAllowed).
I suspect the two different exceptions are a side effect of how AWS implement the Gov-Cloud partition vs the normal partition.
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
s3_bucket
ADDITIONAL INFORMATION
See also: #2266

Reviewed-by: GomathiselviS
(cherry picked from commit db552aa)
@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Aug 29, 2024
Copy link
Contributor

Build succeeded (gate pipeline).
https://ansible.softwarefactory-project.io/zuul/buildset/da8d1fc8b65f4580a0207839e53b0004

✔️ ansible-galaxy-importer SUCCESS in 4m 02s
✔️ build-ansible-collection SUCCESS in 10m 37s
✔️ ansible-test-splitter SUCCESS in 4m 24s
✔️ integration-amazon.aws-1 SUCCESS in 9m 53s
Skipped 43 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 647274c into stable-8 Aug 29, 2024
34 of 37 checks passed
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-8/db552aa17f2056d91e7e4545b55ce14883674b45/pr-2267 branch August 29, 2024 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mergeit Merge the PR (SoftwareFactory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant