[libvirt] [PATCH 2/4] tests: Skip Xen-HVM tests for root on dom0
Eric Blake
eblake at redhat.com
Wed Oct 12 18:31:00 UTC 2011
On 10/12/2011 02:11 AM, Philipp Hahn wrote:
> Several tests fail when run as root on a Xen-dom0-system, since
> virInitialize() then succeeds to open /proc/xen/privcmd and returns the
> actual supported features instead of the faked one when calling
> xenHypervisorMakeCapabilitiesInternal(). Since Xen-4.1 supports
> additional features like "hap" and "viridian", the xencapstest fails.
I'm not sure I like this. I'd much rather fix things to avoid parsing
live /proc/xen/privcmd when run from the testsuite, perhaps by providing
a hook function that only the testsuite overrides (we've done this
elsewhere). It is a bug if a testsuite ever depends on live system
information, and skipping the test just papers over the real bug.
>
> For now disable those 4 tests and return EXIT_AM_SKIP for them.
>
> A better fix would probably just check for the minimum required features
> instead of comparing the two XML documents for bit-equivalence.
Or better yet, fixing the test to parse hard-coded information captured
in a tests/*data/... directory associated with the test rather than
reading /proc/xen/privcmd in the first place.
--
Eric Blake eblake at redhat.com +1-801-349-2682
Libvirt virtualization library http://libvirt.org
More information about the libvir-list
mailing list