Change in vdsm[master]: Move gluster/{cli.py, exception.py} to vdsm rpm

2012-09-21 Thread barumuga
Bala.FA has posted comments on this change. Change subject: Move gluster/{cli.py, exception.py} to vdsm rpm .. Patch Set 2: If gluster/{cli,exception}.py are part of vdsm-python, I think, check of gluster cli import in

Change in vdsm[master]: schema: Remove cpuIdle from HostStats

2012-09-21 Thread lhornyak
Laszlo Hornyak has posted comments on this change. Change subject: schema: Remove cpuIdle from HostStats .. Patch Set 1: Mark, I have left the cpuIdle in the host statistics intentionally, I only removed the cpuIdle from vm

Change in vdsm[master]: Extend the range of Vm's niceness to allow increasing cpu sh...

2012-09-21 Thread lhornyak
Laszlo Hornyak has posted comments on this change. Change subject: Extend the range of Vm's niceness to allow increasing cpu shares .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit

Change in vdsm[master]: bootstrap: use yum API

2012-09-21 Thread bpradip
Pradipta Banerjee has posted comments on this change. Change subject: bootstrap: use yum API .. Patch Set 3: (1 inline comment) File vds_bootstrap/vds_bootstrap.py Line 964:

Change in vdsm[master]: report cpuUser and cpuSys separately

2012-09-21 Thread lhornyak
Laszlo Hornyak has posted comments on this change. Change subject: report cpuUser and cpuSys separately .. Patch Set 3: (1 inline comment) File vdsm/libvirtvm.py Line 172:

Change in vdsm[master]: bootstrap: use yum API

2012-09-21 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: bootstrap: use yum API .. Patch Set 3: (2 inline comments) File vds_bootstrap/vds_bootstrap.py Line 964:

Change in vdsm[master]: schema: Remove cpuIdle from HostStats

2012-09-21 Thread wudxw
Mark Wu has abandoned this change. Change subject: schema: Remove cpuIdle from HostStats .. Patch Set 1: Abandoned Oops! I should have checked it out carefully! Laszlo, thanks for pointing out it. -- To view, visit

Change in vdsm[master]: report cpuUser and cpuSys separately

2012-09-21 Thread wudxw
Mark Wu has posted comments on this change. Change subject: report cpuUser and cpuSys separately .. Patch Set 3: (1 inline comment) File vdsm/libvirtvm.py Line 172: def

Change in vdsm[master]: Move gluster/{cli.py, exception.py} to vdsm rpm

2012-09-21 Thread deepakcs
Deepak C Shetty has posted comments on this change. Change subject: Move gluster/{cli.py, exception.py} to vdsm rpm .. Patch Set 2: No score @Bala, The files are moved to vdsm ( not vdsm-python) rpm. Is there any specific

Change in vdsm[master]: vdsm-upgrade: stop vdsm when complete upgrade

2012-09-21 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: vdsm-upgrade: stop vdsm when complete upgrade .. vdsm-upgrade: stop vdsm when complete upgrade We should stop vdsm daemon to avoid any additional

Change in vdsm[master]: vdsm-upgrade: stop vdsm when complete upgrade

2012-09-21 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: vdsm-upgrade: stop vdsm when complete upgrade .. Patch Set 5: Looks good to me, approved thanks. -- To view, visit http://gerrit.ovirt.org/8029 To

Change in vdsm[master]: pep8ize vdsm/vdsm

2012-09-21 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: pep8ize vdsm/vdsm .. Patch Set 1: I would prefer that you didn't submit this (1 inline comment) File

Change in vdsm[master]: Generalize asynchronous ifupping of network interfaces.

2012-09-21 Thread wudxw
Mark Wu has posted comments on this change. Change subject: Generalize asynchronous ifupping of network interfaces. .. Patch Set 2: Just rebase to latest code. There's no other changes. I am wondering if gerrit can skip this

Change in vdsm[master]: Make nics, vlan and bonding dhcp ifup to be asynchronous.

2012-09-21 Thread wudxw
Mark Wu has posted comments on this change. Change subject: Make nics, vlan and bonding dhcp ifup to be asynchronous. .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit

Change in vdsm[master]: Remove superfluous macros in the specfile

2012-09-21 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: Remove superfluous macros in the specfile .. Remove superfluous macros in the specfile In this patch: * remove the redundant %attr macros for the vdsm

Change in vdsm[master]: Remove superfluous macros in the specfile

2012-09-21 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Remove superfluous macros in the specfile .. Patch Set 1: Looks good to me, approved Since this has been stalled for a long while, I've sampled several hook

Change in vdsm[master]: bootstrap: use yum API

2012-09-21 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: bootstrap: use yum API .. Patch Set 4: Added keep alives during download, if slow link it will trigger the softTimeout. Some minor cleanups. -- To view, visit

Change in vdsm[master]: pep8ize vdsm/vdsm

2012-09-21 Thread deepakcs
Deepak C Shetty has posted comments on this change. Change subject: pep8ize vdsm/vdsm .. Patch Set 1: No score (1 inline comment) File vdsm/vdsm Line 20: from logging

Change in vdsm[master]: pep8ize vdsm/vdsm

2012-09-21 Thread deepakcs
Deepak C Shetty has abandoned this change. Change subject: pep8ize vdsm/vdsm .. Patch Set 1: Abandoned abandoning this, as its already fixed in master. -- To view, visit http://gerrit.ovirt.org/8114 To unsubscribe, visit

Change in vdsm[master]: Fix vdsm-reg service name

2012-09-21 Thread Ryan Harper
Ryan Harper has posted comments on this change. Change subject: Fix vdsm-reg service name .. Patch Set 3: I would prefer that you didn't submit this (1 inline comment) I also agree we have more than one patch 1) fix vdsm-reg

Change in vdsm[master]: [WIP] Add a releaseHostId option to stop the DomainMonitorTh...

2012-09-21 Thread fsimonce
Federico Simoncelli has posted comments on this change. Change subject: [WIP] Add a releaseHostId option to stop the DomainMonitorThread .. Patch Set 1: I would prefer that you didn't submit this (1 inline comment)

Change in vdsm[master]: Validate downloaded CA certificate

2012-09-21 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: Validate downloaded CA certificate .. Patch Set 6: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/8021

Change in vdsm[master]: Validate downloaded SSH public key

2012-09-21 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: Validate downloaded SSH public key .. Patch Set 5: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/8018

Change in vdsm[master]: Replace calling setSchedulerParameters() with filling its XM...

2012-09-21 Thread Ryan Harper
Ryan Harper has posted comments on this change. Change subject: Replace calling setSchedulerParameters() with filling its XML description. .. Patch Set 3: I would prefer that you didn't submit this (2 inline comments)

Change in vdsm[master]: Fix vdsm-reg service name

2012-09-21 Thread Ryan Harper
Ryan Harper has posted comments on this change. Change subject: Fix vdsm-reg service name .. Patch Set 3: Here's how I understand the problem, please correct if I don't have the complete picture 1) node boots (firstboot=1,

Change in vdsm[master]: BZ#854242 - Change low-priority tasks to idle priority

2012-09-21 Thread gpadgett
Greg Padgett has uploaded a new change for review. Change subject: BZ#854242 - Change low-priority tasks to idle priority .. BZ#854242 - Change low-priority tasks to idle priority Some tasks run as best-effort priority level 7,

Change in vdsm[master]: BZ#854242 - Change low-priority tasks to idle priority

2012-09-21 Thread gpadgett
Greg Padgett has posted comments on this change. Change subject: BZ#854242 - Change low-priority tasks to idle priority .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/8121 To unsubscribe, visit