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]: 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]: 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]: 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]: 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]: 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]: 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]: 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]: Fix the charset conversion logic in hooks.py

2013-07-02 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 5: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3030/ : SUCCESS

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

2013-07-02 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 6: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3032/ : SUCCESS

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

2013-07-02 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 7: Build Successful http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3036/ : SUCCESS

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

2013-07-02 Thread msivak
Martin Sivák has posted comments on this change. Change subject: Fix the charset conversion logic in hooks.py .. Patch Set 7: Verified Tests pass even when unicode chars are present in the environment. -- To view, visit

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

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

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

2013-06-26 Thread msivak
Martin Sivák has posted comments on this change. Change subject: Fix the charset conversion logic in hooks.py .. Patch Set 4: (1 inline comment) File vdsm/hooks.py Line 59:

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

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

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

2013-06-25 Thread msivak
Martin Sivák has posted comments on this change. Change subject: Fix the charset conversion logic in hooks.py .. Patch Set 4: Verified Tests pass. The fix ignores lines that throw a UnicodeDecodeException. Jenkins does not

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

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

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

2013-06-24 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 4: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/2902/ : SUCCESS

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

2013-06-19 Thread msivak
Martin Sivák has posted comments on this change. Change subject: Fix the charset conversion logic in hooks.py .. Patch Set 2: Fails (1 inline comment) Back to the drawing board. It works on command line, but the automated

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

2013-06-19 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Fix the charset conversion logic in hooks.py .. Patch Set 2: (1 inline comment) Commit Message Line 7: Fix

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

2013-06-19 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 2: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/2838/ : SUCCESS

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

2013-06-19 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 2: Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/2838/ : SUCCESS

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

2013-06-19 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 2: Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/2838/ : SUCCESS

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

2013-06-19 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 3: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/2847/ : SUCCESS

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

2013-06-19 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 3: Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/2847/ : SUCCESS

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

2013-06-19 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 3: Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/2847/ : SUCCESS

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

2013-06-19 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Fix the charset conversion logic in hooks.py .. Patch Set 3: Looks good to me, but someone else must approve (2 inline comments)

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

2013-06-18 Thread msivak
Martin Sivak has uploaded a new change for review. 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 to unicode

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

2013-06-18 Thread dcaroest
David Caro has posted comments on this change. Change subject: Fix the charset conversion logic in hooks.py .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/15856 To

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

2013-06-18 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 2: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/2824/ : SUCCESS