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_object - Rework list_objects handling #2328

Merged

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Oct 9, 2024

SUMMARY

fixes: #1953

  • Reworks the handling of object listing, ensures that max_keys is respected.
  • Deprecates "mode:list" in s3_object
  • Adds pagination control (max_items) support to s3_object_info
ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

s3_object
s3_object_info

ADDITIONAL INFORMATION

Copy link

github-actions bot commented Oct 9, 2024

Docs Build 📝

Thank you for contribution!✨

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

This comment was marked as resolved.

Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 5m 11s
✔️ build-ansible-collection SUCCESS in 10m 41s
✔️ ansible-test-splitter SUCCESS in 4m 27s
✔️ integration-amazon.aws-1 SUCCESS in 14m 04s
✔️ integration-amazon.aws-2 SUCCESS in 7m 19s
✔️ integration-amazon.aws-3 SUCCESS in 5m 54s
Skipped 41 jobs

Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 4m 47s
✔️ build-ansible-collection SUCCESS in 10m 32s
✔️ ansible-test-splitter SUCCESS in 4m 20s
✔️ integration-amazon.aws-1 SUCCESS in 13m 00s
✔️ integration-amazon.aws-2 SUCCESS in 7m 25s
Skipped 42 jobs

Copy link
Contributor

Build failed.
https://ansible.softwarefactory-project.io/zuul/buildset/b753339feed947319e1c76a0df107274

✔️ ansible-galaxy-importer SUCCESS in 4m 32s
✔️ build-ansible-collection SUCCESS in 10m 36s
✔️ ansible-test-splitter SUCCESS in 4m 21s
integration-amazon.aws-1 FAILURE in 11m 56s
✔️ integration-amazon.aws-2 SUCCESS in 7m 38s
Skipped 42 jobs

Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 5m 19s
✔️ build-ansible-collection SUCCESS in 10m 48s
✔️ ansible-test-splitter SUCCESS in 4m 17s
✔️ integration-amazon.aws-1 SUCCESS in 10m 41s
✔️ integration-amazon.aws-2 SUCCESS in 8m 40s
Skipped 42 jobs

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Oct 10, 2024
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 3m 52s
✔️ build-ansible-collection SUCCESS in 10m 45s
✔️ ansible-test-splitter SUCCESS in 4m 17s
✔️ integration-amazon.aws-1 SUCCESS in 12m 57s
✔️ integration-amazon.aws-2 SUCCESS in 5m 27s
Skipped 42 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 362535e into ansible-collections:main Oct 10, 2024
22 of 35 checks passed
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.

S3 bucket list: max_keys: 0 is ignored
2 participants