Change in vdsm[ovirt-3.2]: vdsm.spec: update python-pthreading

2013-07-09 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: vdsm.spec: update python-pthreading .. vdsm.spec: update python-pthreading F19 and EL7 should require python-pthreading = 1.0.2 Change-Id:

Change in vdsm[master]: spec: finer-grained libvirt requirement

2013-07-09 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: spec: finer-grained libvirt requirement .. Patch Set 1: Could you explain the differences between this change and my http://gerrit.ovirt.org/#/c/15761/ ? --

Change in vdsm[master]: vdsmd.init: do not use previous libvirt logrotate

2013-07-09 Thread sbonazzo
Sandro Bonazzola has posted comments on this change. Change subject: vdsmd.init: do not use previous libvirt logrotate .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit

Change in vdsm[master]: spec: finer-grained libvirt requirement

2013-07-09 Thread wudxw
Mark Wu has posted comments on this change. Change subject: spec: finer-grained libvirt requirement .. Patch Set 1: Dan, Sorry, I didn't notice your patch in http://gerrit.ovirt.org/#/c/15761/ It should have the same result

Change in vdsm[master]: spec: finer-grained libvirt requirement

2013-07-09 Thread wudxw
Mark Wu has posted comments on this change. Change subject: spec: finer-grained libvirt requirement .. Patch Set 1: Your approach should be better than mine, because it remove all unused driver, like storage driver. -- To

Change in vdsm[master]: spec: finer-grained libvirt requirement

2013-07-09 Thread wudxw
Mark Wu has abandoned this change. Change subject: spec: finer-grained libvirt requirement .. Patch Set 1: Abandoned duplicate with http://gerrit.ovirt.org/#/c/15761/ -- To view, visit http://gerrit.ovirt.org/16593 To

Change in vdsm[master]: vdsm: Reduce the migration progress timeout

2013-07-09 Thread vfeenstr
Vinzenz Feenstra has uploaded a new change for review. Change subject: vdsm: Reduce the migration progress timeout .. vdsm: Reduce the migration progress timeout The progress of the migration should indicate a stuck migration

Change in vdsm[master]: vdsm: Reduce the migration progress timeout

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: vdsm: Reduce the migration progress timeout .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3253/ : SUCCESS

Change in vdsm[master]: Remove mac address from host interface statistics

2013-07-09 Thread masayag
Moti Asayag has posted comments on this change. Change subject: Remove mac address from host interface statistics .. Patch Set 3: I agree with Mark. The engine populate the host's mac address by the getVdsCaps, either from

Change in vdsm[master]: Remove mac address from host interface statistics

2013-07-09 Thread masayag
Moti Asayag has posted comments on this change. Change subject: Remove mac address from host interface statistics .. Patch Set 3: (1 inline comment) File vdsm/sampling.py

Change in vdsm[master]: vdsmd.init: do not use previous libvirt logrotate

2013-07-09 Thread sbonazzo
Sandro Bonazzola has posted comments on this change. Change subject: vdsmd.init: do not use previous libvirt logrotate .. Patch Set 2: Verified -- To view, visit http://gerrit.ovirt.org/16528 To unsubscribe, visit

Change in vdsm[master]: Monitor bridges using vdsStats

2013-07-09 Thread masayag
Moti Asayag has posted comments on this change. Change subject: Monitor bridges using vdsStats .. Patch Set 2: Looks good to me, but someone else must approve Need to verify the proper impact on the engine side: If a required

Change in vdsm[master]: Add balloon_min to VM.getVmBalloonInfo

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Add balloon_min to VM.getVmBalloonInfo .. Patch Set 16: Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3254/ : FAILURE

Change in vdsm[master]: RAM snapshots feature

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: RAM snapshots feature .. Patch Set 19: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3255/ : SUCCESS

Change in vdsm[master]: Hidden bonds[2/2]: Refactored and updated nics implementatio...

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Hidden bonds[2/2]: Refactored and updated nics implementation. .. Patch Set 9: Fails Build Failed

Change in vdsm[master]: before/after_update_device hook now receives custom properti...

2013-07-09 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: before/after_update_device hook now receives custom properties .. Patch Set 2: I would prefer that you didn't submit this (1 inline comment)

Change in vdsm[master]: Decoupling libvirtconnection.py from clientIF.py

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Decoupling libvirtconnection.py from clientIF.py .. Patch Set 7: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3257/ :

Change in vdsm[master]: Add balloon_min to VM.getVmBalloonInfo

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Add balloon_min to VM.getVmBalloonInfo .. Patch Set 17: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3258/ : SUCCESS

Change in vdsm[master]: Hidden bonds[2/2]: Refactored and updated nics implementatio...

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Hidden bonds[2/2]: Refactored and updated nics implementation. .. Patch Set 10: Fails Build Failed

Change in vdsm[master]: Decoupling libvirtconnection.py from clientIF.py

2013-07-09 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Decoupling libvirtconnection.py from clientIF.py .. Patch Set 7: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16393 To unsubscribe,

Change in vdsm[master]: Extend vdsm-tool: moving configure libvirt to external shell...

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Extend vdsm-tool: moving configure libvirt to external shell script .. Patch Set 22: Build Successful

Change in vdsm[master]: Extend vdsm-tool: moving configure libvirt to external shell...

2013-07-09 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: Extend vdsm-tool: moving configure libvirt to external shell script .. Patch Set 23: (1 inline comment)

Change in vdsm[master]: Hidden bonds[2/2]: Refactored and updated nics implementatio...

2013-07-09 Thread gvallare
Giuseppe Vallarelli has posted comments on this change. Change subject: Hidden bonds[2/2]: Refactored and updated nics implementation. .. Patch Set 11: New fix reason is that bonding_masters should be filtered out and not

Change in vdsm[master]: Hidden bonds[2/2]: Refactored and updated nics implementatio...

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Hidden bonds[2/2]: Refactored and updated nics implementation. .. Patch Set 11: Fails Build Failed

Change in vdsm[master]: Extend vdsm-tool: moving configure libvirt to external shell...

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Extend vdsm-tool: moving configure libvirt to external shell script .. Patch Set 23: Build Successful

Change in vdsm[master]: spec: restart systemd-modules-load.service on post installat...

2013-07-09 Thread wudxw
Mark Wu has uploaded a new change for review. Change subject: spec: restart systemd-modules-load.service on post installation .. spec: restart systemd-modules-load.service on post installation It's observed that the needed

Change in vdsm[master]: spec: restart systemd-modules-load.service on post installat...

2013-07-09 Thread wudxw
Mark Wu has posted comments on this change. Change subject: spec: restart systemd-modules-load.service on post installation .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/16610 To unsubscribe, visit

Change in vdsm[master]: vdsmd.init.in: break to pieces

2013-07-09 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: vdsmd.init.in: break to pieces .. Patch Set 9: Thanks Alon! I agree on most of your suggestions. I just have 1 different ideas and 1 question. 1. About

Change in vdsm[master]: Remove mac address from host interface statistics

2013-07-09 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Remove mac address from host interface statistics .. Patch Set 3: (1 inline comment) File vdsm/sampling.py

Change in vdsm[master]: vdsmd.init: Add init adapter

2013-07-09 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: vdsmd.init: Add init adapter .. Patch Set 4: (1 inline comment) File vdsm.spec.in Line 757 Line 758 Line

Change in vdsm[master]: spec: restart systemd-modules-load.service on post installat...

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: spec: restart systemd-modules-load.service on post installation .. Patch Set 1: Build Successful

Change in vdsm[master]: vdsmd.init: do not use previous libvirt logrotate

2013-07-09 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: vdsmd.init: do not use previous libvirt logrotate .. Patch Set 2: I would prefer that you didn't submit this (1 inline comment) minor suggestion

Change in vdsm[master]: Extend vdsm-tool: moving configure libvirt to external shell...

2013-07-09 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: Extend vdsm-tool: moving configure libvirt to external shell script .. Patch Set 23: (1 inline comment)

Change in vdsm[master]: RAM snapshots feature

2013-07-09 Thread ahadas
Arik Hadas has posted comments on this change. Change subject: RAM snapshots feature .. Patch Set 17: (7 inline comments) File vdsm/BindingXMLRPC.py Line 287: Line

Change in vdsm[master]: spec: restart systemd-modules-load.service on post installat...

2013-07-09 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: spec: restart systemd-modules-load.service on post installation .. Patch Set 1: (2 inline comments)

Change in vdsm[master]: Remove mac address from host interface statistics

2013-07-09 Thread masayag
Moti Asayag has posted comments on this change. Change subject: Remove mac address from host interface statistics .. Patch Set 3: I would prefer that you didn't submit this (1 inline comment) Since the code is shared between

Change in vdsm[master]: gluster: verbs for managing services

2013-07-09 Thread tjeyasin
Timothy Asir has posted comments on this change. Change subject: gluster: verbs for managing services .. Patch Set 23: (3 inline comments) File client/vdsClientGluster.py

Change in vdsm[master]: Hidden bonds[2/2]: Refactored and updated nics implementatio...

2013-07-09 Thread gvallare
Giuseppe Vallarelli has posted comments on this change. Change subject: Hidden bonds[2/2]: Refactored and updated nics implementation. .. Patch Set 11: Verified Having found some edge cases lately, would be cool if someone

Change in vdsm[master]: gluster: verbs for managing services

2013-07-09 Thread tjeyasin
Timothy Asir has posted comments on this change. Change subject: gluster: verbs for managing services .. Patch Set 24: Verified -- To view, visit http://gerrit.ovirt.org/11094 To unsubscribe, visit

Change in vdsm[master]: Hidden bonds[2/2]: Refactored and updated nics implementatio...

2013-07-09 Thread gvallare
Giuseppe Vallarelli has posted comments on this change. Change subject: Hidden bonds[2/2]: Refactored and updated nics implementation. .. Patch Set 11: Build failure is related to the unhappy pep8 on different modules than the

Change in vdsm[master]: gluster: verbs for managing services

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: gluster: verbs for managing services .. Patch Set 24: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3264/ : SUCCESS

Change in vdsm[master]: Rewrite isOvirt function

2013-07-09 Thread ybronhei
Yaniv Bronhaim has uploaded a new change for review. Change subject: Rewrite isOvirt function .. Rewrite isOvirt function Change-Id: I943594e8d2cb741f5400c8036b5af9f52ec13b38 Signed-off-by: Yaniv Bronhaim ybron...@redhat.com

Change in vdsm[master]: Extend vdsm-tool: moving configure libvirt to external shell...

2013-07-09 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: Extend vdsm-tool: moving configure libvirt to external shell script .. Patch Set 24: (2 inline comments)

Change in vdsm[master]: vdsm-tool usage: Adding path variable to init script

2013-07-09 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: vdsm-tool usage: Adding path variable to init script .. Patch Set 2: Verified -- To view, visit http://gerrit.ovirt.org/16549 To unsubscribe, visit

Change in vdsm[master]: Qos feature: Adds bandwidth definition at configuration leve...

2013-07-09 Thread gvallare
Giuseppe Vallarelli has posted comments on this change. Change subject: Qos feature: Adds bandwidth definition at configuration level. .. Patch Set 14: Verified -- To view, visit http://gerrit.ovirt.org/15724 To unsubscribe,

Change in vdsm[master]: Qos feature: Adds bandwidth definition at configuration leve...

2013-07-09 Thread gvallare
Giuseppe Vallarelli has posted comments on this change. Change subject: Qos feature: Adds bandwidth definition at configuration level. .. Patch Set 14: Tested in this way: Doing: from vdsm import vdscli connection =

Change in vdsm[master]: Cleanup the usages of configWriter in configNetwork

2013-07-09 Thread wudxw
Mark Wu has posted comments on this change. Change subject: Cleanup the usages of configWriter in configNetwork .. Patch Set 2: Verified; I would prefer that you didn't submit this -- To view, visit

Change in vdsm[master]: Hidden bonds[2/2]: Refactored and updated nics implementatio...

2013-07-09 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Hidden bonds[2/2]: Refactored and updated nics implementation. .. Patch Set 12: Fails (1 inline comment) It is important to verify that with the default

Change in vdsm[master]: Hidden bonds[2/2]: Refactored and updated nics implementatio...

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Hidden bonds[2/2]: Refactored and updated nics implementation. .. Patch Set 12: Fails Build Failed

Change in vdsm[master]: Extend vdsm-tool: moving configure libvirt to external shell...

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Extend vdsm-tool: moving configure libvirt to external shell script .. Patch Set 24: Build Successful

Change in vdsm[master]: vdsm-tool usage: Adding path variable to init script

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: vdsm-tool usage: Adding path variable to init script .. Patch Set 2: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3267/ :

Change in vdsm[master]: Rewrite isOvirt function

2013-07-09 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: Rewrite isOvirt function .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/16616 To unsubscribe, visit

Change in vdsm[master]: Cleanup the usages of configWriter in configNetwork

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Cleanup the usages of configWriter in configNetwork .. Patch Set 2: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3269/ :

Change in vdsm[master]: Rewrite isOvirt function

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Rewrite isOvirt function .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3268/ : SUCCESS

Change in vdsm[master]: vdsmd.init: do not use previous libvirt logrotate

2013-07-09 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: vdsmd.init: do not use previous libvirt logrotate .. Patch Set 3: Verified -- To view, visit http://gerrit.ovirt.org/16528 To unsubscribe, visit

Change in vdsm[master]: clientIF: automatically unpause vms in EIO when SD becomes a...

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: clientIF: automatically unpause vms in EIO when SD becomes active .. Patch Set 4: Fails Build Failed

Change in vdsm[master]: vdsmd.init: do not use previous libvirt logrotate

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: vdsmd.init: do not use previous libvirt logrotate .. Patch Set 3: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3271/ :

Change in vdsm[master]: Cleanup the usages of configWriter in configNetwork

2013-07-09 Thread wudxw
Mark Wu has posted comments on this change. Change subject: Cleanup the usages of configWriter in configNetwork .. Patch Set 2: No score -- To view, visit http://gerrit.ovirt.org/16343 To unsubscribe, visit

Change in vdsm[master]: Qos feature: Adds bandwidth definition at configuration leve...

2013-07-09 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Qos feature: Adds bandwidth definition at configuration level. .. Patch Set 14: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/15724 To

Change in vdsm[master]: Multiple Gateways[5/2]: Iproute2 handles exceptions

2013-07-09 Thread amuller
Assaf Muller has uploaded a new change for review. Change subject: Multiple Gateways[5/2]: Iproute2 handles exceptions .. Multiple Gateways[5/2]: Iproute2 handles exceptions Iproute2.configure/removeSourceRoute now don't

Change in vdsm[master]: Cleanup the usages of configWriter in configNetwork

2013-07-09 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: Cleanup the usages of configWriter in configNetwork .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit

Change in vdsm[master]: Multiple Gateways[4/2]: Fixed ifdown bug for DHCP interfaces

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Multiple Gateways[4/2]: Fixed ifdown bug for DHCP interfaces .. Patch Set 3: Build Successful

Change in vdsm[master]: Multiple Gateways[3/2]: Re-did comms between dhclient and vd...

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Multiple Gateways[3/2]: Re-did comms between dhclient and vdsm .. Patch Set 6: Build Successful

Change in vdsm[master]: Multiple Gateways[5/2]: Iproute2 handles exceptions

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Multiple Gateways[5/2]: Iproute2 handles exceptions .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3274/ :

Change in vdsm[master]: vdsmd.init: do not use previous libvirt logrotate

2013-07-09 Thread sbonazzo
Sandro Bonazzola has posted comments on this change. Change subject: vdsmd.init: do not use previous libvirt logrotate .. Patch Set 3: Looks good to me, but someone else must approve -- To view, visit

Change in vdsm[master]: RAM snapshots feature

2013-07-09 Thread ykaplan
Yeela Kaplan has posted comments on this change. Change subject: RAM snapshots feature .. Patch Set 19: I would prefer that you didn't submit this (2 inline comments) File

Change in vdsm[master]: Qos feature: Adds bandwidth definition at configuration leve...

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Qos feature: Adds bandwidth definition at configuration level. .. Patch Set 15: Build Successful

Change in vdsm[master]: RAM snapshots feature

2013-07-09 Thread ahadas
Arik Hadas has posted comments on this change. Change subject: RAM snapshots feature .. Patch Set 19: (2 inline comments) File vdsm/vm.py Line 3573: try: Line

Change in vdsm[master]: Add balloon_min to VM.getVmBalloonInfo

2013-07-09 Thread msivak
Martin Sivák has posted comments on this change. Change subject: Add balloon_min to VM.getVmBalloonInfo .. Patch Set 17: Verified Verified by starting a VM with 312 guaranteed memory (HA tab in VM/edit) and checking domxml on

Change in vdsm[master]: RAM snapshots feature

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: RAM snapshots feature .. Patch Set 20: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3276/ : SUCCESS

Change in vdsm[master]: Qos feature: Adds bandwidth definition at configuration leve...

2013-07-09 Thread gvallare
Giuseppe Vallarelli has posted comments on this change. Change subject: Qos feature: Adds bandwidth definition at configuration level. .. Patch Set 16: Verified Fixing typo in configNetwork qosInboud - qosInbound. -- To

Change in vdsm[master]: Qos feature: Adds bandwidth definition at configuration leve...

2013-07-09 Thread gvallare
Giuseppe Vallarelli has posted comments on this change. Change subject: Qos feature: Adds bandwidth definition at configuration level. .. Patch Set 16: Please Dan give a final look I had to do another rebase where stuff have

Change in vdsm[master]: Qos feature: Adds bandwidth definition at configuration leve...

2013-07-09 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Qos feature: Adds bandwidth definition at configuration level. .. Patch Set 16: Build Successful

Change in vdsm[master]: Qos feature: Adds bandwidth definition at configuration leve...

2013-07-09 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: Qos feature: Adds bandwidth definition at configuration level. .. Qos feature: Adds bandwidth definition at configuration level. Network definition

Change in vdsm[master]: Qos feature: Adds bandwidth definition at configuration leve...

2013-07-09 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Qos feature: Adds bandwidth definition at configuration level. .. Patch Set 16: Looks good to me, approved Simple rebase, copying score. -- To view, visit

Change in vdsm[master]: vdsmd.init: do not use previous libvirt logrotate

2013-07-09 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: vdsmd.init: do not use previous libvirt logrotate .. Patch Set 3: Looks good to me, approved Thanks! -- To view, visit http://gerrit.ovirt.org/16528 To

Change in vdsm[master]: vdsmd.init: do not use previous libvirt logrotate

2013-07-09 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: vdsmd.init: do not use previous libvirt logrotate .. vdsmd.init: do not use previous libvirt logrotate In the past VDSM have been creating

Change in vdsm[master]: vdsm-tool usage: Adding path variable to init script

2013-07-09 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: vdsm-tool usage: Adding path variable to init script .. vdsm-tool usage: Adding path variable to init script Change-Id:

Change in vdsm[master]: vdsm-tool usage: Adding path variable to init script

2013-07-09 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: vdsm-tool usage: Adding path variable to init script .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16549 To

Change in vdsm[master]: Add balloon_min to VM.getVmBalloonInfo

2013-07-09 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Add balloon_min to VM.getVmBalloonInfo .. Patch Set 17: Looks good to me, approved Thanks. -- To view, visit http://gerrit.ovirt.org/15799 To unsubscribe,

Change in vdsm[master]: Add balloon_min to VM.getVmBalloonInfo

2013-07-09 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: Add balloon_min to VM.getVmBalloonInfo .. Add balloon_min to VM.getVmBalloonInfo The balloon_min key will take its value from memGuaranteedSize

Change in vdsm[master]: Rewrite isOvirt function

2013-07-09 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Rewrite isOvirt function .. Patch Set 1: Looks good to me, approved I did not mind the original implementation, but if everybody finds this one better - I do

Change in vdsm[master]: Extend vdsm-tool: moving configure libvirt to external shell...

2013-07-09 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Extend vdsm-tool: moving configure libvirt to external shell script .. Patch Set 24: (1 inline comment)

Change in vdsm[master]: Rewrite isOvirt function

2013-07-09 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Rewrite isOvirt function .. Patch Set 1: but please note - it would be nice if you rename this function isOvirt - isOvirtNode, or something like that. -- To

Change in vdsm[master]: spec: finer-grained libvirt requirement

2013-07-09 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: spec: finer-grained libvirt requirement .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/15761 To unsubscribe,

Change in vdsm[master]: spec: finer-grained libvirt requirement

2013-07-09 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: spec: finer-grained libvirt requirement .. spec: finer-grained libvirt requirement Change-Id: I48e25ca9e307c2cde7f29b65c68ff6e122ec8f64 Signed-off-by:

Change in vdsm[master]: vdsm: /usr/lib/tmpfiles.d/vdsm.conf

2013-07-09 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: vdsm: /usr/lib/tmpfiles.d/vdsm.conf .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/15707 To

Change in vdsm[ovirt-3.2]: vdsmd.init: do not use previous libvirt logrotate

2013-07-09 Thread dougsland
Douglas Schilling Landgraf has uploaded a new change for review. Change subject: vdsmd.init: do not use previous libvirt logrotate .. vdsmd.init: do not use previous libvirt logrotate In the past VDSM have been creating

Change in vdsm[ovirt-3.2]: vdsmd.init: do not use previous libvirt logrotate

2013-07-09 Thread dougsland
Douglas Schilling Landgraf has abandoned this change. Change subject: vdsmd.init: do not use previous libvirt logrotate .. Patch Set 1: Abandoned not required. -- To view, visit http://gerrit.ovirt.org/16647 To unsubscribe,