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

Add SQS FIFO high throughput options #1603

Conversation

redradrat
Copy link
Contributor

SUMMARY

adds high throughput options for SQS fifo queues

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME
  • sqs_topic
ADDITIONAL INFORMATION

@github-actions
Copy link

github-actions bot commented Nov 24, 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 needs_triage new_contributor Help guide this first time contributor plugins plugin (any type) labels Nov 24, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

✔️ ansible-galaxy-importer SUCCESS in 4m 32s
✔️ build-ansible-collection SUCCESS in 5m 52s
ansible-test-sanity-docker-devel FAILURE in 9m 53s (non-voting)
ansible-test-sanity-docker-milestone FAILURE in 8m 28s (non-voting)
ansible-test-sanity-docker-stable-2.12 FAILURE in 11m 45s
ansible-test-sanity-docker-stable-2.13 FAILURE in 8m 55s
ansible-test-sanity-docker-stable-2.14 FAILURE in 9m 06s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 8m 00s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 40s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 23s
ansible-test-changelog FAILURE in 2m 12s
✔️ ansible-test-splitter SUCCESS in 2m 28s
✔️ integration-community.aws-1 SUCCESS in 6m 24s
⚠️ 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
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 38s
✔️ build-ansible-collection SUCCESS in 5m 43s
✔️ ansible-test-sanity-docker-devel SUCCESS in 9m 20s (non-voting)
ansible-test-sanity-docker-milestone FAILURE in 9m 13s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 36s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 33s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 34s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 22s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 47s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 5m 48s
✔️ ansible-test-changelog SUCCESS in 2m 09s
✔️ ansible-test-splitter SUCCESS in 2m 45s
✔️ integration-community.aws-1 SUCCESS in 5m 34s
⚠️ 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
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

Copy link
Contributor

@alinabuzachis alinabuzachis left a comment

Choose a reason for hiding this comment

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

@redradrat Thank you for working on this. Could you please add some integration tests?

plugins/modules/sqs_queue.py Show resolved Hide resolved
plugins/modules/sqs_queue.py Show resolved Hide resolved
plugins/modules/sqs_queue.py Show resolved Hide resolved
plugins/modules/sqs_queue.py Show resolved Hide resolved
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 3m 42s
✔️ build-ansible-collection SUCCESS in 5m 46s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 26s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 11m 05s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 52s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 8m 59s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 8m 55s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 7m 01s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 54s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 13m 26s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 7m 01s
✔️ ansible-test-changelog SUCCESS in 2m 14s
✔️ ansible-test-splitter SUCCESS in 2m 47s
✔️ integration-community.aws-1 SUCCESS in 6m 23s
⚠️ 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
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

@ansibullbot ansibullbot added integration tests/integration tests tests labels Feb 2, 2023
@tremble tremble added mergeit Merge the PR (SoftwareFactory) backport-5 PR should be backported to the stable-5 branch labels Feb 2, 2023
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 26s
✔️ build-ansible-collection SUCCESS in 5m 36s
✔️ ansible-test-sanity-docker-devel SUCCESS in 12m 29s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 16s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 12m 22s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 8m 44s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 42s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 5m 13s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 7m 48s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 06s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 6m 09s
✔️ ansible-test-changelog SUCCESS in 2m 16s
✔️ ansible-test-splitter SUCCESS in 2m 58s
✔️ integration-community.aws-1 SUCCESS in 5m 43s
⚠️ 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
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 231ead9 into ansible-collections:main Feb 2, 2023
@patchback
Copy link

patchback bot commented Feb 2, 2023

Backport to stable-5: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-5/231ead9753a80966e480b4abd859a1daefea61c7/pr-1603

Backported as #1697

🤖 @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 Feb 2, 2023
Add SQS FIFO high throughput options

SUMMARY
adds high throughput options for SQS fifo queues
ISSUE TYPE

Feature Pull Request

COMPONENT NAME

sqs_topic

ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Mark Chappell <None>
(cherry picked from commit 231ead9)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Feb 2, 2023
[PR #1603/231ead97 backport][stable-5] Add SQS FIFO high throughput options

This is a backport of PR #1603 as merged into main (231ead9).
SUMMARY
adds high throughput options for SQS fifo queues
ISSUE TYPE


Feature Pull Request

COMPONENT NAME


sqs_topic

ADDITIONAL INFORMATION

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-5 PR should be backported to the stable-5 branch community_review feature This issue/PR relates to a feature request 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.

4 participants