On May 28, 2015, 10:40 a.m., Adam B wrote:
3rdparty/libprocess/src/help.cpp, line 140
https://reviews.apache.org/r/34655/diff/1/?file=971486#file971486line140
Why does this '/help/' need to be removed?
Marco Massenzio wrote:
because it's wrong - it shouldn't be there.
The only help documentation I know of is what I found in process/help.hpp
https://github.com/apache/mesos/blob/0.22.1/3rdparty/libprocess/include/process/help.hpp#L93
I couldn't find an existing JIRA for updating the missing help pages, but I
did find: MESOS-1267
On May 28, 2015, 5:40 p.m., Adam B wrote:
3rdparty/libprocess/src/help.cpp, line 140
https://reviews.apache.org/r/34655/diff/1/?file=971486#file971486line140
Why does this '/help/' need to be removed?
Marco Massenzio wrote:
because it's wrong - it shouldn't be there.
Adam
On May 28, 2015, 10:40 a.m., Adam B wrote:
Do you clearly understand why this change is needed? I didn't understand
after just reading the JIRA, and had to ask the reporter(s). Mesosphere is
hosting the Mesos UI(s) underneath the DCOS UI behind a reverse proxy, so
that
On Fri, May 29, 2015 at 1:40 AM, Adam B a...@mesosphere.io wrote:
On May 28, 2015, 10:40 a.m., Adam B wrote:
Do you clearly understand why this change is needed? I didn't
understand after just reading the JIRA, and had to ask the reporter(s).
Mesosphere is hosting the Mesos UI(s)
On May 28, 2015, 5:40 p.m., Adam B wrote:
Do you clearly understand why this change is needed? I didn't understand
after just reading the JIRA, and had to ask the reporter(s). Mesosphere is
hosting the Mesos UI(s) underneath the DCOS UI behind a reverse proxy, so
that
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34655/#review85571
---
Do you clearly understand why this change is needed? I didn't
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34655/#review85592
---
Ship it!
Ship It!
- Marco Massenzio
On May 28, 2015, 7:54 a.m.,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34655/#review85544
---
Patch looks great!
Reviews applied: [34655]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34655/
---
Review request for mesos and Adam B.
Bugs: MESOS-2748
10 matches
Mail list logo