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 #794/ed3a7a0b backport][stable-2] Add abort multipart upload and expire obj del markers to s3 lifecycle #864

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Jan 14, 2022

This is a backport of PR #794 as merged into main (ed3a7a0).

Depends-On: ansible/ansible-zuul-jobs#1247

SUMMARY

Fixes #365 #796

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

s3_lifecycle

ADDITIONAL INFORMATION

I have not run integration tests yet because of ansible-collections/amazon.aws#924.

I'm unsure about how to name and structure the new arguments.
Do I nest them to match the API, or flatten them to match existing arguments?

…#794)

Add abort multipart upload and expire obj del markers to s3 lifecycle

Depends-On: ansible/ansible-zuul-jobs#1247
SUMMARY
Fixes #365 #796
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
s3_lifecycle
ADDITIONAL INFORMATION
I have not run integration tests yet because of #793.
I'm unsure about how to name and structure the new arguments.
Do I nest them to match the API, or flatten them to match existing arguments?

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Matthew Davis <None>
Reviewed-by: Mark Chappell <None>
Reviewed-by: None <None>
Reviewed-by: Markus Bergholz <[email protected]>
(cherry picked from commit ed3a7a0)
@marknet15
Copy link
Contributor

recheck

@marknet15
Copy link
Contributor

recheck

@markuman markuman added the mergeit Merge the PR (SoftwareFactory) label Jan 17, 2022
@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 23ae726 into stable-2 Jan 17, 2022
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-2/ed3a7a0b0ae86cbcaed733ada1f88a27917e1a09/pr-794 branch January 17, 2022 17:59
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…listener method to include SSLCertificateId (ansible-collections#864)

elb_classic_lb: Add integration tests for - fix return value _format_listener method to include SSLCertificateId

SUMMARY

Added integration test for ansible-collections#864

ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME

elb_classic_lb

Reviewed-by: Mark Chappell <None>
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.

3 participants