Bug 822331

Summary:	Virt-manager should provide better error message when try to add the 5th IDE
Product:	[Community] Virtualization Tools	Reporter:	Geyang Kong <gkong>
Component:	virt-manager	Assignee:	Cole Robinson <crobinso>
Status:	CLOSED UPSTREAM	QA Contact:
Severity:	low	Docs Contact:
Priority:	low
Version:	unspecified	CC:	acathrow, berrange, crobinso, cwei, dallan, jwu, mjenner, mzhan, zpeng
Target Milestone:	---
Target Release:	---
Hardware:	x86_64
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2014-01-29 16:07:28 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:

Description Geyang Kong 2012-05-17 05:29:37 UTC

Description of problem:
  Virt-manager should provide better error message when try to add the 5th IDE device.

Version-Release number of selected component (if applicable):
virt-manager-0.9.0-14.el6.x86_64

How reproducible:
100%

Steps to Reproduce:
1. Make sure there is a guest.
2. Try to add 5 IDE devices on the guest through virt-manager.
3. Check the error message.
4. Try to add the 5th ide device through libvirt
   [root@KP-T1 ~]# virsh attach-disk --domain T1 --source /var/lib/libvirt/images/1.iso --target hde --sourcetype file --driver qemu --subdriver qcow2 --type cdrom --persistent
   Disk attached successfully
5. Start the domain T1
   [root@KP-T1 ~]# virsh start T1
   error: Failed to start domain T1
   error: internal error Only 1 ide controller is supported

Actual results:
1. After step 3. Following error message appeared
-------------------------------------------------------------------------------
Uncaught error validating hardware input: No more space for disks of type 'hd'
-------------------------------------------------------------------------------
Expected results:
1. Virt-manager should provide more clearer error message, e.g. "Only 4 ide devices are supported by an IDE controller"

Additional info:

Comment 2 Cole Robinson 2014-01-29 16:07:28 UTC

Fixed upstream now