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

[DNM] [PR #1555/b9d5b48b backport][stable-6] s3_object - deprecate leading slash in object keys #1557

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented May 17, 2023

This is a backport of PR #1555 as merged into main (b9d5b48).

SUMMARY

Deprecate support for removing leading / from object names

AWS APIs don't expect it to be there, pruning it may cause pain in future

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

s3_object

ADDITIONAL INFORMATION

Deprecation was split from #1549

#1555)

s3_object - deprecate leading slash in object keys

SUMMARY
Deprecate support for removing leading / from object names
AWS APIs don't expect it to be there, pruning it may cause pain in future
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
s3_object
ADDITIONAL INFORMATION
Deprecation was split from #1549

Reviewed-by: Alina Buzachis
(cherry picked from commit b9d5b48)
@github-actions
Copy link

github-actions bot commented May 17, 2023

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and your docs changes will be incorporated when they are next published.

@tremble tremble changed the title [PR #1555/b9d5b48b backport][stable-6] s3_object - deprecate leading slash in object keys [DNM] [PR #1555/b9d5b48b backport][stable-6] s3_object - deprecate leading slash in object keys May 17, 2023
@tremble
Copy link
Contributor

tremble commented May 17, 2023

This PR should NOT be merged prior to 6.0.1 being released

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/ebb89ffd65504ce991b1181476095406

✔️ ansible-galaxy-importer SUCCESS in 3m 40s
✔️ build-ansible-collection SUCCESS in 13m 07s
✔️ ansible-test-splitter SUCCESS in 4m 56s
✔️ integration-amazon.aws-1 SUCCESS in 11m 20s
Skipped 43 jobs

@alinabuzachis alinabuzachis added the mergeit Merge the PR (SoftwareFactory) label May 22, 2023
@softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 5m 36s
✔️ build-ansible-collection SUCCESS in 12m 38s
✔️ ansible-test-splitter SUCCESS in 4m 54s
✔️ integration-amazon.aws-1 SUCCESS in 13m 01s
Skipped 43 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 1b5ab75 into stable-6 May 22, 2023
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-6/b9d5b48b17c8be9ab171086122eb7e60b75cd901/pr-1555 branch May 22, 2023 17:21
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
…ons#1557)

Add origin shield in cloudfront_distribution module

SUMMARY
Add Origin Shield option to cloudfront_distribution module.
ISSUE TYPE


Feature Pull Request

COMPONENT NAME

cloudfront_distribution.py
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Gonéri Le Bouder <[email protected]>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
…ons#1557)

Add origin shield in cloudfront_distribution module

SUMMARY
Add Origin Shield option to cloudfront_distribution module.
ISSUE TYPE


Feature Pull Request

COMPONENT NAME

cloudfront_distribution.py
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Gonéri Le Bouder <[email protected]>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
…ons#1557)

Add origin shield in cloudfront_distribution module

SUMMARY
Add Origin Shield option to cloudfront_distribution module.
ISSUE TYPE


Feature Pull Request

COMPONENT NAME

cloudfront_distribution.py
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Gonéri Le Bouder <[email protected]>
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.

2 participants