On Fri, Sep 20, 2013 at 05:31:59AM -0400, Timothy Asir Jeyasingh wrote:
> Hi All,
> 
> The vdsm (master branch) rpm build failed with the following error:
> 
> find . -path './.git' -prune -type f -o \
>       -name '*.py' -o -name '*.py.in'  | xargs /usr/bin/pyflakes | \
>       grep -w -v "\./vdsm/storage/lvm\.py.*: list comprehension redefines 
> 'lv' from line .*" | \
>       while read LINE; do echo "$LINE"; false; done
> /usr/bin/pep8 --version
> 1.4.6
> /usr/bin/pep8 --exclude="config.py,constants.py" --filename '*.py,*.py.in' \
>       client lib/cpopen/*.py lib/vdsm/*.py lib/vdsm/*.py.in tests 
> vds_bootstrap vdsm-tool vdsm/*.py vdsm/*.py.in vdsm/netconf 
> vdsm/sos/vdsm.py.in vdsm/storage vdsm/vdsm vdsm_api vdsm_hooks vdsm_reg 
> lib/vdsm/SecureXMLRPCServer.py:143:1: E303 too many blank lines (3)
> make[3]: *** [check-local] Error 1
> make[3]: Leaving directory `/home/timothy/rpmbuild/BUILD/vdsm-4.12.0'
> make[2]: *** [check-am] Error 2
> make[2]: Leaving directory `/home/timothy/rpmbuild/BUILD/vdsm-4.12.0'
> make[1]: *** [check-recursive] Error 1
> make[1]: Leaving directory `/home/timothy/rpmbuild/BUILD/vdsm-4.12.0'
> error: Bad exit status from /var/tmp/rpm-tmp.PcZNhb (%check)

Sorry about that. The pep8 failure was masked by another failure of
apiTests.JsonRawTest.testBadMethod
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/4499/testReport/junit/apiTests/JsonRawTest/testBadMethod/

Saggi, could you look into these recurring breakage?

As David Caro explains in a comment to
http://gerrit.ovirt.org/#/c/19332/ , I carelessly ignored the pep8
failure in Juan's patch.

Problem solved by Juan's http://gerrit.ovirt.org/19408 .

Thanks, and sorry for the noise.
_______________________________________________
vdsm-devel mailing list
vdsm-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel

Reply via email to