-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47822/
-----------------------------------------------------------
(Updated May 25, 2016, 4:27 p.m.)
Review request for mesos, Adam B, Alexander Rukletsov, Anand Mazumdar, Benjamin
Bannier, Greg Mann, Jan Schlicht, Till Toenshoff, and Vinod Kone.
Bugs: MESOS-5293
https://issues.apache.org/jira/browse/MESOS-5293
Repository: mesos
Description (updated)
-------
The master and agent code for routing endpoints are made
consistent. Now they both capture 'this' variable in the
lambda calls in routing function arguments. Continuations
are no more need to be static.
Diffs
-----
src/slave/http.cpp dd1f509e9464af7d02db0182b0eebd63993dd029
src/slave/slave.hpp 0de6a570e8b4699771048295ec3fcedf84593495
src/slave/slave.cpp 2941cf1b6ea1e4deabfcbbe3f4897c06a28531a5
Diff: https://reviews.apache.org/r/47822/diff/
Testing
-------
Unit tests.
On ubuntu 16.04:
sudo GTEST_FILTER="*SlaveEndpointTest*.*" make -j2 check
Manual testing.
1. Ran master with:
sudo ./bin/mesos-master.sh --ip=127.0.0.1 --work_dir=/var/lib/mesos
2. ACL File:
{
"get_endpoints": [
{
"principals": { "type": "NONE" },
"paths": { "values": ["/flags", "/monitor/statistics", "/containers"] }
}
]
}
3. Ran slave with:
sudo ./bin/mesos-slave.sh --master=127.0.0.1:5050 --ip=0.0.0.0
--acls=file:///home/abhishek/testAcl
4. Ran toy-framework with:
sudo ./no-executor-framework --master=master@127.0.0.1:5050 --command="echo
hello"
5. Output while hitting "http://127.0.0.1:5051/slave(1)/containers" - HTTP
error 403: Forbidden
6. Changed ACL to:
{
"get_endpoints": [
{
"principals": { "type": "ANY" },
"paths": { "values": ["/flags", "/monitor/statistics", "/containers"] }
}
]
}
7. Ran slave and framework again.
8. Output:
[{"container_id":"2059c6a5-d168-4db7-912d-1c70b61fdb5d","executor_id":"699","executor_name":"Command
Executor (Task: 699) (Command: sh -c 'echo
hello')","framework_id":"48e2fe51-afba-44c5-958e-0ad3cee6454b-0000","source":"699","statistics":{"cpus_limit":0.2,"cpus_system_time_secs":0.01,"cpus_user_time_secs":0.04,"mem_limit_bytes":67108864,"mem_rss_bytes":88477696,"timestamp":1464168040.50791},"status":{}},{"container_id":"a0be0340-ca6b-4fc4-9862-4af143a7961e",.......
Thanks,
Abhishek Dasgupta
