Change in vdsm[master]: blockSD: Refactor matadataValidity

2014-04-02 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: blockSD: Refactor matadataValidity .. Patch Set 1: ping! -- To view, visit http://gerrit.ovirt.org/25208 To unsubscribe, visit http://gerrit.ovirt.org/settings G

Change in vdsm[master]: resourceManager: Keep resource state if registerResource fails

2014-04-02 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: resourceManager: Keep resource state if registerResource fails .. Patch Set 4: ping! -- To view, visit http://gerrit.ovirt.org/25284 To unsubscribe, visit http://

Change in vdsm[master]: core: Add boot time to the getVdsStats API

2014-04-02 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: core: Add boot time to the getVdsStats API .. Patch Set 11: (1 comment) http://gerrit.ovirt.org/#/c/25877/11//COMMIT_MSG Commit Message: Line 5: CommitDate: 2014-

Change in vdsm[master]: core: Add boot time to the getVdsStats API

2014-04-02 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: core: Add boot time to the getVdsStats API .. Patch Set 11: (1 comment) http://gerrit.ovirt.org/#/c/25877/11/tests/samplingTests.py File tests/samplingTests.py: L

Change in vdsm[master]: netinfo: Retrieve bonding options differing from defaults

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: netinfo: Retrieve bonding options differing from defaults .. Patch Set 26: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7039/

Change in vdsm[master]: core: fixing attribute error on fenceSpmStorage

2014-04-02 Thread Federico Simoncelli
Federico Simoncelli has posted comments on this change. Change subject: core: fixing attribute error on fenceSpmStorage .. Patch Set 2: Code-Review+1 (1 comment) Looks ok. I am not confident enough for a +2 today yet. http://

Change in vdsm[master]: network: Reset bonding options we did not modify to defaults...

2014-04-02 Thread osvoboda
Ondřej Svoboda has posted comments on this change. Change subject: network: Reset bonding options we did not modify to defaults (WIP) .. Patch Set 8: This is completely misguided. VDSM must not persist “polluted” bonding optio

Change in vdsm[master]: network: Reset bonding options we did not modify to defaults...

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: network: Reset bonding options we did not modify to defaults (WIP) .. Patch Set 8: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_

Change in vdsm[master]: netinfo: Retrieve bonding options differing from defaults

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: netinfo: Retrieve bonding options differing from defaults .. Patch Set 25: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7037/

Change in vdsm[master]: network: Reset bonding options we did not modify to defaults...

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: network: Reset bonding options we did not modify to defaults (WIP) .. Patch Set 7: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_

Change in vdsm[master]: network: Reset bonding options we did not modify to defaults...

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: network: Reset bonding options we did not modify to defaults (WIP) .. Patch Set 6: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_

Change in vdsm[master]: network: Reset bonding options we did not modify to defaults...

2014-04-02 Thread osvoboda
Ondřej Svoboda has posted comments on this change. Change subject: network: Reset bonding options we did not modify to defaults (WIP) .. Patch Set 6: (1 comment) Another comment. http://gerrit.ovirt.org/#/c/26374/6/vdsm/netw

Change in vdsm[master]: network: Reset bonding options we did not modify to defaults...

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: network: Reset bonding options we did not modify to defaults (WIP) .. Patch Set 5: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_

Change in vdsm[master]: network: Reset bonding options we did not modify to defaults...

2014-04-02 Thread osvoboda
Ondřej Svoboda has posted comments on this change. Change subject: network: Reset bonding options we did not modify to defaults (WIP) .. Patch Set 6: (2 comments) Comments on the dictionary of defaults. http://gerrit.ovirt.o

Change in vdsm[master]: netinfo: Retrieve bonding options differing from defaults

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: netinfo: Retrieve bonding options differing from defaults .. Patch Set 24: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7033/

Change in vdsm[master]: network: Reset bonding options we did not modify to defaults...

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: network: Reset bonding options we did not modify to defaults (WIP) .. Patch Set 4: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_

Change in vdsm[master]: network: Reset bonding options we did not modify to defaults...

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: network: Reset bonding options we did not modify to defaults (WIP) .. Patch Set 3: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_

Change in vdsm[master]: lvm: Add an option to replace locking type 4

2014-04-02 Thread ykaplan
Yeela Kaplan has posted comments on this change. Change subject: lvm: Add an option to replace locking type 4 .. Patch Set 7: (1 comment) http://gerrit.ovirt.org/#/c/23645/7/vdsm/storage/lvm.py File vdsm/storage/lvm.py: Line

Change in vdsm[ovirt-3.4]: unified pers: fix restoration when moving from ifcfg pers

2014-04-02 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: unified pers: fix restoration when moving from ifcfg pers .. Patch Set 2: The second run of functional tests failure is 'connectivity check' and it is a

Change in vdsm[master]: network: Reset bonding options we did not modify to defaults...

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: network: Reset bonding options we did not modify to defaults (WIP) .. Patch Set 2: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_

Change in vdsm[master]: network: Reset bonding options we did not modify to defaults...

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: network: Reset bonding options we did not modify to defaults (WIP) .. Patch Set 1: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_

Change in vdsm[master]: netinfo: Retrieve bonding options differing from defaults

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: netinfo: Retrieve bonding options differing from defaults .. Patch Set 23: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7028/

Change in vdsm[master]: network: Reset bonding options we did not modify to defaults...

2014-04-02 Thread osvoboda
Ondřej Svoboda has uploaded a new change for review. Change subject: network: Reset bonding options we did not modify to defaults (WIP) .. network: Reset bonding options we did not modify to defaults (WIP) This is an attempt to

Change in vdsm[ovirt-3.4]: unified pers: fix restoration when moving from ifcfg pers

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: unified pers: fix restoration when moving from ifcfg pers .. Patch Set 2: Build Failed http://jenkins.ovirt.org/job/vdsm_network_functional_tests/120

Change in vdsm[master]: vm: add boot menu support

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: vm: add boot menu support .. Patch Set 4: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7027/ : FAILURE http://jenkins.ovirt.o

Change in vdsm[master]: vm: add boot menu support

2014-04-02 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vm: add boot menu support .. Patch Set 4: Verified+1 Patch set 4: * added a couple of tests to catch the corner cased mentioned in the comment of the previou

Change in vdsm[master]: vm: add boot menu support

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: vm: add boot menu support .. Patch Set 3: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7026/ : FAILURE http://jenkins.ovirt.o

Change in vdsm[master]: vm: add boot menu support

2014-04-02 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vm: add boot menu support .. Patch Set 3: Verified+1 Patch set 3: * rebased (needed changes, so no trivial rebase, +1/+2 lost :( ) * added explicit boolean ca

Change in vdsm[ovirt-3.4]: unified pers: fix restoration when moving from ifcfg pers

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: unified pers: fix restoration when moving from ifcfg pers .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/vdsm_network_functional_tests

Change in vdsm[ovirt-3.4]: unified pers: fix restoration when moving from ifcfg pers

2014-04-02 Thread asegurap
Hello Douglas Schilling Landgraf, Dan Kenigsberg, I'd like you to do a code review. Please visit http://gerrit.ovirt.org/26373 to review the following change. Change subject: unified pers: fix restoration when moving from ifcfg pers .

Change in vdsm[ovirt-3.3]: clientIF: Rescan devices if device not found after vmHotplug...

2014-04-02 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: clientIF: Rescan devices if device not found after vmHotplugDisk .. Patch Set 1: Verified+1 Verified on RHEL 6.5 - Create data center/cluster/storage domain - Creat

Change in vdsm[ovirt-3.3]: hsm: Check all devices visibility after multipath rescan

2014-04-02 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: hsm: Check all devices visibility after multipath rescan .. Patch Set 1: Verified+1 Verified on RHEL 6.5 - Create data center/cluster/storage domain - Create vms/st

Change in vdsm[ovirt-3.3]: hsm: Do not hide errors when checking device visibility

2014-04-02 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: hsm: Do not hide errors when checking device visibility .. Patch Set 1: Verified+1 Verified on RHEL 6.5 - Create data center/cluster/storage domain - Create vms/sta

Change in vdsm[ovirt-3.3]: Revert "clientIF: rescan devices on failed hotplugDisk"

2014-04-02 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: Revert "clientIF: rescan devices on failed hotplugDisk" .. Patch Set 1: Verified+1 Verified on RHEL 6.5 - Create data center/cluster/storage domain - Create vms/sta

Change in vdsm[master]: [wip] json-rpc: AMQP test reactor

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: [wip] json-rpc: AMQP test reactor .. Patch Set 3: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7025/ : FAILURE http://jenkins

Change in vdsm[master]: lvm: Add an option to replace locking type 4

2014-04-02 Thread Federico Simoncelli
Federico Simoncelli has posted comments on this change. Change subject: lvm: Add an option to replace locking type 4 .. Patch Set 7: (1 comment) http://gerrit.ovirt.org/#/c/23645/7/vdsm/storage/lvm.py File vdsm/storage/lvm.py:

Change in vdsm[master]: hooks: Add hook for fake VM stats

2014-04-02 Thread mtayer
mooli tayer has posted comments on this change. Change subject: hooks: Add hook for fake VM stats .. Patch Set 10: (3 comments) http://gerrit.ovirt.org/#/c/25927/10/vdsm/API.py File vdsm/API.py: Line 1204: for s in vm

Change in vdsm[master]: core: Add boot time to the getVdsStats API

2014-04-02 Thread dkuznets
Dima Kuznetsov has posted comments on this change. Change subject: core: Add boot time to the getVdsStats API .. Patch Set 11: (1 comment) http://gerrit.ovirt.org/#/c/25877/11/tests/samplingTests.py File tests/samplingTests.py

Change in vdsm[master]: netinfo: Retrieve bonding options differing from defaults

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: netinfo: Retrieve bonding options differing from defaults .. Patch Set 22: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7024/

Change in vdsm[master]: core: fixing attribute error on fenceSpmStorage

2014-04-02 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: core: fixing attribute error on fenceSpmStorage .. Patch Set 2: Code-Review+1 Even worse, but good enough if this fix is urgent. -- To view, visit http://gerrit.o

Change in vdsm[master]: core: Add boot time to the getVdsStats API

2014-04-02 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: core: Add boot time to the getVdsStats API .. Patch Set 11: (1 comment) Nice, but the tests are modifying stuff in unsafe way now. http://gerrit.ovirt.org/#/c/258

Change in vdsm[master]: netinfo: Retrieve bonding options differing from defaults

2014-04-02 Thread osvoboda
Ondřej Svoboda has posted comments on this change. Change subject: netinfo: Retrieve bonding options differing from defaults .. Patch Set 20: (1 comment) http://gerrit.ovirt.org/#/c/24456/20/lib/vdsm/netinfo.py File lib/vdsm/n

Change in vdsm[master]: netinfo: Retrieve bonding options differing from defaults

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: netinfo: Retrieve bonding options differing from defaults .. Patch Set 21: Code-Review-1 Verified-1 Build Failed http://jenkins.ovirt.org/job/vdsm_un

Change in vdsm[ovirt-3.4]: Do not allow empty network names (bridged nor bridgeless)

2014-04-02 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: Do not allow empty network names (bridged nor bridgeless) .. Patch Set 1: Verified+1 Passed the functional tests (jenkins) and tried to create the no n

Change in vdsm[ovirt-3.4]: Do not allow empty network names (bridged nor bridgeless)

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: Do not allow empty network names (bridged nor bridgeless) .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/vdsm_network_functional_tests

Change in vdsm[master]: Allow moving of sparse images to a block domains

2014-04-02 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Allow moving of sparse images to a block domains .. Patch Set 8: Code-Review+1 (1 comment) http://gerrit.ovirt.org/#/c/25778/8/vdsm/storage/image.py File vdsm/

Change in vdsm[master]: lvm: remove unused add/remove tags

2014-04-02 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: lvm: remove unused add/remove tags .. lvm: remove unused add/remove tags Change-Id: Ibffb9138cb28e150f18e2283c24a60c165cde337 Signed-off-by: Yeela Kapl

Change in vdsm[master]: core: Add boot time to the getVdsStats API

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: core: Add boot time to the getVdsStats API .. Patch Set 11: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7022/ : FAILURE http

Change in vdsm[ovirt-3.4]: Do not allow empty network names (bridged nor bridgeless)

2014-04-02 Thread asegurap
Hello Dan Kenigsberg, I'd like you to do a code review. Please visit http://gerrit.ovirt.org/26367 to review the following change. Change subject: Do not allow empty network names (bridged nor bridgeless) .. Do not allow

Change in vdsm[master]: vdsm: [wip] modeling graphics as a device

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: vdsm: [wip] modeling graphics as a device .. Patch Set 8: Code-Review-1 Verified-1 Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_e

Change in vdsm[master]: core: Add boot time to the getVdsStats API

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: core: Add boot time to the getVdsStats API .. Patch Set 10: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7020/ : FAILURE http

Change in vdsm[master]: hooks: Add hook for fake VM stats

2014-04-02 Thread dkuznets
Dima Kuznetsov has posted comments on this change. Change subject: hooks: Add hook for fake VM stats .. Patch Set 10: Ok, will split -- To view, visit http://gerrit.ovirt.org/25927 To unsubscribe, visit http://gerrit.ovirt.or

Change in vdsm[master]: core: fixing attribute error on fenceSpmStorage

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: core: fixing attribute error on fenceSpmStorage .. Patch Set 2: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7019/ : FAILURE

Change in vdsm[master]: hooks: Add hook for fake VM stats

2014-04-02 Thread vfeenstr
Vinzenz Feenstra has posted comments on this change. Change subject: hooks: Add hook for fake VM stats .. Patch Set 10: Code-Review-1 (1 comment) http://gerrit.ovirt.org/#/c/25927/10/vdsm/hooking.py File vdsm/hooking.py: Line

Change in vdsm[master]: core: fixing attribute error on fenceSpmStorage

2014-04-02 Thread Federico Simoncelli
Federico Simoncelli has posted comments on this change. Change subject: core: fixing attribute error on fenceSpmStorage .. Patch Set 1: Code-Review-1 (1 comment) http://gerrit.ovirt.org/#/c/26358/1/vdsm/storage/hsm.py File vds

Change in vdsm[master]: core: fixing attribute error on fenceSpmStorage

2014-04-02 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: core: fixing attribute error on fenceSpmStorage .. Patch Set 1: Code-Review+1 Ugly, but good enough for temporary bug fix. -- To view, visit http://gerrit.ovirt.o

Change in vdsm[master]: core: fixing attribute error on fenceSpmStorage

2014-04-02 Thread laravot
Liron Ar has posted comments on this change. Change subject: core: fixing attribute error on fenceSpmStorage .. Patch Set 1: (1 comment) http://gerrit.ovirt.org/#/c/26358/1/vdsm/storage/hsm.py File vdsm/storage/hsm.py: Line 3

Change in vdsm[master]: hooks: Add hook for fake VM stats

2014-04-02 Thread Federico Simoncelli
Federico Simoncelli has posted comments on this change. Change subject: hooks: Add hook for fake VM stats .. Patch Set 10: (1 comment) http://gerrit.ovirt.org/#/c/25927/10/vdsm/hooking.py File vdsm/hooking.py: Line 56: def wr

Change in vdsm[master]: core: fixing attribute error on fenceSpmStorage

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: core: fixing attribute error on fenceSpmStorage .. Patch Set 1: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7018/ : FAILURE

Change in vdsm[master]: Avoid redundant volume produces.

2014-04-02 Thread Federico Simoncelli
Federico Simoncelli has posted comments on this change. Change subject: Avoid redundant volume produces. .. Patch Set 1: (1 comment) http://gerrit.ovirt.org/#/c/17991/1/vdsm/storage/sd.py File vdsm/storage/sd.py: Line 640:

Change in vdsm[master]: core: fixing attribute error on fenceSpmStorage

2014-04-02 Thread laravot
Liron Ar has uploaded a new change for review. Change subject: core: fixing attribute error on fenceSpmStorage .. core: fixing attribute error on fenceSpmStorage In change I75493d1db60e51cccd5231b516f963c970d24c99 the pool metad

Change in vdsm[master]: core: fixing attribute error on fenceSpmStorage

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: core: fixing attribute error on fenceSpmStorage .. Patch Set 3: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7017/ : FAILURE

Change in vdsm[master]: core: fixing attribute error on fenceSpmStorage

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: core: fixing attribute error on fenceSpmStorage .. Patch Set 2: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7016/ : FAILURE

Change in vdsm[master]: core: fixing attribute error on fenceSpmStorage

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: core: fixing attribute error on fenceSpmStorage .. Patch Set 1: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7015/ : FAILURE

Change in vdsm[master]: core: fixing attribute error on fenceSpmStorage

2014-04-02 Thread laravot
Liron Ar has abandoned this change. Change subject: core: fixing attribute error on fenceSpmStorage .. Abandoned -- To view, visit http://gerrit.ovirt.org/26357 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-Me

Change in vdsm[master]: core: fixing attribute error on fenceSpmStorage

2014-04-02 Thread laravot
Liron Ar has uploaded a new change for review. Change subject: core: fixing attribute error on fenceSpmStorage .. core: fixing attribute error on fenceSpmStorage In change I75493d1db60e51cccd5231b516f963c970d24c99 the pool metad

Change in vdsm[master]: lvm: remove unused add/remove tags

2014-04-02 Thread Federico Simoncelli
Federico Simoncelli has posted comments on this change. Change subject: lvm: remove unused add/remove tags .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/25013 To unsubscribe, visit http://gerrit.ovirt

Change in vdsm[master]: vdsm: [wip] modeling graphics as a device

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: vdsm: [wip] modeling graphics as a device .. Patch Set 7: Code-Review-1 Verified-1 Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_e

Change in vdsm[master]: virt: migration: move threads in a separate module

2014-04-02 Thread Federico Simoncelli
Federico Simoncelli has posted comments on this change. Change subject: virt: migration: move threads in a separate module .. Patch Set 11: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/25970 To unsubscribe, visit ht

Change in vdsm[master]: hooks: Add hook for fake VM stats

2014-04-02 Thread vfeenstr
Vinzenz Feenstra has posted comments on this change. Change subject: hooks: Add hook for fake VM stats .. Patch Set 10: (3 comments) http://gerrit.ovirt.org/#/c/25927/10/vdsm_hooks/fakevmstats/after_get_all_vm_stats.py File vd

Change in vdsm[ovirt-3.4]: vnic QoS: Fix specParams to XML definition conversion

2014-04-02 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: vnic QoS: Fix specParams to XML definition conversion .. Patch Set 1: Verified+1 Passed the functional tests at http://jenkins.ovirt.org/job/vdsm_netwo

Change in vdsm[master]: Do not allow empty network names (bridged nor bridgeless)

2014-04-02 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Do not allow empty network names (bridged nor bridgeless) .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/26263 To unsubscribe, visit h

Change in vdsm[master]: Do not allow empty network names (bridged nor bridgeless)

2014-04-02 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: Do not allow empty network names (bridged nor bridgeless) .. Do not allow empty network names (bridged nor bridgeless) It was possible to create empty

Change in vdsm[master]: vdsm: [wip] modeling graphics as a device

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: vdsm: [wip] modeling graphics as a device .. Patch Set 6: Code-Review-1 Verified-1 Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_e

Change in vdsm[master]: caps: Collect numa information

2014-04-02 Thread michal . skrivanek
Michal Skrivanek has posted comments on this change. Change subject: caps: Collect numa information .. Patch Set 8: (1 comment) http://gerrit.ovirt.org/#/c/23703/8/vdsm/caps.py File vdsm/caps.py: Line 165: Line 166: Line 16

Change in vdsm[ovirt-3.3]: hsm: refresh pool connection on connectStoragePool

2014-04-02 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: hsm: refresh pool connection on connectStoragePool .. Patch Set 1: Code-Review+1 This bug was probably solved before the 3.4 split, so there is no such GERRIT

Change in vdsm[master]: caps: Collect numa information

2014-04-02 Thread msivak
Martin Sivák has posted comments on this change. Change subject: caps: Collect numa information .. Patch Set 8: I like it. Fix Francesco's comments and it should be ready. -- To view, visit http://gerrit.ovirt.org/23703 To un

Change in vdsm[master]: pep8: make pep8-1.5 happy with E265

2014-04-02 Thread eedri
Eyal Edri has posted comments on this change. Change subject: pep8: make pep8-1.5 happy with E265 .. Patch Set 3: still fails on other issues: http://pastebin.com/uMaXEfaT -- To view, visit http://gerrit.ovirt.org/26248 To un

Change in vdsm[master]: virt: migration: always run migration monitor

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: virt: migration: always run migration monitor .. Patch Set 12: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7012/ : FAILURE h

Change in vdsm[master]: virt: migration: move threads in a separate module

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: virt: migration: move threads in a separate module .. Patch Set 11: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7011/ : FAILU

Change in vdsm[master]: virt: migration: decouple monitoring from thread

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: virt: migration: decouple monitoring from thread .. Patch Set 4: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7009/ : FAILURE

Change in vdsm[master]: virt: migration: rename migration threads

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: virt: migration: rename migration threads .. Patch Set 4: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7010/ : FAILURE http:/

Change in vdsm[master]: virt: migration: add monitor thread control loop

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: virt: migration: add monitor thread control loop .. Patch Set 13: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7008/ : FAILURE

Change in vdsm[master]: virt: migration: use contextmanager for monitor

2014-04-02 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: migration: use contextmanager for monitor .. Patch Set 15: Patch set 15: rebased -- To view, visit http://gerrit.ovirt.org/25978 To unsubscribe, visit

Change in vdsm[master]: virt: migration: decouple monitoring from thread

2014-04-02 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: migration: decouple monitoring from thread .. Patch Set 4: Patch set 4: rebased -- To view, visit http://gerrit.ovirt.org/26279 To unsubscribe, visit

Change in vdsm[master]: virt: migration: use contextmanager for monitor

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: virt: migration: use contextmanager for monitor .. Patch Set 15: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7006/ : FAILURE

Change in vdsm[master]: virt: migration: merge monitor and downtime thread

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: virt: migration: merge monitor and downtime thread .. Patch Set 14: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7007/ : FAILU

Change in vdsm[master]: virt: migration: merge monitor and downtime thread

2014-04-02 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: migration: merge monitor and downtime thread .. Patch Set 14: Patch set 14: * rebased * improved the commit message -- To view, visit http://gerrit.ov

Change in vdsm[master]: virt: migration: add monitor thread control loop

2014-04-02 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: migration: add monitor thread control loop .. Patch Set 13: Patch set 13: * rebased -- To view, visit http://gerrit.ovirt.org/25976 To unsubscribe, vi

Change in vdsm[master]: virt: migration: rename migration threads

2014-04-02 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: migration: rename migration threads .. Patch Set 4: Patch set 4: - rebased -- To view, visit http://gerrit.ovirt.org/26260 To unsubscribe, visit http:

Change in vdsm[master]: virt: migration: always run migration monitor

2014-04-02 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: migration: always run migration monitor .. Patch Set 12: Verified+1 Patch set 12: * rebased * improved the commit message re-ticking verified. -- To

Change in vdsm[master]: virt: migration: move threads in a separate module

2014-04-02 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: migration: move threads in a separate module .. Patch Set 11: Verified+1 Patch set 11: * rebased after pep8 1.5.x fixes * verified to be pep8 1.5.x clea

Change in vdsm[ovirt-3.3]: hsm: refresh pool connection on connectStoragePool

2014-04-02 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: hsm: refresh pool connection on connectStoragePool .. Patch Set 1: Note: gerrit is drunk again: $ git checkout ovirt-3.4 $ git log --grep=I46d7c36b48f5d58edbc89

Change in vdsm[ovirt-3.3]: hsm: refresh pool connection on connectStoragePool

2014-04-02 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: hsm: refresh pool connection on connectStoragePool .. Patch Set 1: Verified+1 Verified on RHEL 6.5 with ISCI storage with engine 3.4 - Create data center, cluster,

Change in vdsm[ovirt-3.3]: hsm: refresh pool connection on connectStoragePool

2014-04-02 Thread nsoffer
Hello Ayal Baron, Federico Simoncelli, I'd like you to do a code review. Please visit http://gerrit.ovirt.org/26349 to review the following change. Change subject: hsm: refresh pool connection on connectStoragePool .. hsm

Change in vdsm[master]: pep8: virt cleanups for pep8-1.5

2014-04-02 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: pep8: virt cleanups for pep8-1.5 .. pep8: virt cleanups for pep8-1.5 virt is now pep8-1.5 clean Change-Id: I2ffcb5c7601cad74741fce07301ddb458cd6679f S

Change in vdsm[master]: pep8: virt cleanups for pep8-1.5

2014-04-02 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: pep8: virt cleanups for pep8-1.5 .. Patch Set 1: Verified+1 Code-Review+2 verified with gitpydiff for having not syntactical change. -- To view, visit http://

Change in vdsm[master]: pep8: virt cleanups for pep8-1.5

2014-04-02 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: pep8: virt cleanups for pep8-1.5 .. Patch Set 1: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/7005/ : FAILURE http://jenkins.

Change in vdsm[master]: pep8: virt cleanups for pep8-1.5

2014-04-02 Thread fromani
Francesco Romani has uploaded a new change for review. Change subject: pep8: virt cleanups for pep8-1.5 .. pep8: virt cleanups for pep8-1.5 virt is now pep8-1.5 clean Change-Id: I2ffcb5c7601cad74741fce07301ddb458cd6679f Signed-