09.12.2019 18:15, Max Reitz wrote: > On 03.12.19 14:33, Vladimir Sementsov-Ogievskiy wrote: >> 03.12.2019 16:32, Vladimir Sementsov-Ogievskiy wrote: >>> 11.11.2019 19:02, Max Reitz wrote: >>>> Signed-off-by: Max Reitz<mre...@redhat.com> >>> >>> >>> Reviewed-by: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com> >>> >> >> >> Oops, stop. Why do you remove line "self.vm.shutdown()" ? > > Because we don’t need it. tearDown() does it anyway. I suppose I > should mention it in the commit message. >
Yes... But actually, better to remove extra shutdown from all test cases, not from one, and than it would be separate patch. Extra shutdown is left in (considering only class TestRepairQuorum): test_pause test_cancel_after_ready test_cancel test_complete -- Best regards, Vladimir
