Bug 1545153
| Summary: | No Storage Domain error while restoring a snapshot | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [oVirt] ovirt-engine | Reporter: | Leon Goldberg <lgoldber> | ||||
| Component: | BLL.Storage | Assignee: | Fred Rolland <frolland> | ||||
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Evelina Shames <eshames> | ||||
| Severity: | medium | Docs Contact: | |||||
| Priority: | unspecified | ||||||
| Version: | future | CC: | bugs, ebenahar, eshenitz, frolland, lgoldber, ratamir | ||||
| Target Milestone: | ovirt-4.3.0 | Keywords: | Regression | ||||
| Target Release: | 4.3.0 | Flags: | rule-engine:
ovirt-4.3+
rule-engine: blocker+ |
||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | ovirt-engine-4.3.0_alpha | Doc Type: | If docs needed, set a value | ||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2019-02-13 07:48:09 UTC | Type: | Bug | ||||
| Regression: | --- | Mount Type: | --- | ||||
| Documentation: | --- | CRM: | |||||
| Verified Versions: | Category: | --- | |||||
| oVirt Team: | Storage | RHEL 7.3 requirements from Atomic Host: | |||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||
| Embargoed: | |||||||
| Attachments: |
|
||||||
|
Description
Leon Goldberg
2018-02-14 11:46:44 UTC
(In reply to Leon Goldberg from comment #0) > test_restore_snapshot_with_an_used_mac_implicitly_assigns_new_mac of OST's > networking suite now consistently fails during snapshot restoration with > "Cannot preview Snapshot. Storage Domain doesn't exist." > > Here is an example job: > http://jenkins.ovirt.org/job/ovirt-system-tests_master_check-patch-el7- > x86_64/4009/ And now in English? What's the flow? 1) Create a VM (vm0) 2) Create a vNic for vm0 (vnic0) with a given MAC address (mac_addr_0) 3) Run vm0 4) Take a snapshot of vm0 (snapshot0) 5) Modify vnic0's MAC address to mac_addr_1 6) Create another VM (vm1) 7) Create a vNic for vm1 (vnic1) with mac_addr_0 8) Stop vm0 9) Restore the snapshot --> fails with "Cannot preview Snapshot. Storage Domain doesn't exist." Versions? Logs? Master. All logs can be found in http://jenkins.ovirt.org/job/ovirt-system-tests_master_check-patch-el7-x86_64/4009/artifact/exported-artifacts/network-suite-master__logs/tests.test_mac_pools/ (In reply to Leon Goldberg from comment #4) > Master. Commit hash? Do you expect me to guess? Created attachment 1395903 [details] engine.log Attaching log referenced in comment 4. No sense it relying on another external system... (In reply to Allon Mureinik from comment #6) > (In reply to Leon Goldberg from comment #4) > > Master. > Commit hash? Do you expect me to guess? Precisely! Which commit would you like to see? There is no single patch I can refer to. Here's Engine's hash in the logs git28f336b (In reply to Allon Mureinik from comment #6) > (In reply to Leon Goldberg from comment #4) > > Master. > Commit hash? Do you expect me to guess? Precisely! Which commit would you like to see? There is no single patch I can refer to. Here's Engine's hash in the logs git28f336b (In reply to Leon Goldberg from comment #9) > (In reply to Allon Mureinik from comment #6) > > (In reply to Leon Goldberg from comment #4) > > > Master. > > Commit hash? Do you expect me to guess? > > Precisely! > > Which commit would you like to see? There is no single patch I can refer to. > Here's Engine's hash in the logs git28f336b That's a good start :-) Thanks! Eyal, I'm unable to reproduce this, but I wonder if your recent work may have affected this. Please take a look? (In reply to Allon Mureinik from comment #11) > Eyal, I'm unable to reproduce this, but I wonder if your recent work may > have affected this. > Please take a look? I'm not sure what you were trying to do but we managed to reproduce this via the UI very simply: 1) Take a snaphot 2) Edit a vNic's MAC address (probably irrelevant) 3) Restore the snapshot ---> fails with "Cannot preview Snapshot. Storage Domain doesn't exist." Reproduced in the GUI, moving to Freddy for a quick relief. Eyal - this was indeed a regression from adding the notion of vm leases to the snapshot. Fred's attached patch fixes it, please make sure you take it too when you backport this work. Sure, the fix was added to the patch backport. (In reply to Eyal Shenitzky from comment #15) > Sure, the fix was added to the patch backport. Was this backported? Which BZ / Gerrit? Yes, this was backported as part of - https://gerrit.ovirt.org/#/c/87480/ Are we planning to backport it to 4.2.z? Already backported as mentioned in a comment #17. Verified. Engine version: 4.2.7.3 (In reply to Evelina Shames from comment #20) > Verified. > Engine version: 4.2.7.3 Sorry, Engine version: 4.3.0-0 This bugzilla is included in oVirt 4.2.7 release, published on November 2nd 2018. Since the problem described in this bug report should be resolved in oVirt 4.2.7 release, it has been closed with a resolution of CURRENT RELEASE. If the solution does not work for you, please open a new bug report. Closed by mistake, moving back to qa -> verified This bugzilla is included in oVirt 4.3.0 release, published on February 4th 2019. Since the problem described in this bug report should be resolved in oVirt 4.3.0 release, it has been closed with a resolution of CURRENT RELEASE. If the solution does not work for you, please open a new bug report. |