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

s3-lifecycle: fix remove rule with empty prefix #1398

Conversation

clwluvw
Copy link
Contributor

@clwluvw clwluvw commented Aug 8, 2022

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 [email protected]

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

s3_lifecycle

@ansibullbot
Copy link

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

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 19s
✔️ build-ansible-collection SUCCESS in 5m 23s
✔️ ansible-test-sanity-docker-devel SUCCESS in 9m 44s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 19s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 37s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 13m 02s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 8m 21s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 6m 02s
✔️ ansible-test-splitter SUCCESS in 2m 41s
✔️ integration-community.aws-1 SUCCESS in 20m 57s
⚠️ 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
Copy link
Contributor

tremble commented Aug 8, 2022

Hi,

Thanks for taking the time to submit this PR. If possible please update the integration tests at tests/integration/targets/s3_lifecycle to include coverage for this issue. Doing so helps to avoid accidental regressions in future.

Mark

@ansibullbot ansibullbot added integration tests/integration tests tests and removed needs_triage small_patch Hopefully easy to review labels Aug 8, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 3m 32s
✔️ build-ansible-collection SUCCESS in 5m 15s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 22s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 11m 19s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 11m 51s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 12s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 6m 05s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 5m 10s
✔️ ansible-test-splitter SUCCESS in 2m 28s
✔️ integration-community.aws-1 SUCCESS in 22m 06s
⚠️ 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

clwluvw and others added 2 commits August 8, 2022 14:06
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 <[email protected]>
@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Aug 8, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 21s
✔️ build-ansible-collection SUCCESS in 6m 53s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 34s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 17s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 11m 14s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 18s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 6m 18s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 7m 30s
✔️ ansible-test-splitter SUCCESS in 3m 14s
✔️ integration-community.aws-1 SUCCESS in 22m 38s
⚠️ 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 4c67633 into ansible-collections:main Aug 8, 2022
@tremble tremble added backport-3 PR should be backported to the stable-3 branch backport-4 PR should be backported to the stable-4 branch labels Aug 9, 2022
@patchback
Copy link

patchback bot commented Aug 9, 2022

Backport to stable-3: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-3/4c67633cea15c214f59904ccb49427c8fbaa3435/pr-1398

Backported as #1402

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

@patchback
Copy link

patchback bot commented Aug 9, 2022

Backport to stable-4: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-4/4c67633cea15c214f59904ccb49427c8fbaa3435/pr-1398

Backported as #1401

🤖 @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 Aug 9, 2022
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 [email protected]
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
s3_lifecycle

Reviewed-by: Mark Chappell <None>
(cherry picked from commit 4c67633)
patchback bot pushed a commit that referenced this pull request Aug 9, 2022
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 [email protected]
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
s3_lifecycle

Reviewed-by: Mark Chappell <None>
(cherry picked from commit 4c67633)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Aug 9, 2022
[PR #1398/4c67633c backport][stable-4] s3-lifecycle: fix remove rule with empty prefix

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 [email protected]
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
s3_lifecycle

Reviewed-by: Mark Chappell <None>
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Aug 9, 2022
[PR #1398/4c67633c backport][stable-3] s3-lifecycle: fix remove rule with empty prefix

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 [email protected]
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
s3_lifecycle

Reviewed-by: Mark Chappell <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-3 PR should be backported to the stable-3 branch backport-4 PR should be backported to the stable-4 branch 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