Bug 1309294 - Failed to syncDbRecords after successful live merge because host is down
Summary: Failed to syncDbRecords after successful live merge because host is down
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: BLL.Storage
Version: 3.6.3
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ovirt-3.6.5
: 3.6.5
Assignee: Ala Hino
QA Contact: Elad
URL:
Whiteboard:
: 1309293 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-02-17 12:01 UTC by Ala Hino
Modified: 2016-04-21 14:39 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-04-21 14:39:25 UTC
oVirt Team: Storage
Embargoed:
rule-engine: ovirt-3.6.z+
ylavi: planning_ack+
amureini: devel_ack+
acanan: testing_ack+

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 53606 0 master MERGED core: Handle exception if calling VDSM fails 2016-02-23 14:50:01 UTC
oVirt gerrit 54132 0 ovirt-engine-3.6 MERGED refactor: Split syncDbRecords code 2016-03-02 10:02:00 UTC
oVirt gerrit 54133 0 ovirt-engine-3.6 MERGED core: Wrap syncDbRecords code in a transaction 2016-03-02 10:03:49 UTC
oVirt gerrit 54134 0 ovirt-engine-3.6 MERGED core: Move getImageInfo call outside of transaction boundary 2016-03-02 10:05:20 UTC
oVirt gerrit 54135 0 ovirt-engine-3.6 MERGED core: Handle exception if calling VDSM fails 2016-03-02 10:06:52 UTC
oVirt gerrit 54136 0 ovirt-engine-3.6 MERGED core: Remove index from curly braces 2016-03-02 10:07:37 UTC

Description Ala Hino 2016-02-17 12:01:23 UTC 
Description of problem:
After successful live merge, if during host goes down after completion of live merge but before calling 'syncDbRecords', flow fails because 'syncDbRecords' calls vdsm to get image info.

This bug is to make sure we remove the call to vdsm outside of 'syncDbRecords'.

Version-Release number of selected component (if applicable):


How reproducible:
100%

Steps to Reproduce:
1. (Lively) remove a snapshot
2. After/before DESTROY_IMAGE_CHECK completes, take shutdown host
3. Update the db fails

Actual results:
Db update fails

Expected results:
Db to be updated with the correct data regarding live merge

Additional info:
Comment 1 Allon Mureinik 2016-02-18 11:45:52 UTC 
*** Bug 1309293 has been marked as a duplicate of this bug. ***
Comment 2 Allon Mureinik 2016-02-28 13:26:54 UTC 
This patch should be backported to the ovirt-engine-3.6 branch. Setting the status back to POST.
Comment 3 Elad 2016-04-04 12:26:58 UTC 
Tested the following on 2 builds:
rhevm-3.6.3-0.1.el6.noarch - doesn't contain the fix
rhevm-3.6.5-0.1.el6.noarch - contains the fix


1. Remove a snapshot lively
2. After DESTROY_IMAGE_CHECK completes, stopped vdsm

On the older build, the volume became Illegal and on the latest it got removed.
Note You need to log in before you can comment on or make changes to this bug.