> On March 17, 2015, 11:36 p.m., Aurora ReviewBot wrote: > > Master (a419863) is red with this patch. > > ./build-support/jenkins/build.sh > > > > Installing > > /home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/python/checkstyle.venv/lib/python2.7/site-packages/twitter.common.collections-0.3.0-py2.7-nspkg.pth > > Running setup.py install for twitter.common.util > > Skipping installation of > > /home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/python/checkstyle.venv/lib/python2.7/site-packages/twitter/__init__.py > > (namespace package) > > Skipping installation of > > /home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/python/checkstyle.venv/lib/python2.7/site-packages/twitter/common/__init__.py > > (namespace package) > > Installing > > /home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/python/checkstyle.venv/lib/python2.7/site-packages/twitter.common.util-0.3.0-py2.7-nspkg.pth > > Running setup.py install for twitter.common.log > > Skipping installation of > > /home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/python/checkstyle.venv/lib/python2.7/site-packages/twitter/__init__.py > > (namespace package) > > Skipping installation of > > /home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/python/checkstyle.venv/lib/python2.7/site-packages/twitter/common/__init__.py > > (namespace package) > > Installing > > /home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/python/checkstyle.venv/lib/python2.7/site-packages/twitter.common.log-0.3.0-py2.7-nspkg.pth > > Running setup.py install for twitter.common.process > > Skipping installation of > > /home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/python/checkstyle.venv/lib/python2.7/site-packages/twitter/__init__.py > > (namespace package) > > Skipping installation of > > /home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/python/checkstyle.venv/lib/python2.7/site-packages/twitter/common/__init__.py > > (namespace package) > > Installing > > /home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/python/checkstyle.venv/lib/python2.7/site-packages/twitter.common.process-0.3.0-py2.7-nspkg.pth > > Running setup.py install for gitdb > > building 'gitdb._perf' extension > > x86_64-linux-gnu-gcc -pthread -fno-strict-aliasing -DNDEBUG -g -fwrapv > > -O2 -Wall -Wstrict-prototypes -fPIC -Igitdb -I/usr/include/python2.7 -c > > gitdb/_fun.c -o build/temp.linux-x86_64-2.7/gitdb/_fun.o > > x86_64-linux-gnu-gcc -pthread -fno-strict-aliasing -DNDEBUG -g -fwrapv > > -O2 -Wall -Wstrict-prototypes -fPIC -Igitdb -I/usr/include/python2.7 -c > > gitdb/_delta_apply.c -o build/temp.linux-x86_64-2.7/gitdb/_delta_apply.o > > x86_64-linux-gnu-gcc -pthread -shared -Wl,-O1 -Wl,-Bsymbolic-functions > > -Wl,-Bsymbolic-functions -Wl,-z,relro -fno-strict-aliasing -DNDEBUG -g > > -fwrapv -O2 -Wall -Wstrict-prototypes -D_FORTIFY_SOURCE=2 -g > > -fstack-protector --param=ssp-buffer-size=4 -Wformat > > -Werror=format-security build/temp.linux-x86_64-2.7/gitdb/_fun.o > > build/temp.linux-x86_64-2.7/gitdb/_delta_apply.o -o > > build/lib.linux-x86_64-2.7/gitdb/_perf.so > > Running setup.py install for twitter.common.app > > Skipping installation of > > /home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/python/checkstyle.venv/lib/python2.7/site-packages/twitter/__init__.py > > (namespace package) > > Skipping installation of > > /home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/python/checkstyle.venv/lib/python2.7/site-packages/twitter/common/__init__.py > > (namespace package) > > Installing > > /home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/python/checkstyle.venv/lib/python2.7/site-packages/twitter.common.app-0.3.0-py2.7-nspkg.pth > > Running setup.py install for GitPython > > > > /home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/python/checkstyle.venv/local/lib/python2.7/site-packages/setuptools/dist.py:292: > > UserWarning: The version specified ('0.3.2 RC1') is an invalid version, > > this may not work as expected with newer versions of setuptools, pip, and > > PyPI. Please see PEP 440 for more details. > > "details." % self.metadata.version > > Running setup.py install for pep8 > > Installing pep8 script to > > /home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/python/checkstyle.venv/bin > > Running setup.py install for pyflakes > > Installing pyflakes script to > > /home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/python/checkstyle.venv/bin > > Running setup.py install for twitter.checkstyle > > Skipping installation of > > /home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/python/checkstyle.venv/lib/python2.7/site-packages/twitter/__init__.py > > (namespace package) > > Installing > > /home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/python/checkstyle.venv/lib/python2.7/site-packages/twitter.checkstyle-0.1.0-py2.7-nspkg.pth > > Installing twitterstyle script to > > /home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/python/checkstyle.venv/bin > > 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 > > F841:ERROR src/main/python/apache/aurora/client/cli/update.py:316 local > > variable 'created' is assigned to but never used > > | created = summary.state.createdTimestampMs > > > > F841:ERROR src/main/python/apache/aurora/client/cli/update.py:317 local > > variable 'lastMod' is assigned to but never used > > | lastMod = summary.state.lastModifiedTimestampMs > > > > > > > > I will refresh this build result if you post a review containing > > "@ReviewBot retry"
Doh sorry for the noise. Fixing. - Bill ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32171/#review76845 ----------------------------------------------------------- On March 17, 2015, 11:30 p.m., Bill Farner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/32171/ > ----------------------------------------------------------- > > (Updated March 17, 2015, 11:30 p.m.) > > > Review request for Aurora, David McLaughlin and Zameer Manji. > > > Bugs: AURORA-1168 > https://issues.apache.org/jira/browse/AURORA-1168 > > > Repository: aurora > > > Description > ------- > > NOTE: I also am proposing with this change that the output format of `aurora > update list` be line-oriented and more concise (see test changes for > examples). The idea is to follow up ~immediately with AURORA-1206. The > proposal is for this command to let you retrieve update identifiers, and > `update status` (whose name may change) will allow you to drill into updates. > > I'm open to input on this. > > > Diffs > ----- > > src/main/python/apache/aurora/client/cli/update.py > f025d46d50592156e2455313890e981722ab63a5 > src/test/python/apache/aurora/client/cli/test_supdate.py > cb66439a778349fc5add4985a7395655c9e1328a > > Diff: https://reviews.apache.org/r/32171/diff/ > > > Testing > ------- > > > Thanks, > > Bill Farner > >