Change in ovirt-engine[ovirt-engine-3.6]: sercon: servlet: db: send correct login name
tnisan at redhat.com
tnisan at redhat.com
Thu Oct 8 17:37:23 UTC 2015
Tal Nisan has submitted this change and it was merged.
Change subject: sercon: servlet: db: send correct login name
......................................................................
sercon: servlet: db: send correct login name
According to the ovirt-vmconsole API, the user name
should be provided in entity not username json field.
This patch makes sure this is true.
This patch also includes commit 8bd9459 to fix tests.
Change-Id: I403da73da8cbeb1931ac67909e3a4431a49a2909
Bug-Url: https://bugzilla.redhat.com/1266135
Signed-off-by: Francesco Romani <fromani at redhat.com>
---
M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/UserProfileDaoTest.java
M backend/manager/modules/services/src/main/java/org/ovirt/engine/core/services/VMConsoleProxyServlet.java
M packaging/dbscripts/create_views.sql
3 files changed, 4 insertions(+), 4 deletions(-)
Approvals:
Tal Nisan: Passed CI tests
Francesco Romani: Verified
Arik Hadas: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/46779
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I403da73da8cbeb1931ac67909e3a4431a49a2909
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: David Caro <dcaroest at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
More information about the Engine-commits
mailing list