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(a)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(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org