Bug 771879

Summary: [RHEVM Backend][text] Warning message should be changed in a UI when adding disks to a VM from guide me.
Product: Red Hat Enterprise Virtualization Manager Reporter: Kiril Nesenko <knesenko>
Component: ovirt-engineAssignee: Vered Volansky <vered>
Status: CLOSED WORKSFORME QA Contact:
Severity: low Docs Contact:
Priority: low    
Version: 3.0.0CC: amureini, dfediuck, iheim, lpeer, ofrenkel, Rhev-m-bugs, yeylon, ykaul
Target Milestone: ---   
Target Release: 3.1.0   
Hardware: x86_64   
OS: Linux   
Whiteboard: storage
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-10-03 11:19:17 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Storage RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
logs none

Description Kiril Nesenko 2012-01-05 10:07:01 UTC
Created attachment 550872 [details]
logs

Description of problem:
When creating a new VM 'guide me' window appears. I am creating a disk from that window and VM moves to 'imaged locked'. Trying to add another disk from 'guide me' window and error message appears in a UI ""

2012-01-05 11:41:37,764 WARN  [org.ovirt.engine.core.bll.AddDiskToVmCommand] (http-0.0.0.0-8443-2) CanDoAction of action AddDiskToVm failed. Reasons:ACTION_TYPE_FAILED_VM_IS_NOT_DOWN,VAR__ACTION__ADD,VAR__TYPE__VM_DISK


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

How reproducible:
Always

Steps to Reproduce:
1. Create a new VM 
2. From 'guide me' add disk to a VM  
3. Add another disk while VM is in lock state 
  
Actual results:
Error message in a UI - "Can not add Virtual Macine Disk. VM is not down"

Expected results:
message should be changed to "Can not add Virtual Macine Disk while VM is in lock state" 

Additional info:

Comment 2 Itamar Heim 2012-05-31 09:10:55 UTC
indeed - need to check if adding disk, then another disk is allowed.
if so - close, else need to add validation or warning

Comment 3 Allon Mureinik 2012-09-30 07:54:35 UTC
Vered - IMHO this is no longer relevant.
If it isn't, please close it.
If it is, please fix.

Comment 4 Vered Volansky 2012-10-03 09:45:35 UTC
Doesn't reproduce using commit id 9d34d62525bf51b2b775b456ec789a92798dc7b3

Comment 5 Vered Volansky 2012-10-03 11:19:17 UTC
Works with Change-Id: I61d39f8559bcffdc154b0b2c01d5303cf2ff1e1a