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

rgolan at redhat.com rgolan at redhat.com
Sun Aug 7 12:51:52 UTC 2016


Roy Golan 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>
---
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
  Roy Golan: Looks good to me, but someone else must approve
  Jenny Tokar: Verified
  Yanir Quinn: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I99918ef35099546734cf8835e5abe5bfcef33d08
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Jenny Tokar <jtokar at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Andrej Krejcir <akrejcir at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Phillip Bailey <phbailey at redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan 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