[GitHub] mesos pull request: Upgrade zookeeper to 3.4.8.

2016-04-02 Thread vinodkone
Github user vinodkone commented on a diff in the pull request: https://github.com/apache/mesos/pull/93#discussion_r58094904 --- Diff: 3rdparty/Makefile.am --- @@ -51,7 +51,7 @@ EXTRA_DIST = \ EXTRA_DIST += \ $(LEVELDB).patch -# We

[GitHub] mesos pull request: Upgrade zookeeper to 3.4.8.

2016-04-02 Thread vinodkone
Github user vinodkone commented on the pull request: https://github.com/apache/mesos/pull/93#issuecomment-204558214 libev is committed. You can rebase it now. --- 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

[GitHub] mesos issue #36: replace unsafe "find | xargs" with "find -exec"

2016-08-02 Thread vinodkone
Github user vinodkone commented on the issue: https://github.com/apache/mesos/pull/36 I think this broke the ReviewBot. See https://issues.apache.org/jira/browse/MESOS-5958 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] mesos pull request #168: Update contributors.yaml for gradywang.

2016-09-29 Thread vinodkone
Github user vinodkone commented on a diff in the pull request: https://github.com/apache/mesos/pull/168#discussion_r81236751 --- Diff: docs/contributors.yaml --- @@ -402,13 +402,14 @@ jira_user: yanyanhu reviewboard_user: yanyanhu -- name: Yong Qiao Wang

[GitHub] mesos pull request #168: Update contributors.yaml for gradywang.

2016-10-05 Thread vinodkone
Github user vinodkone commented on a diff in the pull request: https://github.com/apache/mesos/pull/168#discussion_r82063596 --- Diff: docs/contributors.yaml --- @@ -402,13 +402,14 @@ jira_user: yanyanhu reviewboard_user: yanyanhu -- name: Yong Qiao Wang

[GitHub] mesos pull request #168: Update contributors.yaml for gradywang.

2016-09-19 Thread vinodkone
Github user vinodkone commented on a diff in the pull request: https://github.com/apache/mesos/pull/168#discussion_r79516117 --- Diff: docs/contributors.yaml --- @@ -402,13 +402,13 @@ jira_user: yanyanhu reviewboard_user: yanyanhu -- name: Yong Qiao Wang

[GitHub] mesos issue #171: Update cli.py

2016-10-18 Thread vinodkone
Github user vinodkone commented on the issue: https://github.com/apache/mesos/pull/171 Can you close this in favor of the review request? --- 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

[GitHub] mesos pull request #:

2016-10-10 Thread vinodkone
Github user vinodkone commented on the pull request: https://github.com/apache/mesos/commit/aeaec8940c7dd8e17e379fc7aa03fbce60d77b20#commitcomment-19363756 @kamilchm can you file a ticket on issues.apache.org? --- If your project is set up for it, you can reply to this email

[GitHub] mesos issue #190: Ensure curl is present on Ubuntu

2016-11-29 Thread vinodkone
Github user vinodkone commented on the issue: https://github.com/apache/mesos/pull/190 Thanks for the PR. @jieyu what's our recommendation regarding curl dependency? I remember there was a discussion in the mailing list sometime back. --- If your project is set up

[GitHub] mesos issue #205: Clarify existence of FrameworkID in SUBSCRIBE calls.

2017-06-13 Thread vinodkone
Github user vinodkone commented on the issue: https://github.com/apache/mesos/pull/205 @hatred still planning to review this? --- 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

[GitHub] mesos pull request #247: Fixed reference to Mesos paper in the documentation...

2017-11-02 Thread vinodkone
Github user vinodkone commented on a diff in the pull request: https://github.com/apache/mesos/pull/247#discussion_r148613763 --- Diff: docs/architecture.md --- @@ -34,4 +34,4 @@ In addition, this resource offer process repeats when tasks finish and new resou While

[GitHub] mesos pull request #247: Fixed reference to Mesos paper in the documentation...

2017-11-02 Thread vinodkone
Github user vinodkone commented on a diff in the pull request: https://github.com/apache/mesos/pull/247#discussion_r148613391 --- Diff: docs/architecture.md --- @@ -34,4 +34,4 @@ In addition, this resource offer process repeats when tasks finish and new resou While

[GitHub] mesos pull request #294: FaultDomain, conventions for additional hierarchy.

2018-05-31 Thread vinodkone
Github user vinodkone commented on a diff in the pull request: https://github.com/apache/mesos/pull/294#discussion_r192161678 --- Diff: include/mesos/mesos.proto --- @@ -827,6 +827,18 @@ message ExecutorInfo { * in the same zone, in different zones in the same region

[GitHub] mesos issue #216: 1.3.x

2018-01-08 Thread vinodkone
Github user vinodkone commented on the issue: https://github.com/apache/mesos/pull/216 @shendabin Can you please close/delete this PR? ---

[GitHub] mesos issue #213: Update vendored protobuf tar.gz to 3.2.0.

2018-01-18 Thread vinodkone
Github user vinodkone commented on the issue: https://github.com/apache/mesos/pull/213 @zhitaoli Can you close this? Looks we are at protobuf 3.5 already in the repo. ---

[GitHub] mesos issue #255: Add curl examples to operator-http-api

2018-01-18 Thread vinodkone
Github user vinodkone commented on the issue: https://github.com/apache/mesos/pull/255 @nhandler If you can update the PR, I'll commit it ASAP. ---

[GitHub] mesos issue #249: Update presentations.md

2018-01-18 Thread vinodkone
Github user vinodkone commented on the issue: https://github.com/apache/mesos/pull/249 @packtpartner Can you close this please? ---

[GitHub] mesos issue #258: Update api-client-libraries.md

2018-01-18 Thread vinodkone
Github user vinodkone commented on the issue: https://github.com/apache/mesos/pull/258 Thanks for the submission! Are you planning to use this in production? ---

[GitHub] mesos issue #255: Add curl examples to operator-http-api

2018-01-12 Thread vinodkone
Github user vinodkone commented on the issue: https://github.com/apache/mesos/pull/255 @nhandler Thanks for doing this! Awesome. Do I take it that you tested all these commands? Also, is the EOF required in all the curl commands? ---

[GitHub] mesos issue #262: Updating reservation.md

2018-02-05 Thread vinodkone
Github user vinodkone commented on the issue: https://github.com/apache/mesos/pull/262 @mpark @bmahler Can you guys review? ---

[GitHub] mesos pull request #301: Document SUPPRESS HTTP call [MESOS-7211]

2018-07-20 Thread vinodkone
Github user vinodkone commented on a diff in the pull request: https://github.com/apache/mesos/pull/301#discussion_r204166164 --- Diff: docs/scheduler-http-api.md --- @@ -479,6 +479,32 @@ HTTP/1.1 202 Accepted ``` +### SUPPRESS +Sent by the scheduler when

[GitHub] mesos pull request #301: Document SUPPRESS HTTP call [MESOS-7211]

2018-07-20 Thread vinodkone
Github user vinodkone commented on a diff in the pull request: https://github.com/apache/mesos/pull/301#discussion_r204167169 --- Diff: docs/scheduler-http-api.md --- @@ -479,6 +479,32 @@ HTTP/1.1 202 Accepted ``` +### SUPPRESS +Sent by the scheduler when

[GitHub] mesos pull request #301: Document SUPPRESS HTTP call [MESOS-7211]

2018-07-10 Thread vinodkone
Github user vinodkone commented on a diff in the pull request: https://github.com/apache/mesos/pull/301#discussion_r201407700 --- Diff: docs/scheduler-http-api.md --- @@ -128,6 +128,26 @@ TEARDOWN Response: HTTP/1.1 202 Accepted ``` +### SUPPRESS +Sent

[GitHub] mesos pull request #301: Document SUPPRESS HTTP call [MESOS-7211]

2018-07-10 Thread vinodkone
Github user vinodkone commented on a diff in the pull request: https://github.com/apache/mesos/pull/301#discussion_r201401610 --- Diff: docs/scheduler-http-api.md --- @@ -128,6 +128,26 @@ TEARDOWN Response: HTTP/1.1 202 Accepted ``` +### SUPPRESS --- End

[GitHub] mesos pull request #301: Document SUPPRESS HTTP call [MESOS-7211]

2018-07-10 Thread vinodkone
Github user vinodkone commented on a diff in the pull request: https://github.com/apache/mesos/pull/301#discussion_r201406742 --- Diff: docs/scheduler-http-api.md --- @@ -128,6 +128,26 @@ TEARDOWN Response: HTTP/1.1 202 Accepted ``` +### SUPPRESS +Sent

[GitHub] mesos pull request #301: Document SUPPRESS HTTP call [MESOS-7211]

2018-07-10 Thread vinodkone
Github user vinodkone commented on a diff in the pull request: https://github.com/apache/mesos/pull/301#discussion_r201406612 --- Diff: docs/scheduler-http-api.md --- @@ -128,6 +128,26 @@ TEARDOWN Response: HTTP/1.1 202 Accepted ``` +### SUPPRESS +Sent