aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Farner" <wfar...@apache.org>
Subject Re: Review Request 26093: Revert "Run python style checks in jenkins."
Date Fri, 26 Sep 2014 20:33:36 GMT


> On Sept. 26, 2014, 8:29 p.m., Brian Wickman wrote:
> > build-support/jenkins/build.sh, line 27
> > <https://reviews.apache.org/r/26093/diff/1/?file=706245#file706245line27>
> >
> >     without seeing the output, we have no idea.  rather than remove completely,
i'd prefer to change this to something like:
> >     
> >     if ! ./build-support/hooks/pre-commit; then
> >       build-support/python/isort-check || true
> >       build-support/python/checkstyle-check || true
> >       exit 1
> >     fi
> >     
> >     (i know this leaks knowledge of the checks from pre-commit to build.sh, so we
should probably add a build-support/python/check-all that build.sh and pre-commit can each
call.)

>From a manual run, this is the error:

Traceback (most recent call last):
  File "<https://builds.apache.org/job/Aurora/ws/build-support/python/checkstyle.venv/local/lib/python2.7/site-packages/twitter/common/app/application.py",>
line 738, in _wrap_method
    return_code = method()
  File "<https://builds.apache.org/job/Aurora/ws/build-support/python/checkstyle.venv/local/lib/python2.7/site-packages/twitter/common/app/application.py",>
line 760, in <lambda>
    main = lambda: main_method(*args, **kwargs)
  File "<https://builds.apache.org/job/Aurora/ws/build-support/python/checkstyle.venv/local/lib/python2.7/site-packages/twitter/checkstyle/checker.py",>
line 155, in main
    for filename, line_filter in iterator:
  File "<https://builds.apache.org/job/Aurora/ws/build-support/python/checkstyle.venv/local/lib/python2.7/site-packages/twitter/checkstyle/iterators.py",>
line 120, in git_iterator
    diff_commit = repo.rev_parse(options.diff or 'master')
  File "<https://builds.apache.org/job/Aurora/ws/build-support/python/checkstyle.venv/local/lib/python2.7/site-packages/git/repo/fun.py",>
line 274, in rev_parse
    obj = name_to_object(repo, rev)
  File "<https://builds.apache.org/job/Aurora/ws/build-support/python/checkstyle.venv/local/lib/python2.7/site-packages/git/repo/fun.py",>
line 86, in name_to_object
    raise BadObject(name)
BadObject: BadObject: 6d6173746572


- Bill


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26093/#review54724
-----------------------------------------------------------


On Sept. 26, 2014, 7:33 p.m., Bill Farner wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26093/
> -----------------------------------------------------------
> 
> (Updated Sept. 26, 2014, 7:33 p.m.)
> 
> 
> Review request for Aurora and Brian Wickman.
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Not sure why, but this consistently fails in jenkins.  Reverting unless you have any
ideas how to sort it out.
> 
> 
> Diffs
> -----
> 
>   build-support/jenkins/build.sh 602b0bcd050cc8270b1f2c2ff3765f8e9319dd22 
> 
> Diff: https://reviews.apache.org/r/26093/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Bill Farner
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message