27019361523
80%
master: 78%

Ran 05 Jun 2026 02:01PM UTC

Jobs 1

Files 34

Run time 1min

Badge

Embed ▾

Committed 05 Jun 2026 02:00PM UTC coverage: 80.459%. Remained the same

Build # 27019361523

Build Type

push

github

Committed by

psafont

Commit Message

xapi_vm_snapshot: change VM.revert to use VDI.revert

The code now tries to call VDI.revert on all the disk VDIs, and if that
fails, it falls back to the original revert method that destroys VM
disks, clones snapshot disks and then fixes up metadata.

This means that the successfully reverted disks must be excluded from
the original method. This is done by introducing sets of VDIs and VBDs
and using set logic on them.

The CD disks and the suspend VDI are treated like before: destroy +
clone.

The code is more convoluted that I would have liked because of existing
clone infrastructure mixes VBDs with VDIs, so they need to be converted
back and forth to be able to run the previous method correctly,
especially for updating the snapshot_of field of the cloned snapshot
disks.

Quicktests encodes the difference in behaviour from the original method
to the native one: now VDI UUIDs are not changed when reverting.

Signed-off-by: Pau Ruiz Safont <pau.safont@vates.tech>

Coverage Stats

3504 of 4355 relevant lines covered (80.46%)

0.8 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	python3.11 - 27019361523.1	05 Jun 2026 02:01PM UTC	34	80.46	GitHub Action Run

psafont / xen-api / 27019361523
80%
master: 78%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 27019361523

psafont / xen-api / 27019361523 80% master: 78%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 27019361523

psafont / xen-api / 27019361523
80%
master: 78%

README BADGES
x