Re: Review Request 46936: Documented the agent endpoint '/flags'.

2016-05-09 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46936/#review132274 --- Ship it! I'll resolve outstading issues and commit this for

Re: Review Request 46936: Documented the agent endpoint '/flags'.

2016-05-09 Thread Alexander Rukletsov
> On May 5, 2016, 4:15 p.m., Neil Conway wrote: > > src/slave/http.cpp, lines 356-357 > > > > > > +1, although I'd say "The current principal should be ..." I'm ESL, but for me "current" in this context sounds like

Re: Review Request 46936: Documented the agent endpoint '/flags'.

2016-05-05 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46936/#review131848 --- docs/endpoints/slave/flags.md (line 15)

Re: Review Request 46936: Documented the agent endpoint '/flags'.

2016-05-04 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46936/#review131739 --- src/slave/http.cpp (lines 356 - 357)

Re: Review Request 46936: Documented the agent endpoint '/flags'.

2016-05-03 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46936/#review131519 --- Patch looks great! Reviews applied: [46935, 46936] Passed

Review Request 46936: Documented the agent endpoint '/flags'.

2016-05-03 Thread Jan Schlicht
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46936/ --- Review request for mesos, Alexander Rukletsov, Alexander Rojas, and Neil Conway.