-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63953/
-----------------------------------------------------------
(Updated Nov. 22, 2017, 4 p.m.)
Review request for mesos and Alexander Rukletsov.
Changes
-------
Replaced `using` by `include`.
Bugs: MESOS-7361
https://issues.apache.org/jira/browse/MESOS-7361
Repository: mesos
Description
-------
This change enables filtering when printing logs regarding
containers. If its class is `DEBUG` we print the log line at a
verbose level 1 otherwise we print is at the `INFO` level.
We use the added macro in containerizer.cpp so that COMMAND health
checks cause less INFO logs (15 instead of 26 before).
Diffs (updated)
-----
src/Makefile.am 9641ad4cf2a1f7fe56a4dcccb6eeff7130872fa8
src/logging/utils.hpp PRE-CREATION
src/slave/containerizer/mesos/containerizer.cpp
7f3b86d87cf82429c2627d4a32eb0d5adbcc3f29
Diff: https://reviews.apache.org/r/63953/diff/5/
Changes: https://reviews.apache.org/r/63953/diff/4-5/
Testing
-------
Started a Mesos cluster and used `mesos-execute` with this task group to check
that the behaviour after this patch was 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
