Bug 1820016
Summary: | Failed to create a disk only snapshot with --reuse-external to a existing qcow2 image | ||||||
---|---|---|---|---|---|---|---|
Product: | Red Hat Enterprise Linux Advanced Virtualization | Reporter: | yisun | ||||
Component: | libvirt | Assignee: | Peter Krempa <pkrempa> | ||||
Status: | CLOSED ERRATA | QA Contact: | yisun | ||||
Severity: | urgent | Docs Contact: | |||||
Priority: | urgent | ||||||
Version: | 8.2 | CC: | chhu, jdenemar, jen, jsuchane, kanderso, lcheng, lmen, lsvaty, michal.skrivanek, mtessun, pkrempa, virt-maint, xuzhang, yisun, ymankad | ||||
Target Milestone: | rc | Keywords: | Automation, Regression, TestBlockerForLayeredProduct | ||||
Target Release: | 8.3 | ||||||
Hardware: | x86_64 | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Fixed In Version: | libvirt-6.0.0-17.el8 | Doc Type: | If docs needed, set a value | ||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2020-05-05 09:59:02 UTC | Type: | Bug | ||||
Regression: | --- | Mount Type: | --- | ||||
Documentation: | --- | CRM: | |||||
Verified Versions: | Category: | --- | |||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
Cloudforms Team: | --- | Target Upstream Version: | |||||
Embargoed: | |||||||
Bug Depends On: | |||||||
Bug Blocks: | 1818655, 1820068 | ||||||
Attachments: |
|
Description
yisun
2020-04-02 04:42:48 UTC
Oops, this is a regression caused by upstream commit: commit 2ace7a87a8aced68c2504fd4dd4e2df4302c3eeb Author: Peter Krempa <pkrempa> Date: Mon Mar 30 11:18:37 2020 +0200 qemuDomainSnapshotDiskPrepareOne: Don't load the relative path with blockdev Since we are refreshing the relative paths when doing the blockjobs we no longer need to load them upfront when doing the snapshot. Fixed upstream: commit ae64a75a8713cf14b25b40078766c2da93e001ff Author: Peter Krempa <pkrempa> Date: Thu Apr 2 07:29:58 2020 +0200 qemuDomainSnapshotDiskPrepareOne: Fix logic of relative backing store update Commit 2ace7a87a8aced68c250 introduced a logic bug by an improperly modified condition where we'd skip to the else branch when reusing of external images was requested and blockdev is available. The original intentions were to skip the backing store update with blockdev. Fix it by only asserting the boolean which was used to track whether we support update of the backing store only when blockdev is not present along with the appropriate rename. Test with libvirt-6.0.0-17.module+el8.2.0+6257+0d066c28.x86_64 And result is PASS Snapshot auto test: No regression failures https://libvirt-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/view/libvirt/view/RHEL-8.2%20x86_64/job/libvirt-RHEL-8.2-runtest-x86_64-function-snapshot/33/testReport/ (The failed case is a expecting behavior after libvrt commit 045a8e19, not related to current issue, auto case will be updated) Blockcopy auto test: No regression failures https://libvirt-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/view/libvirt/view/RHEL-8.2%20x86_64/job/libvirt-RHEL-8.2-runtest-x86_64-function-block_copy_shallow/27/testReport/ Blockcommit blockpull auto test: No regression failures https://libvirt-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/view/libvirt/view/RHEL-8.2%20x86_64/job/libvirt-RHEL-8.2-runtest-x86_64-function-block_job_commit_pull/34/testReport/ (Well, the result is a little ugly, all failures are automation script issues since we have some libvirt behavior changed in this release, auto owner will modify the script in future, if you're interested about failure reason: https://projects.engineering.redhat.com/browse/LIBVIRTEIG-896) Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHBA-2020:2017 |