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

Multiprocessing Manager resources (Queue) to be freed up during task stop (CherryPick PR#15212 to 202205) #15662

Merged
merged 1 commit into from
Jun 30, 2023

Conversation

sg893052
Copy link
Contributor

Why I did it

Fix for issue #14964

Work item tracking
  • Microsoft ADO (number only):

How I did it

Free up Multiprocessing Manager resource at task stop request
[self.mpmgr.shutdown() in task_stop]

How to verify it

time systemctl stop system-health.service

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@sg893052
Copy link
Contributor Author

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 15662 in repo sonic-net/sonic-buildimage

@sg893052
Copy link
Contributor Author

@yxieca The build failure is irrelevant to this PR change. Please help retrigger the build as I don't have the permission. Thanks!

@yxieca
Copy link
Contributor

yxieca commented Jun 30, 2023

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yxieca yxieca merged commit 9e512bd into sonic-net:202205 Jun 30, 2023
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants