Omer Frenkel has uploaded a new change for review.

Change subject: core: update guest agent info on GetAllVmsQuery (#848048)
......................................................................

core: update guest agent info on GetAllVmsQuery (#848048)

https://bugzilla.redhat.com/show_bug.cgi?id=848048

GetAllVmsQuery did not return guest agent info,
causing user-portal not sending vmLogon, so SSO couldn't work.

Change-Id: I99fdfaba14c2fee767aed48d86cf149c10439175
Signed-off-by: Omer Frenkel <[email protected]>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetAllVmsQuery.java
1 file changed, 9 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/75/7475/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetAllVmsQuery.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetAllVmsQuery.java
index 0bd28c9..c451b3a 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetAllVmsQuery.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetAllVmsQuery.java
@@ -1,5 +1,8 @@
 package org.ovirt.engine.core.bll;
 
+import java.util.List;
+
+import org.ovirt.engine.core.common.businessentities.VM;
 import org.ovirt.engine.core.common.queries.VdcQueryParametersBase;
 
 public class GetAllVmsQuery<P extends VdcQueryParametersBase> extends 
QueriesCommandBase<P> {
@@ -9,7 +12,11 @@
 
     @Override
     protected void executeQueryCommand() {
-        getQueryReturnValue().setReturnValue(getDbFacade()
-                .getVmDAO().getAll(getUserID(), getParameters().isFiltered()));
+        List<VM> vmsList = getDbFacade()
+                .getVmDAO().getAll(getUserID(), getParameters().isFiltered());
+        for (VM vm : vmsList) {
+            VmHandler.UpdateVmGuestAgentVersion(vm);
+        }
+        getQueryReturnValue().setReturnValue(vmsList);
     }
 }


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

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

Reply via email to