[Freeipa-devel] [PATCH 0155] ipatests: Kill winbindd process after uninstall

[Martin Kosek]

On 02/25/2014 07:15 PM, Alexander Bokovoy wrote:
> On Tue, 25 Feb 2014, Tomas Babej wrote:
>> Hi,
>>
>> As a part of a better cleanup procedure in the integration tests,
>> make sure that winbindd is not running after uninstalling the IPA
>> server.
> Better patch 0140  attached. We simply need to stop and disable winbind in
> adtrustinstance.uninstall()

Looks good to me (and a better approach than Tomas' 155 it seems). Since you
are touching this section anyway, can you please also replace bare except with
"except Exception:"?

It will allow admin to CTRL+C the stopping process when needed.

Thanks,
Martin