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.

Sign up for GitHub

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

Jump to bottom

module_utils.s3 - added 501 to the list of tolerated 'NotImplemented' error codes #2508

Merged
merged 1 commit into from
Feb 5, 2025
Merged

module_utils.s3 - added 501 to the list of tolerated 'NotImplemented' error codes #2508

merged 1 commit into from
Feb 5, 2025

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Feb 4, 2025

SUMMARY

As per #2447 "Hetzner Object storage" throws a 501 (HTTP Error - Not Implemented) rather than the standard NotImplemented error.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

module_utlis.s3

ADDITIONAL INFORMATION

@tremble
Copy link
Contributor Author

tremble commented Feb 4, 2025

CC: @pgassmann

@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 6m 49s
✔️ build-ansible-collection SUCCESS in 10m 09s
✔️ ansible-test-splitter SUCCESS in 3m 53s
Skipped 44 jobs

@tremble tremble added the do_not_backport This PR should not be backported to stable- branches unless absolutely necessary label Feb 4, 2025
@tremble
Copy link
Contributor Author

tremble commented Feb 4, 2025

I'll backport the various S3 changes in a block hopefully in time for 9.3.0

@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 3m 41s
✔️ build-ansible-collection SUCCESS in 10m 22s
✔️ ansible-test-splitter SUCCESS in 4m 05s
Skipped 44 jobs

@tremble tremble requested a review from GomathiselviS February 5, 2025 17:57
@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Feb 5, 2025
@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 5m 07s
✔️ build-ansible-collection SUCCESS in 10m 15s
✔️ ansible-test-splitter SUCCESS in 3m 59s
Skipped 44 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit b72c134 into ansible-collections:main Feb 5, 2025
96 checks passed
@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 5, 2025

Docs Build 📝

Thank you for contribution!✨

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

@pgassmann pgassmann mentioned this pull request Feb 11, 2025
Closed
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GomathiselviS GomathiselviS GomathiselviS approved these changes

@alinabuzachis alinabuzachis Awaiting requested review from alinabuzachis

@abikouo abikouo Awaiting requested review from abikouo

Assignees
No one assigned
Labels
do_not_backport This PR should not be backported to stable- branches unless absolutely necessary mergeit Merge the PR (SoftwareFactory)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tremble @GomathiselviS