Bug 1325361 - Vm created without graphics device reports missing display parameters
Summary: Vm created without graphics device reports missing display parameters
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: vdsm
Classification: oVirt
Component: Core
Version: 4.18.0
Hardware: Unspecified
OS: Unspecified
medium
medium vote
Target Milestone: ---
: ---
Assignee: Francesco Romani
QA Contact: meital avital
URL:
Whiteboard: Virt
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-04-08 14:22 UTC by Francesco Romani
Modified: 2016-04-11 08:17 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-04-11 08:17:18 UTC
oVirt Team: Virt
rule-engine: planning_ack?
rule-engine: devel_ack?
rule-engine: testing_ack?


Attachments (Terms of Use)
vdsm log demonstrating the issue (162.11 KB, application/octet-stream)
2016-04-08 14:27 UTC, Francesco Romani
no flags Details
output of vdsClient getAllVmStats (2.34 KB, text/plain)
2016-04-08 14:28 UTC, Francesco Romani
no flags Details
output of vdsClient list (4.80 KB, text/plain)
2016-04-08 14:28 UTC, Francesco Romani
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 55879 0 master MERGED vm: graphics: fill legacy display parameters 2016-04-12 10:25:21 UTC

Description Francesco Romani 2016-04-08 14:22:26 UTC
Description of problem:
Current Vdsm master supports two way of configuring the VM display settings:
1. the new, modern and preferred way, is to send a proper Graphics device from the client (Engine). This is the recommended way started from oVirt 3.6.x
2. the old way is to send the top-level configure display parameters. Vdsm will reconstruct and use a graphic device from those.

When using the old method, Vdsm fails to report back the correct display parameters.

An example of failing creation request is:

Thread-212::DEBUG::2016-04-08 15:57:50,512::bindingxmlrpc::1229::vds::(wrapper) client [::1]::call vmCreate with ({'emulatedMachine': 'pc', 'vmId': 'e42fc2a1-1f2d-486f-a478-badecf2c61a7', 'devices': [{'device': 'scsi', 'model': 'virtio-scsi', 'type': 'controller'}, {'device': 'console', 'specParams': {'enableSocket': 'true'}, 'type': 'console', 'deviceId': 'c7dea132-45e6-4c1f-9e35-50a33fedc014', 'alias': 'console0'}, {'index': '2', 'iface': 'ide', 'specParams': {}, 'readonly': 'true', 'deviceId': '5ee7678a-3056-4344-885d-381753e2c3dd', 'address': {'bus': '1', 'controller': '0', 'type': 'drive', 'target': '0', 'unit': '0'}, 'device': 'cdrom', 'shared': 'false', 'path': '/tmp/tmpf75QbS/seed.iso', 'type': 'disk'}, {'index': '0', 'iface': 'virtio', 'format': 'raw', 'bootOrder': '1', 'poolID': '00000000-0000-0000-0000-000000000000', 'volumeID': '30697d80-3603-48d8-8ced-4dd3ed9692d1', 'imageID': '55c490a0-fdd5-4027-8012-4507013c2fe9', 'specParams': {}, 'readonly': 'false', 'domainID': '0150ca3b-083b-4f7c-8567-600cccc3cddb', 'optional': 'false', 'deviceId': '55c490a0-fdd5-4027-8012-4507013c2fe9', 'address': {'slot': '0x06', 'bus': '0x00', 'domain': '0x0000', 'type': 'pci', 'function': '0x0'}, 'device': 'disk', 'shared': 'exclusive', 'propagateErrors': 'off', 'type': 'disk'}, {'nicModel': 'pv', 'macAddr': '00:16:3e:5e:a6:9f', 'linkActive': 'true', 'network': 'ovirtmgmt', 'filter': 'vdsm-no-mac-spoofing', 'specParams': {}, 'deviceId': 'b2832281-59b7-40ce-8cf8-6bb393e6a9b4', 'address': {'slot': '0x03', 'bus': '0x00', 'domain': '0x0000', 'type': 'pci', 'function': '0x0'}, 'device': 'bridge', 'type': 'interface'}], 'smp': '4', 'memSize': 4096, 'cpuType': 'SandyBridge', 'spiceSecureChannels': 'smain,sdisplay,sinputs,scursor,splayback,srecord,ssmartcard,susbredir', 'vmName': 'HostedEngine', 'display': 'vnc'},) {}



Version-Release number of selected component (if applicable):
4.18.0 (4.0.0 alpha 1)

How reproducible:
100% of times in this flow

Steps to Reproduce:
1. send command like the above from client (e.g. vdscli, [untested] vdsClient)
2. let Vdsm create the VM
3. check the display parameters using e.g. vdsClient -s 0 list

Actual results:
Missing display parameters: displayIp, displayPort, displaySecurePort

Expected results:
Reported correct values for display parameters


Additional info:
1. seems to happen using xmlrpc
2. Engine flow seems ok

Comment 1 Francesco Romani 2016-04-08 14:27:42 UTC
Created attachment 1145148 [details]
vdsm log demonstrating the issue

Comment 2 Francesco Romani 2016-04-08 14:28:26 UTC
Created attachment 1145149 [details]
output of vdsClient getAllVmStats

Comment 3 Francesco Romani 2016-04-08 14:28:46 UTC
Created attachment 1145151 [details]
output of vdsClient list

Comment 4 Michal Skrivanek 2016-04-09 05:55:13 UTC
why bother when we're about to drop xmlrpc and anything <3.6?

Comment 5 Francesco Romani 2016-04-11 07:06:18 UTC
(In reply to Michal Skrivanek from comment #4)
> why bother when we're about to drop xmlrpc and anything <3.6?

Only because I heard we'll still need to keep compatibility with 3.5 in 4.0.
If turns out we don't need, I'll happily CLOSE/WONTFIX :)

Furthermore, I'm not sure Engine 3.6.x is ignoring display* parameters - as it should. I'll take this chance to investigate that.

Comment 6 Michal Skrivanek 2016-04-11 07:42:56 UTC
(In reply to Francesco Romani from comment #5)

The final consensus was that we don't need 3.5 in 4.0. So we have one less bug:)

Comment 7 Francesco Romani 2016-04-11 08:17:18 UTC
(In reply to Michal Skrivanek from comment #6)
> (In reply to Francesco Romani from comment #5)
> 
> The final consensus was that we don't need 3.5 in 4.0. So we have one less
> bug:)

OK, so this bug could be CLOSED WONTFIX


Note You need to log in before you can comment on or make changes to this bug.