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

merge s3_url parameter into endpoint_url #994

Merged

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Sep 6, 2022

SUMMARY

(related to #992)

  • Merges the s3_url parameter into endpoint_url. This makes the modules more consistent with the rest of the collection.
  • Adds rgw as an alias to the ceph parameter in s3_bucket, and renames rgw to ceph (keeping an alias) in s3_object.
ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

plugins/modules/s3_bucket.py
plugins/modules/s3_object.py

ADDITIONAL INFORMATION

@github-actions
Copy link

github-actions bot commented Sep 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.

@ansibullbot
Copy link

@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request module module module_utils module_utils needs_triage plugins plugin (any type) labels Sep 6, 2022
@softwarefactory-project-zuul

This comment was marked as resolved.

@softwarefactory-project-zuul

This comment was marked as resolved.

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 59s
✔️ build-ansible-collection SUCCESS in 5m 23s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 9m 18s
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 10m 36s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 11m 00s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 8m 31s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 47s
✔️ ansible-test-splitter SUCCESS in 2m 34s
✔️ integration-amazon.aws-1 SUCCESS in 16m 59s
⚠️ integration-amazon.aws-2 SKIPPED
⚠️ integration-amazon.aws-3 SKIPPED
⚠️ integration-amazon.aws-4 SKIPPED
⚠️ integration-amazon.aws-5 SKIPPED
⚠️ integration-amazon.aws-6 SKIPPED
⚠️ integration-amazon.aws-7 SKIPPED
⚠️ integration-amazon.aws-8 SKIPPED
⚠️ integration-amazon.aws-9 SKIPPED
⚠️ integration-amazon.aws-10 SKIPPED
⚠️ integration-amazon.aws-11 SKIPPED
⚠️ integration-amazon.aws-12 SKIPPED
⚠️ integration-amazon.aws-13 SKIPPED
✔️ integration-community.aws-1 SUCCESS in 5m 57s
✔️ integration-community.aws-2 SUCCESS in 22m 11s
⚠️ 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
✔️ ansible-test-changelog SUCCESS in 2m 24s

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

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 36s
✔️ build-ansible-collection SUCCESS in 5m 15s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 13m 05s
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 8m 12s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 8m 26s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 7m 15s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 09s
✔️ ansible-test-splitter SUCCESS in 2m 44s
✔️ integration-amazon.aws-1 SUCCESS in 13m 08s
⚠️ integration-amazon.aws-2 SKIPPED
⚠️ integration-amazon.aws-3 SKIPPED
⚠️ integration-amazon.aws-4 SKIPPED
⚠️ integration-amazon.aws-5 SKIPPED
⚠️ integration-amazon.aws-6 SKIPPED
⚠️ integration-amazon.aws-7 SKIPPED
⚠️ integration-amazon.aws-8 SKIPPED
⚠️ integration-amazon.aws-9 SKIPPED
⚠️ integration-amazon.aws-10 SKIPPED
⚠️ integration-amazon.aws-11 SKIPPED
⚠️ integration-amazon.aws-12 SKIPPED
⚠️ integration-amazon.aws-13 SKIPPED
✔️ integration-community.aws-1 SUCCESS in 5m 26s
✔️ integration-community.aws-2 SUCCESS in 22m 58s
⚠️ 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
✔️ ansible-test-changelog SUCCESS in 2m 18s

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community_review feature This issue/PR relates to a feature request mergeit Merge the PR (SoftwareFactory) module_utils module_utils module module needs_triage plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants