You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@aurora.apache.org by Craig Wickesser <co...@gmail.com> on 2015/02/14 03:30:35 UTC
Review Request 31042: Added document describing how to build Aurora
for CentOS
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31042/
-----------------------------------------------------------
Review request for Aurora.
Repository: aurora
Description
-------
Added document describing how to build Aurora for CentOS
Diffs
-----
docs/build-for-centos.md PRE-CREATION
Diff: https://reviews.apache.org/r/31042/diff/
Testing
-------
Thanks,
Craig Wickesser
Re: Review Request 31042: Added document describing how to build
Aurora for CentOS
Posted by Aurora ReviewBot <wf...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31042/#review72649
-----------------------------------------------------------
Master (c599e0c) is red with this patch.
./build-support/jenkins/build.sh
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
F401:ERROR src/main/python/apache/aurora/executor/executor_vars.py:017 'BadZipfile' imported but unused
|from zipfile import BadZipfile
F401:ERROR src/main/python/apache/aurora/executor/executor_vars.py:020 'PexInfo' imported but unused
|from pex.pex import PexInfo
F401:ERROR src/main/python/apache/aurora/executor/executor_vars.py:022 'NamedGauge' imported but unused
|from twitter.common.metrics import LambdaGauge, MutatorGauge, NamedGauge, Observable
F401:ERROR src/main/python/apache/aurora/executor/executor_vars.py:024 'ScanfParser' imported but unused
|from twitter.common.string.scanf import ScanfParser
I will refresh this build result if you post a review containing "@ReviewBot retry"
- Aurora ReviewBot
On Feb. 16, 2015, 7:21 p.m., Craig Wickesser wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31042/
> -----------------------------------------------------------
>
> (Updated Feb. 16, 2015, 7:21 p.m.)
>
>
> Review request for Aurora.
>
>
> Repository: aurora
>
>
> Description
> -------
>
> Added document describing how to build Aurora for CentOS
>
>
> Diffs
> -----
>
> docs/build-for-centos.md PRE-CREATION
>
> Diff: https://reviews.apache.org/r/31042/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Craig Wickesser
>
>
Re: Review Request 31042: Added document describing how to build
Aurora for CentOS
Posted by Craig Wickesser <co...@gmail.com>.
> On Feb. 18, 2015, 7:34 p.m., Zameer Manji wrote:
> > Thanks for your contribution. Would you mind adding myself (zmanji) and Kevin Sweeny (kevints) to the People line of the review?
Thanks, and done.
- Craig
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31042/#review72991
-----------------------------------------------------------
On Feb. 16, 2015, 7:21 p.m., Craig Wickesser wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31042/
> -----------------------------------------------------------
>
> (Updated Feb. 16, 2015, 7:21 p.m.)
>
>
> Review request for Aurora.
>
>
> Repository: aurora
>
>
> Description
> -------
>
> Added document describing how to build Aurora for CentOS
>
>
> Diffs
> -----
>
> docs/build-for-centos.md PRE-CREATION
>
> Diff: https://reviews.apache.org/r/31042/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Craig Wickesser
>
>
Re: Review Request 31042: Added document describing how to build
Aurora for CentOS
Posted by Zameer Manji <zm...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31042/#review72991
-----------------------------------------------------------
Thanks for your contribution. Would you mind adding myself (zmanji) and Kevin Sweeny (kevints) to the People line of the review?
- Zameer Manji
On Feb. 16, 2015, 11:21 a.m., Craig Wickesser wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31042/
> -----------------------------------------------------------
>
> (Updated Feb. 16, 2015, 11:21 a.m.)
>
>
> Review request for Aurora.
>
>
> Repository: aurora
>
>
> Description
> -------
>
> Added document describing how to build Aurora for CentOS
>
>
> Diffs
> -----
>
> docs/build-for-centos.md PRE-CREATION
>
> Diff: https://reviews.apache.org/r/31042/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Craig Wickesser
>
>
Re: Review Request 31042: Added document describing how to build
Aurora for CentOS
Posted by Craig Wickesser <co...@gmail.com>.
> On Feb. 18, 2015, 7:42 p.m., Kevin Sweeney wrote:
> > Hi Craig, thanks for the contribution. I'm wondering if this document might be better suited as a gist or blog post given that it's dated (0.7.1) and we don't have automated test coverage to ensure it doesn't diverge from the rest of the source tree. Alternatively if you'd like to provide a spec file and test coverage (e.g. docker run within our CI build) I think this would be much more palatable to commit. Also, while I personally love Digital Ocean I'm not sure what the rules on endorsing a specific vendor in official project documentation are.
>
> Craig Wickesser wrote:
> I could post it as a blog post, where would a link to a blog post best go in the docs?
>
> Bill Farner wrote:
> I share Kevin's concern over bit rot here, simply because we have nothing to exercise these. However, i think Craig points out a big gap in figuring out _what_ components are needed for a healthy Aurora cluster. More specifically, `README.md` has a link titled 'Deploying Aurora' that is actually a guide for deploying the scheduler, and there is no equivalent for other components. (I've observed three people in IRC this week alone falling into this trap.)
>
> Craig - how would you feel about pulling relevant content from this diff into deploying-aurora-scheduler.md [1] (renaming to deploying-aurora.md)?
> Kevin - does that sound reasonable?
>
>
> [1] https://github.com/apache/incubator-aurora/blob/master/docs/deploying-aurora-scheduler.md
Bill, I think updating the existing "deploying" doc makes sense. If Kevin agrees, I'll take that route.
- Craig
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31042/#review72994
-----------------------------------------------------------
On Feb. 16, 2015, 7:21 p.m., Craig Wickesser wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31042/
> -----------------------------------------------------------
>
> (Updated Feb. 16, 2015, 7:21 p.m.)
>
>
> Review request for Aurora.
>
>
> Repository: aurora
>
>
> Description
> -------
>
> Added document describing how to build Aurora for CentOS
>
>
> Diffs
> -----
>
> docs/build-for-centos.md PRE-CREATION
>
> Diff: https://reviews.apache.org/r/31042/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Craig Wickesser
>
>
Re: Review Request 31042: Added document describing how to build
Aurora for CentOS
Posted by Kevin Sweeney <ke...@apache.org>.
> On Feb. 18, 2015, 11:42 a.m., Kevin Sweeney wrote:
> > Hi Craig, thanks for the contribution. I'm wondering if this document might be better suited as a gist or blog post given that it's dated (0.7.1) and we don't have automated test coverage to ensure it doesn't diverge from the rest of the source tree. Alternatively if you'd like to provide a spec file and test coverage (e.g. docker run within our CI build) I think this would be much more palatable to commit. Also, while I personally love Digital Ocean I'm not sure what the rules on endorsing a specific vendor in official project documentation are.
>
> Craig Wickesser wrote:
> I could post it as a blog post, where would a link to a blog post best go in the docs?
>
> Bill Farner wrote:
> I share Kevin's concern over bit rot here, simply because we have nothing to exercise these. However, i think Craig points out a big gap in figuring out _what_ components are needed for a healthy Aurora cluster. More specifically, `README.md` has a link titled 'Deploying Aurora' that is actually a guide for deploying the scheduler, and there is no equivalent for other components. (I've observed three people in IRC this week alone falling into this trap.)
>
> Craig - how would you feel about pulling relevant content from this diff into deploying-aurora-scheduler.md [1] (renaming to deploying-aurora.md)?
> Kevin - does that sound reasonable?
>
>
> [1] https://github.com/apache/incubator-aurora/blob/master/docs/deploying-aurora-scheduler.md
>
> Craig Wickesser wrote:
> Bill, I think updating the existing "deploying" doc makes sense. If Kevin agrees, I'll take that route.
+1
- Kevin
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31042/#review72994
-----------------------------------------------------------
On Feb. 16, 2015, 11:21 a.m., Craig Wickesser wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31042/
> -----------------------------------------------------------
>
> (Updated Feb. 16, 2015, 11:21 a.m.)
>
>
> Review request for Aurora.
>
>
> Repository: aurora
>
>
> Description
> -------
>
> Added document describing how to build Aurora for CentOS
>
>
> Diffs
> -----
>
> docs/build-for-centos.md PRE-CREATION
>
> Diff: https://reviews.apache.org/r/31042/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Craig Wickesser
>
>
Re: Review Request 31042: Added document describing how to build
Aurora for CentOS
Posted by Bill Farner <wf...@apache.org>.
> On Feb. 18, 2015, 7:42 p.m., Kevin Sweeney wrote:
> > Hi Craig, thanks for the contribution. I'm wondering if this document might be better suited as a gist or blog post given that it's dated (0.7.1) and we don't have automated test coverage to ensure it doesn't diverge from the rest of the source tree. Alternatively if you'd like to provide a spec file and test coverage (e.g. docker run within our CI build) I think this would be much more palatable to commit. Also, while I personally love Digital Ocean I'm not sure what the rules on endorsing a specific vendor in official project documentation are.
>
> Craig Wickesser wrote:
> I could post it as a blog post, where would a link to a blog post best go in the docs?
I share Kevin's concern over bit rot here, simply because we have nothing to exercise these. However, i think Craig points out a big gap in figuring out _what_ components are needed for a healthy Aurora cluster. More specifically, `README.md` has a link titled 'Deploying Aurora' that is actually a guide for deploying the scheduler, and there is no equivalent for other components. (I've observed three people in IRC this week alone falling into this trap.)
Craig - how would you feel about pulling relevant content from this diff into deploying-aurora-scheduler.md [1] (renaming to deploying-aurora.md)?
Kevin - does that sound reasonable?
[1] https://github.com/apache/incubator-aurora/blob/master/docs/deploying-aurora-scheduler.md
- Bill
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31042/#review72994
-----------------------------------------------------------
On Feb. 16, 2015, 7:21 p.m., Craig Wickesser wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31042/
> -----------------------------------------------------------
>
> (Updated Feb. 16, 2015, 7:21 p.m.)
>
>
> Review request for Aurora.
>
>
> Repository: aurora
>
>
> Description
> -------
>
> Added document describing how to build Aurora for CentOS
>
>
> Diffs
> -----
>
> docs/build-for-centos.md PRE-CREATION
>
> Diff: https://reviews.apache.org/r/31042/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Craig Wickesser
>
>
Re: Review Request 31042: Added document describing how to build
Aurora for CentOS
Posted by Craig Wickesser <co...@gmail.com>.
> On Feb. 18, 2015, 7:42 p.m., Kevin Sweeney wrote:
> > Hi Craig, thanks for the contribution. I'm wondering if this document might be better suited as a gist or blog post given that it's dated (0.7.1) and we don't have automated test coverage to ensure it doesn't diverge from the rest of the source tree. Alternatively if you'd like to provide a spec file and test coverage (e.g. docker run within our CI build) I think this would be much more palatable to commit. Also, while I personally love Digital Ocean I'm not sure what the rules on endorsing a specific vendor in official project documentation are.
I could post it as a blog post, where would a link to a blog post best go in the docs?
- Craig
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31042/#review72994
-----------------------------------------------------------
On Feb. 16, 2015, 7:21 p.m., Craig Wickesser wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31042/
> -----------------------------------------------------------
>
> (Updated Feb. 16, 2015, 7:21 p.m.)
>
>
> Review request for Aurora.
>
>
> Repository: aurora
>
>
> Description
> -------
>
> Added document describing how to build Aurora for CentOS
>
>
> Diffs
> -----
>
> docs/build-for-centos.md PRE-CREATION
>
> Diff: https://reviews.apache.org/r/31042/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Craig Wickesser
>
>
Re: Review Request 31042: Added document describing how to build
Aurora for CentOS
Posted by Kevin Sweeney <ke...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31042/#review72994
-----------------------------------------------------------
Hi Craig, thanks for the contribution. I'm wondering if this document might be better suited as a gist or blog post given that it's dated (0.7.1) and we don't have automated test coverage to ensure it doesn't diverge from the rest of the source tree. Alternatively if you'd like to provide a spec file and test coverage (e.g. docker run within our CI build) I think this would be much more palatable to commit. Also, while I personally love Digital Ocean I'm not sure what the rules on endorsing a specific vendor in official project documentation are.
- Kevin Sweeney
On Feb. 16, 2015, 11:21 a.m., Craig Wickesser wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31042/
> -----------------------------------------------------------
>
> (Updated Feb. 16, 2015, 11:21 a.m.)
>
>
> Review request for Aurora.
>
>
> Repository: aurora
>
>
> Description
> -------
>
> Added document describing how to build Aurora for CentOS
>
>
> Diffs
> -----
>
> docs/build-for-centos.md PRE-CREATION
>
> Diff: https://reviews.apache.org/r/31042/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Craig Wickesser
>
>
Re: Review Request 31042: Added document describing how to build
Aurora for CentOS
Posted by Maxim Khutornenko <ma...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31042/#review109972
-----------------------------------------------------------
Discarding this diff as it appears to be stale. Feel free to reopen if you feel otherwise!
- Maxim Khutornenko
On Feb. 16, 2015, 7:21 p.m., Craig Wickesser wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31042/
> -----------------------------------------------------------
>
> (Updated Feb. 16, 2015, 7:21 p.m.)
>
>
> Review request for Aurora.
>
>
> Repository: aurora
>
>
> Description
> -------
>
> Added document describing how to build Aurora for CentOS
>
>
> Diffs
> -----
>
> docs/build-for-centos.md PRE-CREATION
>
> Diff: https://reviews.apache.org/r/31042/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Craig Wickesser
>
>
Re: Review Request 31042: Added document describing how to build
Aurora for CentOS
Posted by Craig Wickesser <co...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31042/
-----------------------------------------------------------
(Updated Feb. 16, 2015, 7:21 p.m.)
Review request for Aurora.
Changes
-------
Removed installation of Gradle since the Gradle wrapper is already included.
Repository: aurora
Description
-------
Added document describing how to build Aurora for CentOS
Diffs (updated)
-----
docs/build-for-centos.md PRE-CREATION
Diff: https://reviews.apache.org/r/31042/diff/
Testing
-------
Thanks,
Craig Wickesser
Re: Review Request 31042: Added document describing how to build
Aurora for CentOS
Posted by Zameer Manji <zm...@apache.org>.
> On Feb. 14, 2015, 11:39 a.m., Florian Pfeiffer wrote:
> > docs/build-for-centos.md, line 40
> > <https://reviews.apache.org/r/31042/diff/1/?file=864093#file864093line40>
> >
> > Installing gradle is probably not needed, since the gradle wrapper is included in the aurora project?
>
> Zameer Manji wrote:
> The gradle wrapper is not apart of the source dists because of licensing issues. So if you grab the source release from Apache you will need to install gradle.
Derp, I now see that the document points to the source on github and not an Apache release.
- Zameer
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31042/#review72512
-----------------------------------------------------------
On Feb. 16, 2015, 11:21 a.m., Craig Wickesser wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31042/
> -----------------------------------------------------------
>
> (Updated Feb. 16, 2015, 11:21 a.m.)
>
>
> Review request for Aurora.
>
>
> Repository: aurora
>
>
> Description
> -------
>
> Added document describing how to build Aurora for CentOS
>
>
> Diffs
> -----
>
> docs/build-for-centos.md PRE-CREATION
>
> Diff: https://reviews.apache.org/r/31042/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Craig Wickesser
>
>
Re: Review Request 31042: Added document describing how to build
Aurora for CentOS
Posted by Zameer Manji <zm...@apache.org>.
> On Feb. 14, 2015, 11:39 a.m., Florian Pfeiffer wrote:
> > docs/build-for-centos.md, line 40
> > <https://reviews.apache.org/r/31042/diff/1/?file=864093#file864093line40>
> >
> > Installing gradle is probably not needed, since the gradle wrapper is included in the aurora project?
The gradle wrapper is not apart of the source dists because of licensing issues. So if you grab the source release from Apache you will need to install gradle.
- Zameer
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31042/#review72512
-----------------------------------------------------------
On Feb. 16, 2015, 11:21 a.m., Craig Wickesser wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31042/
> -----------------------------------------------------------
>
> (Updated Feb. 16, 2015, 11:21 a.m.)
>
>
> Review request for Aurora.
>
>
> Repository: aurora
>
>
> Description
> -------
>
> Added document describing how to build Aurora for CentOS
>
>
> Diffs
> -----
>
> docs/build-for-centos.md PRE-CREATION
>
> Diff: https://reviews.apache.org/r/31042/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Craig Wickesser
>
>
Re: Review Request 31042: Added document describing how to build
Aurora for CentOS
Posted by Florian Pfeiffer <ap...@x8s.de>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31042/#review72512
-----------------------------------------------------------
docs/build-for-centos.md
<https://reviews.apache.org/r/31042/#comment118575>
Installing gradle is probably not needed, since the gradle wrapper is included in the aurora project?
- Florian Pfeiffer
On Feb. 14, 2015, 2:30 vorm., Craig Wickesser wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31042/
> -----------------------------------------------------------
>
> (Updated Feb. 14, 2015, 2:30 vorm.)
>
>
> Review request for Aurora.
>
>
> Repository: aurora
>
>
> Description
> -------
>
> Added document describing how to build Aurora for CentOS
>
>
> Diffs
> -----
>
> docs/build-for-centos.md PRE-CREATION
>
> Diff: https://reviews.apache.org/r/31042/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Craig Wickesser
>
>
Re: Review Request 31042: Added document describing how to build
Aurora for CentOS
Posted by Aurora ReviewBot <wf...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31042/#review72483
-----------------------------------------------------------
Master (c599e0c) is red with this patch.
./build-support/jenkins/build.sh
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
F401:ERROR src/main/python/apache/aurora/executor/executor_vars.py:017 'BadZipfile' imported but unused
|from zipfile import BadZipfile
F401:ERROR src/main/python/apache/aurora/executor/executor_vars.py:020 'PexInfo' imported but unused
|from pex.pex import PexInfo
F401:ERROR src/main/python/apache/aurora/executor/executor_vars.py:022 'NamedGauge' imported but unused
|from twitter.common.metrics import LambdaGauge, MutatorGauge, NamedGauge, Observable
F401:ERROR src/main/python/apache/aurora/executor/executor_vars.py:024 'ScanfParser' imported but unused
|from twitter.common.string.scanf import ScanfParser
I will refresh this build result if you post a review containing "@ReviewBot retry"
- Aurora ReviewBot
On Feb. 14, 2015, 2:30 a.m., Craig Wickesser wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31042/
> -----------------------------------------------------------
>
> (Updated Feb. 14, 2015, 2:30 a.m.)
>
>
> Review request for Aurora.
>
>
> Repository: aurora
>
>
> Description
> -------
>
> Added document describing how to build Aurora for CentOS
>
>
> Diffs
> -----
>
> docs/build-for-centos.md PRE-CREATION
>
> Diff: https://reviews.apache.org/r/31042/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Craig Wickesser
>
>