Change in ovirt-engine[master]: frontend: Properly enable instance images on New VM from Bla...

Code Review gerrit at ovirt.org
Thu Nov 24 12:50:44 UTC 2016


>From Tomas Jelinek <tjelinek at redhat.com>:

Tomas Jelinek has submitted this change and it was merged.

Change subject: frontend: Properly enable instance images on New VM from Blank tmpl.
......................................................................


frontend: Properly enable instance images on New VM from Blank tmpl.

In the case the user switches to blank template the initDisks() method which
among other things also updates the availability of instance images editor is
not called therefore after switching from template with 1+ disks the editor
stayed disabled even for the blank template.

Fixed by addding enablement of the instance images editor also to the specific
flow for the blank template in the NewVmModelBehavior.

Change-Id: I3cd612ac0a961a79e9a87ca46e28f6c877388d71
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1376339
Signed-off-by: Martin Betak <mbetak at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/NewVmModelBehavior.java
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Tomas Jelinek: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Martin Betak: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3cd612ac0a961a79e9a87ca46e28f6c877388d71
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Betak <mbetak at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Betak <mbetak at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list