Change in ovirt-engine[ovirt-engine-4.0]: webadmin: userportal: Fix first vm creation in session

piotr.kliczewski at gmail.com piotr.kliczewski at gmail.com
Mon Aug 8 13:23:17 UTC 2016


Piotr Kliczewski has submitted this change and it was merged.

Change subject: webadmin: userportal: Fix first vm creation in session
......................................................................


webadmin: userportal: Fix first vm creation in session

When creating a vm on a local storage domain it should be
pinned to a specific host that is available for this domain.

When creating a vm on local storage for the first time it wouldn't
be created with a pinned host due to an async code that would not run
in the expected order. When editing such a vm the user would get an
error since it seems that he is editing the pinned host.

To ensure the correct order of execution the handling of the local
storage was moved to the end of the block of code that is run
asynchronously.

Change-Id: I99918ef35099546734cf8835e5abe5bfcef33d08
Bug-Url: https://bugzilla.redhat.com/1284472
Signed-off-by: Jenny Tokar <jtokar at redhat.com>
(cherry picked from commit c12e312)
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/NewVmModelBehavior.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java
2 files changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Alexander Wels: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Jenny Tokar: Verified
  Yanir Quinn: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I99918ef35099546734cf8835e5abe5bfcef33d08
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Jenny Tokar <jtokar at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Yanir Quinn <yquinn at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list