yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #49329
[Bug 1569404] [NEW] Remove threading before process forking
Public bug reported:
Forking processes when a few threads are already running is potentially
unsafe operation and could cause a lot of problems because only current
thread will continue working in child thread. Any locked by other thread
resource will remain locked forever.
We faced with this problem during oslo.messaging development and added
workaround to hide this problem:
https://review.openstack.org/#/c/274255/ I tried to fix this problem in
oslo.service: https://review.openstack.org/#/c/270832/ but oslo folks
said that this fix is ugly and it is wrong way to add workarounds to
common libraries because projects use them incorrectly. I think that is
fare.
** Affects: neutron
Importance: Undecided
Assignee: Dmitriy Ukhlov (dukhlov)
Status: In Progress
** Changed in: neutron
Assignee: (unassigned) => Dmitriy Ukhlov (dukhlov)
** Changed in: neutron
Status: New => In Progress
--
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1569404
Title:
Remove threading before process forking
Status in neutron:
In Progress
Bug description:
Forking processes when a few threads are already running is
potentially unsafe operation and could cause a lot of problems because
only current thread will continue working in child thread. Any locked
by other thread resource will remain locked forever.
We faced with this problem during oslo.messaging development and added
workaround to hide this problem:
https://review.openstack.org/#/c/274255/ I tried to fix this problem
in oslo.service: https://review.openstack.org/#/c/270832/ but oslo
folks said that this fix is ugly and it is wrong way to add
workarounds to common libraries because projects use them incorrectly.
I think that is fare.
To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1569404/+subscriptions
Follow ups