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(a)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(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org