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

Dan Kenigsberg has posted comments on this change. Change subject: storage: prepare iscsi for IPv6 targets .. Patch Set 6: (2 comments) https://gerrit.ovirt.org/#/c/65696/6/vdsm/storage/iscsi.py File vdsm/storage/iscsi.py:

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

gerrit-hooks has posted comments on this change. Change subject: storage: iscsiadm parses IPv6 iSCSI addresses .. Patch Set 6: * Update Tracker::#1308306::OK, status: NEW * Check Bug-Url::IGNORE, not relevant for branch:

Change in vdsm[master]: storage: RPC support for IPv6 iSCSI targets

gerrit-hooks has posted comments on this change. Change subject: storage: RPC support for IPv6 iSCSI targets .. Patch Set 2: * Update Tracker::#1308306::OK, status: NEW * Check Bug-Url::IGNORE, not relevant for branch: master

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

gerrit-hooks has posted comments on this change. Change subject: storage: prepare iscsi for IPv6 targets .. Patch Set 6: * Update Tracker::#1308306::OK, status: NEW * Check Bug-Url::IGNORE, not relevant for branch: master *

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

Dominik Holler has posted comments on this change. Change subject: storage: prepare iscsi for IPv6 targets .. Patch Set 5: Verified+1 -- To view, visit https://gerrit.ovirt.org/65696 To unsubscribe, visit

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

Dominik Holler has posted comments on this change. Change subject: storage: iscsiadm parses IPv6 iSCSI addresses .. Patch Set 5: Verified+1 -- To view, visit https://gerrit.ovirt.org/65707 To unsubscribe, visit

Change in vdsm[master]: storage: RPC support for IPv6 iSCSI targets

Dominik Holler has posted comments on this change. Change subject: storage: RPC support for IPv6 iSCSI targets .. Patch Set 1: Verified+1 -- To view, visit https://gerrit.ovirt.org/65736 To unsubscribe, visit

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

Dominik Holler has uploaded a new change for review. Change subject: storage: prepare iscsi for IPv6 targets .. storage: prepare iscsi for IPv6 targets The used iscsiadm tool expects IPv6 addresses in square brackets if a port

Change in vdsm[master]: storage: RPC support for IPv6 iSCSI targets

Dominik Holler has uploaded a new change for review. Change subject: storage: RPC support for IPv6 iSCSI targets .. storage: RPC support for IPv6 iSCSI targets HSM.discoverSendTargets uses iscsi.getTargetString, which already

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

Dominik Holler has uploaded a new change for review. Change subject: storage: iscsiadm parses IPv6 iSCSI addresses .. storage: iscsiadm parses IPv6 iSCSI addresses Enables parsing of IPv6 iSCSI addresses during discovery.

Change in vdsm[master]: tests: bootstrap the livemerge tests

gerrit-hooks has posted comments on this change. Change subject: tests: bootstrap the livemerge tests .. Patch Set 10: * Update Tracker::IGNORE, no bug url/s found * Check Bug-Url::IGNORE, not relevant for branch: master *

Change in vdsm[master]: clientIF: rename recovery thread

gerrit-hooks has posted comments on this change. Change subject: clientIF: rename recovery thread .. Patch Set 2: * Update Tracker::#1141422::OK, status: NEW * Check Bug-Url::IGNORE, not relevant for branch: master * Check

Change in vdsm[master]: migration: coalesce join() into stop()

gerrit-hooks has posted comments on this change. Change subject: migration: coalesce join() into stop() .. Patch Set 7: * Update Tracker::IGNORE, no bug url/s found * Check Bug-Url::IGNORE, not relevant for branch: master *

Change in vdsm[master]: virt: vm: migration: introduce virt subloggers

gerrit-hooks has posted comments on this change. Change subject: virt: vm: migration: introduce virt subloggers .. Patch Set 15: * Update Tracker::IGNORE, no bug url/s found * Check Bug-Url::IGNORE, not relevant for branch:

Change in vdsm[master]: lib: shorten name of libvirt event thread

gerrit-hooks has posted comments on this change. Change subject: lib: shorten name of libvirt event thread .. Patch Set 2: * Update Tracker::#1141422::OK, status: NEW * Check Bug-Url::IGNORE, not relevant for branch: master *

Change in vdsm[master]: migration: use system thread names

gerrit-hooks has posted comments on this change. Change subject: migration: use system thread names .. Patch Set 2: * Update Tracker::#1141422::OK, status: NEW * Check Bug-Url::IGNORE, not relevant for branch: master * Check

Change in vdsm[master]: API: modernize VM methods

gerrit-hooks has posted comments on this change. Change subject: API: modernize VM methods .. Patch Set 20: * Update Tracker::IGNORE, no bug url/s found * Check Bug-Url::IGNORE, not relevant for branch: master * Check Public

Change in vdsm[master]: API: drop the bogus _getStats helper method

gerrit-hooks has posted comments on this change. Change subject: API: drop the bogus _getStats helper method .. Patch Set 2: * Update Tracker::IGNORE, no bug url/s found * Check Bug-Url::IGNORE, not relevant for branch: master

Change in vdsm[master]: vm: a Down domain is not ready for commands

Francesco Romani has abandoned this change. Change subject: vm: a Down domain is not ready for commands .. Abandoned so let's stick with 65131 for the time being -- To view, visit https://gerrit.ovirt.org/65130 To

Change in vdsm[master]: vm: refactor the monitorResponse handling

Francesco Romani has abandoned this change. Change subject: vm: refactor the monitorResponse handling .. Abandoned overkill, we may want to use the new constants - but not now -- To view, visit https://gerrit.ovirt.org/65539

Change in vdsm[master]: vm: check commands timeout only on steady state

gerrit-hooks has posted comments on this change. Change subject: vm: check commands timeout only on steady state .. Patch Set 5: * update_tracker: OK -- To view, visit https://gerrit.ovirt.org/65505 To unsubscribe, visit

Change in vdsm[master]: copy_data: Add qcow2_compat on convert.

Maor Lipchuk has posted comments on this change. Change subject: copy_data: Add qcow2_compat on convert. .. Patch Set 25: Verified+1 -- To view, visit https://gerrit.ovirt.org/64373 To unsubscribe, visit

Change in vdsm[master]: image: Calculate qcow volume size on import.

Maor Lipchuk has posted comments on this change. Change subject: image: Calculate qcow volume size on import. .. Patch Set 2: Verified+1 -- To view, visit https://gerrit.ovirt.org/65039 To unsubscribe, visit

Change in vdsm[master]: image: Calculate qcow volume size on import.

Maor Lipchuk has posted comments on this change. Change subject: image: Calculate qcow volume size on import. .. Patch Set 2: -Verified -- To view, visit https://gerrit.ovirt.org/65039 To unsubscribe, visit

Change in vdsm[master]: virt: move valid states list in vmstatus.py

Francesco Romani has abandoned this change. Change subject: virt: move valid states list in vmstatus.py .. Abandoned not now -- To view, visit https://gerrit.ovirt.org/65538 To unsubscribe, visit

Change in vdsm[master]: virt: move valid states list in vmstatus.py

gerrit-hooks has posted comments on this change. Change subject: virt: move valid states list in vmstatus.py .. Patch Set 1: * update_tracker: OK -- To view, visit https://gerrit.ovirt.org/65538 To unsubscribe, visit

Change in vdsm[master]: vm: a Down domain is not ready for commands

gerrit-hooks has posted comments on this change. Change subject: vm: a Down domain is not ready for commands .. Patch Set 3: * update_tracker: OK -- To view, visit https://gerrit.ovirt.org/65130 To unsubscribe, visit

Change in vdsm[master]: vm: refactor the monitorResponse handling

gerrit-hooks has posted comments on this change. Change subject: vm: refactor the monitorResponse handling .. Patch Set 1: * update_tracker: OK -- To view, visit https://gerrit.ovirt.org/65539 To unsubscribe, visit

Change in vdsm[master]: vm: check commands timeout only on steady state

Francesco Romani has abandoned this change. Change subject: vm: check commands timeout only on steady state .. Abandoned https://gerrit.ovirt.org/65727 is a better fix. -- To view, visit https://gerrit.ovirt.org/65505 To

Change in vdsm[master]: hsm:Use sd compat instead of qemu conf compat.

Maor Lipchuk has posted comments on this change. Change subject: hsm:Use sd compat instead of qemu conf compat. .. Patch Set 11: Verified+1 -- To view, visit https://gerrit.ovirt.org/64951 To unsubscribe, visit

Change in vdsm[master]: hsm:Use sd compat instead of qemu conf compat.

gerrit-hooks has posted comments on this change. Change subject: hsm:Use sd compat instead of qemu conf compat. .. Patch Set 11: * Update Tracker::IGNORE, no bug url/s found * Check Bug-Url::IGNORE, not relevant for branch:

Change in vdsm[master]: vm: check stats timeout only for monitorable VMs

gerrit-hooks has posted comments on this change. Change subject: vm: check stats timeout only for monitorable VMs .. Patch Set 3: * Update Tracker::#1382578::IGNORE, not relevant for Red Hat classification * Update

Change in vdsm[master]: vm: check stats timeout only for monitorable VMs

Jenkins CI has posted comments on this change. Change subject: vm: check stats timeout only for monitorable VMs .. Patch Set 3: Propagate review hook: Continuous Integration value inherited from patch 2 -- To view, visit

Change in vdsm[master]: volume_artifacts: Add qcow2_compat on create.

gerrit-hooks has posted comments on this change. Change subject: volume_artifacts: Add qcow2_compat on create. .. Patch Set 25: * Update Tracker::IGNORE, no bug url/s found * Check Bug-Url::IGNORE, not relevant for branch:

Change in vdsm[master]: copy_data: Add qcow2_compat on convert.

gerrit-hooks has posted comments on this change. Change subject: copy_data: Add qcow2_compat on convert. .. Patch Set 25: * Update Tracker::IGNORE, no bug url/s found * Check Bug-Url::IGNORE, not relevant for branch: master *

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

gerrit-hooks has posted comments on this change. Change subject: net: switch type switch .. Patch Set 11: * Update Tracker::IGNORE, not relevant for branch: master * Check Bug-Url::IGNORE, not relevant for branch: master *

Change in vdsm[master]: storagetestlib: Add sdVersion param to the test API.

gerrit-hooks has posted comments on this change. Change subject: storagetestlib: Add sdVersion param to the test API. .. Patch Set 4: * Update Tracker::IGNORE, no bug url/s found * Check Bug-Url::IGNORE, not relevant for

Change in vdsm[master]: vm: check stats timeout only for monitorable VMs

Jenkins CI has posted comments on this change. Change subject: vm: check stats timeout only for monitorable VMs .. Patch Set 3: Continuous-Integration+1 Propagate review hook: Continuous Integration value inherited from patch

Change in vdsm[master]: vm: check stats timeout only for monitorable VMs

Francesco Romani has posted comments on this change. Change subject: vm: check stats timeout only for monitorable VMs .. Patch Set 3: Verified+1 verified injecting sleep() in the domDependentInit method, and checked that

Change in vdsm[master]: vm: introduce a `monitorable' attribute

gerrit-hooks has posted comments on this change. Change subject: vm: introduce a `monitorable' attribute .. Patch Set 6: * Update Tracker::#1382578::IGNORE, not relevant for Red Hat classification * Update

Change in vdsm[master]: vm: rename statsAge parameter

Francesco Romani has posted comments on this change. Change subject: vm: rename statsAge parameter .. Patch Set 7: Verified+1 verified with 65590 -- To view, visit https://gerrit.ovirt.org/65508 To unsubscribe, visit

Change in vdsm[master]: vm: ignore more errors in isDomainReadyForCommands

gerrit-hooks has posted comments on this change. Change subject: vm: ignore more errors in isDomainReadyForCommands .. Patch Set 9: * Update Tracker::#1382578::IGNORE, not relevant for Red Hat classification * Check

Change in vdsm[master]: vm: rename statsAge parameter

gerrit-hooks has posted comments on this change. Change subject: vm: rename statsAge parameter .. Patch Set 8: * Update Tracker::#1382578::IGNORE, not relevant for Red Hat classification * Update Tracker::#1382583::IGNORE, not

Change in vdsm[master]: vm: reformat setUnresponsiveIfTimeout

gerrit-hooks has posted comments on this change. Change subject: vm: reformat setUnresponsiveIfTimeout .. Patch Set 8: * Update Tracker::#1382578::IGNORE, not relevant for Red Hat classification * Update

Change in vdsm[master]: vm: reformat setUnresponsiveIfTimeout

Francesco Romani has posted comments on this change. Change subject: vm: reformat setUnresponsiveIfTimeout .. Patch Set 7: Verified+1 verified with 65590 -- To view, visit https://gerrit.ovirt.org/65504 To unsubscribe, visit

Change in vdsm[master]: copy_data: Add qcow2_compat on convert.

Maor Lipchuk has posted comments on this change. Change subject: copy_data: Add qcow2_compat on convert. .. Patch Set 24: (3 comments) https://gerrit.ovirt.org/#/c/64373/24/tests/storage_sdm_copy_data_test.py File

Change in vdsm[master]: vm: check stats timeout only for monitorable VMs

gerrit-hooks has posted comments on this change. Change subject: vm: check stats timeout only for monitorable VMs .. Patch Set 2: * Update Tracker::#1382578::IGNORE, not relevant for Red Hat classification * Update

Change in vdsm[master]: vm: introduce a `monitorable' attribute

Francesco Romani has posted comments on this change. Change subject: vm: introduce a `monitorable' attribute .. Patch Set 5: Verified+1 works, but it is not enough to fix the bugs. -- To view, visit

Change in vdsm[master]: vm: check stats timeout only for monitorable VMs

gerrit-hooks has posted comments on this change. Change subject: vm: check stats timeout only for monitorable VMs .. Patch Set 1: * Update Tracker::#1382578::IGNORE, not relevant for Red Hat classification * Update

Change in vdsm[master]: vm: reformat setUnresponsiveIfTimeout

gerrit-hooks has posted comments on this change. Change subject: vm: reformat setUnresponsiveIfTimeout .. Patch Set 7: * Update Tracker::#1382578::IGNORE, not relevant for Red Hat classification * Update

Change in vdsm[master]: vm: ignore more errors in isDomainReadyForCommands

gerrit-hooks has posted comments on this change. Change subject: vm: ignore more errors in isDomainReadyForCommands .. Patch Set 8: * Update Tracker::#1382578::IGNORE, not relevant for Red Hat classification * Check

Change in vdsm[master]: vm: rename statsAge parameter

gerrit-hooks has posted comments on this change. Change subject: vm: rename statsAge parameter .. Patch Set 7: * Update Tracker::#1382578::IGNORE, not relevant for Red Hat classification * Update Tracker::#1382583::IGNORE, not

Change in vdsm[master]: vm: check stats timeout only for monitorable VMs

Francesco Romani has uploaded a new change for review. Change subject: vm: check stats timeout only for monitorable VMs .. vm: check stats timeout only for monitorable VMs We should adjust responsiveness for stats too old only

Change in vdsm[master]: vm: introduce a `monitorable' attribute

gerrit-hooks has posted comments on this change. Change subject: vm: introduce a `monitorable' attribute .. Patch Set 5: * Update Tracker::#1382578::IGNORE, not relevant for Red Hat classification * Update

Change in vdsm[master]: caps: report nested virtualization capabilities

gerrit-hooks has posted comments on this change. Change subject: caps: report nested virtualization capabilities .. Patch Set 2: * Update Tracker::IGNORE, no bug url/s found * Check Bug-Url::IGNORE, not relevant for branch:

Change in vdsm[master]: osinfo: memoize runtime_kernel_flags

gerrit-hooks has posted comments on this change. Change subject: osinfo: memoize runtime_kernel_flags .. Patch Set 1: * Update Tracker::IGNORE, no bug url/s found * Check Bug-Url::IGNORE, not relevant for branch: master *

Change in vdsm[master]: caps: report nested virtualization capabilities

Martin Polednik has uploaded a new change for review. Change subject: caps: report nested virtualization capabilities .. caps: report nested virtualization capabilities Nested virtualization capability can be easily queried

Change in vdsm[master]: caps: report nested virtualization capabilities

gerrit-hooks has posted comments on this change. Change subject: caps: report nested virtualization capabilities .. Patch Set 1: * Update Tracker::IGNORE, no bug url/s found * Check Bug-Url::IGNORE, not relevant for branch:

Change in vdsm[master]: caps: report realtime kernel status

gerrit-hooks has posted comments on this change. Change subject: caps: report realtime kernel status .. Patch Set 1: * Update Tracker::IGNORE, no bug url/s found * Check Bug-Url::IGNORE, not relevant for branch: master * Check

Change in vdsm[master]: caps: report realtime kernel status

Martin Polednik has uploaded a new change for review. Change subject: caps: report realtime kernel status .. caps: report realtime kernel status Determining whether system is suitable for RT operation is complex task consisting

Change in vdsm[master]: osinfo: memoize runtime_kernel_flags

Martin Polednik has uploaded a new change for review. Change subject: osinfo: memoize runtime_kernel_flags .. osinfo: memoize runtime_kernel_flags Unless magic is involved, we don't expect the underlying kernel to change.

Change in vdsm[master]: vm: ignore more errors in isDomainReadyForCommands

Francesco Romani has posted comments on this change. Change subject: vm: ignore more errors in isDomainReadyForCommands .. Patch Set 7: The test failure is interesting but unrelated 00:15:56.269

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

Irit Goihman has posted comments on this change. Change subject: jsonrpc: introduce new client .. Patch Set 10: Verified+1 (3 comments) https://gerrit.ovirt.org/#/c/64502/8/lib/vdsm/client.py File lib/vdsm/client.py: Line

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

gerrit-hooks has posted comments on this change. Change subject: jsonrpc: introduce new client .. Patch Set 10: * Update Tracker::IGNORE, no bug url/s found * Check Bug-Url::IGNORE, not relevant for branch: master * Check

Change in vdsm[master]: hsm:Use sd compat instead of qemu conf compat.

Maor Lipchuk has posted comments on this change. Change subject: hsm:Use sd compat instead of qemu conf compat. .. Patch Set 10: (4 comments) https://gerrit.ovirt.org/#/c/64951/10/tests/storage_hsm_test.py File

Change in vdsm[master]: net: support nameserver address with %iface tail

Petr Horáček has posted comments on this change. Change subject: net: support nameserver address with %iface tail .. Patch Set 3: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/65693 To unsubscribe, visit

Change in vdsm[master]: Add new api to get the IO tune policies for all vms

Andrej Krejcir has posted comments on this change. Change subject: Add new api to get the IO tune policies for all vms .. Patch Set 11: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/63748 To unsubscribe, visit

Change in vdsm[master]: net: support nameserver address with %iface tail

Edward Haas has posted comments on this change. Change subject: net: support nameserver address with %iface tail .. Patch Set 3: Nothing changed in the reporting or setup itself, only in the validation. It is not practical to

Change in vdsm[master]: tests: Refine TerminationTests names

Yaniv Bronhaim has posted comments on this change. Change subject: tests: Refine TerminationTests names .. Patch Set 6: (4 comments) https://gerrit.ovirt.org/#/c/65326/6//COMMIT_MSG Commit Message: Line 5: CommitDate:

Change in vdsm[master]: infra tests: added functional test for upgrading vdsm

Irit Goihman has posted comments on this change. Change subject: infra tests: added functional test for upgrading vdsm .. Patch Set 15: http://jenkins.ovirt.org/job/vdsm_master_check-merged-el7-x86_64/297/ job ended

Change in vdsm[master]: utils: Raise detectable error if termination fail

Yaniv Bronhaim has posted comments on this change. Change subject: utils: Raise detectable error if termination fail .. Patch Set 6: (1 comment) https://gerrit.ovirt.org/#/c/65325/6/lib/vdsm/utils.py File lib/vdsm/utils.py:

Change in vdsm[master]: utils: Wait for terminated process

Yaniv Bronhaim has posted comments on this change. Change subject: utils: Wait for terminated process .. Patch Set 5: we don't use terminating context anywhere yet anyway, only in execmd itself when calling communicate which

Change in vdsm[master]: net: support nameserver address with %iface tail

Petr Horáček has posted comments on this change. Change subject: net: support nameserver address with %iface tail .. Patch Set 3: Code-Review-1 We should add a test to func_dns_test.py, to check if both ifcfg and our caps

Change in vdsm[master]: net: support nameserver address with %iface tail

Edward Haas has posted comments on this change. Change subject: net: support nameserver address with %iface tail .. Patch Set 3: Verified+1 -- To view, visit https://gerrit.ovirt.org/65693 To unsubscribe, visit

Change in vdsm[master]: net: support nameserver address with %iface tail

gerrit-hooks has posted comments on this change. Change subject: net: support nameserver address with %iface tail .. Patch Set 3: * Update Tracker::IGNORE, no bug url/s found * Check Bug-Url::IGNORE, not relevant for branch:

Change in vdsm[master]: net: support nameserver address with %iface tail

gerrit-hooks has posted comments on this change. Change subject: net: support nameserver address with %iface tail .. Patch Set 2: * Update Tracker::IGNORE, no bug url/s found * Check Bug-Url::IGNORE, not relevant for branch:

Change in vdsm[master]: net: support nameserver address with %iface tail

Edward Haas has posted comments on this change. Change subject: net: support nameserver address with %iface tail .. Patch Set 2: (1 comment) https://gerrit.ovirt.org/#/c/65693/1/tests/network/ip_test.py File

Change in vdsm[master]: net: support nameserver address with %iface tail

Dominik Holler has posted comments on this change. Change subject: net: support nameserver address with %iface tail .. Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/65693/1/tests/network/ip_test.py File

Change in vdsm[master]: net: support nameserver address with %iface tail

Edward Haas has posted comments on this change. Change subject: net: support nameserver address with %iface tail .. Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/65693/1/tests/network/ip_test.py File

Change in vdsm[master]: infra tests: added functional test for upgrading vdsm

gerrit-hooks has posted comments on this change. Change subject: infra tests: added functional test for upgrading vdsm .. Patch Set 15: * Update Tracker::IGNORE, no bug url/s found * Check Bug-Url::IGNORE, not relevant for

Change in vdsm[master]: net: support nameserver address with %iface tail

Dominik Holler has posted comments on this change. Change subject: net: support nameserver address with %iface tail .. Patch Set 1: Code-Review+1 (2 comments) Currently I only know that the zone id is used for local link

Change in vdsm[master]: utils: Wait for terminated process

Dan Kenigsberg has posted comments on this change. Change subject: utils: Wait for terminated process .. Patch Set 5: Code-Review+1 what you say here is very clear; but since https://gerrit.ovirt.org/#/c/51407/ was very

Change in vdsm[master]: tests: Use Popen.poll() for running state

gerrit-hooks has posted comments on this change. Change subject: tests: Use Popen.poll() for running state .. Patch Set 6: * Update Tracker::IGNORE, no bug url/s found * Set MODIFIED::IGNORE, no Bug-Url found. -- To view,

Change in vdsm[master]: tests: Use Popen.poll() for running state

Dan Kenigsberg has submitted this change and it was merged. Change subject: tests: Use Popen.poll() for running state .. tests: Use Popen.poll() for running state Checking if /proc/pid exists works, but using poll() is simpler