> On Dec. 8, 2017, 6:05 p.m., Alexander Rukletsov wrote: > > src/slave/containerizer/mesos/containerizer.hpp > > Line 356 (original), 361-368 (patched) > > <https://reviews.apache.org/r/63953/diff/6/?file=1909541#file1909541line361> > > > > Let's pull this is a separate patch.
https://reviews.apache.org/r/64575/ > On Dec. 8, 2017, 6:05 p.m., Alexander Rukletsov wrote: > > src/slave/containerizer/mesos/containerizer.cpp > > Lines 1630-1632 (original), 1632-1633 (patched) > > <https://reviews.apache.org/r/63953/diff/6/?file=1909542#file1909542line1632> > > > > This is not related to logging and hence belongs to the separate patch > > you will create for `containerClass()` https://reviews.apache.org/r/64576/ - Armand ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63953/#review193267 ----------------------------------------------------------- On Dec. 6, 2017, 2:47 p.m., Armand Grillet wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63953/ > ----------------------------------------------------------- > > (Updated Dec. 6, 2017, 2:47 p.m.) > > > Review request for mesos and Alexander Rukletsov. > > > 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 > ----- > > src/slave/containerizer/mesos/containerizer.hpp > e2739e017cb8dda37d94ad809ca1bd461f308bfb > src/slave/containerizer/mesos/containerizer.cpp > 7f3b86d87cf82429c2627d4a32eb0d5adbcc3f29 > > > Diff: https://reviews.apache.org/r/63953/diff/6/ > > > 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 > >
