Re: Review Request 46319: Added authorization to agents' `/monitor/statistics` endpoints.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46319/#review130549 --- Fix it, then Ship it! Only minor nits left beyond my two remain

Re: Review Request 46319: Added authorization to agents' `/monitor/statistics` endpoints.

> On April 22, 2016, 1:22 a.m., Adam B wrote: > > src/slave/slave.hpp, lines 471-472 > > > > > > How did this come up? The original `_statistics()` is not static, and > > it had no issues. > > Benjamin Bannier wrot

Re: Review Request 46319: Added authorization to agents' `/monitor/statistics` endpoints.

> On April 22, 2016, 1:22 a.m., Adam B wrote: > > src/tests/slave_tests.cpp, line 1895 > > > > > > Should we create a TYPED_TEST that tests this ACL in the local > > authorizer (direct and as a module), or do we on

Re: Review Request 46203: Added authorization of the '/flags' endpoint.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46203/#review130538 --- Looks great! I think we just need to pass the GET/POST verb into

Review Request 46676: Slave/Agent Rename Phase I: Rename '/include/mesos/slave' folder.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46676/ --- Review request for mesos, Kevin Klues and Vinod Kone. Bugs: mesos-5230 http

Re: Review Request 46220: Added documentation for Nvidia GPU support.

> On April 22, 2016, 5:55 p.m., Qian Zhang wrote: > > docs/gpu-support.md, line 440 > > > > > > Does this limitation mean that currently we do not support container > > with an image (e.g., Docker image, Appc image)

Re: Review Request 46501: Updated authorization.md to reflect current changes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46501/#review130540 --- docs/authorization.md (line 12)

Re: Review Request 46618: Enhanced the log message when there are duplicate volumes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46618/#review130543 --- Ship it! Ship It! - Gilbert Song On April 25, 2016, 10:56 a.

Re: Review Request 46670: Added deprecated alias for `--authenticate_frameworks` master flag.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46670/ --- (Updated April 26, 2016, 2:24 a.m.) Review request for mesos and Ben Mahler.

Re: Review Request 46670: Added deprecated alias for `--authenticate_frameworks` master flag.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46670/#review130536 --- Bad review! Reviews applied: [] Error: No reviewers specified. P

Re: Review Request 46370: Introduced linux capabilities API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46370/ --- (Updated April 26, 2016, 1:23 a.m.) Review request for mesos, Jie Yu and Kevin

Review Request 46670: Added deprecated alias for `--authenticate_frameworks` master flag.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46670/ --- Review request for mesos. Bugs: MESOS-4386 https://issues.apache.org/jira/b

Review Request 46669: Added deprecation support to Flag name.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46669/ --- Review request for mesos and Ben Mahler. Bugs: MESOS-5271 https://issues.ap

Re: Review Request 46621: Added alias support for flags.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46621/ --- (Updated April 26, 2016, 1:04 a.m.) Review request for mesos and Ben Mahler.

Re: Review Request 46621: Add alias support for flags.

> On April 25, 2016, 2:48 p.m., Benjamin Bannier wrote: > > Did a quick review since I spent some time on working on a fix for > > MESOS-3335. It would be great if we wouldn't aggreviate that problem > > further. > > > > What seems unclear to me ATM is how alias'ed flags can override each othe

Re: Review Request 46588: Added URI struct to stout.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46588/ --- (Updated April 25, 2016, 5:59 p.m.) Review request for mesos, Gilbert Song, Art

Re: Review Request 46620: Refactored Flags to store name in `Name` struct instead of string.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46620/ --- (Updated April 26, 2016, 12:54 a.m.) Review request for mesos and Ben Mahler.

Re: Review Request 46620: Refactored Flags to store name in `Name` struct instead of string.

> On April 25, 2016, 2:48 p.m., Benjamin Bannier wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/flags/flag.hpp, line 45 > > > > > > If this was `const` it would be much easier to reason how this works >

Re: Review Request 45958: Updated protobuf Resource to mark the resource as shareable.

> On April 18, 2016, 6:47 a.m., Jiang Yan Xu wrote: > > include/mesos/mesos.proto, line 762 > > > > > > How about `optional SharedInfo shared`? > > > > [shareable](http://www.merriam-webster.com/dictionary/s

Re: Review Request 46665: Added 0.26.2 to the CHANGELOG.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46665/#review130527 --- Patch looks great! Reviews applied: [46663, 46664, 46665] Passed

Re: Review Request 46665: Added 0.26.2 to the CHANGELOG.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46665/#review130519 --- Ship it! Ship It! - Vinod Kone On April 25, 2016, 11:08 p.m.

Re: Review Request 46664: Added 0.27.3 to the CHANGELOG.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46664/#review130518 --- Ship it! Ship It! - Vinod Kone On April 25, 2016, 11:08 p.m.

Re: Review Request 46663: Added 0.28.2 to the CHANGELOG.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46663/#review130517 --- Ship it! Ship It! - Vinod Kone On April 25, 2016, 11:08 p.m.

Re: Review Request 46580: Added uriparser as a bundled dependency.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46580/ --- (Updated April 25, 2016, 4:12 p.m.) Review request for mesos, Gilbert Song, Art

Re: Review Request 46588: Added URI struct to stout.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46588/ --- (Updated April 25, 2016, 4:13 p.m.) Review request for mesos, Gilbert Song, Art

Review Request 46663: Added 0.28.2 to the CHANGELOG.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46663/ --- Review request for mesos and Vinod Kone. Repository: mesos Description --

Review Request 46664: Added 0.27.3 to the CHANGELOG.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46664/ --- Review request for mesos and Vinod Kone. Repository: mesos Description --

Review Request 46665: Added 0.26.2 to the CHANGELOG.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46665/ --- Review request for mesos and Vinod Kone. Repository: mesos Description --

Re: Review Request 40266: Libprocess Reinit: Cleanup SocketManager alongside ProcessManager.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40266/#review130502 --- Some inline TODO's from an offline review (with Joris & Neil). 3

Re: Review Request 45668: Enable CMake build for Linux as a BUILDTOOL option.

> On April 18, 2016, 6:18 p.m., Vinod Kone wrote: > > support/docker_build.sh, lines 127-140 > > > > > > Hmm. I was hoping for something more generic than hard coding these 3 > > configurations. But if it's not ea

Re: Review Request 46187: Terminate when receiving the ACK of terminal status update.

> On April 14, 2016, 4:49 p.m., Vinod Kone wrote: > > src/launcher/http_command_executor.cpp, line 749 > > > > > > Looking at slave::statusUpdate() code there are several scenarios where > > the slave ignores a stat

Re: Review Request 44379: Use tokens size to parse perf stat format.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44379/#review130504 --- Ship it! Thanks for the comment! I've added a commit descripti

Re: Review Request 46641: Added tests for the Profiler to libprocess.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46641/ --- (Updated April 25, 2016, 8:47 p.m.) Review request for mesos, Alexander Rojas a

Re: Review Request 46260: Added a MetricsTest with authentication to libprocess.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46260/ --- (Updated April 25, 2016, 8:47 p.m.) Review request for mesos, Adam B and Alexan

Re: Review Request 40266: Libprocess Reinit: Cleanup SocketManager alongside ProcessManager.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40266/#review130501 --- 3rdparty/libprocess/src/process.cpp (line 1356)

Re: Review Request 40512: Libprocess Reinit: Add a test-only method to reinitialize libprocess.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40512/#review130500 --- 3rdparty/libprocess/src/process.cpp (line 475)

Re: Review Request 46187: Terminate when receiving the ACK of terminal status update.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46187/ --- (Updated April 25, 2016, 8:34 p.m.) Review request for mesos, Anand Mazumdar an

Re: Review Request 46618: Enhanced the log message when there are duplicate volumes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46618/#review130497 --- Patch looks great! Reviews applied: [46618] Passed command: expo

Re: Review Request 42806: Added the fetcher plugin module interface.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42806/#review130490 --- FYI: I will be making some changes to the fetcher underneath your

Re: Review Request 46647: Added the '--enable-perftools' flag to configure docs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46647/#review130482 --- Ship it! Ship It! - Kapil Arya On April 25, 2016, 1:45 p.m.,

Re: Review Request 46643: Fixed the build when perftools is enabled.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46643/#review130481 --- Ship it! Ship It! - Kapil Arya On April 25, 2016, 1:19 p.m.,

Re: Review Request 46618: Enhanced the log message when there are duplicate volumes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46618/ --- (Updated 四月 25, 2016, 5:56 p.m.) Review request for mesos, Gilbert Song and Jie

Re: Review Request 46618: Enhanced the log message when there are duplicate volumes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46618/ --- (Updated 四月 25, 2016, 5:56 p.m.) Review request for mesos, Gilbert Song and Jie

Review Request 46647: Added the '--enable-perftools' flag to configure docs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46647/ --- Review request for mesos and Kapil Arya. Bugs: MESOS-3319 https://issues.ap

Re: Review Request 46261: Added new MetricsTests with authentication to Mesos tests.

> On April 25, 2016, 3:32 p.m., Kapil Arya wrote: > > src/tests/metrics_tests.cpp, line 85 > > > > > > Newline? Good catch! Fixed. - Greg --- This is an aut

Re: Review Request 46261: Added new MetricsTests with authentication to Mesos tests.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46261/ --- (Updated April 25, 2016, 5:37 p.m.) Review request for mesos, Adam B and Alexan

Review Request 46643: Fixed the build when perftools is enabled.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46643/ --- Review request for mesos and Kapil Arya. Bugs: MESOS-3319 https://issues.ap

Re: Review Request 46461: Updated gperftools to version 2.5 (libprocess).

> On April 25, 2016, 4:01 p.m., Kapil Arya wrote: > > 3rdparty/libprocess/configure.ac, line 853 > > > > > > I am wondering if we should split out this patch into two: one that > > updates bundled version, and secon

Re: Review Request 46569: Parameterized agent endpoint authorization tests on tested endpoint.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46569/#review130452 --- Patch looks great! Reviews applied: [45922, 46318, 46203, 46319,

Re: Review Request 46462: Updated gperftools to version 2.5 (Mesos).

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46462/ --- (Updated April 25, 2016, 5:26 p.m.) Review request for mesos and Kapil Arya.

Re: Review Request 46461: Updated gperftools to version 2.5 (libprocess).

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46461/ --- (Updated April 25, 2016, 5:25 p.m.) Review request for mesos and Kapil Arya.

Re: Review Request 46618: Enhanced the log message when there are duplicate volumes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46618/#review130445 --- src/slave/containerizer/mesos/isolators/docker/volume/isolator.cp

Review Request 46641: Added tests for the Profiler to libprocess.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46641/ --- Review request for mesos, Alexander Rojas and Kapil Arya. Bugs: MESOS-4902

Re: Review Request 46497: Added authentication to the '/profiler/*' endpoints.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46497/#review130438 --- Patch looks great! Reviews applied: [46254, 46255, 46258, 46259,

Re: Review Request 45958: Updated protobuf Resource to mark the resource as shareable.

> On April 18, 2016, 6:47 a.m., Jiang Yan Xu wrote: > > include/mesos/mesos.proto, line 762 > > > > > > How about `optional SharedInfo shared`? > > > > [shareable](http://www.merriam-webster.com/dictionary/s

Re: Review Request 46462: Updated gperftools to version 2.5 (Mesos).

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46462/#review130437 --- Ship it! Ship It! - Kapil Arya On April 21, 2016, 12:22 p.m.

Re: Review Request 46461: Updated gperftools to version 2.5 (libprocess).

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46461/#review130436 --- LGTM modulo the comment. Also, have you done a grep for HAS_GPERFT

Re: Review Request 46497: Added authentication to the '/profiler/*' endpoints.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46497/#review130429 --- Ship it! Ship It! - Kapil Arya On April 25, 2016, 10:02 a.m.

Re: Review Request 46262: Added a LoggingTest with authentication.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46262/#review130427 --- Ship it! Ship It! - Kapil Arya On April 15, 2016, 3:01 a.m.,

Re: Review Request 46260: Added a MetricsTest with authentication to libprocess.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46260/#review130426 --- Ship it! Ship It! - Kapil Arya On April 22, 2016, 3:14 p.m.,

Re: Review Request 46261: Added new MetricsTests with authentication to Mesos tests.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46261/#review130425 --- Ship it! src/tests/metrics_tests.cpp (line 85)

Re: Review Request 46259: Added authentication to `/metrics/snapshot` endpoint.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46259/#review130423 --- Ship it! Ship It! - Kapil Arya On April 25, 2016, 10:21 a.m.

Re: Review Request 46621: Add alias support for flags.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46621/#review130421 --- Did a quick review since I spent some time on working on a fix for

Re: Review Request 46620: Refactored Flags to store name in `Name` struct instead of string.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46620/#review130422 --- 3rdparty/libprocess/3rdparty/stout/include/stout/flags/flag.hpp (

Re: Review Request 45922: Added agent authorization flags.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45922/#review130419 --- Ship it! Ship It! - Alexander Rojas On April 22, 2016, 12:11

Re: Review Request 46593: Added test for containerizer destroy while provisioning race.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46593/#review130418 --- Fix it, then Ship it! src/tests/containerizer/mesos_containeri

Re: Review Request 46259: Added authentication to `/metrics/snapshot` endpoint.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46259/ --- (Updated April 25, 2016, 2:21 p.m.) Review request for mesos, Adam B and Alexan

Re: Review Request 46258: Added authentication to `/logging/toggle` endpoint.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46258/ --- (Updated April 25, 2016, 2:20 p.m.) Review request for mesos, Adam B and Alexan

Re: Review Request 46319: Added authorization to agents' `/monitor/statistics` endpoints.

> On April 22, 2016, 10:22 a.m., Adam B wrote: > > src/tests/slave_tests.cpp, lines 1926-1927 > > > > > > I would expect you to await/validate the response after validating the > > request. > > Benjamin Bannier wr

Re: Review Request 46569: Parameterized agent endpoint authorization tests on tested endpoint.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46569/ --- (Updated April 25, 2016, 4:16 p.m.) Review request for mesos and Adam B. Chan

Re: Review Request 46319: Added authorization to agents' `/monitor/statistics` endpoints.

> On April 25, 2016, 12:57 p.m., Alexander Rojas wrote: > > src/slave/http.cpp, lines 647-649 > > > > > > I was thinking that it makes sense to pull up the limiter related logic > > to `statistics()` so that method

Re: Review Request 46319: Added authorization to agents' `/monitor/statistics` endpoints.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46319/ --- (Updated April 25, 2016, 4:16 p.m.) Review request for mesos, Adam B, Alexander

Re: Review Request 46569: Parameterized agent endpoint authorization tests on tested endpoint.

> On April 25, 2016, 1:31 p.m., Adam B wrote: > > src/tests/slave_authorization_tests.cpp, line 176 > > > > > > Shouldn't you give this a more generic name now that it can handle any > > (agent) endpoint? Done. -

Re: Review Request 46497: Added authentication to the '/profiler/*' endpoints.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46497/#review130415 --- Ship it! Ship It! - Alexander Rojas On April 25, 2016, 4:02

Re: Review Request 46497: Added authentication to the '/profiler/*' endpoints.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46497/ --- (Updated April 25, 2016, 2:02 p.m.) Review request for mesos, Alexander Rojas a

Re: Review Request 46497: Added authentication to the '/profiler/*' endpoints.

> On April 25, 2016, 12:37 p.m., Alexander Rojas wrote: > > Whoops! Thanks Alexander :-) - Greg --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46497/#review130395 ---

Re: Review Request 46203: Added authorization of the '/flags' endpoint.

> On April 20, 2016, 10:35 a.m., Adam B wrote: > > src/slave/http.cpp, lines 658-660 > > > > > > Where did you come up with the magic number 3? What if we reorganize > > the operator endpoints in the (1.0) future? H

Re: Review Request 46203: Added authorization of the '/flags' endpoint.

> On April 20, 2016, 10:35 a.m., Adam B wrote: > > src/slave/http.cpp, lines 658-660 > > > > > > Where did you come up with the magic number 3? What if we reorganize > > the operator endpoints in the (1.0) future? H

Re: Review Request 46203: Added authorization of the '/flags' endpoint.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46203/ --- (Updated April 25, 2016, 2:50 p.m.) Review request for mesos, Adam B, Alexander

Re: Review Request 46203: Added authorization of the '/flags' endpoint.

> On April 20, 2016, 10:35 a.m., Adam B wrote: > > src/slave/http.cpp, lines 658-660 > > > > > > Where did you come up with the magic number 3? What if we reorganize > > the operator endpoints in the (1.0) future? H

Re: Review Request 46497: Added authentication to the '/profiler/*' endpoints.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46497/#review130395 --- 3rdparty/libprocess/include/process/profiler.hpp (line 71)

Re: Review Request 46264: Fixed a typo in docker_containerizer_tests.cpp.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46264/#review130393 --- Patch looks great! Reviews applied: [46264] Passed command: expo

Re: Review Request 46569: Parameterized agent endpoint authorization tests on tested endpoint.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46569/#review130391 --- Fix it, then Ship it! Looks like a great generalization! What m

Re: Review Request 46139: Add positive tests for /weights endpoint.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46139/#review130390 --- Ship it! Ship It! - Adam B On April 19, 2016, 6:42 a.m., Yon

Re: Review Request 46319: Added authorization to agents' `/monitor/statistics` endpoints.

> On April 22, 2016, 1:22 a.m., Adam B wrote: > > src/tests/slave_tests.cpp, lines 1926-1927 > > > > > > I would expect you to await/validate the response after validating the > > request. > > Benjamin Bannier wro

Re: Review Request 46203: Added authorization of the '/flags' endpoint.

> On April 20, 2016, 1:35 a.m., Adam B wrote: > > include/mesos/authorizer/acls.proto, line 151 > > > > > > Let's consider calling this `GetEndpoint`, to match the HTTP verb? > > There may be some users that are all

Re: Review Request 46203: Added authorization of the '/flags' endpoint.

> On April 20, 2016, 1:35 a.m., Adam B wrote: > > src/tests/slave_authorization_tests.cpp, lines 110-114 > > > > > > This seems wrong. You don't even bother to reset the authenticator > > after you're done? > >

Re: Review Request 46318: Added helper to create test agent with injected `Authorizer`.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46318/#review130386 --- Ship it! Ship It! - Adam B On April 25, 2016, 1:19 a.m., Ben

Re: Review Request 46319: Added authorization to agents' `/monitor/statistics` endpoints.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46319/#review130385 --- src/slave/http.cpp (lines 647 - 649)

Re: Review Request 45922: Added agent authorization flags.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45922/#review130384 --- Ship it! Ship It! - Adam B On April 22, 2016, 3:11 a.m., Jan

Re: Review Request 46318: Added helper to create test agent with injected `Authorizer`.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46318/#review130383 --- Ship it! Ship It! - Alexander Rojas On April 25, 2016, 10:19

Re: Review Request 46203: Added authorization of the '/flags' endpoint.

> On April 20, 2016, 10:35 a.m., Adam B wrote: > > src/slave/http.cpp, lines 658-660 > > > > > > Where did you come up with the magic number 3? What if we reorganize > > the operator endpoints in the (1.0) future? H

Re: Review Request 46564: Fixed a typo in Docker containerizer.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46564/#review130378 --- Patch looks great! Reviews applied: [46564] Passed command: expo

Re: Review Request 46569: Parameterized agent endpoint authorization tests on tested endpoint.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46569/ --- (Updated April 25, 2016, 11:52 a.m.) Review request for mesos and Adam B. Cha

Re: Review Request 46319: Added authorization to agents' `/monitor/statistics` endpoints.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46319/ --- (Updated April 25, 2016, 11:52 a.m.) Review request for mesos, Adam B, Alexande

Re: Review Request 46203: Added authorization of the '/flags' endpoint.

> On April 20, 2016, 10:35 a.m., Adam B wrote: > > src/slave/http.cpp, lines 658-660 > > > > > > Where did you come up with the magic number 3? What if we reorganize > > the operator endpoints in the (1.0) future? H

Re: Review Request 46203: Added authorization of the '/flags' endpoint.

> On April 20, 2016, 10:35 a.m., Adam B wrote: > > src/slave/http.cpp, lines 658-660 > > > > > > Where did you come up with the magic number 3? What if we reorganize > > the operator endpoints in the (1.0) future? H

Re: Review Request 46203: Added authorization of the '/flags' endpoint.

> On April 20, 2016, 10:35 a.m., Adam B wrote: > > src/slave/http.cpp, lines 658-660 > > > > > > Where did you come up with the magic number 3? What if we reorganize > > the operator endpoints in the (1.0) future? H

Re: Review Request 46203: Added authorization of the '/flags' endpoint.

> On April 20, 2016, 10:35 a.m., Adam B wrote: > > docs/configuration.md, line 897 > > > > > > We're going to have to start documenting which endpoints can/must be > > authorized this way, similar to how Joerg added

Re: Review Request 46624: Added support to mesos to work with flag alias.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46624/#review130368 --- Patch looks great! Reviews applied: [46620, 46621, 46622, 46623,

  1   2   >