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