Change in vdsm[master]: getVolumeInfo() new implementation.

2013-09-17 Thread ewarszaw
Eduardo has posted comments on this change. Change subject: getVolumeInfo() new implementation. .. Patch Set 3: (2 comments) Commit Message Line 5: CommitDate: 2013-09-16

Change in vdsm[master]: Making getVSize and getVTrueSize SD methods.

2013-09-17 Thread ewarszaw
Eduardo has posted comments on this change. Change subject: Making getVSize and getVTrueSize SD methods. .. Patch Set 4: (3 comments) Commit Message Line 12: Line 13:

Change in vdsm[master]: Use the full prefix when removing file images.

2013-09-17 Thread abaron
Ayal Baron has posted comments on this change. Change subject: Use the full prefix when removing file images. .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/19107 To unsubscribe, visit

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

2013-09-17 Thread abaron
Ayal Baron has posted comments on this change. Change subject: Avoid redundant volume produces. .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/17991 To unsubscribe, visit

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

2013-09-17 Thread abaron
Ayal Baron has posted comments on this change. Change subject: Rename *Volume.extend method to *Volume.enlarge .. Patch Set 1: Code-Review-2 I see no point for this change (which is not explained in the commit message

Change in vdsm[master]: Avoid hsm image deletions.

2013-09-17 Thread abaron
Ayal Baron has posted comments on this change. Change subject: Avoid hsm image deletions. .. Patch Set 4: Code-Review-2 -2 for visibility since you keep on rebasing without addressing the comments -- To view, visit

Change in vdsm[master]: Fix parent volume rollback after failure to create snapshot

2013-09-17 Thread abaron
Ayal Baron has posted comments on this change. Change subject: Fix parent volume rollback after failure to create snapshot .. Patch Set 6: (1 comment) File

Change in vdsm[master]: hsm: deleteImage fails because of wrong dictionary use

2013-09-17 Thread abaron
Ayal Baron has posted comments on this change. Change subject: hsm: deleteImage fails because of wrong dictionary use .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/17383 To unsubscribe, visit

Change in vdsm[master]: reduce the time VM is paused in snapshot with RAM creation

2013-09-17 Thread ahadas
Arik Hadas has posted comments on this change. Change subject: reduce the time VM is paused in snapshot with RAM creation .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.ovirt.org/19296 To unsubscribe, visit

Change in vdsm[master]: Introduce lvm short filters.

2013-09-17 Thread abaron
Ayal Baron has posted comments on this change. Change subject: Introduce lvm short filters. .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/17968 To unsubscribe, visit http://gerrit.ovirt.org/settings

Change in vdsm[master]: Use the full prefix when removing file images.

2013-09-17 Thread sgotliv
Sergey Gotliv has posted comments on this change. Change subject: Use the full prefix when removing file images. .. Patch Set 1: Code-Review+1 (1 comment) The patch is good, but descriptions should be changed?

Change in vdsm[master]: reduce the time VM is paused in snapshot with RAM creation

2013-09-17 Thread michal . skrivanek
Michal Skrivanek has posted comments on this change. Change subject: reduce the time VM is paused in snapshot with RAM creation .. Patch Set 1: (1 comment) File vdsm/vm.py

Change in vdsm[master]: hsm: deleteImage fails because of wrong dictionary use

2013-09-17 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: hsm: deleteImage fails because of wrong dictionary use .. hsm: deleteImage fails because of wrong dictionary use also, get fake template params before

Change in vdsm[master]: glusterSD: BZ 988299: Deduce gluster volume name from domain...

2013-09-17 Thread fsimonce
Federico Simoncelli has posted comments on this change. Change subject: glusterSD: BZ 988299: Deduce gluster volume name from domain metadata .. Patch Set 1: Code-Review-1 (2 comments) -1 for visibility

Change in vdsm[master]: Fix parent volume rollback after failure to create snapshot

2013-09-17 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: Fix parent volume rollback after failure to create snapshot .. Patch Set 6: -Code-Review (2 comments) Reply to comments

Change in vdsm[master]: glusterSD: BZ 988299: Deduce gluster volume name from domain...

2013-09-17 Thread deepakcs
Deepak C Shetty has posted comments on this change. Change subject: glusterSD: BZ 988299: Deduce gluster volume name from domain metadata .. Patch Set 1: (1 comment) File

Change in vdsm[master]: Report bond speed as function of slaved nics speed and bond ...

2013-09-17 Thread wudxw
Mark Wu has posted comments on this change. Change subject: Report bond speed as function of slaved nics speed and bond mode .. Patch Set 1: Code-Review-1 (4 comments) File

Change in vdsm[master]: multipath.getMPDevNamesIter() return /dev/mapper.

2013-09-17 Thread smizrahi
Saggi Mizrahi has posted comments on this change. Change subject: multipath.getMPDevNamesIter() return /dev/mapper. .. Patch Set 2: One of Saggi's automated scripts discovered that you are changing a file he deeply cares

Change in vdsm[master]: sp: update lease parameters on masterMigrate

2013-09-17 Thread fsimonce
Federico Simoncelli has posted comments on this change. Change subject: sp: update lease parameters on masterMigrate .. Patch Set 2: Verified+1 Verified migrating the master from one SD to another. -- To view, visit

Change in vdsm[master]: packaging: spec: own /etc/pki/vdsm/libvirt-spice

2013-09-17 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: packaging: spec: own /etc/pki/vdsm/libvirt-spice .. packaging: spec: own /etc/pki/vdsm/libvirt-spice Change-Id: If939a60a916c1cc8c529d91e4bc379032048d42e

Change in vdsm[master]: packaging: spec: own /etc/pki/vdsm/libvirt-spice

2013-09-17 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: packaging: spec: own /etc/pki/vdsm/libvirt-spice .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/4491/ :

Change in vdsm[master]: vdsm: move balloon logic out of buildConfDevices

2013-09-17 Thread mpoledni
Martin Polednik has uploaded a new change for review. Change subject: vdsm: move balloon logic out of buildConfDevices .. vdsm: move balloon logic out of buildConfDevices Encapsulates empty balloon creation logic outside of

Change in vdsm[master]: vdsm: move balloon logic out of buildConfDevices

2013-09-17 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: vdsm: move balloon logic out of buildConfDevices .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/4492/ :

Change in vdsm[master]: constants: unify the BLANK_UUID definition

2013-09-17 Thread smizrahi
Saggi Mizrahi has posted comments on this change. Change subject: constants: unify the BLANK_UUID definition .. Patch Set 2: Code-Review-1 (1 comment) I agree that code duplication is an issue but constants.py is not the

Change in vdsm[master]: Raise specific exception when local path not exist

2013-09-17 Thread smizrahi
Saggi Mizrahi has posted comments on this change. Change subject: Raise specific exception when local path not exist .. Patch Set 3: Code-Review-1 (1 comment) File

Change in vdsm[master]: Change mom policy related interface to support multi-policy

2013-09-17 Thread smizrahi
Saggi Mizrahi has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 11: I reviewed *only* the API change and I approve. I have not looked at the

Change in vdsm[master]: Report bond speed as function of slaved nics speed and bond ...

2013-09-17 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: Report bond speed as function of slaved nics speed and bond mode .. Patch Set 2: Build Successful

Change in vdsm[master]: Report bond speed as function of slaved nics speed and bond ...

2013-09-17 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: Report bond speed as function of slaved nics speed and bond mode .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/19297 To

Change in vdsm[master]: Make HSM service self containted

2013-09-17 Thread smizrahi
Saggi Mizrahi has posted comments on this change. Change subject: Make HSM service self containted .. Patch Set 5: Code-Review-2 This is not the way to go. We are trying to move away from the mismatch between where files are

Change in vdsm[master]: Report bond speed as function of slaved nics speed and bond ...

2013-09-17 Thread apahim
Amador Pahim has posted comments on this change. Change subject: Report bond speed as function of slaved nics speed and bond mode .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.ovirt.org/19297 To unsubscribe,

Change in vdsm[master]: Raise specific exception when local path not exist

2013-09-17 Thread ewarszaw
Eduardo has posted comments on this change. Change subject: Raise specific exception when local path not exist .. Patch Set 3: Code-Review-1 (1 comment) Please explain which issue are you trying to solve. Raising specific

Change in vdsm[master]: packaging: spec: own /etc/pki/vdsm/libvirt-spice

2013-09-17 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: packaging: spec: own /etc/pki/vdsm/libvirt-spice .. Patch Set 2: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/4494/ :

Change in vdsm[master]: packaging: spec: own /etc/pki/vdsm/libvirt-spice

2013-09-17 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: packaging: spec: own /etc/pki/vdsm/libvirt-spice .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/19318 To unsubscribe, visit

Change in vdsm[master]: packaging: spec: own /etc/pki/vdsm/libvirt-spice

2013-09-17 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: spec: own /etc/pki/vdsm/libvirt-spice .. Patch Set 2: waiting for: testStressTest to verify... -- To view, visit http://gerrit.ovirt.org/19318 To

Change in vdsm[master]: Change of ConfigWriter._createConfFile parameters

2013-09-17 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Change of ConfigWriter._createConfFile parameters .. Patch Set 6: Code-Review-1 -- To view, visit http://gerrit.ovirt.org/18331 To unsubscribe, visit

Change in vdsm[master]: packaging: spec: own /etc/pki/vdsm/libvirt-spice

2013-09-17 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: spec: own /etc/pki/vdsm/libvirt-spice .. Patch Set 2: Verified+1 this stress test never ends... had to kill it. -- To view, visit

Change in vdsm[master]: reduce the time VM is paused in snapshot with RAM creation

2013-09-17 Thread ahadas
Arik Hadas has posted comments on this change. Change subject: reduce the time VM is paused in snapshot with RAM creation .. Patch Set 1: (1 comment) File vdsm/vm.py Line

Change in vdsm[master]: sp: update lease parameters on masterMigrate

2013-09-17 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: sp: update lease parameters on masterMigrate .. Patch Set 2: danken: what if one storage domain has considerably different latency? paraphrasing fsimonce's

Change in vdsm[master]: More precise exception when multipath failed.

2013-09-17 Thread ykaplan
Yeela Kaplan has posted comments on this change. Change subject: More precise exception when multipath failed. .. Patch Set 4: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/17145 To unsubscribe, visit

Change in vdsm[master]: reduce the time VM is paused in snapshot with RAM creation

2013-09-17 Thread michal . skrivanek
Michal Skrivanek has posted comments on this change. Change subject: reduce the time VM is paused in snapshot with RAM creation .. Patch Set 1: (1 comment) File vdsm/vm.py

Change in vdsm[master]: sp: update lease parameters on masterMigrate

2013-09-17 Thread fsimonce
Federico Simoncelli has posted comments on this change. Change subject: sp: update lease parameters on masterMigrate .. Patch Set 3: Verified+1 Commit change only. -- To view, visit http://gerrit.ovirt.org/19176 To

Change in vdsm[master]: sp: update lease parameters on masterMigrate

2013-09-17 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: sp: update lease parameters on masterMigrate .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/19176 To unsubscribe, visit

Change in vdsm[master]: make misc.pgrep for general usage

2013-09-17 Thread smizrahi
Saggi Mizrahi has posted comments on this change. Change subject: make misc.pgrep for general usage .. Patch Set 3: Code-Review-1 (1 comment) File vdsm/storage/misc.py Line

Change in vdsm[master]: sp: update lease parameters on masterMigrate

2013-09-17 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: sp: update lease parameters on masterMigrate .. sp: update lease parameters on masterMigrate Given the current API the lease parameters should be

Change in vdsm[master]: sp: update lease parameters on masterMigrate

2013-09-17 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: sp: update lease parameters on masterMigrate .. Patch Set 3: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/4495/ : SUCCESS

Change in vdsm[master]: Fix getStorageDomainInfo() logic.

2013-09-17 Thread abaron
Ayal Baron has posted comments on this change. Change subject: Fix getStorageDomainInfo() logic. .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/14671 To unsubscribe, visit

Change in vdsm[master]: Read pool metadata once in StoragePool.getInfo()

2013-09-17 Thread abaron
Ayal Baron has posted comments on this change. Change subject: Read pool metadata once in StoragePool.getInfo() .. Patch Set 5: Code-Review+2 The subsequent patch addresses my comment so +2 -- To view, visit

Change in vdsm[master]: packaging: spec: own /etc/pki/vdsm/libvirt-spice

2013-09-17 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: packaging: spec: own /etc/pki/vdsm/libvirt-spice .. Patch Set 2: Yes, this is a delicate dangerous change. -- To view, visit http://gerrit.ovirt.org/19318 To

Change in vdsm[master]: supervdsmServer was not passing kwargs to fuser

2013-09-17 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: supervdsmServer was not passing kwargs to fuser .. Patch Set 1: Yaniv, supervdsmProxy should better protect itself from unexpected extra arguments. -- To

Change in vdsm[master]: packaging: spec: own /etc/pki/vdsm/libvirt-spice

2013-09-17 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: packaging: spec: own /etc/pki/vdsm/libvirt-spice .. packaging: spec: own /etc/pki/vdsm/libvirt-spice Change-Id:

Change in vdsm[master]: vdsm: move watchdog default params to WatchdogDevice

2013-09-17 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: vdsm: move watchdog default params to WatchdogDevice .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/4496/ :

Change in vdsm[master]: Log client address on SSL errors

2013-09-17 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: Log client address on SSL errors .. Patch Set 3: Verified-1 Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/4499/ : FAILURE

Change in vdsm[master]: Log client address on SSL errors

2013-09-17 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: Log client address on SSL errors .. Patch Set 3: I verified that this works correctly, but the tests are failing in Jenkins, and I don't understand why. I

Change in vdsm[master]: Log client address on SSL errors

2013-09-17 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: Log client address on SSL errors .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.ovirt.org/19332 To unsubscribe, visit

Change in vdsm[master]: Log client address on SSL errors

2013-09-17 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: Log client address on SSL errors .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.ovirt.org/19332 To unsubscribe, visit

Change in vdsm[master]: Log client address on SSL errors

2013-09-17 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: Log client address on SSL errors .. Patch Set 1: Code-Review-1 Verified-1 Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/4497/ :

Change in vdsm[master]: Normalize lvm._buildFilter().

2013-09-17 Thread abaron
Ayal Baron has posted comments on this change. Change subject: Normalize lvm._buildFilter(). .. Patch Set 2: (1 comment) File vdsm/storage/lvm.py Line 131: strippeds =

Change in vdsm[master]: vdsm: move watchdog default params to WatchdogDevice

2013-09-17 Thread mpoledni
Martin Polednik has uploaded a new change for review. Change subject: vdsm: move watchdog default params to WatchdogDevice .. vdsm: move watchdog default params to WatchdogDevice Watchdog device creation is currently for no

Change in vdsm[master]: supervdsmServer was not passing kwargs to fuser

2013-09-17 Thread abaron
Ayal Baron has posted comments on this change. Change subject: supervdsmServer was not passing kwargs to fuser .. Patch Set 1: +1 on what Dan said. unexpected extra arguments should throw an exception so that such bugs would

Change in vdsm[master]: Packaging: VDSM v4.12.0 packaged on Ubuntu

2013-09-17 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: Packaging: VDSM v4.12.0 packaged on Ubuntu .. Patch Set 6: Verified+1 Since Ubuntu and Debian are among the most popular distributions, I think it's good to

Change in vdsm[master]: vdsm: refactor buildConfDevices and getConfDevices into sing...

2013-09-17 Thread mpoledni
Martin Polednik has uploaded a new change for review. Change subject: vdsm: refactor buildConfDevices and getConfDevices into single function .. vdsm: refactor buildConfDevices and getConfDevices into single function Removes

Change in vdsm[master]: vdsm: refactor buildConfDevices and getConfDevices into sing...

2013-09-17 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: vdsm: refactor buildConfDevices and getConfDevices into single function .. Patch Set 1: Build Successful

Change in vdsm[master]: vdsm: centralize device limit checking

2013-09-17 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: vdsm: centralize device limit checking .. Patch Set 14: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/4501/ : SUCCESS