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