----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63953/ -----------------------------------------------------------
(Updated Dec. 18, 2017, 10:05 a.m.) Review request for mesos and Alexander Rukletsov. Changes ------- Fixed issues. Bugs: MESOS-7361 https://issues.apache.org/jira/browse/MESOS-7361 Repository: mesos Description ------- This change adjusts log level based on the container class. If the class is `DEBUG` we print the log entry at a verbose level 1, otherwise we print it at the `INFO` level. We use the added macro in mesos containerizer so that COMMAND checks cause less INFO logs (15 lines instead of 26 before). Diffs (updated) ----- src/slave/containerizer/mesos/containerizer.hpp 85cb325fba2e49b78a1f381183a61f03ccd5194a src/slave/containerizer/mesos/containerizer.cpp 1a398a8ea7f3e49273459b318ae2e8a3eaae2d9c Diff: https://reviews.apache.org/r/63953/diff/9/ Changes: https://reviews.apache.org/r/63953/diff/8-9/ Testing ------- Started a Mesos cluster and used `mesos-execute` with this task group to check that the behaviour after this patch is the one expected: ``` { "tasks": [ { "name": "Name of the task", "task_id": { "value": "task-group" }, "agent_id": { "value": "" }, "resources": [ { "name": "cpus", "type": "SCALAR", "scalar": { "value": 0.01 } }, { "name": "mem", "type": "SCALAR", "scalar": { "value": 2 } } ], "command": { "value": "sleep 1000" }, "check": { "type": "COMMAND", "command": { "command": { "value": "echo \"Bonjour\"" }, "uris": [] } } } ] } ``` And: ``` $ nice make check ``` Thanks, Armand Grillet