Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1136808

Summary: [ImportDomain][LOG]Confusing ERROR logs "Could not get populated disk" when importing a domain
Product: [Retired] oVirt Reporter: Ori Gofen <ogofen>
Component: ovirt-engine-coreAssignee: Maor <mlipchuk>
Status: CLOSED CURRENTRELEASE QA Contact: Aharon Canan <acanan>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 3.5CC: acanan, amureini, bugs, ecohen, gklein, iheim, ogofen, rbalakri, s.kieske, tnisan, yeylon
Target Milestone: ---   
Target Release: 3.5.1   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: storage
Fixed In Version: ovirt-3.5.1_rc1 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-01-19 07:09:45 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:
Bug Depends On:    
Bug Blocks: 1193195    
Attachments:
Description Flags
vdsm+engine logs none

Description Ori Gofen 2014-09-03 11:11:50 UTC
Created attachment 934044 [details]
vdsm+engine logs

Description of problem:
The operation of importing a populated storage domain to a new setup,fails every time on logs:
2014-09-03 13:08:59,953 ERROR [org.ovirt.engine.core.bll.GetUnregisteredDisksQuery] (ajp--127.0.0.1-8702-2) [3cf8853c] Could not get populated disk, reason: null
2014-09-03 13:08:59,962 INFO  [org.ovirt.engine.core.vdsbroker.irsbroker.GetVolumesListVDSCommand] (ajp--127.0.0.1-8702-2) [3cf8853c] START, GetVolumesListVDSCommand( storagePoolId = 00000002-0002-0002-0002-000000000389, ignoreFailoverLimit = false, storageDomainId = ffca62e7-397c-4533-9df3-446cc6580ee6, imageGroupId = 989efd20-f30d-44ae-9237-b627a1a86a6e), log id: 523f6582

the operation itself is successful

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

How reproducible:
100%

Steps to Reproduce:
1.import a populated domain to a new setup(not as master domain)

Actual results:
the operation fails on logs

Expected results:
there is no use for confusing ERRORs if the operation is successful,otherwise, if there's a problem with the import flow,it should be fixed

Additional info:

Comment 1 Ori Gofen 2014-09-03 12:06:44 UTC
*** Bug 1136809 has been marked as a duplicate of this bug. ***

Comment 2 Allon Mureinik 2014-09-04 10:11:08 UTC
I'm not sure I understand the description of this bug - is it about redundant ERRORs in the log for a successful operation, or does the operation just FAIL?

Comment 3 Ori Gofen 2014-09-07 07:49:48 UTC
Allon,option 1.
False Errors can be very confusing,we shouldn't allow it in a sensitive operation like recovering a domain.

(In reply to Allon Mureinik from comment #2)
> I'm not sure I understand the description of this bug - is it about
> redundant ERRORs in the log for a successful operation, or does the
> operation just FAIL?

Comment 4 Allon Mureinik 2014-09-08 17:02:33 UTC
(In reply to Ori from comment #3)
> Allon,option 1.
> False Errors can be very confusing,we shouldn't allow it in a sensitive
> operation like recovering a domain.
Agreed, completely.
Just needed to understand what exactly the problem is before we can fix it :-)

Comment 5 Maor 2014-09-21 18:08:42 UTC
The origin of the ERROR is that we try to register a disk with snapshots:
GetVolumesList for image 558be124-aad0-4da8-950c-339a7892319d
returns two volumes: [a49a902c-1edf-489c-8edd-0fce8be07bde, 370d21cd-aeed-4063-930b-3c36fc8b0ee6]
That is why the registration fails.

The fix is on the way

Comment 6 Sandro Bonazzola 2015-01-15 14:15:15 UTC
This is an automated message: 
This bug should be fixed in oVirt 3.5.1 RC1, moving to QA