> On June 11, 2015, 10:16 a.m., Aurora ReviewBot wrote:
> > Master (fab6637) is red with this patch.
> >   ./build-support/jenkins/build.sh
> > 
> >   Using cached twitter.common.process-0.3.0.tar.gz
> > Collecting twitter.common.log==0.3.0 (from 
> > twitter.common.app==0.3.0->twitter.checkstyle==0.1.0)
> >   Using cached twitter.common.log-0.3.0.tar.gz
> > Collecting twitter.common.util==0.3.0 (from 
> > twitter.common.app==0.3.0->twitter.checkstyle==0.1.0)
> >   Using cached twitter.common.util-0.3.0.tar.gz
> > Collecting twitter.common.collections==0.3.0 (from 
> > twitter.common.app==0.3.0->twitter.checkstyle==0.1.0)
> >   Using cached twitter.common.collections-0.3.0.tar.gz
> > Collecting smmap>=0.8.5 (from 
> > gitdb>=0.5.1->GitPython==0.3.2.RC1->twitter.checkstyle==0.1.0)
> >   Using cached smmap-0.9.0.tar.gz
> > Collecting twitter.common.string==0.3.0 (from 
> > twitter.common.process==0.3.0->twitter.common.app==0.3.0->twitter.checkstyle==0.1.0)
> >   Using cached twitter.common.string-0.3.0.tar.gz
> > Collecting twitter.common.options==0.3.0 (from 
> > twitter.common.log==0.3.0->twitter.common.app==0.3.0->twitter.checkstyle==0.1.0)
> >   Using cached twitter.common.options-0.3.0.tar.gz
> > Collecting twitter.common.dirutil==0.3.0 (from 
> > twitter.common.log==0.3.0->twitter.common.app==0.3.0->twitter.checkstyle==0.1.0)
> >   Using cached twitter.common.dirutil-0.3.0.tar.gz
> > Collecting twitter.common.contextutil==0.3.0 (from 
> > twitter.common.util==0.3.0->twitter.common.app==0.3.0->twitter.checkstyle==0.1.0)
> >   Using cached twitter.common.contextutil-0.3.0.tar.gz
> > Collecting twitter.common.lang==0.3.0 (from 
> > twitter.common.collections==0.3.0->twitter.common.app==0.3.0->twitter.checkstyle==0.1.0)
> >   Using cached twitter.common.lang-0.3.0.tar.gz
> > Installing collected packages: pyflakes, pep8, smmap, gitdb, GitPython, 
> > twitter.common.lang, twitter.common.string, twitter.common.process, 
> > twitter.common.options, twitter.common.dirutil, twitter.common.log, 
> > twitter.common.contextutil, twitter.common.util, 
> > twitter.common.collections, twitter.common.app, twitter.checkstyle
> >   Running setup.py install for pyflakes
> >   Running setup.py install for pep8
> >   Running setup.py install for smmap
> >   Running setup.py install for gitdb
> >   Running setup.py install for GitPython
> >   Running setup.py install for twitter.common.lang
> >   Running setup.py install for twitter.common.string
> >   Running setup.py install for twitter.common.process
> >   Running setup.py install for twitter.common.options
> >   Running setup.py install for twitter.common.dirutil
> >   Running setup.py install for twitter.common.log
> >   Running setup.py install for twitter.common.contextutil
> >   Running setup.py install for twitter.common.util
> >   Running setup.py install for twitter.common.collections
> >   Running setup.py install for twitter.common.app
> >   Running setup.py install for twitter.checkstyle
> > Successfully installed GitPython-0.3.2rc1 gitdb-0.6.4 pep8-1.4.5 
> > pyflakes-0.7.2 smmap-0.9.0 twitter.checkstyle-0.1.0 
> > twitter.common.app-0.3.0 twitter.common.collections-0.3.0 
> > twitter.common.contextutil-0.3.0 twitter.common.dirutil-0.3.0 
> > twitter.common.lang-0.3.0 twitter.common.log-0.3.0 
> > twitter.common.options-0.3.0 twitter.common.process-0.3.0 
> > twitter.common.string-0.3.0 twitter.common.util-0.3.0
> > F401:ERROR   
> > src/test/python/apache/aurora/executor/test_thermos_task_runner.py:024 
> > 'pytest' imported but unused
> >      |import pytest
> > 
> > 
> > 
> > I will refresh this build result if you post a review containing 
> > "@ReviewBot retry"

@ReviewBot retry

Hmm, still had incubator lurking in my repo in places. I can run 
checkstyle-again after some tweaking.


- Brian


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


On June 11, 2015, 10:39 a.m., Brian Brazil wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/32973/
> -----------------------------------------------------------
> 
> (Updated June 11, 2015, 10:39 a.m.)
> 
> 
> Review request for Aurora, Kevin Sweeney and Maxim Khutornenko.
> 
> 
> Bugs: AURORA-1257
>     https://issues.apache.org/jira/browse/AURORA-1257
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Fix name of test for sigterm being ignored.
> 
> Based on 
> https://github.com/tellapart/incubator-aurora/commit/c90532d69d69045c545004451e3660f5282e6e34
> 
> 
> Diffs
> -----
> 
>   docs/configuration-reference.md fb753ead94fcd1ed76352306391d356652934a4e 
>   src/main/python/apache/aurora/common/http_signaler.py 
> 531f1fecbcbf8015175c7f1cb1e1c4d3e7d1268a 
>   src/main/python/apache/aurora/executor/thermos_task_runner.py 
> 837ad5c55b4a02215a8d509c211e80529536f705 
>   src/main/python/apache/thermos/config/schema_base.py 
> f9143cc1b83143d6147f59d90c79435d055d0518 
>   src/test/python/apache/aurora/common/test_http_signaler.py 
> c6a21708eb997f213f4cca038891a84f6c5218d4 
>   src/test/python/apache/aurora/executor/test_thermos_task_runner.py 
> 1e21a112efcbd0a575fb8d90f78ff1e8c7f034cf 
> 
> Diff: https://reviews.apache.org/r/32973/diff/
> 
> 
> Testing
> -------
> 
> Unittests and manually in vagrant cluster. Also verified that the flaky tests 
> pass.
> 
> 
> Thanks,
> 
> Brian Brazil
> 
>

Reply via email to