Change in vdsm[ovirt-3.5]: fence: Fix description and JSONRPC marshalling for fenceNode...

2014-09-22 Thread mperina
Martin Peřina has posted comments on this change. Change subject: fence: Fix description and JSONRPC marshalling for fenceNode API .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.ovirt.org/33156 To unsubscribe,

Change in vdsm[ovirt-3.5]: fence: Fix description and JSONRPC marshalling for fenceNode...

2014-09-22 Thread mperina
Hello Piotr Kliczewski, Yaniv Bronhaim, Dan Kenigsberg, I'd like you to do a code review. Please visit http://gerrit.ovirt.org/33156 to review the following change. Change subject: fence: Fix description and JSONRPC marshalling for fenceNode API

Change in vdsm[ovirt-3.5]: fence: Fix description and JSONRPC marshalling for fenceNode...

2014-09-22 Thread oourfali
Oved Ourfali has posted comments on this change. Change subject: fence: Fix description and JSONRPC marshalling for fenceNode API .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/33156 To unsubscribe,

Change in vdsm[master]: domainMonitor: Cleanup logging

2014-09-22 Thread Federico Simoncelli
Federico Simoncelli has posted comments on this change. Change subject: domainMonitor: Cleanup logging .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/32505 To unsubscribe, visit

Change in vdsm[master]: storage.monitor: Streamline names

2014-09-22 Thread Federico Simoncelli
Federico Simoncelli has posted comments on this change. Change subject: storage.monitor: Streamline names .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/32507 To unsubscribe, visit

Change in vdsm[master]: storage.monitor: Use lowercase module name

2014-09-22 Thread Federico Simoncelli
Federico Simoncelli has posted comments on this change. Change subject: storage.monitor: Use lowercase module name .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/32506 To unsubscribe, visit

Change in vdsm[master]: fileSD: Move private constant near the only call site

2014-09-22 Thread Federico Simoncelli
Federico Simoncelli has posted comments on this change. Change subject: fileSD: Move private constant near the only call site .. Patch Set 1: Code-Review-1 Every time you use a (private) constant you would have to check if it

Change in vdsm[ovirt-3.5]: fence: Fix description and JSONRPC marshalling for fenceNode...

2014-09-22 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: fence: Fix description and JSONRPC marshalling for fenceNode API .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/33156 To unsubscribe,

Change in vdsm[ovirt-3.5]: fence: Fix description and JSONRPC marshalling for fenceNode...

2014-09-22 Thread ybronhei
Yaniv Bronhaim has submitted this change and it was merged. Change subject: fence: Fix description and JSONRPC marshalling for fenceNode API .. fence: Fix description and JSONRPC marshalling for fenceNode API 1) Fixes

Change in vdsm[master]: fileSD: Move private constant near the only call site

2014-09-22 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: fileSD: Move private constant near the only call site .. Patch Set 1: When we add the second use, we can move the constant up - for now this is the best location.

Change in vdsm[ovirt-3.5]: oop: Use Storage.oop logger

2014-09-22 Thread ykaplan
Hello Nir Soffer, Saggi Mizrahi, mooli tayer, Dima Kuznetsov, Dan Kenigsberg, I'd like you to do a code review. Please visit http://gerrit.ovirt.org/33163 to review the following change. Change subject: oop: Use Storage.oop logger

Change in vdsm[ovirt-3.5]: oop: Use Storage.oop logger

2014-09-22 Thread ykaplan
Yeela Kaplan has posted comments on this change. Change subject: oop: Use Storage.oop logger .. Patch Set 1: Verified+1 Same as upstream. build is ok. -- To view, visit http://gerrit.ovirt.org/33163 To unsubscribe, visit

Change in vdsm[ovirt-3.5]: fence: Fix description and JSONRPC marshalling for fenceNode...

2014-09-22 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: fence: Fix description and JSONRPC marshalling for fenceNode API .. Patch Set 2: Build Failed

Change in vdsm[master]: [WIP] change qosOutbound for a nested 'hostQoS':{'out': band...

2014-09-22 Thread asegurap
Antoni Segura Puimedon has uploaded a new change for review. Change subject: [WIP] change qosOutbound for a nested 'hostQoS':{'out': bandwidth_params} .. [WIP] change qosOutbound for a nested 'hostQoS':{'out': bandwidth_params}

Change in vdsm[master]: [WIP] change qosOutbound for a nested 'hostQoS':{'out': band...

2014-09-22 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: [WIP] change qosOutbound for a nested 'hostQoS':{'out': bandwidth_params} .. Patch Set 1: Build Failed

Change in vdsm[master]: [WIP] change qosOutbound for a nested 'hostQoS':{'out': band...

2014-09-22 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: [WIP] change qosOutbound for a nested 'hostQoS':{'out': bandwidth_params} .. Patch Set 2: Build Successful

Change in vdsm[ovirt-3.5]: Conflict with firewalld and NetworkManager and use iptables

2014-09-22 Thread asegurap
Antoni Segura Puimedon has uploaded a new change for review. Change subject: Conflict with firewalld and NetworkManager and use iptables .. Conflict with firewalld and NetworkManager and use iptables As shown in the bug, we do

Change in vdsm[ovirt-3.5]: oop: Use Storage.oop logger

2014-09-22 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: oop: Use Storage.oop logger .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/33163 To unsubscribe, visit

Change in vdsm[ovirt-3.5]: oop: Use Storage.oop logger

2014-09-22 Thread ybronhei
Yaniv Bronhaim has submitted this change and it was merged. Change subject: oop: Use Storage.oop logger .. oop: Use Storage.oop logger Use storage logger to avoid oop logs in /var/log/messages. Storage logger is configured to

Change in vdsm[master]: vm, guestagent: return hash should include also the disk map...

2014-09-22 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: vm, guestagent: return hash should include also the disk mapping .. Patch Set 14: Build Failed

Change in vdsm[ovirt-3.5]: vm: return diskMapping on status()

2014-09-22 Thread laravot
Liron Aravot has uploaded a new change for review. Change subject: vm: return diskMapping on status() .. vm: return diskMapping on status() status() is used by API.Global.getVMList and should return also the disk mapping

Change in vdsm[ovirt-3.5]: vm, guestagent: return hash should include also the disk map...

2014-09-22 Thread laravot
Liron Aravot has uploaded a new change for review. Change subject: vm, guestagent: return hash should include also the disk mapping .. vm, guestagent: return hash should include also the disk mapping Change-Id:

Change in vdsm[ovirt-3.5]: Extract the DOM to Drive name, alias and path logic to reusa...

2014-09-22 Thread msivak
Martin Sivák has abandoned this change. Change subject: Extract the DOM to Drive name, alias and path logic to reusable method .. Abandoned Won't make it into 3.5. -- To view, visit http://gerrit.ovirt.org/30066 To

Change in vdsm[ovirt-3.5]: tests: make api check support falsey values

2014-09-22 Thread msivak
Martin Sivák has abandoned this change. Change subject: tests: make api check support falsey values .. Abandoned Won't make it into 3.5. -- To view, visit http://gerrit.ovirt.org/30067 To unsubscribe, visit

Change in vdsm[ovirt-3.5]: Fix the API definition for cpu tune methods

2014-09-22 Thread msivak
Martin Sivák has abandoned this change. Change subject: Fix the API definition for cpu tune methods .. Abandoned Won't make it into 3.5. -- To view, visit http://gerrit.ovirt.org/30068 To unsubscribe, visit

Change in vdsm[ovirt-3.5]: Improve the _validateIoTuneParams so the params are passed a...

2014-09-22 Thread msivak
Martin Sivák has abandoned this change. Change subject: Improve the _validateIoTuneParams so the params are passed as argument .. Abandoned Won't make it into 3.5. -- To view, visit http://gerrit.ovirt.org/30069 To

Change in vdsm[ovirt-3.5]: Collect current QoS settings for IO devices and report throu...

2014-09-22 Thread msivak
Martin Sivák has abandoned this change. Change subject: Collect current QoS settings for IO devices and report through RunningVmStats .. Abandoned Won't make it into 3.5. -- To view, visit http://gerrit.ovirt.org/30175 To

Change in vdsm[ovirt-3.5]: Refactor updateVmPolicy to use DOM manipulation

2014-09-22 Thread msivak
Martin Sivák has abandoned this change. Change subject: Refactor updateVmPolicy to use DOM manipulation .. Abandoned Won't make it into 3.5. -- To view, visit http://gerrit.ovirt.org/30070 To unsubscribe, visit

Change in vdsm[ovirt-3.5]: janitorial: move isVdsmImage into utils

2014-09-22 Thread msivak
Martin Sivák has abandoned this change. Change subject: janitorial: move isVdsmImage into utils .. Abandoned Won't make it into 3.5. -- To view, visit http://gerrit.ovirt.org/30099 To unsubscribe, visit

Change in vdsm[ovirt-3.5]: vm: fix getVmPolicy return type.

2014-09-22 Thread msivak
Martin Sivák has abandoned this change. Change subject: vm: fix getVmPolicy return type. .. Abandoned Won't make it into 3.5. -- To view, visit http://gerrit.ovirt.org/30072 To unsubscribe, visit

Change in vdsm[ovirt-3.5]: Refactor XMLElement to virt.utils

2014-09-22 Thread msivak
Martin Sivák has abandoned this change. Change subject: Refactor XMLElement to virt.utils .. Abandoned Won't make it into 3.5. -- To view, visit http://gerrit.ovirt.org/30071 To unsubscribe, visit

Change in vdsm[ovirt-3.5]: Add IO tunables support to updateVmPolicy

2014-09-22 Thread msivak
Martin Sivák has abandoned this change. Change subject: Add IO tunables support to updateVmPolicy .. Abandoned Won't make it into 3.5. -- To view, visit http://gerrit.ovirt.org/30074 To unsubscribe, visit

Change in vdsm[ovirt-3.5]: Change method names in vmtune.py to the PEP8 style

2014-09-22 Thread msivak
Martin Sivák has abandoned this change. Change subject: Change method names in vmtune.py to the PEP8 style .. Abandoned Won't make it into 3.5. -- To view, visit http://gerrit.ovirt.org/30078 To unsubscribe, visit

Change in vdsm[ovirt-3.5]: Add API.VM.setIoTune

2014-09-22 Thread msivak
Martin Sivák has abandoned this change. Change subject: Add API.VM.setIoTune .. Abandoned Won't make it into 3.5. -- To view, visit http://gerrit.ovirt.org/30076 To unsubscribe, visit http://gerrit.ovirt.org/settings

Change in vdsm[ovirt-3.5]: Add setIoTune and getIoTunePolicy to the xml-rpc API

2014-09-22 Thread msivak
Martin Sivák has abandoned this change. Change subject: Add setIoTune and getIoTunePolicy to the xml-rpc API .. Abandoned Won't make it into 3.5. -- To view, visit http://gerrit.ovirt.org/30075 To unsubscribe, visit

Change in vdsm[ovirt-3.5]: Remove noisy log message from CPU metadata collector

2014-09-22 Thread msivak
Martin Sivák has abandoned this change. Change subject: Remove noisy log message from CPU metadata collector .. Abandoned Won't make it into 3.5. -- To view, visit http://gerrit.ovirt.org/30079 To unsubscribe, visit

Change in vdsm[ovirt-3.5]: Support ioTune values 2^31 in getStats over xml-rpc

2014-09-22 Thread msivak
Martin Sivák has abandoned this change. Change subject: Support ioTune values 2^31 in getStats over xml-rpc .. Abandoned Won't make it into 3.5. -- To view, visit http://gerrit.ovirt.org/30233 To unsubscribe, visit

Change in vdsm[ovirt-3.5]: Add API.VM.getIoTunePolicy

2014-09-22 Thread msivak
Martin Sivák has abandoned this change. Change subject: Add API.VM.getIoTunePolicy .. Abandoned Won't make it into 3.5. -- To view, visit http://gerrit.ovirt.org/30077 To unsubscribe, visit http://gerrit.ovirt.org/settings

Change in vdsm[ovirt-3.5]: vm, guestagent: return hash should include also the disk map...

2014-09-22 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: vm, guestagent: return hash should include also the disk mapping .. Patch Set 3: Rerun-Hooks: run -- To view, visit http://gerrit.ovirt.org/33198 To unsubscribe,

Change in vdsm[ovirt-3.5]: oop: Use Storage.oop logger

2014-09-22 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: oop: Use Storage.oop logger .. Patch Set 2: Build Failed http://jenkins.ovirt.org/job/vdsm_3.5_create-rpms_merged/170/ : SUCCESS

Change in vdsm[ovirt-3.5]: vm: return diskMapping on status()

2014-09-22 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: vm: return diskMapping on status() .. Patch Set 3: Rerun-Hooks: all -- To view, visit http://gerrit.ovirt.org/33197 To unsubscribe, visit

Change in vdsm[ovirt-3.5]: vm, guestagent: return hash should include also the disk map...

2014-09-22 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: vm, guestagent: return hash should include also the disk mapping .. Patch Set 3: Rerun-Hooks: all -- To view, visit http://gerrit.ovirt.org/33198 To unsubscribe,

Change in vdsm[master]: vm: return diskMapping on status()

2014-09-22 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: vm: return diskMapping on status() .. Patch Set 10: Build Failed http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/11597/ : SUCCESS

Change in vdsm[master]: vm: return diskMapping on status()

2014-09-22 Thread laravot
Liron Aravot has posted comments on this change. Change subject: vm: return diskMapping on status() .. Patch Set 10: Verified+1 -- To view, visit http://gerrit.ovirt.org/31700 To unsubscribe, visit

Change in vdsm[master]: vm, guestagent: return hash should include also the disk map...

2014-09-22 Thread laravot
Liron Aravot has posted comments on this change. Change subject: vm, guestagent: return hash should include also the disk mapping .. Patch Set 14: Verified+1 -- To view, visit http://gerrit.ovirt.org/31701 To unsubscribe,

Change in vdsm[master]: [WIP] change qosOutbound for a nested 'hostQoS':{'out': band...

2014-09-22 Thread lvernia
Lior Vernia has posted comments on this change. Change subject: [WIP] change qosOutbound for a nested 'hostQoS':{'out': bandwidth_params} .. Patch Set 2: Code-Review+1 Ack, will implement serialization/deserialization

Change in vdsm[ovirt-3.5]: vm: return diskMapping on status()

2014-09-22 Thread laravot
Liron Aravot has posted comments on this change. Change subject: vm: return diskMapping on status() .. Patch Set 6: Verified+1 -- To view, visit http://gerrit.ovirt.org/33197 To unsubscribe, visit

Change in vdsm[ovirt-3.5]: vm, guestagent: return hash should include also the disk map...

2014-09-22 Thread laravot
Liron Aravot has posted comments on this change. Change subject: vm, guestagent: return hash should include also the disk mapping .. Patch Set 7: Verified+1 -- To view, visit http://gerrit.ovirt.org/33198 To unsubscribe,

Change in vdsm[master]: Move multipath configuration to vdsm-tool configurator

2014-09-22 Thread ykaplan
Yeela Kaplan has posted comments on this change. Change subject: Move multipath configuration to vdsm-tool configurator .. Patch Set 9: (4 comments) http://gerrit.ovirt.org/#/c/30909/9//COMMIT_MSG Commit Message: Line 11:

Change in vdsm[master]: Move multipath configuration to vdsm-tool configurator

2014-09-22 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: Move multipath configuration to vdsm-tool configurator .. Patch Set 10: Build Successful

Change in vdsm[master]: Revert 'Create default multipath.conf during install'

2014-09-22 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: Revert 'Create default multipath.conf during install' .. Patch Set 9: Build Failed

Change in vdsm[master]: utils: Add logs to rotateFiles

2014-09-22 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: utils: Add logs to rotateFiles .. Patch Set 3: Build Successful http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/11600/ : SUCCESS

Change in vdsm[master]: utils: Use ovirt node for persist and unpersist files

2014-09-22 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: utils: Use ovirt node for persist and unpersist files .. Patch Set 4: Build Successful

Change in vdsm[master]: utils: Move functionality from storage.misc

2014-09-22 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: utils: Move functionality from storage.misc .. Patch Set 3: Build Successful http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/11602/ :

Change in vdsm[master]: change qosOutbound for a nested 'hostQoS':{'out': bandwidth_...

2014-09-22 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: change qosOutbound for a nested 'hostQoS':{'out': bandwidth_params} .. Patch Set 3: Build Failed

Change in vdsm[master]: vdsm: making PM 'on' and 'off' sync

2014-09-22 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: vdsm: making PM 'on' and 'off' sync .. Patch Set 5: Code-Review-1 (1 comment) http://gerrit.ovirt.org/#/c/32695/5/vdsm/API.py File vdsm/API.py: Line 1234:

Change in vdsm[master]: vm, guestagent: return hash should include also the disk map...

2014-09-22 Thread Federico Simoncelli
Federico Simoncelli has posted comments on this change. Change subject: vm, guestagent: return hash should include also the disk mapping .. Patch Set 14: Code-Review+2 Same as patch set 13. -- To view, visit

Change in vdsm[master]: vm: return diskMapping on status()

2014-09-22 Thread Federico Simoncelli
Federico Simoncelli has posted comments on this change. Change subject: vm: return diskMapping on status() .. Patch Set 10: Code-Review+2 Same as patch set 9. -- To view, visit http://gerrit.ovirt.org/31700 To unsubscribe,

Change in vdsm[master]: vm: return diskMapping on status()

2014-09-22 Thread Federico Simoncelli
Federico Simoncelli has submitted this change and it was merged. Change subject: vm: return diskMapping on status() .. vm: return diskMapping on status() status() is used by API.Global.getVMList and should return also the disk

Change in vdsm[master]: vm, guestagent: return hash should include also the disk map...

2014-09-22 Thread Federico Simoncelli
Federico Simoncelli has submitted this change and it was merged. Change subject: vm, guestagent: return hash should include also the disk mapping .. vm, guestagent: return hash should include also the disk mapping Change-Id:

Change in vdsm[master]: vm: return diskMapping on status()

2014-09-22 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: vm: return diskMapping on status() .. Patch Set 11: Build Failed http://jenkins.ovirt.org/job/vdsm_master_create-rpms_merged/1878/ : SUCCESS

Change in vdsm[master]: vm, guestagent: return hash should include also the disk map...

2014-09-22 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: vm, guestagent: return hash should include also the disk mapping .. Patch Set 15: Build Failed

Change in vdsm[master]: vdsm: making PM 'on' and 'off' sync

2014-09-22 Thread emesika
Eli Mesika has posted comments on this change. Change subject: vdsm: making PM 'on' and 'off' sync .. Patch Set 5: (1 comment) http://gerrit.ovirt.org/#/c/32695/5/vdsm/API.py File vdsm/API.py: Line 1234:

Change in vdsm[master]: vdsm: making PM 'on' and 'off' sync

2014-09-22 Thread emesika
Eli Mesika has posted comments on this change. Change subject: vdsm: making PM 'on' and 'off' sync .. Patch Set 6: Verified+1 -- To view, visit http://gerrit.ovirt.org/32695 To unsubscribe, visit

Change in vdsm[master]: vdsm: making PM 'on' and 'off' sync

2014-09-22 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: vdsm: making PM 'on' and 'off' sync .. Patch Set 6: Build Successful http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/11604/ : SUCCESS

Change in vdsm[master]: Move multipath configuration to vdsm-tool configurator

2014-09-22 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: Move multipath configuration to vdsm-tool configurator .. Patch Set 10: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/30909 To unsubscribe, visit

Change in vdsm[ovirt-3.5]: vm: return diskMapping on status()

2014-09-22 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: vm: return diskMapping on status() .. Patch Set 6: Code-Review-1 We are way past feature freeze. 3.5.0 is very much delayed already. We should not take this

Change in vdsm[master]: utils: Move namedTemporaryDir here from testlib

2014-09-22 Thread osvoboda
Ondřej Svoboda has uploaded a new change for review. Change subject: utils: Move namedTemporaryDir here from testlib .. utils: Move namedTemporaryDir here from testlib To be used in subsequent work. Change-Id:

Change in vdsm[master]: netinfo: Stop NetworkManager from ifup'ing our reference bond

2014-09-22 Thread osvoboda
Ondřej Svoboda has uploaded a new change for review. Change subject: netinfo: Stop NetworkManager from ifup'ing our reference bond .. netinfo: Stop NetworkManager from ifup'ing our reference bond This is hopefully only a

Change in vdsm[master]: netinfo: Error out if we failed to change a bond's mode

2014-09-22 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: netinfo: Error out if we failed to change a bond's mode .. Patch Set 2: Build Failed

Change in vdsm[master]: utils: Move namedTemporaryDir here from testlib

2014-09-22 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: utils: Move namedTemporaryDir here from testlib .. Patch Set 1: Build Failed http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/11606/ :

Change in vdsm[master]: netinfo: Stop NetworkManager from ifup'ing our reference bond

2014-09-22 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: netinfo: Stop NetworkManager from ifup'ing our reference bond .. Patch Set 1: Build Failed

Change in vdsm[master]: vdsm: making PM 'on' and 'off' sync

2014-09-22 Thread mperina
Martin Peřina has posted comments on this change. Change subject: vdsm: making PM 'on' and 'off' sync .. Patch Set 6: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/32695 To unsubscribe, visit