Skip to content

Implement for driving libvirt via virsh

Ian Campbell xen-devel 2 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v3 Latest19 patches 19 repliesIn review
  2. v218 patches 42 repliesSuperseded

Patches in v3 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/19TestSupport: Add helper to wait for a guest to shutdownOsstest/TestSupport.pm+18 -61 reply
2/19apt: lock osstest's usages of apt-get against each otherOsstest/Debian.pm Osstest/TestSupport.pm+6 -81 reply
3/19ts-logs-capture: Collect some libvirt logs and capabilitiests-logs-capture+4 -00 replies
4/19Pass host to toolstack()Osstest/TestSupport.pm ts-debian-fixup ts-debian-hvm-install +11 more+34 -300 replies
5/19ts-rumpuserxen-demo-xenstorels: Use standard functions for thingsts-rumpuserxen-demo-xenstorels+2 -40 replies
6/19Toolstack: use get_host_method_object() to manage toolstack selectionOsstest/TestSupport.pm Osstest/Toolstack/libvirt.pm Osstest/Toolstack/xend.pm +1 more+111 -330 replies
7/19TestSupport: always use xl for generic operations.Osstest/TestSupport.pm ts-debian-fixup+12 -43 replies
8/19TestSupport: guest_create and guest_destroy take only a $gho.Osstest/TestSupport.pm ts-debian-hvm-install ts-guest-destroy +3 more+16 -201 reply
9/19Toolstack: Refactor guest lifecycle.Osstest/TestSupport.pm Osstest/Toolstack/libvirt.pm Osstest/Toolstack/xl.pm +2 more+37 -111 reply
10/19Toolstack: Refactor consolecmd handlingOsstest/Toolstack/libvirt.pm Osstest/Toolstack/xl.pm ts-logs-capture+13 -11 reply
11/19Toolstack: Refactor shutdown supportOsstest/Toolstack/libvirt.pm Osstest/Toolstack/xl.pm ts-guest-stop+14 -41 reply
12/19Toolstack: Refactor migration support check.Osstest/Toolstack/libvirt.pm Osstest/Toolstack/xend.pm Osstest/Toolstack/xl.pm +1 more+18 -90 replies
13/19Toolstack: Refactor migration support.Osstest/Toolstack/libvirt.pm Osstest/Toolstack/xl.pm ts-guest-localmigrate +1 more+18 -81 reply
14/19Toolstack: Refactor save/restore supportOsstest/Toolstack/libvirt.pm Osstest/Toolstack/xl.pm ts-guest-saverestore+31 -101 reply
15/19libvirt: Implement initscript restart which has some hope of working.ts-libvirt-build+15 -51 reply
16/19libvirt: Implement shutdown_waitOsstest/TestSupport.pm Osstest/Toolstack/libvirt.pm+10 -20 replies
17/19Toolstack: Remove Command field for all toolstacks.Osstest/Toolstack/libvirt.pm Osstest/Toolstack/xend.pm Osstest/Toolstack/xl.pm+10 -110 replies
18/19ts-guest-start: Use guest_createOsstest/Toolstack/libvirt.pm Osstest/Toolstack/xend.pm Osstest/Toolstack/xl.pm +1 more+1 -42 replies
19/19Toolstack: Pass $gho to create methodOsstest/TestSupport.pm Osstest/Toolstack/libvirt.pm Osstest/Toolstack/xl.pm+5 -31 reply

Reviews trailers per patch

All 19 patches have a Reviewed-by or Acked-by.

PatchIan Jackson
1/19A-b
2/19A-b
3/19A-b
4/19A-b
5/19A-b
6/19A-b
7/19A-b
8/19A-b
9/19A-b
10/19A-b
11/19A-b
12/19A-b
13/19A-b
14/19A-b
15/19A-b
16/19A-b
17/19A-b
18/19A-b
19/19A-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v3 thread

Cover letter [PATCH v3 OSSTEST 0/19] Implement for driving libvirt via virsh

The following series switches osstest to implement the toolstack via get_host_method_object()->method rather than toolstack()->{Command}." method" etc.

This is needed because virsh differs from xm/xl in a few commands.

It also implements partial virsh support (simple lifecycle stuff, but not e.g. migration yet). Due to the ts-migration-check logic this means that the libvirt sequence works in so far as it skips/ignores the migration/save+restore related tests. Wei has a followup series (or at least a WIP?) which adds migration support. I've dropped my WIP patch from last time in favour of that.

Since last time I've addressed all of Ian's review on v2. Lots of passing $gho to things instead of individual properties, switch to a better inheritance scheme for Osstest::Toolstack::xend,

* TestSupport: Add helper to wait for a guest to shutdown
apt: lock osstest's usages of apt-get against each other
A ts-logs-capture: Collect some libvirt logs and capabilities
A Pass host to toolstack()
A ts-rumpuserxen-demo-xenstorels: Use standard functions for things
Toolstack: use get_host_method_object() to manage toolstack selection
TestSupport: always use xl for generic operations.
TestSupport: guest_create and guest_destroy take only a $gho.
Toolstack: Refactor guest lifecycle.
Toolstack: Refactor consolecmd handling
Toolstack: Refactor shutdown support
A Toolstack: Refactor migration support check.
Toolstack: Refactor migration support.
Toolstack: Refactor save/restore support
libvirt: Implement initscript restart which has some hope of working.
A libvirt: Implement shutdown_wait
A Toolstack: Remove Command field for all toolstacks.
A ts-guest-start: Use guest_create
N Toolstack: Pass $gho to create method

* == "TestSupport: Add helper to wait for a guest to shutdown") wasn't posted last time, it was actually part of the "add distro domU testing flight" series, but it is needed here too and I expect this one to land first.

N == New
A == Acked

Ian.