Re: [ovirt-devel] Ensure processes death by terminating decorator - https://gerrit.ovirt.org/51407

2016-01-24 Thread Yaniv Bronheim
On Sat, Jan 23, 2016 at 8:05 PM, Nir Soffer wrote: > On Sat, Jan 23, 2016 at 7:11 PM, Yaniv Bronheim > wrote: > > any updates about that? > > > > https://gerrit.ovirt.org/#/c/52357/ - this can be verified and get it > > Should wait for execCmd (it should

Re: [ovirt-devel] Ensure processes death by terminating decorator - https://gerrit.ovirt.org/51407

2016-01-23 Thread Yaniv Bronheim
any updates about that? https://gerrit.ovirt.org/#/c/52357/ - this can be verified and get it https://gerrit.ovirt.org/#/c/52349/ - got tiny comment about unneeded kill call https://gerrit.ovirt.org/51407 - Nir can merge Nir - please review https://gerrit.ovirt.org/52646 or take over and update

Re: [ovirt-devel] Ensure processes death by terminating decorator - https://gerrit.ovirt.org/51407

2016-01-23 Thread Nir Soffer
On Sat, Jan 23, 2016 at 7:11 PM, Yaniv Bronheim wrote: > any updates about that? > > https://gerrit.ovirt.org/#/c/52357/ - this can be verified and get it Should wait for execCmd (it should use the new terminating() context manager to ensure process is killed on errors). >

Re: [ovirt-devel] Ensure processes death by terminating decorator - https://gerrit.ovirt.org/51407

2016-01-18 Thread Francesco Romani
- Original Message - > From: "Yaniv Bronheim" > To: "devel" , "Shahar Havivi" , > "Francesco Romani" , "Nir > Soffer" > Sent: Monday, January 18, 2016 11:01:10 AM > Subject: Ensure

[ovirt-devel] Ensure processes death by terminating decorator - https://gerrit.ovirt.org/51407

2016-01-18 Thread Yaniv Bronheim
Hi guys, Following the work to omit deathSignal attribute from our cpopen implementation we posted https://gerrit.ovirt.org/51407 which is ready for use. Currently locations that should use it are: (I wrote above who I expect to check the area and post a patch for that - we'll discuss it during