Bug 1331333

Summary:	VNC Console Keyboard layout/mapping not used
Product:	[oVirt] ovirt-engine	Reporter:	Jonas Israelsson <jonas>
Component:	BLL.Virt	Assignee:	Francesco Romani <fromani>
Status:	CLOSED CURRENTRELEASE	QA Contact:	meital avital <mavital>
Severity:	high	Docs Contact:
Priority:	high
Version:	3.6.4	CC:	bugs, mgoldboi, michal.skrivanek, sbonazzo, s.kieske
Target Milestone:	ovirt-3.6.7	Keywords:	Regression
Target Release:	3.6.7	Flags:	rule-engine: ovirt-3.6.z+ rule-engine: blocker+ mgoldboi: planning_ack+ michal.skrivanek: devel_ack+ rule-engine: testing_ack+
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-07-04 12:31:10 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	Virt	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Jonas Israelsson 2016-04-28 10:24:33 UTC

Description of problem:
The VNC keyboard layout set in the web-iu never reach the vm.

Version-Release number of selected component (if applicable):
3.6.x

How reproducible:
100%

Steps to Reproduce:
1. Configure a vm to use VNC and set some keyboard layout, like Swedish.
2. Start the vm
3. In the host/node look at the vm's xml definition, the keyboard layout is missing, hence the -k switch to kvm-qemu is not set.

Actual results:
The default keyboard layout (probably US) is always used.

Expected results:
The keyboard layout set in the web-ui being used.

Additional info:
http://lists.ovirt.org/pipermail/users/2016-April/039001.html

Comment 1 Michal Skrivanek 2016-04-29 06:45:08 UTC

*** Bug 1331274 has been marked as a duplicate of this bug. ***

Comment 3 Francesco Romani 2016-04-29 12:58:06 UTC

This fix is Engine side.
During the investigation and the fixing, we discovered few Vdsm improvements which are attached to this BZ, but they are secondary.

Comment 4 Francesco Romani 2016-05-18 15:50:36 UTC

doc_text not needed, this bug is simple and self-explanatory.

Comment 5 Francesco Romani 2016-05-24 16:00:32 UTC

The additional Vdsm patch (57932) is a further improvement, the fix is engine side, already merged, so reverting to MODIFIED.

Comment 6 meital avital 2016-06-19 12:42:11 UTC

Verified on version: 3.6.7.2-0.1.el6


Verified steps:
1. Configure a vm to use VNC and set some keyboard layout -> 'sv'
2. Start the vm

In the host: 
<graphics type='vnc' port='5903' autoport='yes' listen='10.35.5.1' keymap='sv'