openSUSE / salt

openSUSE and SUSE patches and backports for SaltStack
Apache License 2.0
22 stars 52 forks source link

Refactor batch_async implementation #640

Closed vzhestkov closed 5 months ago

vzhestkov commented 6 months ago

What does this PR do?

Refactors batch_async implementation to make it using single Event Publisher stream per MWorker. In this case we don't really need to open and close in time such amount of streams as it was made before and fixed with https://github.com/openSUSE/salt/pull/632

This refactoring also implements the threshold config parameter for batch_async to control if the batch async call is really required in runtime.

What issues does this PR fix or reference?

Tracks: https://github.com/SUSE/spacewalk/issues/23526 Supersedes: https://github.com/openSUSE/salt/pull/632

Previous Behavior

Remove this section if not relevant

New Behavior

Remove this section if not relevant

Merge requirements satisfied?

[NOTICE] Bug fixes or features added to Salt require tests.

Commits signed with GPG?

Yes/No

Please review Salt's Contributing Guide for best practices.

See GitHub's page on GPG signing for more information about signing commits with GPG.