Change in vdsm[master]: stats: Report running status of HE components

2016-11-28 Thread Code Review
From Dan Kenigsberg : Dan Kenigsberg has posted comments on this change. Change subject: stats: Report running status of HE components .. Patch Set 9: Martin, Philip: I understand that you'd like to report

Change in vdsm[master]: after_vm_destroy.py: migrate to jsonrpc

2016-11-28 Thread Code Review
From Dan Kenigsberg : Dan Kenigsberg has posted comments on this change. Change subject: after_vm_destroy.py: migrate to jsonrpc .. Patch Set 10: Code-Review-1 (1 comment)

Change in vdsm[master]: vm_migrate_hook: skip interface handling for customized VMs

2016-11-28 Thread Code Review
From Dan Kenigsberg : Dan Kenigsberg has posted comments on this change. Change subject: vm_migrate_hook: skip interface handling for customized VMs .. Patch Set 2: Code-Review+1 horrible, but that's the

Change in vdsm[master]: client: construct namespaces from schema

2016-11-28 Thread Code Review
From Dan Kenigsberg : Dan Kenigsberg has submitted this change and it was merged. Change subject: client: construct namespaces from schema .. client: construct namespaces from schema namepaces are now

Change in vdsm[master]: Adding simple client for sending gauge metrics to statsd por...

2016-11-28 Thread Code Review
From Dan Kenigsberg : Dan Kenigsberg has posted comments on this change. Change subject: Adding simple client for sending gauge metrics to statsd port using udp .. Patch Set 8: (3 comments)

Change in vdsm[master]: virt network: correctly check if element is present

2016-11-28 Thread Code Review
From Dan Kenigsberg : Dan Kenigsberg has posted comments on this change. Change subject: virt network: correctly check if element is present .. Patch Set 1: This should have been caught by a unit test. Can

Change in vdsm[master]: spec: bump up to RHEL 7.3 GA version of qemu and libvirt

2016-11-28 Thread Code Review
From Dan Kenigsberg : Dan Kenigsberg has submitted this change and it was merged. Change subject: spec: bump up to RHEL 7.3 GA version of qemu and libvirt .. spec: bump up to RHEL 7.3 GA version of qemu and

Change in vdsm[master]: net test: increase monitor timeout

2016-11-28 Thread Code Review
From Dan Kenigsberg : Dan Kenigsberg has submitted this change and it was merged. Change subject: net test: increase monitor timeout .. net test: increase monitor timeout Monitor tests sometimes fail with

Change in vdsm[master]: Adding simple client for sending gauge metrics to statsd por...

2016-11-28 Thread Code Review
From Yaniv Bronhaim : Yaniv Bronhaim has posted comments on this change. Change subject: Adding simple client for sending gauge metrics to statsd port using udp .. Patch Set 8: Verified+1 -- To view,

Change in vdsm[master]: net test: increase monitor timeout

2016-11-28 Thread Code Review
From Dan Kenigsberg : Dan Kenigsberg has posted comments on this change. Change subject: net test: increase monitor timeout .. Patch Set 1: Code-Review+2 I hate it, but let's try it out. -- To view, visit

Change in vdsm[master]: tests: rename api_test.py -> api_response_test.py

2016-11-28 Thread Code Review
From Dan Kenigsberg : Dan Kenigsberg has submitted this change and it was merged. Change subject: tests: rename api_test.py -> api_response_test.py .. tests: rename api_test.py -> api_response_test.py

Change in vdsm[master]: tests: rename api_test.py -> api_response_test.py

2016-11-28 Thread Code Review
From Dan Kenigsberg : Dan Kenigsberg has posted comments on this change. Change subject: tests: rename api_test.py -> api_response_test.py .. Patch Set 1: Code-Review+2 -- To view, visit

Change in vdsm[master]: dumpStorageTable: remove module and all references

2016-11-28 Thread Code Review
From Dan Kenigsberg : Dan Kenigsberg has submitted this change and it was merged. Change subject: dumpStorageTable: remove module and all references .. dumpStorageTable: remove module and all references

Change in vdsm[master]: client: construct namespaces from schema

2016-11-28 Thread Code Review
From Yaniv Bronhaim : Yaniv Bronhaim has posted comments on this change. Change subject: client: construct namespaces from schema .. Patch Set 3:  -- To view, visit https://gerrit.ovirt.org/67373 To

Change in vdsm[master]: client: construct namespaces from schema

2016-11-28 Thread Code Review
From Yaniv Bronhaim : Yaniv Bronhaim has posted comments on this change. Change subject: client: construct namespaces from schema .. Patch Set 3: the code we change here didn't expose anything related to

Change in vdsm[master]: Adding simple client for sending gauge metrics to statsd por...

2016-11-28 Thread Code Review
From Yaniv Bronhaim : Yaniv Bronhaim has posted comments on this change. Change subject: Adding simple client for sending gauge metrics to statsd port using udp .. Patch Set 7: (1 comment)

Change in vdsm[master]: testValidation: Add @skipif decorator

2016-11-28 Thread Code Review
From Yaniv Bronhaim : Yaniv Bronhaim has posted comments on this change. Change subject: testValidation: Add @skipif decorator .. Patch Set 1: Code-Review-1 for visibility -- To view, visit

Change in vdsm[master]: dumpStorageTable: remove module and all references

2016-11-28 Thread Code Review
From Dan Kenigsberg : Dan Kenigsberg has posted comments on this change. Change subject: dumpStorageTable: remove module and all references .. Patch Set 4: Code-Review+2 -- To view, visit

Change in vdsm[master]: client: construct namespaces from schema

2016-11-28 Thread Code Review
From Yaniv Bronhaim : Yaniv Bronhaim has posted comments on this change. Change subject: client: construct namespaces from schema .. Patch Set 3: Code-Review+1 -- To view, visit

Change in vdsm[master]: dumpStorageTable: remove module and all references

2016-11-28 Thread Code Review
From Yaniv Bronhaim : Yaniv Bronhaim has posted comments on this change. Change subject: dumpStorageTable: remove module and all references .. Patch Set 3: Code-Review+1 -- To view, visit

Change in vdsm[master]: Adding simple client for sending gauge metrics to statsd por...

2016-11-28 Thread Code Review
From Yaniv Bronhaim : Yaniv Bronhaim has posted comments on this change. Change subject: Adding simple client for sending gauge metrics to statsd port using udp .. Patch Set 7: Verified+1 -- To view,

Change in vdsm[master]: tests: Remove invalid pyc file tests

2016-11-28 Thread Code Review
From Yaniv Bronhaim : Yaniv Bronhaim has posted comments on this change. Change subject: tests: Remove invalid pyc file tests .. Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/67339/1//COMMIT_MSG