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

2014-11-19 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: vm: fix getVmPolicy return type. .. Patch Set 9: Build Failed http://jenkins.ovirt.org/job/vdsm_3.5_create-rpms-fc20-x86_64_merged/97/ : FAILURE

Change in vdsm[master]: utils: add CommandStream class

2014-11-19 Thread fromani
Francesco Romani has posted comments on this change. Change subject: utils: add CommandStream class .. Patch Set 2: (9 comments) preliminary review - easy parts first http://gerrit.ovirt.org/#/c/33909/2/lib/vdsm/utils.py

Change in vdsm[master]: task: add the support for abort_callback

2014-11-19 Thread fromani
Francesco Romani has posted comments on this change. Change subject: task: add the support for abort_callback .. Patch Set 3: (1 comment) VERY preliminary review - good chance for me to learn more about the task framework

Change in vdsm[ovirt-3.5]: tests: fix pep8 errors

2014-11-19 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: tests: fix pep8 errors .. Patch Set 9: Build Failed http://jenkins.ovirt.org/job/vdsm_3.5_create-rpms-fc20-x86_64_merged/98/ : FAILURE

Change in vdsm[ovirt-3.5]: caps: Report online logical CPUs

2014-11-19 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: caps: Report online logical CPUs .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/34768 To unsubscribe, visit

Change in vdsm[ovirt-3.5]: caps: Report online logical CPUs

2014-11-19 Thread ybronhei
Yaniv Bronhaim has submitted this change and it was merged. Change subject: caps: Report online logical CPUs .. caps: Report online logical CPUs Introduces a new information in the capabilities command which reports the list

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

2014-11-19 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: Add IO tunables support to updateVmPolicy .. Patch Set 9: Build Failed http://jenkins.ovirt.org/job/vdsm_3.5_create-rpms-fc20-x86_64_merged/99/ :

Change in vdsm[master]: contrib: Simple jsonrpc client

2014-11-19 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: contrib: Simple jsonrpc client .. Patch Set 4: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/35181 To unsubscribe, visit

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

2014-11-19 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: Add setIoTune and getIoTunePolicy to the xml-rpc API .. Patch Set 9: Build Failed

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

2014-11-19 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: Add API.VM.setIoTune .. Patch Set 9: Build Failed http://jenkins.ovirt.org/job/vdsm_3.5_create-rpms-fc20-x86_64_merged/101/ : FAILURE

Change in vdsm[master]: jsonrpc: have simplejson as soft dependency

2014-11-19 Thread fromani
Francesco Romani has uploaded a new change for review. Change subject: jsonrpc: have simplejson as soft dependency .. jsonrpc: have simplejson as soft dependency In commit ec58bca5cb18 we added dependency to the external

Change in vdsm[ovirt-3.5]: jsonrpc: Wrong key name for getDevicesVisibility

2014-11-19 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: jsonrpc: Wrong key name for getDevicesVisibility .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/35306 To unsubscribe, visit

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

2014-11-19 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: Add API.VM.getIoTunePolicy .. Patch Set 9: Build Failed http://jenkins.ovirt.org/job/vdsm_3.5_create-rpms-fc20-x86_64_merged/102/ : FAILURE

Change in vdsm[master]: jsonrpc: have simplejson as soft dependency

2014-11-19 Thread fromani
Francesco Romani has posted comments on this change. Change subject: jsonrpc: have simplejson as soft dependency .. Patch Set 2: fixed comment in compat.py. -- To view, visit http://gerrit.ovirt.org/35354 To unsubscribe,

Change in vdsm[master]: jsonrpc: have simplejson as soft dependency

2014-11-19 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: jsonrpc: have simplejson as soft dependency .. Patch Set 1: Build Failed http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12734/ :

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

2014-11-19 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change method names in vmtune.py to the PEP8 style .. Patch Set 9: Build Failed

Re: [oVirt Jenkins] vdsm_master_unit-tests_merged - Build # 4202 - Still Failing!

2014-11-19 Thread Dan Kenigsberg
On Mon, Nov 17, 2014 at 04:36:38PM +0200, David Caro wrote: On 11/17, Dan Kenigsberg wrote: On Sun, Nov 16, 2014 at 09:21:18AM +, Jenkins ci oVirt Server wrote: Project: http://jenkins.ovirt.org/job/vdsm_master_unit-tests_merged/ Build:

Change in vdsm[master]: jsonrpc: have simplejson as soft dependency

2014-11-19 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: jsonrpc: have simplejson as soft dependency .. Patch Set 2: Build Failed http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12735/ :

Change in vdsm[master]: netinfo: Replace getVlanDevice/ID in bootstrap with ipwrappe...

2014-11-19 Thread osvoboda
Ondřej Svoboda has uploaded a new change for review. Change subject: netinfo: Replace getVlanDevice/ID in bootstrap with ipwrapper.getLink .. netinfo: Replace getVlanDevice/ID in bootstrap with ipwrapper.getLink Change-Id:

Change in vdsm[master]: netinfo: Replace getVlanDevice/ID in bootstrap with ipwrappe...

2014-11-19 Thread osvoboda
Ondřej Svoboda has posted comments on this change. Change subject: netinfo: Replace getVlanDevice/ID in bootstrap with ipwrapper.getLink .. Patch Set 1: Code-Review-1 (1 comment) Questioning the original code.

Change in vdsm[master]: netinfo: Replace getVlanDevice/ID in bootstrap with ipwrappe...

2014-11-19 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: netinfo: Replace getVlanDevice/ID in bootstrap with ipwrapper.getLink .. Patch Set 1: Build Failed

Change in vdsm[master]: netinfo: Replace misused asserts with InvalidConfiguration

2014-11-19 Thread osvoboda
Ondřej Svoboda has posted comments on this change. Change subject: netinfo: Replace misused asserts with InvalidConfiguration .. Patch Set 3: Code-Review-1 Nir, guys, I don't think we need to correct this old code. Instead,

Change in vdsm[master]: clientIF: Make domain state callback a regular method

2014-11-19 Thread alitke
Adam Litke has uploaded a new change for review. Change subject: clientIF: Make domain state callback a regular method .. clientIF: Make domain state callback a regular method Our storage Event class is taking a weakref on the

Change in vdsm[master]: irs: Directly register domain state change callbacks

2014-11-19 Thread alitke
Adam Litke has uploaded a new change for review. Change subject: irs: Directly register domain state change callbacks .. irs: Directly register domain state change callbacks Currently, domain state change callbacks aren't

Change in vdsm[ovirt-3.5]: irs: Directly register domain state change callbacks

2014-11-19 Thread alitke
Adam Litke has uploaded a new change for review. Change subject: irs: Directly register domain state change callbacks .. irs: Directly register domain state change callbacks Currently, domain state change callbacks aren't

Change in vdsm[ovirt-3.5]: clientIF: Make domain state callback a regular method

2014-11-19 Thread alitke
Adam Litke has uploaded a new change for review. Change subject: clientIF: Make domain state callback a regular method .. clientIF: Make domain state callback a regular method Our storage Event class is taking a weakref on the

Change in vdsm[ovirt-3.5]: Fix ballooning behaviour of freshly received VM

2014-11-19 Thread msivak
Martin Sivák has posted comments on this change. Change subject: Fix ballooning behaviour of freshly received VM .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.ovirt.org/35266 To unsubscribe, visit

Change in vdsm[master]: irs: Directly register domain state change callbacks

2014-11-19 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: irs: Directly register domain state change callbacks .. Patch Set 1: Build Failed

Change in vdsm[master]: clientIF: Make domain state callback a regular method

2014-11-19 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: clientIF: Make domain state callback a regular method .. Patch Set 1: Build Failed

Change in vdsm[master]: networkTests: Extend a test to also consider DHCPv6

2014-11-19 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: networkTests: Extend a test to also consider DHCPv6 .. Patch Set 33: Build Failed

Change in vdsm[master]: jsonrpc: have simplejson as soft dependency

2014-11-19 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: jsonrpc: have simplejson as soft dependency .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/35354 To unsubscribe, visit

Change in vdsm[master]: Sysctl to allow iSCSI multipath with multiple NICs in the sa...

2014-11-19 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Sysctl to allow iSCSI multipath with multiple NICs in the same subnet .. Patch Set 19: Code-Review-1 (1 comment)

Change in vdsm[ovirt-3.5]: Fix ballooning behaviour of freshly received VM

2014-11-19 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: Fix ballooning behaviour of freshly received VM .. Fix ballooning behaviour of freshly received VM There is an issue with freshly received VMs. MOM

Change in vdsm[ovirt-3.5]: Fix ballooning behaviour of freshly received VM

2014-11-19 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Fix ballooning behaviour of freshly received VM .. Patch Set 2: Code-Review+2 Raising score -- To view, visit http://gerrit.ovirt.org/35266 To unsubscribe,

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

2014-11-19 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: change qosOutbound for a nested 'hostQos':{'out': bandwidth_params} .. Patch Set 5: Code-Review+2 (1 comment)

Change in vdsm[master]: host QoS: have service curve parameters be integers

2014-11-19 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: host QoS: have service curve parameters be integers .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/34822 To unsubscribe, visit

Change in vdsm[master]: host QoS: have service curve parameters be integers

2014-11-19 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: host QoS: have service curve parameters be integers .. host QoS: have service curve parameters be integers It does not make sense to represent

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

2014-11-19 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: change qosOutbound for a nested 'hostQos':{'out': bandwidth_params} .. change qosOutbound for a nested 'hostQos':{'out': bandwidth_params} The engine

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

2014-11-19 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 6: Build Failed

Change in vdsm[master]: host QoS: have service curve parameters be integers

2014-11-19 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: host QoS: have service curve parameters be integers .. Patch Set 3: Build Failed http://jenkins.ovirt.org/job/vdsm_master_unit-tests_merged/4219/ :

Change in vdsm[master]: Provide device details using python-blivet module

2014-11-19 Thread tjeyasin
Timothy Asir has posted comments on this change. Change subject: Provide device details using python-blivet module .. Patch Set 5: (5 comments) http://gerrit.ovirt.org/#/c/35028/5//COMMIT_MSG Commit Message: Line 3:

Change in vdsm[master]: gluster: add glusterStorageDevicesList verb

2014-11-19 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: gluster: add glusterStorageDevicesList verb .. Patch Set 6: Build Failed http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12741/ :

Change in vdsm[master]: gluster: add glusterStorageDevicesList verb

2014-11-19 Thread tjeyasin
Timothy Asir has posted comments on this change. Change subject: gluster: add glusterStorageDevicesList verb .. Patch Set 6: Verified+1 -- To view, visit http://gerrit.ovirt.org/35028 To unsubscribe, visit

Change in vdsm[master]: libnl: make link cache optional for link/route/addr_info

2014-11-19 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: libnl: make link cache optional for link/route/addr_info .. Patch Set 22: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/32949 To unsubscribe, visit

Change in vdsm[master]: libnl: make link cache optional for link/route/addr_info

2014-11-19 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: libnl: make link cache optional for link/route/addr_info .. libnl: make link cache optional for link/route/addr_info Change-Id:

Change in vdsm[master]: libnl: non-existing link exception

2014-11-19 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: libnl: non-existing link exception .. Patch Set 12: Code-Review-1 (1 comment) http://gerrit.ovirt.org/#/c/34690/12/lib/vdsm/netlink/link.py File

Change in vdsm[master]: make isVDSMInterface and it's helpers static

2014-11-19 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: make isVDSMInterface and it's helpers static .. make isVDSMInterface and it's helpers static Change-Id: I6878c3b98e6acbeae440fb29d71bddf6cadc5fae

Change in vdsm[master]: make isVDSMInterface and it's helpers static

2014-11-19 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: make isVDSMInterface and it's helpers static .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/34139 To unsubscribe, visit

Change in vdsm[master]: libnl: make link cache optional for link/route/addr_info

2014-11-19 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: libnl: make link cache optional for link/route/addr_info .. Patch Set 23: Build Failed

Change in vdsm[master]: tool: change configurators from classes to modules.

2014-11-19 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: tool: change configurators from classes to modules. .. Patch Set 3: Code-Review-1 please rebase this branch -- To view, visit http://gerrit.ovirt.org/34047

Change in vdsm[master]: tool: configurators package should only contain configurators.

2014-11-19 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: tool: configurators package should only contain configurators. .. Patch Set 3: Code-Review-1 -- To view, visit http://gerrit.ovirt.org/34048 To unsubscribe,

Change in vdsm[master]: tool: autoload modules.

2014-11-19 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: tool: autoload modules. .. Patch Set 3: Code-Review-1 -- To view, visit http://gerrit.ovirt.org/34049 To unsubscribe, visit http://gerrit.ovirt.org/settings

Change in vdsm[master]: tool: change configure verb logic.

2014-11-19 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: tool: change configure verb logic. .. Patch Set 2: Code-Review-1 -- To view, visit http://gerrit.ovirt.org/34100 To unsubscribe, visit

Change in vdsm[master]: tool: configurator module names optional, defaults to file n...

2014-11-19 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: tool: configurator module names optional, defaults to file name. .. Patch Set 2: Code-Review-1 -- To view, visit http://gerrit.ovirt.org/34099 To unsubscribe,

Change in vdsm[master]: make isVDSMInterface and it's helpers static

2014-11-19 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: make isVDSMInterface and it's helpers static .. Patch Set 7: Build Failed http://jenkins.ovirt.org/job/vdsm_master_unit-tests_merged/4221/ : FAILURE

Change in vdsm[master]: caps: Do not memoize CPU topology

2014-11-19 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: caps: Do not memoize CPU topology .. Patch Set 4: Code-Review-1 Please consider rebasing on top of http://gerrit.ovirt.org/#/c/34709/4 and using its

Change in vdsm[master]: cache: Add caching decorator with invalidation

2014-11-19 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: cache: Add caching decorator with invalidation .. Patch Set 4: (2 comments) http://gerrit.ovirt.org/#/c/34709/4/lib/vdsm/cache.py File lib/vdsm/cache.py:

Change in vdsm[master]: networkTests: Extend a test to also consider DHCPv6

2014-11-19 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: networkTests: Extend a test to also consider DHCPv6 .. Patch Set 34: Build Failed

Change in vdsm[master]: gluster: Add geo-replication start and stop session

2014-11-19 Thread dnarayan
Darshan N has posted comments on this change. Change subject: gluster: Add geo-replication start and stop session .. Patch Set 7: (1 comment) http://gerrit.ovirt.org/#/c/17766/7/vdsm/gluster/exception.py File

Change in vdsm[master]: gluster: Add geo-replication start and stop session

2014-11-19 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: gluster: Add geo-replication start and stop session .. Patch Set 10: Build Failed

Change in vdsm[master]: gluster: Added geo-replication status and status detail

2014-11-19 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: gluster: Added geo-replication status and status detail .. Patch Set 14: Build Failed

Change in vdsm[master]: gluster: volume snapshot create and delete verbs.

2014-11-19 Thread dnarayan
Darshan N has posted comments on this change. Change subject: gluster: volume snapshot create and delete verbs. .. Patch Set 4: (4 comments) http://gerrit.ovirt.org/#/c/34535/4/vdsm.spec.in File vdsm.spec.in: Line 237:

Change in vdsm[master]: gluster: volume snapshot create and delete verbs.

2014-11-19 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: gluster: volume snapshot create and delete verbs. .. Patch Set 5: Build Failed http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/12745/

Change in vdsm[master]: gluster: Add geo-replication start and stop session

2014-11-19 Thread dnarayan
Darshan N has posted comments on this change. Change subject: gluster: Add geo-replication start and stop session .. Patch Set 10: Verified+1 -- To view, visit http://gerrit.ovirt.org/17766 To unsubscribe, visit

Change in vdsm[master]: gluster: Added geo-replication status and status detail

2014-11-19 Thread dnarayan
Darshan N has posted comments on this change. Change subject: gluster: Added geo-replication status and status detail .. Patch Set 14: Verified+1 -- To view, visit http://gerrit.ovirt.org/18414 To unsubscribe, visit

Change in vdsm[master]: gluster: volume snapshot create and delete verbs.

2014-11-19 Thread dnarayan
Darshan N has posted comments on this change. Change subject: gluster: volume snapshot create and delete verbs. .. Patch Set 5: Verified+1 -- To view, visit http://gerrit.ovirt.org/34535 To unsubscribe, visit

<    1   2