Skip to content

CA-425192/CP-311612/CP-311613: backport quicktest SMAPIv3 fixes to LCM branch#6959

Merged
edwintorok merged 2 commits intoxapi-project:26.1-lcmfrom
edwintorok:26.1-lcm
Mar 20, 2026
Merged

CA-425192/CP-311612/CP-311613: backport quicktest SMAPIv3 fixes to LCM branch#6959
edwintorok merged 2 commits intoxapi-project:26.1-lcmfrom
edwintorok:26.1-lcm

Conversation

@edwintorok
Copy link
Copy Markdown
Member

Backport #6908 to the LCM branch.
These latent bugs have existed in the tests for a long time, but previously our test system (XenRT) picked machines randomly, so it would've been unlikely to pick a machine which has a local 4K SR (SMAPIv3 instead of SMAPIv1), and even if it did, the test would seemingly pass on a rerun (because another machine would get picked).
We are now explicitly running the test multiple times on different hardware categories, and some of those tests are now failing 100% of the time.

This seems to be a SMAPIv1 specific implementation detail (content_id in
VDI.other-config), that is missing on SMAPIv3 SRs.

The rest of the test works on SMAPIv3, so skip just this step.

Signed-off-by: Edwin Török <edwin.torok@citrix.com>
(cherry picked from commit ceebe4b)
VHD export takes various shortcuts, and delta export only works if the source is also VHD.
Otherwise it fails with:
`Failure("Cannot compute differences on non-VHD images")`

If the source is already qcow2, like for SMAPIv3, then this is not (yet)
expected to work, so skip those tests on SMAPIv3 for now.

Note that regular export works, it is just the delta export that fails (specifying `base=`
in `xe vdi-export`).

Signed-off-by: Edwin Török <edwin.torok@citrix.com>
(cherry picked from commit 8ac4476)
@edwintorok edwintorok added this pull request to the merge queue Mar 20, 2026
Merged via the queue into xapi-project:26.1-lcm with commit 0678df0 Mar 20, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants