8000 Azure servicebus - add async functionality by ptoscano · Pull Request #2017 · ansible-collections/azure · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Azure servicebus - add async functionality #2017

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
Merged
merged 1 commit into from
Jul 16, 2025
Merged

Azure servicebus - add async functionality #2017

merged 1 commit into from
Jul 16, 2025
+80 −60

Conversation

ptoscano
Copy link
Contributor

Forwarding ansible/event-driven-ansible#451 to the version of the azure_service_bus.py EDA source in this collection; quoting from that PR:

This PR is to fix the issue ansible/event-driven-ansible#401 , in which the async functionality was not working as expected. I've rewritten this module to leverage the Azure Service Bus async functionality.

In my testing, this resolves the QueueFull message that was being received when a large number of events needed to be processed.

@susanhooks @pre-commit-ci
@Alex-Izquierdo @ptoscano
Azure servicebus - add async functionality (ansible-collections#451)
59fb6c9 
* add async support

* added tests

* mypy typing

* chore: auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* updated test format

* chore: auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Alex <aizquier@redhat.com>
@ptoscano ptoscano mentioned this pull request Jul 16, 2025
Merged
@p3ck p3ck self-requested a review July 16, 2025 13:33
p3ck
p3ck approved these changes Jul 16, 2025
View reviewed changes
Copy link
Collaborator
@p3ck p3ck left a comment

Choose a reason for hiding this comment

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

Thank you for this! And thank you for re-submitting the PR while we transition this plugin from the EDA repo to here.

@p3ck p3ck merged commit a54cd2b into ansible-collections:dev Jul 16, 2025
@ptoscano ptoscano deleted the eda-pr451 branch July 16, 2025 13:40
@susanhooks
Copy link
Contributor

Thanks all!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@p3ck p3ck p3ck approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ptoscano @susanhooks @p3ck
0