-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63953/
-----------------------------------------------------------
(Updated Dec. 13, 2017, 3:26 p.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
965e183bb5c54f31d90e910edd35313ab380cea9
src/slave/containerizer/mesos/containerizer.cpp
7ab0b07f689f872573ca458ae47cd6426ebc0365
Diff: https://reviews.apache.org/r/63953/diff/7/
Changes: https://reviews.apache.org/r/63953/diff/6-7/
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
