Change in vdsm[master]: net: init: wait for ipv4 with monitoring

2015-12-29 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: net: init: wait for ipv4 with monitoring .. net: init: wait for ipv4 with monitoring Now we are polling devices until all required IPv4 addresses are

Change in vdsm[master]: v2v: refactor get_external_vms

2015-12-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: v2v: refactor get_external_vms .. Patch Set 9: * Update tracker: IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found. -- To view, visit

Change in vdsm[master]: v2v: refactor get_external_vms

2015-12-29 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: v2v: refactor get_external_vms .. v2v: refactor get_external_vms break to smaller methods Change-Id: If5c331b3136b61b0ee81829d7e71eedbbc1fa6b9

Change in vdsm[master]: v2v: add test testSuccessfulImportOVA

2015-12-29 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: v2v: add test testSuccessfulImportOVA .. v2v: add test testSuccessfulImportOVA Change-Id: I465d68e5f42f6f5829d33bf62c2d40412a8aead4 Signed-off-by:

Change in vdsm[master]: net: netconf owner change should ignore gid

2015-12-29 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: net: netconf owner change should ignore gid .. Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/51148 To unsubscribe, visit

Change in vdsm[master]: net: init: wait for ipv4 with monitoring

2015-12-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: net: init: wait for ipv4 with monitoring .. Patch Set 13: * Update tracker: IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found. -- To view,

Change in vdsm[master]: jobs: Return job id in Host.getJobs response

2015-12-29 Thread laravot
Liron Aravot has posted comments on this change. Change subject: jobs: Return job id in Host.getJobs response .. Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/51147 To unsubscribe, visit

Change in vdsm[master]: jobs: Fix job error format

2015-12-29 Thread nsoffer
Nir Soffer has uploaded a new change for review. Change subject: jobs: Fix job error format .. jobs: Fix job error format The schema species that job's error (ErrorInfo) is a map with "code" and "message" keys, but we returned

Change in vdsm[master]: jobs: Fix job error format

2015-12-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: jobs: Fix job error format .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url:

Change in vdsm[master]: jobs: Return job id in Host.getJobs response

2015-12-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: jobs: Return job id in Host.getJobs response .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: network: tests: mirror drop inital ip tests

2015-12-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: network: tests: mirror drop inital ip tests .. Patch Set 4: * Update tracker: IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found. -- To view,

Change in vdsm[master]: v2v: add test testSuccessfulImportOVA

2015-12-29 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: v2v: add test testSuccessfulImportOVA .. Patch Set 5: Continuous-Integration+1 unrelated failure 09:36:12 ERROR: testLoopMount (mountTests.MountTests)

Change in vdsm[master]: v2v: add test testSuccessfulImportOVA

2015-12-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: v2v: add test testSuccessfulImportOVA .. Patch Set 6: * Update tracker: IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found. -- To view, visit

Change in vdsm[master]: net: netconf owner change should ignore gid

2015-12-29 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: net: netconf owner change should ignore gid .. Patch Set 1: Code-Review+2 Verified+1 -- To view, visit https://gerrit.ovirt.org/51148 To unsubscribe, visit

Change in vdsm[master]: net: netconf owner change should ignore gid

2015-12-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: net: netconf owner change should ignore gid .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found. -- To view,

Change in vdsm[master]: net: netconf owner change should ignore gid

2015-12-29 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: net: netconf owner change should ignore gid .. net: netconf owner change should ignore gid When changing ownership of the directories/files the group

Change in vdsm[master]: Import CPopen and IOStream only in python2 to allow test runs

2015-12-29 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: Import CPopen and IOStream only in python2 to allow test runs .. Patch Set 4: Verified+1 -- To view, visit https://gerrit.ovirt.org/50902 To unsubscribe,

Change in vdsm[master]: network: tests: mirror drop inital ip tests

2015-12-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: network: tests: mirror drop inital ip tests .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: vm: snapshot - adding appropriate logs

2015-12-29 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: vm: snapshot - adding appropriate logs .. Patch Set 2: (1 comment) https://gerrit.ovirt.org/#/c/50775/2/vdsm/virt/vm.py File vdsm/virt/vm.py: Line 3247:

Change in vdsm[master]: network: tests: mirror drop inital ip tests

2015-12-29 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: network: tests: mirror drop inital ip tests .. network: tests: mirror drop inital ip tests Rename test_remove_initial_network_nic_ip_config to

Change in vdsm[master]: Added Python 3 support for sslTests.py

2015-12-29 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: Added Python 3 support for sslTests.py .. Patch Set 5: does it pass make check ? you can remove the draft and jenkins will run it automatically for each new

Change in vdsm[master]: jobs: Return job id in Host.getJobs response

2015-12-29 Thread nsoffer
Nir Soffer has uploaded a new change for review. Change subject: jobs: Return job id in Host.getJobs response .. jobs: Return job id in Host.getJobs response The schema promise this value, but we did not return it. Change-Id:

Change in vdsm[ovirt-3.6]: net: netconf owner change should ignore gid

2015-12-29 Thread edwardh
Edward Haas has posted comments on this change. Change subject: net: netconf owner change should ignore gid .. Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/51149 To unsubscribe, visit

Change in vdsm[master]: net: netconf owner change should ignore gid

2015-12-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: net: netconf owner change should ignore gid .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: net: netconf owner change should ignore gid

2015-12-29 Thread edwardh
Edward Haas has uploaded a new change for review. Change subject: net: netconf owner change should ignore gid .. net: netconf owner change should ignore gid When changing ownership of the directories/files the group id does not

Change in vdsm[master]: v2v: refactor testSuccessfulImport test

2015-12-29 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: v2v: refactor testSuccessfulImport test .. v2v: refactor testSuccessfulImport test - test convert_external_vm api instead of internal ImportVM class -

Change in vdsm[master]: v2v: refactor testSuccessfulImport test

2015-12-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: v2v: refactor testSuccessfulImport test .. Patch Set 6: * Update tracker: IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found. -- To view, visit

Change in vdsm[master]: jobs: Return job id in Host.getJobs response

2015-12-29 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: jobs: Return job id in Host.getJobs response .. Patch Set 1: Continuous-Integration+1 CI failure is not relevant (mount tests failing again). -- To view, visit

Change in vdsm[ovirt-3.6]: net: netconf owner change should ignore gid

2015-12-29 Thread danken
Dan Kenigsberg has uploaded a new change for review. Change subject: net: netconf owner change should ignore gid .. net: netconf owner change should ignore gid When changing ownership of the directories/files the group id does

Change in vdsm[ovirt-3.6]: net: netconf owner change should ignore gid

2015-12-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: net: netconf owner change should ignore gid .. Patch Set 1: Verified-1 * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::ERROR, At least one bug-url

Change in vdsm[master]: jobs: Fix job error format

2015-12-29 Thread laravot
Liron Aravot has posted comments on this change. Change subject: jobs: Fix job error format .. Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/51145 To unsubscribe, visit

Change in vdsm[master]: jobs: Return job id in Host.getJobs response

2015-12-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: jobs: Return job id in Host.getJobs response .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: jobs: Fix job error format

2015-12-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: jobs: Fix job error format .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url:

Change in vdsm[master]: register: handle non uuid machines

2015-12-29 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: register: handle non uuid machines .. Patch Set 2: (1 comment) https://gerrit.ovirt.org/#/c/47385/2/lib/vdsm/tool/register.py File lib/vdsm/tool/register.py:

Change in vdsm[master]: register: handle non uuid machines

2015-12-29 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: register: handle non uuid machines .. Patch Set 2: (1 comment) https://gerrit.ovirt.org/#/c/47385/2/lib/vdsm/tool/register.py File lib/vdsm/tool/register.py:

Change in vdsm[master]: spec: replace deprecated define to global macro

2015-12-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: spec: replace deprecated define to global macro .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure

Change in vdsm[master]: spec: replace deprecated define to global macro

2015-12-29 Thread dougsland
Douglas Schilling Landgraf has uploaded a new change for review. Change subject: spec: replace deprecated define to global macro .. spec: replace deprecated define to global macro

Change in vdsm[master]: spec: replace deprecated define to global macro

2015-12-29 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: spec: replace deprecated define to global macro .. Patch Set 1: Verified+1 -- To view, visit https://gerrit.ovirt.org/51155 To unsubscribe, visit

Change in vdsm[master]: spec: replace deprecated define to global macro

2015-12-29 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: spec: replace deprecated define to global macro .. Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/51155/1/vdsm.spec.in File vdsm.spec.in: Line 20: Line

Change in vdsm[master]: vm: snapshot - adding appropriate logs

2015-12-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: vm: snapshot - adding appropriate logs .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: vm: snapshot - adding appropriate logs

2015-12-29 Thread derez
Daniel Erez has posted comments on this change. Change subject: vm: snapshot - adding appropriate logs .. Patch Set 1: (7 comments) https://gerrit.ovirt.org/#/c/50775/1/vdsm/virt/vm.py File vdsm/virt/vm.py: Line 3243 Line

Change in vdsm[ovirt-3.5]: netconf: own /var/run/vdsm/netconf as well

2015-12-29 Thread tlitovsk
Anatoly Litovsky has posted comments on this change. Change subject: netconf: own /var/run/vdsm/netconf as well .. Patch Set 4: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/51104 To unsubscribe, visit

Change in vdsm[ovirt-3.5]: netconf: own /var/run/vdsm/netconf as well

2015-12-29 Thread tlitovsk
Anatoly Litovsky has posted comments on this change. Change subject: netconf: own /var/run/vdsm/netconf as well .. Patch Set 4: Verified+1 -- To view, visit https://gerrit.ovirt.org/51104 To unsubscribe, visit

Change in vdsm[ovirt-3.5]: netconf: own /var/run/vdsm/netconf as well

2015-12-29 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: netconf: own /var/run/vdsm/netconf as well .. netconf: own /var/run/vdsm/netconf as well commit 1fa3b3 made /var/run/vdsm/netconf/nets owned by vdsm,

Change in vdsm[ovirt-3.5]: netconf: own /var/run/vdsm/netconf as well

2015-12-29 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: netconf: own /var/run/vdsm/netconf as well .. Patch Set 4: Code-Review+2 Continuous-Integration+1 Unrelated test failure ERROR: testBridgeEthtoolDrvinfo

Change in vdsm[ovirt-3.5]: netconf: own /var/run/vdsm/netconf as well

2015-12-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: netconf: own /var/run/vdsm/netconf as well .. Patch Set 5: * #1294075::Update tracker: OK * Set MODIFIED::bug 1294075#1294075IGNORE, not oVirt

Change in vdsm[master]: untested: notify Engine when host connectivity changes

2015-12-29 Thread danken
Dan Kenigsberg has uploaded a new change for review. Change subject: untested: notify Engine when host connectivity changes .. untested: notify Engine when host connectivity changes Change-Id:

Change in vdsm[master]: untested: notify Engine when host connectivity changes

2015-12-29 Thread automation
gerrit-hooks has posted comments on this change. Change subject: untested: notify Engine when host connectivity changes .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make