Change in ovirt-engine[master]: webadmin: Check for VM status when activating IDE disks

derez at redhat.com derez at redhat.com
Thu Jun 25 08:13:00 UTC 2015


Daniel Erez has submitted this change and it was merged.

Change subject: webadmin: Check for VM status when activating IDE disks
......................................................................


webadmin: Check for VM status when activating IDE disks

When the VM is down and one tries to attach and activate an IDE disk, he
gets an error message that tells him he can't activate it and that he
should turn off the VM first. In addition, the disk is attached but not
activated although the VM is down.
This patch adds a check for the VM status to fix this issue.

Change-Id: Ic2fe5e5974890ee591f6a903d9ea9ae024a22ac3
Bug-Url: https://bugzilla.redhat.com/1167248
Signed-off-by: Idan Shaby <ishaby at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/AttachDiskModel.java
1 file changed, 13 insertions(+), 3 deletions(-)

Approvals:
  Daniel Erez: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Idan Shaby: Verified



-- 
To view, visit https://gerrit.ovirt.org/42547
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic2fe5e5974890ee591f6a903d9ea9ae024a22ac3
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list