11:13 a.m.
Omer Frenkel has submitted this change and it was merged.
Change subject: core: [RFE] Display of both the console user and the guest user
......................................................................
core: [RFE] Display of both the console user and the guest user
In 3.0 we displayed the guest logged-in user in the engine. In 3.1 we display
the console user instead. This patch makes the engine to display both of these
values.
- removed guest_user_id column from vm_dynamic table (it is not used) and
corresponding field in entities (VM, VmDynamic)
- added new column console_cur_user_name and a corresponding field in the
entities (VM, VmDynamic)
- NOTE: the semantics of existing guest_cur_user_name of VmDynamic is changed
to its original state (as in 3.0 - the value is reported by GA), the console
user name will be tracked in the console_cur_user_name, which makes more
sense
- removed VmLogoffCommand which has not been used
- added new command UpdateVmConsoleDataCommand Vm console data
- vdsbroker package - changed behavior so that guest logged-in user information
is not discarded
Change-Id: Icd5b178402b32435ceb79120ea1e411f0cc83606
Signed-off-by: Frantisek Kobzik <fkobzik(a)redhat.com>
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=879586
---
M backend/manager/dbscripts/create_views.sql
A backend/manager/dbscripts/upgrade/03_02_0150_add_console_cur_user_name.sql
A backend/manager/dbscripts/upgrade/03_02_0160_drop_guest_user_id_column.sql
M backend/manager/dbscripts/vms_sp.sql
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/SetVmTicketCommand.java
A
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmConsoleDataCommand.java
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsEventListener.java
D
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmLogoffCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmLogonCommand.java
A
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/UpdateVmConsoleDataParameters.java
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionType.java
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VM.java
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmDynamic.java
M
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDAODbFacadeImpl.java
M
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDynamicDAODbFacadeImpl.java
M backend/manager/modules/dal/src/test/resources/fixtures.xml
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResourceManager.java
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/UpdateVmDynamicDataVDSCommand.java
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java
20 files changed, 132 insertions(+), 132 deletions(-)
Approvals:
Omer Frenkel: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/10219
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Icd5b178402b32435ceb79120ea1e411f0cc83606
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Frank Kobzik <fkobzik(a)redhat.com>
Gerrit-Reviewer: Frank Kobzik <fkobzik(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>