Giuseppe Vallarelli has posted comments on this change.

Change subject: before/after_update_device hook now receives custom properties
......................................................................


Patch Set 1: I would prefer that you didn't submit this

I'm afraid there are still some problems:
Thread-125::DEBUG::2013-07-04 17:04:16,892::BindingXMLRPC::929::vds::(wrapper) 
client [10.34.63.66]::call vmUpdateDevice with 
('67ca68e3-60b4-4aaa-9b49-9ce2f17cb9fc', {'linkActive': 'true', 'network': 
'ovirtmgmt', 'custom': {'duplex': 'full', 'speed': '10000'}, 'alias': 'net4', 
'deviceType': 'interface', 'portMirroring': []}) {} flowID [75012c60]
Thread-125::DEBUG::2013-07-04 
17:04:16,893::vm::2986::vm.Vm::(setLinkAndNetwork) 
vmId=`67ca68e3-60b4-4aaa-9b49-9ce2f17cb9fc`::SET LINK AND NETWORK


Thread-125::DEBUG::2013-07-04 
17:04:16,893::vm::2987::vm.Vm::(setLinkAndNetwork) 
vmId=`67ca68e3-60b4-4aaa-9b49-9ce2f17cb9fc`::########################
Thread-125::DEBUG::2013-07-04 
17:04:16,893::vm::2988::vm.Vm::(setLinkAndNetwork) 
vmId=`67ca68e3-60b4-4aaa-9b49-9ce2f17cb9fc`::CUSTOM PROPS

Thread-125::DEBUG::2013-07-04 
17:04:16,893::vm::2989::vm.Vm::(setLinkAndNetwork) 
vmId=`67ca68e3-60b4-4aaa-9b49-9ce2f17cb9fc`::{}
Thread-125::DEBUG::2013-07-04 
17:04:16,893::vm::2990::vm.Vm::(setLinkAndNetwork) 
vmId=`67ca68e3-60b4-4aaa-9b49-9ce2f17cb9fc`::########################
Thread-125::DEBUG::2013-07-04 17:04:16,893::hooks::49::root::(_runHooksDir) 
#################################################
Thread-125::DEBUG::2013-07-04 17:04:16,893::hooks::50::root::(_runHooksDir) 
_runHooksDir
Thread-125::DEBUG::2013-07-04 17:04:16,894::hooks::51::root::(_runHooksDir) DIR 
before_update_device
Thread-125::DEBUG::2013-07-04 17:04:16,894::hooks::52::root::(_runHooksDir) 
PARAMS {}
Thread-125::DEBUG::2013-07-04 17:04:16,894::hooks::53::root::(_runHooksDir) 
VMCONF {'status': 'Up', 'acpiEnable': 'true', 'emulatedMachine': 'pc-0.14', 
'vmId': '67ca68e3-60b4-4aaa-9b49-9ce2f17cb9fc', 'pid': '16927', 'timeOffset': 
'0', 'keyboardLayout': 'en-us', 'displayPort': '-1', 'displaySecurePort': 
u'5900', 'spiceSslCipherSuite': 'DEFAULT', 'cpuType': 'SandyBridge', 'custom': 
{'device_ff351073-8c4f-426e-acdb-c7a145768c29': 'VmDevice 
{vmId=67ca68e3-60b4-4aaa-9b49-9ce2f17cb9fc, 
deviceId=ff351073-8c4f-426e-acdb-c7a145768c29, device=scsi, type=CONTROLLER, 
bootOrder=0, specParams={}, address={bus=0x00, domain=0x0000, type=pci, 
slot=0x05, function=0x0}, managed=false, plugged=true, readOnly=false, 
deviceAlias=scsi0, customProperties={}}', 
'device_ff351073-8c4f-426e-acdb-c7a145768c29device_5f3fccbf-5905-4a79-8970-9f975734cdf9device_7470ed26-d967-4717-a5bd-f364584eaafedevice_8945557f-d4fa-4aea-b467-a4458b56bfcadevice_85d2d0ab-3603-4143-8d69-e9143f5a3d30device_fd3c16c0-93be-4890-!
 8ae7-93a0a567ed9e': 'VmDevice {vmId=67ca68e3-60b4-4aaa-9b49-9ce2f17cb9fc, 
deviceId=fd3c16c0-93be-4890-8ae7-93a0a567ed9e, device=unix, type=CHANNEL, 
bootOrder=0, specParams={}, address={port=2, bus=0, controller=0, 
type=virtio-serial}, managed=false, plugged=true, readOnly=false, 
deviceAlias=channel1, customProperties={}}', 
'device_ff351073-8c4f-426e-acdb-c7a145768c29device_5f3fccbf-5905-4a79-8970-9f975734cdf9device_7470ed26-d967-4717-a5bd-f364584eaafe':
 'VmDevice {vmId=67ca68e3-60b4-4aaa-9b49-9ce2f17cb9fc, 
deviceId=7470ed26-d967-4717-a5bd-f364584eaafe, device=ide, type=CONTROLLER, 
bootOrder=0, specParams={}, address={bus=0x00, domain=0x0000, type=pci, 
slot=0x01, function=0x1}, managed=false, plugged=true, readOnly=false, 
deviceAlias=ide0, customProperties={}}', 
'device_ff351073-8c4f-426e-acdb-c7a145768c29device_5f3fccbf-5905-4a79-8970-9f975734cdf9device_7470ed26-d967-4717-a5bd-f364584eaafedevice_8945557f-d4fa-4aea-b467-a4458b56bfca':
 'VmDevice {vmId=67ca68e3-60b4-4aaa-9b49-!
 9ce2f17cb9fc, deviceId=8945557f-d4fa-4aea-b467-a4458b56bfca, 
device=virtio-serial, type=CONTROLLER, bootOrder=0, specParams={}, 
address={bus=0x00, domain=0x0000, type=pci, slot=0x06, function=0x0}, 
managed=false, plugged=true, readOnly=false, deviceAlias=virtio-serial0, 
customProperties={}}', 
'device_ff351073-8c4f-426e-acdb-c7a145768c29device_5f3fccbf-5905-4a79-8970-9f975734cdf9device_7470ed26-d967-4717-a5bd-f364584eaafedevice_8945557f-d4fa-4aea-b467-a4458b56bfcadevice_85d2d0ab-3603-4143-8d69-e9143f5a3d30':
 'VmDevice {vmId=67ca68e3-60b4-4aaa-9b49-9ce2f17cb9fc, 
deviceId=85d2d0ab-3603-4143-8d69-e9143f5a3d30, device=unix, type=CHANNEL, 
bootOrder=0, specParams={}, address={port=1, bus=0, controller=0, 
type=virtio-serial}, managed=false, plugged=true, readOnly=false, 
deviceAlias=channel0, customProperties={}}', 
'device_ff351073-8c4f-426e-acdb-c7a145768c29device_5f3fccbf-5905-4a79-8970-9f975734cdf9':
 'VmDevice {vmId=67ca68e3-60b4-4aaa-9b49-9ce2f17cb9fc, 
deviceId=5f3fccbf-5905-4a79-8970-9f975734cdf9, device=memballoon, type=BALLOON, 
bootOrder=0, spec!
 Params={model=virtio}, address={bus=0x00, domain=0x0000, type=pci, slot=0x07, 
function=0x0}, managed=false, plugged=true, readOnly=false, 
deviceAlias=balloon0, customProperties={}}'}, 'pauseCode': 'NOERR', 'clientIp': 
'', 'nicModel': 'rtl8139,pv', 'smartcardEnable': 'false', 'kvmEnable': 'true', 
'transparentHugePages': 'true', 'devices': [{'device': 'scsi', 'alias': 
u'scsi0', 'model': 'virtio-scsi', 'type': 'controller', 'address': {u'slot': 
u'0x05', u'bus': u'0x00', u'domain': u'0x0000', u'type': u'pci', u'function': 
u'0x0'}}, {'alias': u'video0', 'specParams': {'vram': '65536'}, 'deviceId': 
'296a10ac-526f-413c-9e72-2ef0b267b84e', 'address': {u'slot': u'0x02', u'bus': 
u'0x00', u'domain': u'0x0000', u'type': u'pci', u'function': u'0x0'}, 'device': 
'qxl', 'type': 'video'}, {'nicModel': 'pv', 'macAddr': '00:1a:4a:22:3f:db', 
'linkActive': True, 'network': 'ovirtmgmt', 'alias': u'net0', 'bootOrder': '1', 
'custom': {'duplex': 'full', 'speed': '9000'}, 'filter': 'vdsm-no-mac-spoo!
 fing', 'specParams': {}, 'deviceId': '1e551538-1fce-40de-92be-3201d7ef24a4', 
'address': {u'slot': u'0x08', u'bus': u'0x00', u'domain': u'0x0000', u'type': 
u'pci', u'function': u'0x0'}, 'device': 'bridge', 'type': 'interface', 'name': 
u'vnet0'}, {'nicModel': 'pv', 'macAddr': '00:1a:4a:22:3f:dd', 'linkActive': 
True, 'network': 'ovirtmgmt', 'alias': u'net1', 'bootOrder': '2', 'custom': 
{'duplex': 'full', 'speed': '1000'}, 'filter': 'vdsm-no-mac-spoofing', 
'specParams': {}, 'deviceId': '9aecae91-4a7f-4e5d-b5ca-6f287384ca06', 
'address': {u'slot': u'0x03', u'bus': u'0x00', u'domain': u'0x0000', u'type': 
u'pci', u'function': u'0x0'}, 'device': 'bridge', 'type': 'interface', 'name': 
u'vnet1'}, {'nicModel': 'pv', 'macAddr': '00:1a:4a:22:3f:de', 'linkActive': 
True, 'network': 'ovirtmgmt', 'alias': u'net2', 'bootOrder': '3', 'custom': 
{'duplex': 'half', 'speed': '999'}, 'filter': 'vdsm-no-mac-spoofing', 
'specParams': {}, 'deviceId': 'a0b14989-4bd5-49cf-8b68-1e88e10221af', 
'address': {u'slot': u'0x04', u'bus': u'0x00', u'domain': u'0x0000', u'type': 
u'pci!
 ', u'function': u'0x0'}, 'device': 'bridge', 'type': 'interface', 'name': 
u'vnet2'}, {'nicModel': 'pv', 'macAddr': '00:1a:4a:22:3f:df', 'linkActive': 
True, 'network': 'ovirtmgmt', 'alias': u'net3', 'bootOrder': '4', 'custom': 
{'duplex': 'full', 'speed': '6666'}, 'filter': 'vdsm-no-mac-spoofing', 
'specParams': {}, 'deviceId': '5f35db52-744c-4558-ab5e-646cae39b244', 
'address': {u'slot': u'0x09', u'bus': u'0x00', u'domain': u'0x0000', u'type': 
u'pci', u'function': u'0x0'}, 'device': 'bridge', 'type': 'interface', 'name': 
u'vnet3'}, {'index': '2', 'iface': 'ide', 'name': u'hdc', 'alias': u'ide0-1-0', 
'address': {u'bus': u'1', u'controller': u'0', u'type': u'drive', u'target': 
u'0', u'unit': u'0'}, 'specParams': {'path': ''}, 'readonly': 'True', 
'deviceId': 'd368dadd-dcd8-47d2-bc74-9f1f2a24cdfc', 'path': '', 'device': 
'cdrom', 'shared': 'false', 'type': 'disk'}, {'target': 1048576, 'alias': 
u'balloon0', 'specParams': {'model': 'virtio'}, 'deviceId': 
'a8cdd5de-badb-44e5-a9b5-6d74!
 9477aa72', 'address': {u'slot': u'0x07', u'bus': u'0x00', u'domain': 
u'0x0000', u'type': u'pci', u'function': u'0x0'}, 'device': 'memballoon', 
'type': 'balloon'}, {'device': u'usb', 'alias': u'usb0', 'type': 'controller', 
'address': {u'slot': u'0x01', u'bus': u'0x00', u'domain': u'0x0000', u'type': 
u'pci', u'function': u'0x2'}}, {'device': u'ide', 'alias': u'ide0', 'type': 
'controller', 'address': {u'slot': u'0x01', u'bus': u'0x00', u'domain': 
u'0x0000', u'type': u'pci', u'function': u'0x1'}}, {'device': u'virtio-serial', 
'alias': u'virtio-serial0', 'type': 'controller', 'address': {u'slot': u'0x06', 
u'bus': u'0x00', u'domain': u'0x0000', u'type': u'pci', u'function': u'0x0'}}, 
{'device': u'unix', 'alias': u'channel0', 'type': u'channel', 'address': 
{u'bus': u'0', u'controller': u'0', u'type': u'virtio-serial', u'port': u'1'}}, 
{'device': u'unix', 'alias': u'channel1', 'type': u'channel', 'address': 
{u'bus': u'0', u'controller': u'0', u'type': u'virtio-serial', u'port': u'2'}}, 
{'device': u'spicevmc', 'alias': u'channel2', 'type': u'channel', !
 'address': {u'bus': u'0', u'controller': u'0', u'type': u'virtio-serial', 
u'port': u'3'}}, {'nicModel': 'pv', 'macAddr': '00:1a:4a:22:3f:29', 
'linkActive': True, 'network': 'ovirtmgmt', 'alias': u'net4', 'custom': {}, 
'filter': 'vdsm-no-mac-spoofing', 'specParams': {}, 'deviceId': 
'a9e57888-2f78-412c-9570-2fc4117a67d8', 'address': {u'slot': u'0x0a', u'bus': 
u'0x00', u'domain': u'0x0000', u'type': u'pci', u'function': u'0x0'}, 'device': 
'bridge', 'type': 'interface', 'name': u'vnet4'}], 'smp': '1', 'vmType': 'kvm', 
'memSize': 1024, 'displayIp': '0', 'spiceSecureChannels': 
'smain,sinputs,scursor,splayback,srecord,sdisplay,susbredir,ssmartcard', 
'smpCoresPerSocket': '1', 'vmName': 'bla', 'display': 'qxl', 'nice': '0'}

-- 
To view, visit http://gerrit.ovirt.org/16408
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ife0f617fe7e40104de3dc5bcad28b2de04f1d446
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Assaf Muller <amul...@redhat.com>
Gerrit-Reviewer: Assaf Muller <amul...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Giuseppe Vallarelli <gvall...@redhat.com>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to