Nir Soffer has posted comments on this change.
Change subject: vdsm: Shuting down protocol detector fails
......................................................................
Patch Set 1:
I looked at the log myself, and there was no double stop.
This looks like a race in the shutdown procedure:
1. vdsm get SIGTERM
2. main thread wakes up and start shutdown procedure
3. main thread invoke MultiProtocolAcceptor.stop setting
self._is_running to False
4. acceptor threads wakes up, probably because of the received
signal, and find that self._is_running is False.
5. acceptor thread cleans up, closing the pipe
6. main thread try to write to closed pipe
So it seems that the correct solution would be to ignore EPIPE which
means "already stopped".
--
To view, visit
http://gerrit.ovirt.org/29556
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I82119a61835fe335f2aa5da29fb8d3f2b8ae33fc
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No