----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40197/#review106147 -----------------------------------------------------------
Ship it! Master (0b025a7) is green with this patch. ./build-support/jenkins/build.sh I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On Nov. 11, 2015, 8:30 p.m., John Sirois wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40197/ > ----------------------------------------------------------- > > (Updated Nov. 11, 2015, 8:30 p.m.) > > > Review request for Aurora, Joshua Cohen and Bill Farner. > > > Bugs: AURORA-547 > https://issues.apache.org/jira/browse/AURORA-547 > > > Repository: aurora > > > Description > ------- > > The Aurora python tests cannot currently all be run together; so the ci > script passes --no-fast and devs need to remember to do this too. > Improve the dev experience by defaulting this option. > > build-support/jenkins/build.sh | 2 +- > pants.ini | 7 +++++++ > 2 files changed, 8 insertions(+), 1 deletion(-) > > > Diffs > ----- > > build-support/jenkins/build.sh 5cd5242a2e4551e356e5ce5d15a0d27beb7e2d4e > pants.ini 0bd8ec829ae65e2296a122166dea13f315323c9f > > Diff: https://reviews.apache.org/r/40197/diff/ > > > Testing > ------- > > Ran into this working https://issues.apache.org/jira/browse/AURORA-547. > After the fix, locally ran `./pants test src/test/python:: -- -v` green. > Previously this would lead to test failures and CPU starvation. > > > Thanks, > > John Sirois > >
