-
Notifications
You must be signed in to change notification settings - Fork 23.8k
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
Fix issue where using listen parameter for handlers in collections could lead to memory issues #83393
Fix issue where using listen parameter for handlers in collections could lead to memory issues #83393
Conversation
…uld lead to memory issues
@ShawnHardwick CI failures are known and being handled in a different PR. |
@Akasurde could you link to that PR? I'd like to follow it |
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
@@ -558,7 +558,7 @@ def search_handlers_by_notification(self, notification: str, iterator: PlayItera | |||
handler.fattributes.get('listen'), | |||
listeners, | |||
templar, | |||
) | |||
).copy() | |||
if handler._role is not None: | |||
for listener in listeners.copy(): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This second copy probably isn't needed anymore
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is still needed as listeners
are mutated within the for
loop.
Superseded by #83400 |
Fix issue where using
listen
parameter for handlers in collections could lead to memory issues on Ansible controller.SUMMARY
Fixes #83392
ISSUE TYPE
ADDITIONAL INFORMATION
See issue for more info.