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

Event subscribe fix #25832

Merged
merged 2 commits into from
Jul 29, 2015
Merged

Conversation

DmitryKuzmenko
Copy link
Contributor

Fix memory consumption issue produced by unwanted subscription to bus messages.
This is for #25557

Dmitry Kuzmenko added 2 commits July 29, 2015 17:44
Once subscribed to publisher SUB socket gets collecting all incoming
messages that is unwanted behavior for fire-only events.
Fixed by using listen=<True|False> constructor argument.
cachedout pushed a commit that referenced this pull request Jul 29, 2015
@cachedout cachedout merged commit 3849790 into saltstack:develop Jul 29, 2015
@basepi basepi added the bugfix-bckport will be be back-ported to an older release branch by creating a PR against that branch label Aug 3, 2015
@basepi
Copy link
Contributor

basepi commented Aug 3, 2015

Backport to 2015.8

@rallytime rallytime added ZZZ[Done]-back-ported-bf RETIRED The pull request has been back-ported to an older branch. and removed bugfix-bckport will be be back-ported to an older release branch by creating a PR against that branch labels Aug 3, 2015
cachedout pushed a commit that referenced this pull request Aug 4, 2015
@DmitryKuzmenko DmitryKuzmenko deleted the event_subscribe_fix branch August 13, 2015 08:43
@mimianddaniel
Copy link

Can this be backport to 2015.5 as well?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ZZZ[Done]-back-ported-bf RETIRED The pull request has been back-ported to an older branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants