[GitHub] incubator-aurora pull request: Formatted table of contents

2015-01-06 Thread mohitsoni
Github user mohitsoni commented on the pull request: https://github.com/apache/incubator-aurora/pull/4#issuecomment-68984059 Thanks for updating the documentation. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your p

[GitHub] incubator-aurora pull request: Formatted table of contents

2015-01-06 Thread mohitsoni
Github user mohitsoni closed the pull request at: https://github.com/apache/incubator-aurora/pull/4 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feat

[GitHub] incubator-aurora pull request: Fix bad indentation causing bad lay...

2015-01-06 Thread davelester
Github user davelester commented on the pull request: https://github.com/apache/incubator-aurora/pull/2#issuecomment-68981531 @isomer Thanks for contributing this patch! Apologies for the delayed comment, we have been been using Review Board to accept patches and this slipped through

[GitHub] incubator-aurora pull request: Formatted table of contents

2015-01-06 Thread davelester
Github user davelester commented on the pull request: https://github.com/apache/incubator-aurora/pull/4#issuecomment-68981078 @mohitsoni thanks for the pull request! The configuration reference documentation has been revised since this PR and now includes proper formatting for its tab

[GitHub] incubator-aurora pull request: Aurora 475 Remove Copyright Apache ...

2015-01-06 Thread hsaputra
Github user hsaputra closed the pull request at: https://github.com/apache/incubator-aurora/pull/10 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feat

[GitHub] incubator-aurora pull request: Aurora 475 Remove Copyright Apache ...

2015-01-06 Thread hsaputra
Github user hsaputra commented on the pull request: https://github.com/apache/incubator-aurora/pull/10#issuecomment-68980762 Not sure why this one was reopened. Closing it again. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] incubator-aurora pull request: Aurora 475 Remove Copyright Apache ...

2015-01-06 Thread davelester
Github user davelester commented on the pull request: https://github.com/apache/incubator-aurora/pull/10#issuecomment-68980431 @hsaputra thanks for the contribution! As @Yasumoto noted, this was reviewed on Review Board and submitted in May. Could you close this pull request? --- If

[GitHub] incubator-aurora pull request: Use the correct IP for the schedule...

2015-01-06 Thread davelester
Github user davelester commented on the pull request: https://github.com/apache/incubator-aurora/pull/5#issuecomment-68980345 Hi @brianm, thanks for your contribution! It looks like your Review Board patch was previously submitted. https://reviews.apache.org/r/18306/ Could you close t

[GitHub] incubator-aurora pull request: Correct python executable name

2015-01-06 Thread davelester
Github user davelester commented on the pull request: https://github.com/apache/incubator-aurora/pull/6#issuecomment-68980332 Hi @brianm, thanks for your contribution! It looks like your Review Board patch was previously submitted. https://reviews.apache.org/r/18306/ Could you close t

Re: Should we continue to maintain fine-grained python BUILD targets?

2015-01-06 Thread Zameer Manji
If we decide to simplify the targets then I'm also in favor of going all in and removing pants in favour of setup.py and tox. On Tue, Jan 6, 2015 at 2:43 PM, Brian Wickman wrote: > 4) Enforcing sensible software hygiene. > > The pants project itself for the longest time had a single root rglobs

Jenkins build is back to normal : Aurora #805

2015-01-06 Thread Apache Jenkins Server
See

Re: Should we continue to maintain fine-grained python BUILD targets?

2015-01-06 Thread Brian Wickman
4) Enforcing sensible software hygiene. The pants project itself for the longest time had a single root rglobs python_library target. I spent a few days splitting up that project into fine-grained targets and it uncovered a number of cyclical dependencies and abstraction leaks that simply would'v

Should we continue to maintain fine-grained python BUILD targets?

2015-01-06 Thread Bill Farner
We currently have ~111 BUILD targets serving our python code, and ~106 test targets: $ find . -name BUILD | xargs grep python_library | wc -l 111 $ find . -name BUILD | xargs grep python_test | wc -l 106 This presents a non-trivial maintenance burden, and i'm not convinced the benefits are worth

Build failed in Jenkins: Aurora #804

2015-01-06 Thread Apache Jenkins Server
See Changes: [wfarner] Begin dismantling client v1. -- [...truncated 2823 lines...] Running setup.py (path:

Re: Build failed in Jenkins: Aurora #803

2015-01-06 Thread Bill Farner
This was an artifact fetch failure: Exception message: Failed to fetch https://pypi.python.org/packages/source/w/wheel/wheel-0.23.0.tar.gz#md5=273374d28127b821d61cdfe56e7c35dd to /tmp/user/2395/tmp5mleoN: Could not reach https://pypi.python.org/packages/source/w/wheel/wheel-0.23.0.tar.gz#md5=27337

[GitHub] incubator-aurora pull request: [AURORA-986] RequestLoggerTest fail...

2015-01-06 Thread ielatif
Github user ielatif closed the pull request at: https://github.com/apache/incubator-aurora/pull/14 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featu

Build failed in Jenkins: Aurora #803

2015-01-06 Thread Apache Jenkins Server
See Changes: [wfarner] Remove SANDBOX_DELETED state. -- [...truncated 2781 lines...] Downloading/unpacking ansicolors==1.0.2 (from -r

[GitHub] incubator-aurora pull request: [AURORA-986] RequestLoggerTest fail...

2015-01-06 Thread davelester
Github user davelester commented on the pull request: https://github.com/apache/incubator-aurora/pull/14#issuecomment-68925588 Thanks for this contribution! Since this contribution has been reviewed and committed review board [1], could you close this pull request? _ [1] https:

Build failed in Jenkins: Aurora #802

2015-01-06 Thread Apache Jenkins Server
See Changes: [kevints] [AURORA-986] RequestLoggerTest failed for Locale.FRANCE -- [...truncated 2825 lines...] Downloading/unpacking twitter.common.config==0.3.1 (from -r