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 #1398/4c67633c backport][stable-3] s3-lifecycle: fix remove rule with empty prefix #1402

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Aug 9, 2022

This is a backport of PR #1398 as merged into main (4c67633).

SUMMARY

In case of removing a lifecycle policy without a prefix, there will be no prefix key in the existing_rule filter

Signed-off-by: Seena Fallah seenafallah@gmail.com

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

s3_lifecycle

s3-lifecycle: fix remove rule with empty prefix

SUMMARY
In case of removing a lifecycle policy without a prefix, there will be no prefix key in the existing_rule filter
Signed-off-by: Seena Fallah seenafallah@gmail.com
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
s3_lifecycle

Reviewed-by: Mark Chappell <None>
(cherry picked from commit 4c67633)
@ansibullbot
Copy link

@ansibullbot ansibullbot added bug This issue/PR relates to a bug community_review integration tests/integration module module needs_triage new_contributor Help guide this first time contributor plugins plugin (any type) tests tests labels Aug 9, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 45s
✔️ build-ansible-collection SUCCESS in 5m 30s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 04s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 18s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 36s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 11m 17s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 6m 24s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 7m 08s
✔️ ansible-test-splitter SUCCESS in 2m 32s
✔️ integration-community.aws-1 SUCCESS in 24m 20s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ 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

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Aug 9, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 6m 36s
✔️ build-ansible-collection SUCCESS in 5m 31s
✔️ ansible-test-sanity-docker-devel SUCCESS in 13m 36s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 13m 16s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 13m 08s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 13m 57s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 9m 00s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 8m 02s
✔️ ansible-test-splitter SUCCESS in 2m 35s
✔️ integration-community.aws-1 SUCCESS in 24m 08s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ 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

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 9d55fba into stable-3 Aug 9, 2022
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-3/4c67633cea15c214f59904ccb49427c8fbaa3435/pr-1398 branch August 9, 2022 10:54
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
… when cancelling request (ansible-collections#1402)

ec2_spot_instance: add parameter to enable terminating spot instances when cancelling request

SUMMARY

This PR adds a new parameter terminate_instances (true | false) to enable terminating spot instances when cancelling request.
Can be used only when state=absent.

Fixes ansible-collections#1360
ISSUE TYPE


Feature Pull Request

COMPONENT NAME

ec2_spot_instance
ADDITIONAL INFORMATION

Reviewed-by: Bikouo Aubin
Reviewed-by: Mandar Kulkarni <mandar242@gmail.com>
Reviewed-by: Alina Buzachis
Reviewed-by: Mark Chappell
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 integration tests/integration mergeit Merge the PR (SoftwareFactory) module module 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