[GitHub] incubator-aurora pull request: add mesos role feature

2015-02-05 Thread davelester
Github user davelester commented on the pull request: https://github.com/apache/incubator-aurora/pull/16#issuecomment-73168745 Thanks for this patch! Would you mind moving this patch to ReviewBoard? You'll find contributions guidelines in the Aurora documentation:

[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

[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

[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

[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

[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

[GitHub] incubator-aurora pull request: Added a dummy Tutorial heading

2014-12-13 Thread davelester
Github user davelester commented on the pull request: https://github.com/apache/incubator-aurora/pull/3#issuecomment-66905277 Hey @mgargenta, can you close this pull request since it was originally meant as a test? The vagrant tutorial is currently linked within the docs README