On Wed, Jan 29, 2020 at 10:23:33PM +0100, Philippe Mathieu-Daudé wrote: > Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com> > --- > tests/acceptance/virtio_check_params.py | 8 ++++++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > > diff --git a/tests/acceptance/virtio_check_params.py > b/tests/acceptance/virtio_check_params.py > index 4a417b8ef5..51a2dd76e8 100755 > --- a/tests/acceptance/virtio_check_params.py > +++ b/tests/acceptance/virtio_check_params.py > @@ -77,8 +77,12 @@ class VirtioMaxSegSettingsCheck(Test): > vm.set_machine(mt["name"]) > for s in VM_DEV_PARAMS[dev_type_name]: > vm.add_args(s) > - vm.launch() > - query_ok, props, error = self.query_virtqueue(vm, dev_type_name) > + try: > + vm.launch() > + query_ok, props, error = self.query_virtqueue(vm, > dev_type_name) > + except: > + query_ok = False > + error = sys.exc_info()[0]
I would prefer to do this inside query_virtqueue(), but: Reviewed-by: Eduardo Habkost <ehabk...@redhat.com> -- Eduardo
