Change in ovirt-engine[master]: core: fix single sign on for all guests
ofrenkel at redhat.com
ofrenkel at redhat.com
Wed Apr 17 08:00:39 UTC 2013
Omer Frenkel has submitted this change and it was merged.
Change subject: core: fix single sign on for all guests
......................................................................
core: fix single sign on for all guests
This patch fix a problem that caused single-sign-on capability not to
work for some guests: vdsm sends the name of the guest agent application
in the applications list. the engine expects to receive RHEV-Agent as
the guest agent application name, but that's true only for guest agent
running on Windows and in d/s version.
This solution is:
1. set all the possible names for the guest agent application, on all
operating systems, in the configuration so that the engine will look for
all of them (instead of only RHEV-Agent).
2. set the relevant names of the guest agent application in u/s, in
ovirt's engine configuration.
Change-Id: If256ddda586eb53af0ca55d6b97dd98e10139d8a
Bug-Url: https://bugzilla.redhat.com/922398
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/dbscripts/upgrade/pre_upgrade/0000_config.sql
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
3 files changed, 13 insertions(+), 12 deletions(-)
Approvals:
Omer Frenkel: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/13667
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If256ddda586eb53af0ca55d6b97dd98e10139d8a
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
More information about the Engine-commits
mailing list