Change in vdsm[master]: virt: Make DomainDescriptor use XML helpers

2016-10-31 Thread mzamazal
Milan Zamazal has posted comments on this change. Change subject: virt: Make DomainDescriptor use XML helpers .. Patch Set 9: Any other suggestion besides renaming export_xml for the upcoming rebase? -- To view, visit https:/

Change in vdsm[master]: storage: iscsiadm parses IPv6 iSCSI addresses

2016-10-31 Thread automation
gerrit-hooks has posted comments on this change. Change subject: storage: iscsiadm parses IPv6 iSCSI addresses .. Patch Set 17: * Update Tracker::#1308306::OK, status: POST * Check Bug-Url::OK * Check Public Bug::#1308306::OK,

Change in vdsm[master]: storage: Extend the API for IPv6 iSCSI targets

2016-10-31 Thread automation
gerrit-hooks has posted comments on this change. Change subject: storage: Extend the API for IPv6 iSCSI targets .. Patch Set 5: * Update Tracker::#1308306::OK, status: POST * Check Bug-Url::OK * Check Public Bug::#1308306::OK,

Change in vdsm[master]: build: Remove build requires needed for the tests

2016-10-31 Thread eedri
Eyal Edri has posted comments on this change. Change subject: build: Remove build requires needed for the tests .. Patch Set 1: You're assuming you'll always have .packages file. What will happen if you'll run vdsm build outsid

Change in vdsm[master]: storage: prepare iscsi for IPv6 targets

2016-10-31 Thread automation
gerrit-hooks has posted comments on this change. Change subject: storage: prepare iscsi for IPv6 targets .. Patch Set 13: * Update Tracker::#1308306::OK, status: POST * Check Bug-Url::OK * Check Public Bug::#1308306::OK, public

Change in vdsm[master]: storage: Enable IPv6 encapsulation in HSM

2016-10-31 Thread automation
gerrit-hooks has posted comments on this change. Change subject: storage: Enable IPv6 encapsulation in HSM .. Patch Set 11: * Update Tracker::#1308306::OK, status: POST * Check Bug-Url::OK * Check Public Bug::#1308306::OK, publ

Change in vdsm[master]: storage: iscsiadm parses IPv6 iSCSI addresses

2016-10-31 Thread automation
gerrit-hooks has posted comments on this change. Change subject: storage: iscsiadm parses IPv6 iSCSI addresses .. Patch Set 16: * Update Tracker::#1308306::OK, status: POST * Check Bug-Url::OK * Check Public Bug::#1308306::OK,

Change in vdsm[master]: storage: Extend the API for IPv6 iSCSI targets

2016-10-31 Thread automation
gerrit-hooks has posted comments on this change. Change subject: storage: Extend the API for IPv6 iSCSI targets .. Patch Set 4: * Update Tracker::#1308306::OK, status: POST * Check Bug-Url::OK * Check Public Bug::#1308306::OK,

Change in vdsm[master]: net: switch type switch

2016-10-31 Thread phoracek
Petr Horáček has posted comments on this change. Change subject: net: switch type switch .. Patch Set 13: Verified+1 Passed all network/*_test.py OK -- To view, visit https://gerrit.ovirt.org/62330 To unsubscribe, visit https

Change in vdsm[master]: net: switch type switch

2016-10-31 Thread automation
gerrit-hooks has posted comments on this change. Change subject: net: switch type switch .. Patch Set 13: * Update Tracker::#1362399::OK, status: POST * Check Bug-Url::OK * Check Public Bug::#1362399::OK, public bug * Check Pro

Change in vdsm[master]: logging: Remove connectivity log

2016-10-31 Thread automation
gerrit-hooks has posted comments on this change. Change subject: logging: Remove connectivity log .. Patch Set 1: * Update Tracker::IGNORE, no bug url/s found * Check Bug-Url::IGNORE, no bug url/s found * Check Public Bug::WARN

Change in vdsm[master]: logging: Remove connectivity log

2016-10-31 Thread mzamazal
Milan Zamazal has uploaded a new change for review. Change subject: logging: Remove connectivity log .. logging: Remove connectivity log There is no known use or user of this log. So the log just unnecessarily produces another

Change in vdsm[master]: readme: Fix build status url

2016-10-31 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: readme: Fix build status url .. Patch Set 1: Verified+1 Verified on github: https://github.com/nirs/vdsm/blob/build-status/README.md -- To view, visit https://ger

Change in vdsm[master]: jsonrpc: introduce new client

2016-10-31 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: jsonrpc: introduce new client .. Patch Set 10: (5 comments) Partial review. https://gerrit.ovirt.org/#/c/64502/10/lib/vdsm/client.py File lib/vdsm/client.py: Lin

Change in vdsm[master]: jsonrpc: introduce new client

2016-10-31 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: jsonrpc: introduce new client .. Patch Set 10: (1 comment) https://gerrit.ovirt.org/#/c/64502/10/lib/vdsm/client.py File lib/vdsm/client.py: Line 108: ServerError

Change in vdsm[master]: readme: Fix build status url

2016-10-31 Thread automation
gerrit-hooks has posted comments on this change. Change subject: readme: Fix build status url .. Patch Set 1: * Update Tracker::IGNORE, no bug url/s found * Check Bug-Url::IGNORE, no bug url/s found * Check Public Bug::WARN, no

Change in vdsm[master]: readme: Fix build status url

2016-10-31 Thread nsoffer
Nir Soffer has uploaded a new change for review. Change subject: readme: Fix build status url .. readme: Fix build status url Previously we pointed to Nir's private fork, now that oVirt's github is triggering builds we should po

Change in vdsm[master]: net: switch type switch

2016-10-31 Thread automation
gerrit-hooks has posted comments on this change. Change subject: net: switch type switch .. Patch Set 12: * Update Tracker::#1362399::OK, status: NEW * Check Bug-Url::OK * Check Public Bug::#1362399::OK, public bug * Check Prod

Change in vdsm[master]: stomp: simplified client

2016-10-31 Thread automation
gerrit-hooks has posted comments on this change. Change subject: stomp: simplified client .. Patch Set 14: * update_tracker: OK * Set MODIFIED::IGNORE, no Bug-Url found. -- To view, visit https://gerrit.ovirt.org/59151 To uns

Change in vdsm[master]: utils: build cert paths in single place

2016-10-31 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: utils: build cert paths in single place .. utils: build cert paths in single place We make sure that all the references to certificates location are in

Change in vdsm[master]: stomp: simplified client

2016-10-31 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: stomp: simplified client .. Patch Set 13: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/59151 To unsubscribe, visit https://gerrit.ovirt.org/setting

Change in vdsm[master]: utils: build cert paths in single place

2016-10-31 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: utils: build cert paths in single place .. Patch Set 14: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/52354 To unsubscribe, visit https://gerrit.ov

Change in vdsm[master]: utils: build cert paths in single place

2016-10-31 Thread automation
gerrit-hooks has posted comments on this change. Change subject: utils: build cert paths in single place .. Patch Set 15: * update_tracker: OK * Set MODIFIED::IGNORE, no Bug-Url found. -- To view, visit https://gerrit.ovirt.o

Change in vdsm[master]: stomp: simplified client

2016-10-31 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: stomp: simplified client .. stomp: simplified client Change-Id: I2b9b33474e60ab349885a7de18eeacc8f648011f Signed-off-by: Piotr Kliczewski Signed-off-b

Change in vdsm[master]: stomp: simplified client

2016-10-31 Thread igoihman
Irit Goihman has posted comments on this change. Change subject: stomp: simplified client .. Patch Set 13: Verified+1 (1 comment) https://gerrit.ovirt.org/#/c/59151/10/lib/yajsonrpc/stompreactor.py File lib/yajsonrpc/stompreac

Change in vdsm[master]: utils: build cert paths in single place

2016-10-31 Thread igoihman
Irit Goihman has posted comments on this change. Change subject: utils: build cert paths in single place .. Patch Set 14: Verified+1 -- To view, visit https://gerrit.ovirt.org/52354 To unsubscribe, visit https://gerrit.ovirt.o

Change in vdsm[master]: virt: Prevent running stuck VM operations

2016-10-31 Thread automation
gerrit-hooks has posted comments on this change. Change subject: virt: Prevent running stuck VM operations .. Patch Set 2: * update_tracker: OK -- To view, visit https://gerrit.ovirt.org/59735 To unsubscribe, visit https://ge

Change in vdsm[master]: virt: Prevent running stuck VM operations

2016-10-31 Thread mzamazal
Milan Zamazal has abandoned this change. Change subject: virt: Prevent running stuck VM operations .. Abandoned -- To view, visit https://gerrit.ovirt.org/59735 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-M

Change in vdsm[master]: storage: iscsiadm parses IPv6 iSCSI addresses

2016-10-31 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: storage: iscsiadm parses IPv6 iSCSI addresses .. Patch Set 14: (1 comment) pep8 is unhappy: 20:01:55 ./vdsm/storage/hsm.py:2943:80: E501 line too long (82 > 7

Change in vdsm[master]: utils: build cert paths in single place

2016-10-31 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: utils: build cert paths in single place .. Patch Set 13: (1 comment) https://gerrit.ovirt.org/#/c/52354/13/lib/vdsm/constants.py.in File lib/vdsm/constants.py.

Change in vdsm[master]: utils: build cert paths in single place

2016-10-31 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: utils: build cert paths in single place .. Patch Set 14: -Code-Review -- To view, visit https://gerrit.ovirt.org/52354 To unsubscribe, visit https://gerrit.ovi

Change in vdsm[master]: Allow pinning of VM NUMA nodes to host NUMA nodes.

2016-10-31 Thread automation
gerrit-hooks has posted comments on this change. Change subject: Allow pinning of VM NUMA nodes to host NUMA nodes. .. Patch Set 8: * update_tracker: OK * Set MODIFIED::bug 1306698#1306698IGNORE, not all related patches

Change in vdsm[master]: Allow pinning of VM NUMA nodes to host NUMA nodes.

2016-10-31 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: Allow pinning of VM NUMA nodes to host NUMA nodes. .. Allow pinning of VM NUMA nodes to host NUMA nodes. Adds elements to the element in the libvirt