Change in vdsm[master]: lint: providing a default pylintrc

2013-08-08 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: lint: providing a default pylintrc .. Patch Set 4: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/17772 To unsubscribe, visit http://gerrit.ov

Change in vdsm[master]: refactoring: rename variable in getVlanID function

2013-08-08 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: refactoring: rename variable in getVlanID function .. Patch Set 1: (1 comment) File lib/vdsm/netin

Change in vdsm[master]: refactoring: rename variable in getVlanID function

2013-08-08 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: refactoring: rename variable in getVlanID function .. Patch Set 1: Code-Review-1 (1 comment) File

Change in vdsm[master]: refactoring: rename variable in getVlanID function

2013-08-08 Thread wudxw
Mark Wu has posted comments on this change. Change subject: refactoring: rename variable in getVlanID function .. Patch Set 1: Code-Review-1 -- To view, visit http://gerrit.ovirt.org/17830 To unsubscribe, visit http://gerrit.o

Change in vdsm[master]: refactoring: rename variable in getVlanID function

2013-08-08 Thread wudxw
Mark Wu has posted comments on this change. Change subject: refactoring: rename variable in getVlanID function .. Patch Set 1: (1 comment) I agree with that id is not a good variable name in python, but it's also ok because w

Change in vdsm[master]: libvirt configure: drop "--copy" option from "sed" invocation

2013-08-08 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: libvirt configure: drop "--copy" option from "sed" invocation .. Patch Set 1: Code-Review-1 --copy was required for bind-mouted files on ovirt-node. Please cons

Change in vdsm[master]: Management network is now kept in main routing table

2013-08-08 Thread amuller
Assaf Muller has posted comments on this change. Change subject: Management network is now kept in main routing table .. Patch Set 11: -Verified Found an issue with a very specific flow. Removing the verified flag. -- To view

Change in vdsm[ovirt-3.3]: Management network is now kept in main routing table

2013-08-08 Thread amuller
Assaf Muller has uploaded a new change for review. Change subject: Management network is now kept in main routing table .. Management network is now kept in main routing table Only the management network should add its default r

Change in vdsm[master]: Fixing return value of function 'start' in init script

2013-08-08 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: Fixing return value of function 'start' in init script .. Fixing return value of function 'start' in init script During fixing lock mechanism (http://g

Change in vdsm[master]: Fixing return value of function 'start' in init script

2013-08-08 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Fixing return value of function 'start' in init script .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/17828 To unsubscribe, visit http

Change in vdsm[master]: Fixing return value of function 'start' in init script

2013-08-08 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: Fixing return value of function 'start' in init script .. Patch Set 3: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3819/ : S

Change in vdsm[master]: Fixing return value of function 'start' in init script

2013-08-08 Thread obasan
Ohad Basan has posted comments on this change. Change subject: Fixing return value of function 'start' in init script .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.ovirt.org/17828 To unsubscribe, visit http://gerr

Change in vdsm[master]: Fixing return value of function 'start' in init script

2013-08-08 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: Fixing return value of function 'start' in init script .. Patch Set 3: Verified+1 Code-Review+1 -- To view, visit http://gerrit.ovirt.org/17828 To unsubscribe,

Change in vdsm[master]: vdsm: support VIR_MIGRATE_ABORT_ON_ERROR

2013-08-08 Thread michal . skrivanek
Michal Skrivanek has posted comments on this change. Change subject: vdsm: support VIR_MIGRATE_ABORT_ON_ERROR .. Patch Set 2: -Code-Review (1 comment) File vdsm/vm.py Line 3

Change in vdsm[master]: vdsm: return start retval if success instead of always 1

2013-08-08 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: vdsm: return start retval if success instead of always 1 .. Patch Set 2: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3818/ :

Change in vdsm[master]: vdsm: fix a bug that prevents vdsm from starting

2013-08-08 Thread obasan
Ohad Basan has posted comments on this change. Change subject: vdsm: fix a bug that prevents vdsm from starting .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.ovirt.org/17828 To unsubscribe, visit http://gerrit.ovi

Change in vdsm[master]: refactoring: rename variable in getVlanID function

2013-08-08 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: refactoring: rename variable in getVlanID function .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3817/ : SUCCE

Change in vdsm[master]: refactoring: rename variable in getVlanID function

2013-08-08 Thread gvallare
Giuseppe Vallarelli has posted comments on this change. Change subject: refactoring: rename variable in getVlanID function .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.ovirt.org/17830 To unsubscribe, visit http:/

Change in vdsm[master]: refactoring: rename variable in getVlanID function

2013-08-08 Thread gvallare
Giuseppe Vallarelli has uploaded a new change for review. Change subject: refactoring: rename variable in getVlanID function .. refactoring: rename variable in getVlanID function In the function getVlanID we're using id as a var

Change in vdsm[master]: vdsm: fix a bug that prevents vdsm from starting

2013-08-08 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: vdsm: fix a bug that prevents vdsm from starting .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3816/ : SUCCESS

Change in vdsm[master]: vdsm: fix a bug that prevents vdsm from starting

2013-08-08 Thread obasan
Ohad Basan has uploaded a new change for review. Change subject: vdsm: fix a bug that prevents vdsm from starting .. vdsm: fix a bug that prevents vdsm from starting Change-Id: If9a9e89594f136d8e7b41b713e43d64f3702d454 Signed-of

Change in vdsm[master]: Don't reconfigure vlan devs unless necessary.

2013-08-08 Thread psebek
Petr Ĺ ebek has posted comments on this change. Change subject: Don't reconfigure vlan devs unless necessary. .. Patch Set 4: (1 comment) File vdsm/netmodels.py Line 186:

Change in vdsm[master]: gluster: [WIP]Get size information of gluster volume

2013-08-08 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: gluster: [WIP]Get size information of gluster volume .. Patch Set 1: Code-Review-1 Verified-1 Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tes

Change in vdsm[master]: gluster: [WIP]Get size information of gluster volume

2013-08-08 Thread avishwan
Aravinda VK has uploaded a new change for review. Change subject: gluster: [WIP]Get size information of gluster volume .. gluster: [WIP]Get size information of gluster volume Change-Id: I358d4f3bf793ecc1a01e0592d68919d1405f6e19

Change in vdsm[master]: Management network is now kept in main routing table

2013-08-08 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: Management network is now kept in main routing table .. Patch Set 11: Code-Review+1 Thanks! -- To view, visit http://gerrit.ovirt.org/17575 To unsubsc

Change in vdsm[master]: Management network is now kept in main routing table

2013-08-08 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: Management network is now kept in main routing table .. Patch Set 11: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3814/ : SU

Change in vdsm[master]: Lock init script when vdsmd starts up until the service is s...

2013-08-08 Thread sbonazzo
Sandro Bonazzola has posted comments on this change. Change subject: Lock init script when vdsmd starts up until the service is stopped .. Patch Set 11: This patch doesn't work for me. On Fedora 19 I can't start vdsmd service

Change in vdsm[master]: Management network is now kept in main routing table

2013-08-08 Thread amuller
Assaf Muller has posted comments on this change. Change subject: Management network is now kept in main routing table .. Patch Set 11: Verified+1 -- To view, visit http://gerrit.ovirt.org/17575 To unsubscribe, visit http://ger

Change in vdsm[master]: lint: providing a default pylintrc

2013-08-08 Thread gvallare
Giuseppe Vallarelli has posted comments on this change. Change subject: lint: providing a default pylintrc .. Patch Set 4: Hi Alon, I think you might need to customize the pylint for the specific project. Clearly there might b

Change in vdsm[master]: lint: providing a default pylintrc

2013-08-08 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: lint: providing a default pylintrc .. Patch Set 4: I tried this lint in the past, there were many false positives. I retried just now it with this configuration

Change in vdsm[master]: lint: providing a default pylintrc

2013-08-08 Thread sbonazzo
Sandro Bonazzola has posted comments on this change. Change subject: lint: providing a default pylintrc .. Patch Set 4: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/17772 To unsubscribe, visit http://gerrit.ovirt.or

Change in vdsm[master]: lint: providing a default pylintrc

2013-08-08 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: lint: providing a default pylintrc .. Patch Set 4: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3813/ : SUCCESS http://jenki

Change in vdsm[master]: lint: providing a default pylintrc

2013-08-08 Thread gvallare
Giuseppe Vallarelli has posted comments on this change. Change subject: lint: providing a default pylintrc .. Patch Set 4: Verified+1 -- To view, visit http://gerrit.ovirt.org/17772 To unsubscribe, visit http://gerrit.ovirt.or

Change in vdsm[master]: libvirt configure: drop "--copy" option from "sed" invocation

2013-08-08 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: libvirt configure: drop "--copy" option from "sed" invocation .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/17813 To unsubscribe, vis

Change in vdsm[master]: Rename *Volume.extend method to *Volume.enlarge

2013-08-08 Thread ykaplan
Yeela Kaplan has posted comments on this change. Change subject: Rename *Volume.extend method to *Volume.enlarge .. Patch Set 1: Why? -- To view, visit http://gerrit.ovirt.org/17802 To unsubscribe, visit http://gerrit.ovirt.o

Change in vdsm[master]: make our udev rules take effect later than default Ubuntu lv...

2013-08-08 Thread zhshzhou
Zhou Zheng Sheng has abandoned this change. Change subject: make our udev rules take effect later than default Ubuntu lvm rules .. Abandoned This change will be moved to Ubuntu packaging rules. -- To view, visit http://gerri

Change in vdsm[master]: vdsm: Load softdog if hardware watchdog is not available

2013-08-08 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: vdsm: Load softdog if hardware watchdog is not available .. Patch Set 1: Verified+1 Test on Fedora, RHEL and Ubuntu. This patch logically depends on http://g

Change in vdsm[master]: libvirt configure: drop "--copy" option from "sed" invocation

2013-08-08 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: libvirt configure: drop "--copy" option from "sed" invocation .. Patch Set 1: Verified+1 Test on Fedora, RHEL and Ubuntu. -- To view, visit http://gerrit.ov

Change in vdsm[master]: Add vdsm upstart jobs

2013-08-08 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: Add vdsm upstart jobs .. Patch Set 1: Verified+1 Test on Fedora, RHEL and Ubuntu by running xmlrpc functional tests. To test this patch, we need some extra c

Change in vdsm[master]: xmlrpcTests: Detect Ubuntu initramfs when booting VM

2013-08-08 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: xmlrpcTests: Detect Ubuntu initramfs when booting VM .. Patch Set 1: Verified+1 Test on Fedora, RHEL and Ubuntu by running xmlrpc functional tests. -- To vi

Change in vdsm[master]: xmlrpcTests: Mount configfs automatically

2013-08-08 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: xmlrpcTests: Mount configfs automatically .. Patch Set 1: Verified+1 Test on Fedora, RHEL and Ubuntu by running xmlrpc functional tests. -- To view, visit h

Change in vdsm[master]: SecureXMLRPCServer: Work with Official M2Crypto Release

2013-08-08 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: SecureXMLRPCServer: Work with Official M2Crypto Release .. Patch Set 1: Verified+1 Test on Fedora, RHEL and Ubuntu by running xmlrpc functional tests. -- To

Change in vdsm[master]: packaging: rename supervdsmServer.py to supervdsmServer and ...

2013-08-08 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: packaging: rename supervdsmServer.py to supervdsmServer and add environment file .. Patch Set 4: Verified+1 Test on Fedora, RHEL and Ubuntu. The new patch

Change in vdsm[master]: vdsmd and supervdsmd: extract common init tasks and add init...

2013-08-08 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: vdsmd and supervdsmd: extract common init tasks and add init adapter .. Patch Set 13: Verified+1 Test on Fedora, RHEL and Ubuntu. The new patch set is reba