[GitHub] mesos issue #284: Remove accent from first name of clems4ever contributor.

2018-04-20 Thread bbannier
Github user bbannier commented on the issue: https://github.com/apache/mesos/pull/284 @clems4ever I don't believe fixing that database is going to make `support/apply_reviews.py` work. I think you need to reconfigure the name you use to create git commit you put onto reviewboard

[GitHub] mesos pull request #264: WIP: docker image build automation

2018-02-12 Thread bbannier
Github user bbannier closed the pull request at: https://github.com/apache/mesos/pull/264 ---

[GitHub] mesos pull request #264: WIP: docker image build automation

2018-02-12 Thread bbannier
GitHub user bbannier opened a pull request: https://github.com/apache/mesos/pull/264 WIP: docker image build automation You can merge this pull request into a Git repository by running: $ git pull https://github.com/bbannier/mesos t/docker_image_build Alternatively you can

[GitHub] mesos pull request #256: Correct spelling errors in the docs

2018-01-12 Thread bbannier
Github user bbannier commented on a diff in the pull request: https://github.com/apache/mesos/pull/256#discussion_r161343683 --- Diff: docs/endpoints/files/browse.json.md --- @@ -20,7 +20,7 @@ Query parameters: ### AUTHENTICATION ### -This endpoint requires

[GitHub] mesos pull request #223: fixes broken link in docs

2017-07-10 Thread bbannier
Github user bbannier commented on a diff in the pull request: https://github.com/apache/mesos/pull/223#discussion_r126453567 --- Diff: docs/containerizer-internals.md --- @@ -32,7 +32,7 @@ Mesos currently supports the following containerizers: * Composing * [Docker

[GitHub] mesos issue #223: fixes broken link in docs

2017-07-10 Thread bbannier
Github user bbannier commented on the issue: https://github.com/apache/mesos/pull/223 Thanks for the patch @conradkdotcom! Grepping over docs tree I see there are in total four mentions of `(containerizer.md)`. It would be great if you could clean up all of them in one swoop

[GitHub] mesos issue #218: MESOS-7654 Fix owner of fetched files in docker containeri...

2017-06-13 Thread bbannier
Github user bbannier commented on the issue: https://github.com/apache/mesos/pull/218 For code changes like this one we prefer working with our JIRA and reviewboard, https://mesos.apache.org/documentation/latest/submitting-a-patch/. Could you submit a review request to reviewboard

[GitHub] mesos pull request #:

2016-10-04 Thread bbannier
Github user bbannier commented on the pull request: https://github.com/apache/mesos/commit/498d14e934233e4501597b43da3924bfe8b2de20#commitcomment-19287295 In src/slave/containerizer/mesos/containerizer.cpp: In src/slave/containerizer/mesos/containerizer.cpp on line 1945

[GitHub] mesos issue #118: Make Mesos healtchecks compatible with Marathon

2016-07-07 Thread bbannier
Github user bbannier commented on the issue: https://github.com/apache/mesos/pull/118 Could you please move this to reviewboard so we got this growing discussion in the expected place? @drexin @haosdent @brat002 @ConnorDoyle --- If your project is set up for it, you can reply