Bug 710913

Summary: [vdsm] in case virtIO & IDE are used, VDSM builds XML with both disks as 'hda'
Product: Red Hat Enterprise Linux 6 Reporter: Haim <hateya>
Component: vdsmAssignee: Federico Simoncelli <fsimonce>
Status: CLOSED ERRATA QA Contact: Kiril Nesenko <knesenko>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.2CC: abaron, bazulay, dfediuck, iheim, ilvovsky, knesenko, mgoldboi, yeylon, ykaul, yzaslavs
Target Milestone: rcKeywords: Regression
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: vdsm-4.9-75.el6 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-12-06 07:20:29 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 709732    

Description Haim 2011-06-05 18:46:21 UTC
Description of problem:

case: 

- VM with 2 disks 
  1) one IDE
  2) one virtIO
- VDSM builds XML with both disks symboled as 'hda'.
- VDSM then reports both disk as 'hda' which causes several problems such as backend doesn't update actual disk size of second disk.


                <disk device="disk" type="file">
                        <source file="/rhev/data-center/7afb7b98-8b71-11e0-8474-00262df77985/60139b36-5583-4510-9ab1-323467cd793d/images/9c44e5d3-b59d-4e63-8cd
0-42434f29e99c/d7c309fa-1cbe-46f6-b2d2-bdba1ceea069"/>
                        <target bus="ide" dev="hda"/>
                        <serial>63-8cd0-42434f29e99c</serial>
                        <driver cache="writeback" error_policy="stop" io="threads" name="qemu" type="raw"/>
                </disk>
                <disk device="disk" type="file">
                        <source file="/rhev/data-center/7afb7b98-8b71-11e0-8474-00262df77985/60139b36-5583-4510-9ab1-323467cd793d/images/ca361874-019a-486e-945
9-c7a81dac5f5e/5f5a865c-80e0-4212-9f29-34c3cc7bbe82"/>
                        <target bus="virtio" dev="hda"/>
                        <serial>6e-9459-c7a81dac5f5e</serial>
                        <driver cache="writeback" error_policy="stop" io="threads" name="qemu" type="raw"/>
                </disk>

repro steps: mentioned above. 
repro rate: 100%
attached logs

Comment 2 Dan Kenigsberg 2011-06-05 20:22:06 UTC
The problem is not the reuse of "hda" as disk index in libvirt xml, but the fact that that name (instead of disk alias?) is used as key to statistics dictionary.

Comment 6 Federico Simoncelli 2011-06-07 07:45:43 UTC
commit af385bf9a06777b1c3203d3564c5c2b7453dc145
Author: Federico Simoncelli <fsimonce>
Date:   Mon Jun 6 09:48:02 2011 -0400

    BZ#710913 Use different naming for virtio disks
    
    Change-Id: Icb3464ae98d6e20b73a4f4e9623f366f19c5b167

http://gerrit.usersys/545

Comment 9 Kiril Nesenko 2011-06-20 09:18:54 UTC
Different names are used for virtio disks (vda,vdb...) 
Checked on vdsm-4.9-75.el6.x86_64.
Verified
ic123

Comment 10 errata-xmlrpc 2011-12-06 07:20:29 UTC
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.

http://rhn.redhat.com/errata/RHEA-2011-1782.html