[libvirt] [PATCH] tests: Fix detection of expected errors

Jiri Denemark jdenemar at redhat.com
Mon Nov 29 15:47:53 UTC 2010


> >  tests/nwfilterxml2xmltest.c |   15 +++++----------
> >  tests/qemuxml2argvtest.c    |    8 +++-----
> >  2 files changed, 8 insertions(+), 15 deletions(-)
> 
> ACK to the patch as is. It fixes the problem

Thanks, I pushed it.

> qemudBuildCommandLine returns 0 and reports and error. Actually it
> doesn't report an error, because then it should return non-zero. What
> it does in the 3 cases where the test expects and "error" it reports
> actually a warning. For example this one:
> 
> qemuReportError(VIR_ERR_CONFIG_UNSUPPORTED,
>                 _("the QEMU binary %s does not support kvm"),
>                 emulator);
> 
> This is currently reported at error level (VIR_ERR_ERROR) but it
> should actually be reported at warning level (VIR_ERR_WARNING),
> because it's treated as non-fatal.

Yeah, there's a whole bunch of such errors in qemudBuildCommandLine which
don't really result in returning an error code. These should all be changed to
warnings.

I'll prepare a patch for this after the release.

Jirka




More information about the libvir-list mailing list