Agree.. Until we'll upload better solution for the zombies issue I prefer to merge that patch.
----- Original Message ----- From: "Dan Kenigsberg" danken@redhat.com To: "Dead Horse" deadhorseconsulting@gmail.com Cc: engine-devel@ovirt.org, "Yaniv Bronhaim" ybronhei@redhat.com, "Royce Lv" lvroyce@linux.vnet.ibm.com, "ShaoHe Feng" shaohef@linux.vnet.ibm.com, vdsm-devel@fedorahosted.org Sent: Tuesday, February 19, 2013 3:33:05 PM Subject: Re: [Engine-devel] vdsm/zombiereaper
On Mon, Feb 18, 2013 at 04:21:16PM -0600, Dead Horse wrote:
Any movement or thoughts yet on this one: http://gerrit.ovirt.org/#/c/11492/
It still has vdsm master broken.
Guys, I think that DHC is right - we cannot keep deliberating forever. Having a process leak is bad, but keeping master broken for so long is even worse.
So unless anyone violently objects, I'll revert the offending patch.
Dan.