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: reassure that configuration is complete before returning. #1085

Conversation

lrubaszewski
Copy link
Contributor

Get bucket lifecycle configuration a few times to make sure it is stable.

It was observed that shortly (~30s) after setting the rules a method get-bucket-lifecycle-configuration
returns alternatively new and old rules in a random manner.
Similar issue reported for boto3 library:
boto/boto3#2491

SUMMARY

Fixes #1084

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 integration tests/integration module module needs_triage new_contributor Help guide this first time contributor plugins plugin (any type) tests tests labels Apr 20, 2022
@tremble
Copy link
Contributor

tremble commented Apr 20, 2022

Thanks for taking the time to submit this PR. One minor niggle on the formatting of the changelog entry

Get bucket lifecycle configuration a few times to make sure it is stable.

It was observed that shortly (~30s) after setting the rules a method get-bucket-lifecycle-configuration
returns alternatively new and old rules in a random manner.
Similar issue reported for boto3 library:
boto/boto3#2491
@lrubaszewski
Copy link
Contributor Author

@tremble I have applied changes according to the requests.
Is there anything else I should do in order to proceed with this PR?

@markuman markuman added backport-2 PR should be backported to the stable-2 branch backport-3 PR should be backported to the stable-3 branch labels May 5, 2022
@markuman markuman removed the request for review from tremble May 5, 2022 19:00
Copy link
Contributor

@jatorcasso jatorcasso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks @lrubaszewski

@jatorcasso jatorcasso added the mergeit Merge the PR (SoftwareFactory) label May 6, 2022
@goneri
Copy link
Member

goneri commented May 6, 2022

regate

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

ansible-galaxy-importer FAILURE in 5m 08s (non-voting)
✔️ build-ansible-collection SUCCESS in 4m 16s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 52s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 38s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 15m 01s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 9m 55s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 54s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 6m 04s
✔️ ansible-test-splitter SUCCESS in 3m 42s
✔️ 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

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit d802749 into ansible-collections:main May 6, 2022
@patchback
Copy link

patchback bot commented May 6, 2022

Backport to stable-2: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-2/d802749febd7dbf30e92a6eac174aebf0ba73308/pr-1085

Backported as #1131

🤖 @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 May 6, 2022
…1085)

s3_lifecycle: reassure that configuration is complete before returning.

Get bucket lifecycle configuration a few times to make sure it is stable.
It was observed that shortly (~30s) after setting the rules a method get-bucket-lifecycle-configuration
returns alternatively new and old rules in a random manner.
Similar issue reported for boto3 library:
boto/boto3#2491
SUMMARY
Fixes #1084
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
s3_lifecycle

Reviewed-by: Mark Chappell <None>
Reviewed-by: Lukasz Rubaszewski <None>
Reviewed-by: Mark Woolley <mw@marknet15.com>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Joseph Torcasso <None>
(cherry picked from commit d802749)
@patchback
Copy link

patchback bot commented May 6, 2022

Backport to stable-3: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-3/d802749febd7dbf30e92a6eac174aebf0ba73308/pr-1085

Backported as #1132

🤖 @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 May 6, 2022
…1085)

s3_lifecycle: reassure that configuration is complete before returning.

Get bucket lifecycle configuration a few times to make sure it is stable.
It was observed that shortly (~30s) after setting the rules a method get-bucket-lifecycle-configuration
returns alternatively new and old rules in a random manner.
Similar issue reported for boto3 library:
boto/boto3#2491
SUMMARY
Fixes #1084
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
s3_lifecycle

Reviewed-by: Mark Chappell <None>
Reviewed-by: Lukasz Rubaszewski <None>
Reviewed-by: Mark Woolley <mw@marknet15.com>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Joseph Torcasso <None>
(cherry picked from commit d802749)
@github-actions
Copy link

github-actions bot commented May 6, 2022

Docs Build 📝

Thank you for contribution!✨

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

softwarefactory-project-zuul bot pushed a commit that referenced this pull request May 9, 2022
…1085) (#1132)

[PR #1085/d802749f backport][stable-3] s3_lifecycle: reassure that configuration is complete before returning.

This is a backport of PR #1085 as merged into main (d802749).
Get bucket lifecycle configuration a few times to make sure it is stable.
It was observed that shortly (~30s) after setting the rules a method get-bucket-lifecycle-configuration
returns alternatively new and old rules in a random manner.
Similar issue reported for boto3 library:
boto/boto3#2491
SUMMARY
Fixes #1084
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
s3_lifecycle

Reviewed-by: Alina Buzachis <None>
softwarefactory-project-zuul bot pushed a commit that referenced this pull request May 9, 2022
…1085) (#1131)

[PR #1085/d802749f backport][stable-2] s3_lifecycle: reassure that configuration is complete before returning.

This is a backport of PR #1085 as merged into main (d802749).
Get bucket lifecycle configuration a few times to make sure it is stable.
It was observed that shortly (~30s) after setting the rules a method get-bucket-lifecycle-configuration
returns alternatively new and old rules in a random manner.
Similar issue reported for boto3 library:
boto/boto3#2491
SUMMARY
Fixes #1084
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
s3_lifecycle

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

s3_lifecycle - occasionally fails to add/remove all rules from playbook
8 participants