]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
service child process normal SIGTERM exit
authorBrant Knudson <bknudson@us.ibm.com>
Wed, 22 Apr 2015 19:57:53 +0000 (14:57 -0500)
committerBrant Knudson <bknudson@us.ibm.com>
Wed, 22 Apr 2015 20:01:07 +0000 (15:01 -0500)
commitb05274c96bc48e749e6ad21633b39158838c313e
treed418e04bbfd8fe0a9d322def27ede0bf9c0dbed9
parente1f92186656f91b0686264b0a8640ae13c9a0bcd
service child process normal SIGTERM exit

service.py had some code where the child process would catch the
SIGTERM from the parent just so it could exit with 1 status rather
than with an indication that it exited due to SIGTERM. When
shutting down the parent doesn't care in what way the child ended,
only that they're all gone, so this code is unnecessary.

Also, for some reason this caused the child to never exit while
there was an open connection from a client. Probably something
with eventlet and signal handling.

This is a cherry-pick of oslo-incubator commit
702bc569987854b602ef189655c201c348de84cb .

Change-Id: I87f3ca4da64fb8070e4d6c3876a2f1ce1a3ca71d
Closes-Bug: #1446583
(cherry picked from commit d73ac96d18c66aa4dd5b7d7f8d7c22e8f8434683)
cinder/openstack/common/service.py