-----------------------------------------------------------
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
