Tomas Jelinek has uploaded a new change for review.

Change subject: webadmin: if next run configuration exists the edit VM may load 
forever
......................................................................

webadmin: if next run configuration exists the edit VM may load forever

If the nextRunConfiguration exists, the model.stopProgress(); was not called in
updateVirtioScsi() which caused the progress bar to show forever.

Change-Id: Iea63c3557d5b962d56ba2c0e1f97356f41474aef
Bug-Url: https://bugzilla.redhat.com/1180038
Signed-off-by: Tomas Jelinek <[email protected]>
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/instancetypes/InstanceTypeManager.java
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/79/36679/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/instancetypes/InstanceTypeManager.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/instancetypes/InstanceTypeManager.java
index b4ca8e8..2e0d9f3 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/instancetypes/InstanceTypeManager.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/instancetypes/InstanceTypeManager.java
@@ -417,6 +417,7 @@
     private void updateVirtioScsi(VmBase vmBase) {
         if (isNextRunConfigurationExists()) {
             
getModel().getIsVirtioScsiEnabled().setEntity(isVmDeviceExists(vmBase.getManagedDeviceMap(),
 VmDeviceType.VIRTIOSCSI.getName()));
+            model.stopProgress();
             return;
         }
 


-- 
To view, visit http://gerrit.ovirt.org/36679
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iea63c3557d5b962d56ba2c0e1f97356f41474aef
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to