Change in vdsm[master]: vdsm: add support for hostdev passthrough

2014-06-04 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vdsm: add support for hostdev passthrough .. Patch Set 10: Was version 10 just a rebase? -- To view, visit http://gerrit.ovirt.org/22462 To unsubscribe, vis

Change in vdsm[master]: vdsm: add support for hostdev passthrough

2014-06-04 Thread mpoledni
Martin Polednik has posted comments on this change. Change subject: vdsm: add support for hostdev passthrough .. Patch Set 10: No, we realized that the libvirt device XML is not usable as hostdev XML, therefore adding the need

Change in vdsm[master]: lvm: ignore LVs with special tag

2014-06-04 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: lvm: ignore LVs with special tag .. Patch Set 8: Build Failed http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit/9683/ : FAILURE http://jenk

Change in vdsm[master]: xen: libvirtconnection: connect to xen, if it is available

2014-06-04 Thread danken
Dan Kenigsberg has uploaded a new change for review. Change subject: xen: libvirtconnection: connect to xen, if it is available .. xen: libvirtconnection: connect to xen, if it is available Change-Id: I6cc6474b012d900fe3c5e1dab7

Change in vdsm[master]: xen: do not attempt to read underlying device info

2014-06-04 Thread danken
Dan Kenigsberg has uploaded a new change for review. Change subject: xen: do not attempt to read underlying device info .. xen: do not attempt to read underlying device info Xen's device information is quite different than what

Change in vdsm[master]: libvirtconnection: refactor: open a new connection in specia...

2014-06-04 Thread danken
Dan Kenigsberg has uploaded a new change for review. Change subject: libvirtconnection: refactor: open a new connection in special function .. libvirtconnection: refactor: open a new connection in special function This patch sp

Change in vdsm[master]: _findLiveSnapshotSupport: do not assume that guest

2014-06-04 Thread danken
Dan Kenigsberg has uploaded a new change for review. Change subject: _findLiveSnapshotSupport: do not assume that guest exist .. _findLiveSnapshotSupport: do not assume that guest exist Change-Id: I8e6d1ae6d020bedcfa8f2986fce

Change in vdsm[master]: xen: hook: let oVirt run VMs on Xen hosts

2014-06-04 Thread danken
Dan Kenigsberg has uploaded a new change for review. Change subject: xen: hook: let oVirt run VMs on Xen hosts .. xen: hook: let oVirt run VMs on Xen hosts With this hook installed (and previous hacks applied) it is possible to

Change in vdsm[master]: xen: use qemu connection, as compareCPU is unavailable via xen

2014-06-04 Thread danken
Dan Kenigsberg has uploaded a new change for review. Change subject: xen: use qemu connection, as compareCPU is unavailable via xen .. xen: use qemu connection, as compareCPU is unavailable via xen This patch is required until c

Change in vdsm[master]: upgrade_unified: fix for nicless and bridges with vnics

2014-06-04 Thread asegurap
Antoni Segura Puimedon has uploaded a new change for review. Change subject: upgrade_unified: fix for nicless and bridges with vnics .. upgrade_unified: fix for nicless and bridges with vnics The upgrade script had a mistake whe

Change in vdsm[master]: upgrade_unified: fix for nicless and bridges with vnics

2014-06-04 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: upgrade_unified: fix for nicless and bridges with vnics .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.ovirt.org/28354 To unsubscribe, visi

Change in vdsm[master]: upgrade_unified: fix for nicless and bridges with vnics

2014-06-04 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: upgrade_unified: fix for nicless and bridges with vnics .. Patch Set 1: Verified manually trying several network configurations: Add nicless net to

Change in vdsm[master]: virt: migration: replace magic number

2014-06-04 Thread fromani
Francesco Romani has uploaded a new change for review. Change subject: virt: migration: replace magic number .. virt: migration: replace magic number replace a magic constant with the symbolic name to clarify the intent of the c

Change in vdsm[master]: _findLiveSnapshotSupport: do not assume that guest

2014-06-04 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: _findLiveSnapshotSupport: do not assume that guest exist .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerri

Change in vdsm[master]: virt: migration: rename helper function

2014-06-04 Thread fromani
Francesco Romani has uploaded a new change for review. Change subject: virt: migration: rename helper function .. virt: migration: rename helper function rename helper function to be a bit more consistent with the new style guid

Change in vdsm[master]: libvirtconnection: refactor: open a new connection in specia...

2014-06-04 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: libvirtconnection: refactor: open a new connection in special function .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/vdsm_master_uni

Change in vdsm[master]: faqemu hook: move memory limitation to config

2014-06-04 Thread mpoledni
Martin Polednik has posted comments on this change. Change subject: faqemu hook: move memory limitation to config .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.ovirt.org/28320 To unsubscribe, visit http://gerrit.o

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

2014-06-04 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: migration: add monitor thread control loop .. Patch Set 16: Verified+1 fixed a rebase glitch which cause an AttributeError (MONITOR_INTERVAL vs MIGRATI

Change in vdsm[master]: faqemu hook: move memory limitation to config

2014-06-04 Thread michal . skrivanek
Michal Skrivanek has posted comments on this change. Change subject: faqemu hook: move memory limitation to config .. Patch Set 1: Code-Review+1 i believe this is much better than the last modification for ppc. faqemu still ha

Change in vdsm[master]: virt: migration: replace magic number

2014-06-04 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: migration: replace magic number .. Patch Set 1: Verified+1 trivial. Verified by doing a migration, just to rule out spelling mistakes (seems OK) -- To

Change in vdsm[master]: virt: migration: rename helper function

2014-06-04 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: migration: rename helper function .. Patch Set 1: Verified+1 trivial. Verified by running a migration just to rule out spelling mistakes. -- To view,

Change in vdsm[master]: xen: libvirtconnection: connect to xen, if it is available

2014-06-04 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: xen: libvirtconnection: connect to xen, if it is available .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerri

Change in vdsm[master]: xen: use qemu connection, as compareCPU is unavailable via xen

2014-06-04 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: xen: use qemu connection, as compareCPU is unavailable via xen .. Patch Set 1: Build Failed http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerri

Change in vdsm[master]: faqemu hook: move memory limitation to config

2014-06-04 Thread fromani
Francesco Romani has posted comments on this change. Change subject: faqemu hook: move memory limitation to config .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/28320 To unsubscribe, visit http://gerr

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

2014-06-04 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: migration: merge monitor and downtime thread .. Patch Set 17: Verified+1 Verified with 25976, 25978, 25979 by running migrations and by inspecting the

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

2014-06-04 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: migration: use contextmanager for monitor .. Patch Set 18: Verified+1 Verified with 25976, 25977 and 25979 by running a few migrations. -- To view, vi

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

2014-06-04 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: migration: merge monitor and downtime thread .. Patch Set 17: s/25979/26279/ -- To view, visit http://gerrit.ovirt.org/25977 To unsubscribe, visit htt

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

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

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

2014-06-04 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: migration: add monitor thread control loop .. Patch Set 16: s/25979/26279/ -- To view, visit http://gerrit.ovirt.org/25976 To unsubscribe, visit http:

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

2014-06-04 Thread fromani
Francesco Romani has posted comments on this change. Change subject: virt: migration: decouple monitoring from thread .. Patch Set 7: Verified+1 Verified with 25976, 25977, 25978 running migrations a few times and observing th

Change in vdsm[master]: xen: do not attempt to read underlying device info

2014-06-04 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: xen: do not attempt to read underlying device info .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit/9688/

Change in vdsm[master]: libvirtconnection: refactor: open a new connection in specia...

2014-06-04 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: libvirtconnection: refactor: open a new connection in special function .. Patch Set 1: Verified+1 Verified by running unit tests. Has no functional effect. --

Change in vdsm[master]: xen: hook: let oVirt run VMs on Xen hosts

2014-06-04 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: xen: hook: let oVirt run VMs on Xen hosts .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit/9689/ : SUCCESS

Change in vdsm[master]: _findLiveSnapshotSupport: do not assume that guest

2014-06-04 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: _findLiveSnapshotSupport: do not assume that guest exist .. Patch Set 1: Verified+1 Verified to have no effect on qemu:///system connections, and to return No

Change in vdsm[master]: _findLiveSnapshotSupport: do not assume that guest

2014-06-04 Thread fromani
Francesco Romani has posted comments on this change. Change subject: _findLiveSnapshotSupport: do not assume that guest exist .. Patch Set 1: Code-Review+1 there is probably room for a beautification, but this can surely wait

Change in vdsm[master]: upgrade_unified: fix for nicless and bridges with vnics

2014-06-04 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: upgrade_unified: fix for nicless and bridges with vnics .. Patch Set 1: Build Failed http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit/9690/

Change in vdsm[master]: virt: migration: replace magic number

2014-06-04 Thread vfeenstr
Vinzenz Feenstra has posted comments on this change. Change subject: virt: migration: replace magic number .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/28356 To unsubscribe, visit http://gerrit.ovirt

Change in vdsm[master]: virt: migration: rename helper function

2014-06-04 Thread vfeenstr
Vinzenz Feenstra has posted comments on this change. Change subject: virt: migration: rename helper function .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/28355 To unsubscribe, visit http://gerrit.ovi

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

2014-06-04 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 18: Build Successful http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit/9691/ :

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

2014-06-04 Thread vfeenstr
Vinzenz Feenstra has posted comments on this change. Change subject: virt: migration: use contextmanager for monitor .. Patch Set 18: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/25978 To unsubscribe, visit http://g

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

2014-06-04 Thread vfeenstr
Vinzenz Feenstra has posted comments on this change. Change subject: virt: migration: merge monitor and downtime thread .. Patch Set 17: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/25977 To unsubscribe, visit http:

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

2014-06-04 Thread vfeenstr
Vinzenz Feenstra has posted comments on this change. Change subject: virt: migration: decouple monitoring from thread .. Patch Set 7: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/26279 To unsubscribe, visit http://g

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

2014-06-04 Thread vfeenstr
Vinzenz Feenstra has posted comments on this change. Change subject: virt: migration: add monitor thread control loop .. Patch Set 16: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/25976 To unsubscribe, visit http://

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

2014-06-04 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 17: Build Successful http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit/9692/

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

2014-06-04 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 16: Build Successful http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit/9693/ :

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

2014-06-04 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 7: Build Failed http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit/9694/ : FAIL

Change in vdsm[master]: virt: migration: rename helper function

2014-06-04 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: virt: migration: rename helper function .. Patch Set 1: Build Failed http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit/9695/ : FAILURE http

Change in vdsm[ovirt-3.4]: vm: fix odd migration progress reporting

2014-06-04 Thread fromani
Francesco Romani has uploaded a new change for review. Change subject: vm: fix odd migration progress reporting .. vm: fix odd migration progress reporting migration progress should not start from 100, otherwise the user will se

Change in vdsm[ovirt-3.4]: vm: fix odd migration progress reporting

2014-06-04 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vm: fix odd migration progress reporting .. Patch Set 1: this patch is not the same of master due to recent refactorings. -- To view, visit http://gerrit.ov

Change in vdsm[ovirt-3.4]: vm: fix odd migration progress reporting

2014-06-04 Thread fromani
Francesco Romani has abandoned this change. Change subject: vm: fix odd migration progress reporting .. Abandoned will do proper cherry-pick to make automation happy -- To view, visit http://gerrit.ovirt.org/28362 To unsubscr

Change in vdsm[ovirt-3.4]: vm: fix odd migration progress reporting

2014-06-04 Thread fromani
Hello Tomas Jelinek, Dan Kenigsberg, I'd like you to do a code review. Please visit http://gerrit.ovirt.org/28363 to review the following change. Change subject: vm: fix odd migration progress reporting .. vm: fix odd mig

Change in vdsm[master]: virt: migration: replace magic number

2014-06-04 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: virt: migration: replace magic number .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit/9696/ : SUCCESS ht

Change in vdsm[master]: sp: ensure that master domain is active

2014-06-04 Thread Federico Simoncelli
Federico Simoncelli has posted comments on this change. Change subject: sp: ensure that master domain is active .. Patch Set 2: Verified+1 Verified: * it is impossible to connect to a pool where the master domain is attached

Change in vdsm[master]: sp: prevent master demotion on activation

2014-06-04 Thread Federico Simoncelli
Federico Simoncelli has posted comments on this change. Change subject: sp: prevent master demotion on activation .. Patch Set 2: Verified+1 Verified (regular flow only): * it is possible to activate a storage domain -- To v

Change in vdsm[master]: sp: ensure that master domain is active

2014-06-04 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: sp: ensure that master domain is active .. Patch Set 2: Build Failed http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit/9697/ : FAILURE http

Change in vdsm[master]: sp: ensure that master domain is active

2014-06-04 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: sp: ensure that master domain is active .. sp: ensure that master domain is active It's not possible to be connected to a pool where the master domain

Change in vdsm[master]: sp: ensure that master domain is active

2014-06-04 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: sp: ensure that master domain is active .. Patch Set 2: Code-Review+2 Seems reasonable to me. -- To view, visit http://gerrit.ovirt.org/28331 To unsubscribe,

Change in vdsm[master]: sp: ensure that master domain is active

2014-06-04 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: sp: ensure that master domain is active .. Patch Set 3: Build Successful http://jenkins.ovirt.org/job/vdsm_master_create-rpms_merged/1385/ : SUCCESS

Change in vdsm[master]: sp: prevent master demotion on activation

2014-06-04 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: sp: prevent master demotion on activation .. Patch Set 2: Build Successful http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit/9698/ : SUCCESS

Change in vdsm[master]: mutipath: Remove unneeded and dangerous -r parameter

2014-06-04 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: mutipath: Remove unneeded and dangerous -r parameter .. mutipath: Remove unneeded and dangerous -r parameter Since commit dbf2089488 (Jul 9 2013) multi

Change in vdsm[master]: mutipath: Remove unneeded and dangerous -r parameter

2014-06-04 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: mutipath: Remove unneeded and dangerous -r parameter .. Patch Set 3: Build Successful http://jenkins.ovirt.org/job/vdsm_master_create-rpms_merged/138

Change in vdsm[ovirt-3.3]: Fix memShared units when reported from MOM

2014-06-04 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Fix memShared units when reported from MOM .. Patch Set 1: I do not really mind this patch, but strictly speaking, the ovirt-3.3 branch is no longer supported.

Change in vdsm[master]: IGNORE: testing gerrit triggered jobs

2014-06-04 Thread dcaroest
David Caro has abandoned this change. Change subject: IGNORE: testing gerrit triggered jobs .. Abandoned -- To view, visit http://gerrit.ovirt.org/26570 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageTyp

Change in vdsm[ovirt-3.4]: hsm: unify vm ovf management lock

2014-06-04 Thread Federico Simoncelli
Hello Dan Kenigsberg, I'd like you to do a code review. Please visit http://gerrit.ovirt.org/28379 to review the following change. Change subject: hsm: unify vm ovf management lock .. hsm: unify vm ovf management lock Ta

Change in vdsm[master]: faqemu hook: move memory limitation to config

2014-06-04 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: faqemu hook: move memory limitation to config .. Patch Set 1: Code-Review-1 (2 comments) http://gerrit.ovirt.org/#/c/28320/1//COMMIT_MSG Commit Message: Line

Change in vdsm[master]: sp: prevent master demotion on activation

2014-06-04 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: sp: prevent master demotion on activation .. Patch Set 2: (1 comment) http://gerrit.ovirt.org/#/c/28332/2/vdsm/storage/sp.py File vdsm/storage/sp.py: Line 104

Change in vdsm[master]: drop ominous log for libvirt errors

2014-06-04 Thread danken
Dan Kenigsberg has abandoned this change. Change subject: drop ominous log for libvirt errors .. Abandoned I reserve the right to restore this patch, and remove the needless log line. -- To view, visit http://gerrit.ovirt.org

Change in vdsm[master]: net: janitorial: format log messages lazily

2014-06-04 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: net: janitorial: format log messages lazily .. Patch Set 2: Build Failed http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit/9699/ : FAILURE

Change in vdsm[master]: faqemu: move hook logic out of vdsm

2014-06-04 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: faqemu: move hook logic out of vdsm .. Patch Set 4: Build Failed http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit/9700/ : FAILURE http://j

Change in vdsm[master]: net: janitorial: format log messages lazily

2014-06-04 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: net: janitorial: format log messages lazily .. Patch Set 2: Verified+1 Passes the network functional tests, no functional change. -- To view, visit http://ger