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

zhshzhou at linux.vnet.ibm.com zhshzhou at linux.vnet.ibm.com
Thu Jul 4 08:43:09 UTC 2013


Zhou Zheng Sheng has posted comments on this change.

Change subject: PEP8: fix variours errors
......................................................................


Patch Set 3:

The Jenkins job configuration for vdsm_unit_tests_gerrit_el is wrong, it always checkout the master but not the gerrit ref. You can see from its the log.

http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/2278/console

http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3169/console

>From the above you can see, the unit_tests_gerrit_el's git revision hash is 4d92abd525, which is "Fix the charset conversion logic in hooks.py" in the latest master. vdsm_unit_tests_gerrit's revision hash is correct.

-- 
To view, visit http://gerrit.ovirt.org/16424
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib6f8843f7bf9f57c040b4747e89ce90ee6122823
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Zhou Zheng Sheng <zhshzhou at linux.vnet.ibm.com>
Gerrit-Reviewer: Kiril Nesenko <knesenko at redhat.com>
Gerrit-Reviewer: Ohad Basan <obasan at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Zhou Zheng Sheng <zhshzhou at linux.vnet.ibm.com>


More information about the vdsm-patches mailing list