Change in vdsm[master]: make vdsm/libvirtvm.py PEP 8 clean

2012-08-23 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: make vdsm/libvirtvm.py PEP 8 clean .. Patch Set 5: (2 inline comments) This one is quite close to it. Just a couple of changes and I'll gladly give a

Change in vdsm[master]: make vdsm/libvirtvm.py PEP 8 clean

2012-08-23 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: make vdsm/libvirtvm.py PEP 8 clean .. Patch Set 6: Thanks Antoni! -- To view, visit http://gerrit.ovirt.org/6561 To unsubscribe, visit

Change in vdsm[master]: make vdsm/libvirtvm.py PEP 8 clean

2012-08-23 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: make vdsm/libvirtvm.py PEP 8 clean .. Patch Set 6: Build Successful http://jenkins.ovirt.info/job/patch_vdsm_unit_tests/618/ : SUCCESS -- To view,

Change in vdsm[master]: make vdsm/libvirtvm.py PEP 8 clean

2012-08-23 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: make vdsm/libvirtvm.py PEP 8 clean .. Patch Set 6: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/6561

Change in vdsm[master]: make vdsm/libvirtvm.py PEP 8 clean

2012-08-22 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: make vdsm/libvirtvm.py PEP 8 clean .. Patch Set 5: Verified; Looks good to me, but someone else must approve Rebase to latest master. Run through

Change in vdsm[master]: make vdsm/libvirtvm.py PEP 8 clean

2012-08-22 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: make vdsm/libvirtvm.py PEP 8 clean .. Patch Set 5: Build Successful http://jenkins.ovirt.info/job/patch_vdsm_unit_tests/607/ : SUCCESS -- To view,

Change in vdsm[master]: make vdsm/libvirtvm.py PEP 8 clean

2012-08-16 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: make vdsm/libvirtvm.py PEP 8 clean .. Patch Set 4: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/6561 To

Change in vdsm[master]: make vdsm/libvirtvm.py PEP 8 clean

2012-08-15 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: make vdsm/libvirtvm.py PEP 8 clean .. Patch Set 4: Build Successful http://jenkins.ovirt.info/job/patch_vdsm_unit_tests/471/ : SUCCESS -- To view,

Change in vdsm[master]: make vdsm/libvirtvm.py PEP 8 clean

2012-08-15 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: make vdsm/libvirtvm.py PEP 8 clean .. Patch Set 4: Verified Rebase to latest master. Run through autobuild.sh. pydiff reports some changes but they are

Change in vdsm[master]: make vdsm/libvirtvm.py PEP 8 clean

2012-07-25 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: make vdsm/libvirtvm.py PEP 8 clean .. Patch Set 2: I would prefer that you didn't submit this (1 inline comment)

Change in vdsm[master]: make vdsm/libvirtvm.py PEP 8 clean

2012-07-25 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: make vdsm/libvirtvm.py PEP 8 clean .. Patch Set 2: (1 inline comment) File vdsm/libvirtvm.py Line 578:

Change in vdsm[master]: make vdsm/libvirtvm.py PEP 8 clean

2012-07-24 Thread zhshzhou
Hello Mark Wu, I'd like you to do a code review. Please visit http://gerrit.ovirt.org/6561 to review the following change. Change subject: make vdsm/libvirtvm.py PEP 8 clean .. make vdsm/libvirtvm.py PEP 8 clean

Change in vdsm[master]: make vdsm/libvirtvm.py PEP 8 clean

2012-07-24 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: make vdsm/libvirtvm.py PEP 8 clean .. Patch Set 2: Verified; Looks good to me, but someone else must approve Runs through autobuild.sh. pydiff reports 11