Change in vdsm[master]: RAM snapshots feature

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

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

2013-07-03 Thread gvallare
Giuseppe Vallarelli has posted comments on this change. Change subject: Hidden bonds[2/2]: Refactored and updated nics implementation. .. Patch Set 5: (3 inline comments)

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

2013-07-03 Thread gvallare
Giuseppe Vallarelli has posted comments on this change. Change subject: Hidden bonds[2/2]: Refactored and updated nics implementation. .. Patch Set 5: (1 inline comment)

Change in vdsm[master]: Fixing pep8 and pyflake errors after updating to new pep8 ve...

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Fixing pep8 and pyflake errors after updating to new pep8 version .. Patch Set 3: Build Successful

Change in vdsm[master]: vdsm: [WIP] Reboot capability for VM

2013-07-03 Thread vfeenstr
Vinzenz Feenstra has posted comments on this change. Change subject: vdsm: [WIP] Reboot capability for VM .. Patch Set 5: I would prefer that you didn't submit this (1 inline comment) Agreeing with tony on vm.py

Change in vdsm[master]: Fixing spec file for python-ethtool requirements

2013-07-03 Thread vfeenstr
Vinzenz Feenstra has posted comments on this change. Change subject: Fixing spec file for python-ethtool requirements .. Patch Set 2: (1 inline comment) File vdsm.spec.in

Change in vdsm[master]: RAM snapshots feature

2013-07-03 Thread ahadas
Arik Hadas has posted comments on this change. Change subject: RAM snapshots feature .. Patch Set 12: (8 inline comments) File vdsm/vm.py Line 2852: if

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

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

Change in vdsm[master]: Fix the charset conversion logic in hooks.py

2013-07-03 Thread msivak
Martin Sivák has posted comments on this change. Change subject: Fix the charset conversion logic in hooks.py .. Patch Set 8: Verified Tests pass. -- To view, visit http://gerrit.ovirt.org/15856 To unsubscribe, visit

Change in vdsm[master]: Clean the imports a bit

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Clean the imports a bit .. Patch Set 3: Build Successful http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3046/ : SUCCESS

Change in vdsm[master]: Fix the charset conversion logic in hooks.py

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Fix the charset conversion logic in hooks.py .. Patch Set 8: Build Successful http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3047/ : SUCCESS

Change in vdsm[master]: API: Add free memory to stats

2013-07-03 Thread omasad
ofri masad has posted comments on this change. Change subject: API: Add free memory to stats .. Patch Set 3: Verified -- To view, visit http://gerrit.ovirt.org/16253 To unsubscribe, visit http://gerrit.ovirt.org/settings

Change in vdsm[master]: API: Add free memory to stats

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: API: Add free memory to stats .. Patch Set 3: Build Successful http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3048/ : SUCCESS

Change in vdsm[master]: vdsm-tool: service management work with previous systemd rel...

2013-07-03 Thread zhshzhou
Zhou Zheng Sheng has uploaded a new change for review. Change subject: vdsm-tool: service management work with previous systemd release .. vdsm-tool: service management work with previous systemd release The service management

Change in vdsm[master]: vdsm-tool: service management work with previous systemd rel...

2013-07-03 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: vdsm-tool: service management work with previous systemd release .. Patch Set 1: Verified Test on F17 -- To view, visit http://gerrit.ovirt.org/16379 To

Change in vdsm[master]: vdsClient: check getFileList arguments

2013-07-03 Thread peet
Peter V. Saveliev has uploaded a new change for review. Change subject: vdsClient: check getFileList arguments .. vdsClient: check getFileList arguments A simple check of getFileList arguments on the function call. Bug-Url:

Change in vdsm[master]: vdsClient: check getFileList arguments

2013-07-03 Thread peet
Peter V. Saveliev has posted comments on this change. Change subject: vdsClient: check getFileList arguments .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/16380 To unsubscribe, visit

Change in vdsm[master]: vdsClient: check getFileList arguments

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: vdsClient: check getFileList arguments .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3050/ : SUCCESS

Change in vdsm[master]: vdsm: fix config comment for migration_timeout

2013-07-03 Thread peet
Peter V. Saveliev has uploaded a new change for review. Change subject: vdsm: fix config comment for migration_timeout .. vdsm: fix config comment for migration_timeout The migration_timeout variable means not overall migration

Change in vdsm[master]: vdsm: fix config comment for migration_timeout

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: vdsm: fix config comment for migration_timeout .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3051/ : SUCCESS

Change in vdsm[master]: vdsm-tool: service management work with previous systemd rel...

2013-07-03 Thread deepakcs
Deepak C Shetty has posted comments on this change. Change subject: vdsm-tool: service management work with previous systemd release .. Patch Set 1: Looks good to me, but someone else must approve (1 inline comment)

Change in vdsm[master]: vdsClient: check getFileList arguments

2013-07-03 Thread vfeenstr
Vinzenz Feenstra has posted comments on this change. Change subject: vdsClient: check getFileList arguments .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/16380 To

Change in vdsm[master]: Fixing pep8 and pyflake errors after updating to new pep8 ve...

2013-07-03 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Fixing pep8 and pyflake errors after updating to new pep8 version .. Patch Set 3: Looks good to me, approved I have no idea why you wouldn't state the version

Change in vdsm[master]: Fixing pep8 and pyflake errors after updating to new pep8 ve...

2013-07-03 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Fixing pep8 and pyflake errors after updating to new pep8 version .. Patch Set 3: Verified -- To view, visit http://gerrit.ovirt.org/16339 To unsubscribe,

Change in vdsm[master]: Fixing pep8 and pyflake errors after updating to new pep8 ve...

2013-07-03 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: Fixing pep8 and pyflake errors after updating to new pep8 version .. Fixing pep8 and pyflake errors after updating to new pep8 version Change-Id:

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

2013-07-03 Thread mtayer
mooli tayer has uploaded a new change for review. Change subject: Decoupling libvirtconnection.py from clintIf.py .. Decoupling libvirtconnection.py from clintIf.py 1.) Decoupling libvirtconnection.py from clientIF.py adding

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

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Decoupling libvirtconnection.py from clintIf.py .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3052/ : SUCCESS

Change in vdsm[master]: Fix pep8 compliance after updating to latest pep8

2013-07-03 Thread knesenko
Kiril Nesenko has posted comments on this change. Change subject: Fix pep8 compliance after updating to latest pep8 .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit

Change in vdsm[master]: Fix pep8 compliance after updating to latest pep8

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Fix pep8 compliance after updating to latest pep8 .. Patch Set 1: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3053/ : SUCCESS

Change in vdsm[master]: Fix pep8 compliance after updating to latest pep8

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Fix pep8 compliance after updating to latest pep8 .. Patch Set 1: No score Build Successful http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3053/ :

Change in vdsm[master]: Fix pep8 compliance after updating to latest pep8

2013-07-03 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: Fix pep8 compliance after updating to latest pep8 .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit

Change in vdsm[master]: vm: Fix custom device properties on hot operation

2013-07-03 Thread amuller
Assaf Muller has posted comments on this change. Change subject: vm: Fix custom device properties on hot operation .. Patch Set 2: Verified; Looks good to me, but someone else must approve Verified: Used the engine GUI to set

Change in vdsm[master]: vm: Fix custom device properties on hot operation

2013-07-03 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: vm: Fix custom device properties on hot operation .. vm: Fix custom device properties on hot operation On hot plug unplug of a vNIC/vDisk the custom

Change in vdsm[master]: Fix pep8 compliance after updating to latest pep8

2013-07-03 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Fix pep8 compliance after updating to latest pep8 .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16394 To unsubscribe,

Change in vdsm[master]: Fix pep8 compliance after updating to latest pep8

2013-07-03 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: Fix pep8 compliance after updating to latest pep8 .. Fix pep8 compliance after updating to latest pep8 Change-Id:

Change in vdsm[master]: Fixed rollback of port-mirroring enabled nic during hotplugN...

2013-07-03 Thread amuller
Assaf Muller has uploaded a new change for review. Change subject: Fixed rollback of port-mirroring enabled nic during hotplugNic .. Fixed rollback of port-mirroring enabled nic during hotplugNic During vm.py:hotplugNic, if the

Change in vdsm[master]: Fixed rollback of port-mirroring enabled nic during hotplugN...

2013-07-03 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Fixed rollback of port-mirroring enabled nic during hotplugNic .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16400 To

Change in vdsm[master]: Fixed rollback of port-mirroring enabled nic during hotplugN...

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Fixed rollback of port-mirroring enabled nic during hotplugNic .. Patch Set 1: Fails Build Failed

Change in vdsm[master]: Fixed rollback of port-mirroring enabled nic during hotplugN...

2013-07-03 Thread amuller
Assaf Muller has posted comments on this change. Change subject: Fixed rollback of port-mirroring enabled nic during hotplugNic .. Patch Set 1: Verified Verification - I checked 'Verified', however I did not ran the code

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

2013-07-03 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Decoupling libvirtconnection.py from clintIf.py .. Patch Set 1: I would prefer that you didn't submit this (7 inline comments) Thanks Mooli, I really want to

Change in vdsm[master]: vdsClient: check getFileList arguments

2013-07-03 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: vdsClient: check getFileList arguments .. vdsClient: check getFileList arguments A simple check of getFileList arguments on the function call.

Change in vdsm[master]: vdsClient: check getFileList arguments

2013-07-03 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: vdsClient: check getFileList arguments .. Patch Set 1: Looks good to me, approved Fine by me. -- To view, visit http://gerrit.ovirt.org/16380 To unsubscribe,

Change in vdsm[master]: Fix the charset conversion logic in hooks.py

2013-07-03 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Fix the charset conversion logic in hooks.py .. Patch Set 8: (1 inline comment) File vdsm/hooks.py Line

Change in vdsm[master]: Fix the charset conversion logic in hooks.py

2013-07-03 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Fix the charset conversion logic in hooks.py .. Patch Set 8: I would prefer that you didn't submit this -- To view, visit http://gerrit.ovirt.org/15856 To

Change in vdsm[master]: Fix the charset conversion logic in hooks.py

2013-07-03 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Fix the charset conversion logic in hooks.py .. Patch Set 9: Looks good to me, approved Martin, would you please verify that this actually passes custom

Change in vdsm[master]: Fixed rollback of port-mirroring enabled nic during hotplugN...

2013-07-03 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Fixed rollback of port-mirroring enabled nic during hotplugNic .. Patch Set 1: I would prefer that you didn't submit this Please rebase on top of recent

Change in vdsm[master]: Fixed rollback of port-mirroring enabled nic during hotplugN...

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Fixed rollback of port-mirroring enabled nic during hotplugNic .. Patch Set 2: Fails Build Failed

Change in vdsm[master]: Fix the charset conversion logic in hooks.py

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Fix the charset conversion logic in hooks.py .. Patch Set 9: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3056/ : SUCCESS

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

2013-07-03 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: gluster: verbs for managing services .. Patch Set 23: (1 inline comment) File vdsm/gluster/services.py

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

2013-07-03 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: gluster: verbs for managing services .. Patch Set 23: (1 inline comment) File

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

2013-07-03 Thread amuller
Assaf Muller has uploaded a new change for review. Change subject: before/after_update_device hook now receives custom properties .. before/after_update_device hook now receives custom properties Change-Id:

Change in vdsm[master]: Fixed rollback of port-mirroring enabled nic during hotplugN...

2013-07-03 Thread amuller
Assaf Muller has posted comments on this change. Change subject: Fixed rollback of port-mirroring enabled nic during hotplugNic .. Patch Set 2: Verified Copied verified flag from last patch set. -- To view, visit

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

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: before/after_update_device hook now receives custom properties .. Patch Set 1: Fails Build Failed

Change in vdsm[master]: vdsm: Reboot capability for VM

2013-07-03 Thread mbetak
Martin Betak has posted comments on this change. Change subject: vdsm: Reboot capability for VM .. Patch Set 5: (3 inline comments) File vdsm_api/vdsmapi-schema.json Line

Change in vdsm[master]: vdsm: Reboot capability for VM

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: vdsm: Reboot capability for VM .. Patch Set 6: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3058/ : SUCCESS

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

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Add balloon_min to VM.getVmBalloonInfo .. Patch Set 12: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3059/ : SUCCESS

Change in vdsm[master]: Clean the imports a bit

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Clean the imports a bit .. Patch Set 4: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3060/ : SUCCESS

Change in vdsm[master]: Fix the charset conversion logic in hooks.py

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Fix the charset conversion logic in hooks.py .. Patch Set 10: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3061/ : SUCCESS

Change in vdsm[master]: Fixed rollback of port-mirroring enabled nic during hotplugN...

2013-07-03 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Fixed rollback of port-mirroring enabled nic during hotplugNic .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16400 To

Change in vdsm[master]: Fixed rollback of port-mirroring enabled nic during hotplugN...

2013-07-03 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: Fixed rollback of port-mirroring enabled nic during hotplugNic .. Fixed rollback of port-mirroring enabled nic during hotplugNic During

Change in vdsm[master]: Use more fine lock for hsm.deleteImage().

2013-07-03 Thread ewarszaw
Eduardo has uploaded a new change for review. Change subject: Use more fine lock for hsm.deleteImage(). .. Use more fine lock for hsm.deleteImage(). When removing an image from a specific SD, lock the image in this SD

Change in vdsm[master]: Remove unused hsm._hsmSchedule()

2013-07-03 Thread ewarszaw
Eduardo has uploaded a new change for review. Change subject: Remove unused hsm._hsmSchedule() .. Remove unused hsm._hsmSchedule() Related to: BZ#960952. Change-Id: I92b9fcd899463157951fb76ab7ac6dfb4b47caf9 Signed-off-by:

Change in vdsm[master]: Avoid running scheduled jobs when not SPM.

2013-07-03 Thread ewarszaw
Eduardo has uploaded a new change for review. Change subject: Avoid running scheduled jobs when not SPM. .. Avoid running scheduled jobs when not SPM. Related to: BZ#960952. Change-Id: I0402817df9c2c08a5a945082edfbb03e832facbe

Change in vdsm[master]: Remove unused hsm._hsmSchedule()

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Remove unused hsm._hsmSchedule() .. Patch Set 1: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3064/ : SUCCESS

Change in vdsm[master]: Remove StoragePool.createVolume().

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Remove StoragePool.createVolume(). .. Patch Set 2: Fails; I would prefer that you didn't submit this Build Failed

Change in vdsm[master]: Avoid running scheduled jobs when not SPM.

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Avoid running scheduled jobs when not SPM. .. Patch Set 1: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3063/ : SUCCESS

Change in vdsm[master]: Improve StoragePool.detachAllDomains() monitoring.

2013-07-03 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: Improve StoragePool.detachAllDomains() monitoring. .. Improve StoragePool.detachAllDomains() monitoring. Change-Id:

Change in vdsm[master]: Use more fine lock for hsm.deleteImage().

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Use more fine lock for hsm.deleteImage(). .. Patch Set 1: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3065/ : SUCCESS

Change in vdsm[master]: Fix the charset conversion logic in hooks.py

2013-07-03 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Fix the charset conversion logic in hooks.py .. Patch Set 10: I would prefer that you didn't submit this (1 inline comment)

Change in vdsm[master]: Adding intellij project files to .gitignore

2013-07-03 Thread mtayer
mooli tayer has uploaded a new change for review. Change subject: Adding intellij project files to .gitignore .. Adding intellij project files to .gitignore Change-Id: I337ff8ea0e950ae94effb226a5170570b489d68a Signed-off-by:

Change in vdsm[master]: Adding intellij project files to .gitignore

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Adding intellij project files to .gitignore .. Patch Set 1: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3066/ : SUCCESS

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

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Add balloon_min to VM.getVmBalloonInfo .. Patch Set 13: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3067/ : SUCCESS

Change in vdsm[master]: Fix the charset conversion logic in hooks.py

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Fix the charset conversion logic in hooks.py .. Patch Set 11: Fails; I would prefer that you didn't submit this Build Failed

Change in vdsm[master]: Clean the imports a bit

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Clean the imports a bit .. Patch Set 5: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3069/ : SUCCESS

Change in vdsm[master]: RAM snapshots feature

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: RAM snapshots feature .. Patch Set 15: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3070/ : SUCCESS

Change in vdsm[master]: Simplify configWriter to use netmodels

2013-07-03 Thread wudxw
Mark Wu has posted comments on this change. Change subject: Simplify configWriter to use netmodels .. Patch Set 4: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/16324 To

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

2013-07-03 Thread gvallare
Giuseppe Vallarelli has posted comments on this change. Change subject: before/after_update_device hook now receives custom properties .. Patch Set 1: I would prefer that you didn't submit this Hi Assaf you got a pep8 issue:

Change in vdsm[master]: netconf: Add a base configurator class

2013-07-03 Thread wudxw
Mark Wu has posted comments on this change. Change subject: netconf: Add a base configurator class .. Patch Set 2: Verified -- To view, visit http://gerrit.ovirt.org/16178 To unsubscribe, visit

Change in vdsm[master]: netconf: Add a base configurator class

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: netconf: Add a base configurator class .. Patch Set 2: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3073/ : SUCCESS

Change in vdsm[master]: netconf: Add dhcp support for iproute2 configurator

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: netconf: Add dhcp support for iproute2 configurator .. Patch Set 3: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3072/ : SUCCESS

Change in vdsm[master]: netconf: enable multiple gateways for iproute2 configurator

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: netconf: enable multiple gateways for iproute2 configurator .. Patch Set 2: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3071/ :

Change in vdsm[master]: Add iproute2 configurator

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Add iproute2 configurator .. Patch Set 4: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3074/ : SUCCESS

Change in vdsm[master]: Simplify configWriter to use netmodels

2013-07-03 Thread gvallare
Giuseppe Vallarelli has posted comments on this change. Change subject: Simplify configWriter to use netmodels .. Patch Set 4: I would prefer that you didn't submit this (1 inline comment) See comment.

Change in vdsm[master]: Failing to deactivate LVs is a serious issue.

2013-07-03 Thread ewarszaw
Eduardo has uploaded a new change for review. Change subject: Failing to deactivate LVs is a serious issue. .. Failing to deactivate LVs is a serious issue. Changing volumes that are active in multiple hosts can lead to DC.

Change in vdsm[master]: Failing to deactivate LVs is a serious issue.

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Failing to deactivate LVs is a serious issue. .. Patch Set 1: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3076/ : SUCCESS

Change in vdsm[master]: One shot teardown.

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: One shot teardown. .. Patch Set 7: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3075/ : SUCCESS

Change in vdsm[master]: Fix the charset conversion logic in hooks.py

2013-07-03 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: Fix the charset conversion logic in hooks.py .. Fix the charset conversion logic in hooks.py The proper way to convert text from byte oriented string

Change in vdsm[master]: Fix the charset conversion logic in hooks.py

2013-07-03 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Fix the charset conversion logic in hooks.py .. Patch Set 11: Verified; Looks good to me, approved I do not know what is wrong with our slaves' pep8 tool, and

Change in vdsm[master]: vdsm-tool: service management work with previous systemd rel...

2013-07-03 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: vdsm-tool: service management work with previous systemd release .. Patch Set 1: (1 inline comment) File

Change in vdsm[master]: vdsm-tool: service management work with previous systemd rel...

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: vdsm-tool: service management work with previous systemd release .. Patch Set 2: Fails Build Failed

Change in vdsm[master]: vdsm-tool: service management work with previous systemd rel...

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: vdsm-tool: service management work with previous systemd release .. Patch Set 3: Fails Build Failed

Change in vdsm[master]: PEP8: fix tests/hooksTests.py

2013-07-03 Thread zhshzhou
Zhou Zheng Sheng has uploaded a new change for review. Change subject: PEP8: fix tests/hooksTests.py .. PEP8: fix tests/hooksTests.py pep8 1.4.6 gives the following error tests/hooksTests.py:140:23: E127 continuation line

Change in vdsm[master]: PEP8: fix variours errors

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: PEP8: fix variours errors .. Patch Set 2: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/2269/ : FAILURE

Change in vdsm[master]: vdsm-tool: service management work with previous systemd rel...

2013-07-03 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: vdsm-tool: service management work with previous systemd release .. Patch Set 5: Fails Build Failed

Change in vdsm[master]: PEP8: fix variours errors

2013-07-03 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: PEP8: fix variours errors .. Patch Set 2: Verified Test on RHEL6 and Fedora. It seems the RHEL6 slave needs update the python-pep8 tool. -- To view, visit

Change in vdsm[master]: vdsm-tool: service management work with previous systemd rel...

2013-07-03 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: vdsm-tool: service management work with previous systemd release .. Patch Set 5: Verified Jenkins error does not related to this patch. It seems the RHEL6

Change in vdsm[master]: vdsm-tool: service management work with previous systemd rel...

2013-07-03 Thread deepakcs
Deepak C Shetty has posted comments on this change. Change subject: vdsm-tool: service management work with previous systemd release .. Patch Set 5: Looks good to me, but someone else must approve -- To view, visit