[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 17/29] tests/acceptance/virtio_check_params: Improve excep
From: |
Eduardo Habkost |
Subject: |
Re: [PATCH v2 17/29] tests/acceptance/virtio_check_params: Improve exception logging |
Date: |
Thu, 6 Feb 2020 14:54:41 -0500 |
On Wed, Jan 29, 2020 at 10:23:33PM +0100, Philippe Mathieu-Daudé wrote:
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> 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 <address@hidden>
--
Eduardo
- Re: [PATCH v2 17/29] tests/acceptance/virtio_check_params: Improve exception logging,
Eduardo Habkost <=