Re: [Engine-devel] vdsm/zombiereaper

2013-02-19 Thread Dan Kenigsberg
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.
___
Engine-devel mailing list
Engine-devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-devel


Re: [Engine-devel] vdsm/zombiereaper

2013-02-19 Thread Yaniv Bronheim
Agree.. Until we'll upload better solution for the zombies issue I prefer to 
merge that patch.


- Original Message -
From: Dan Kenigsberg dan...@redhat.com
To: Dead Horse deadhorseconsult...@gmail.com
Cc: engine-devel@ovirt.org, Yaniv Bronhaim ybron...@redhat.com, Royce Lv 
lvro...@linux.vnet.ibm.com, ShaoHe Feng shao...@linux.vnet.ibm.com, 
vdsm-de...@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.
___
Engine-devel mailing list
Engine-devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-devel