mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Alexander Rukletsov <ruklet...@gmail.com>
Subject Re: Review Request 63953: Added logging based on container class.
Date Fri, 15 Dec 2017 10:19:51 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63953/#review193897
-----------------------------------------------------------


Fix it, then Ship it!





src/slave/containerizer/mesos/containerizer.hpp
Lines 54-55 (patched)
<https://reviews.apache.org/r/63953/#comment272564>

    Please backtick class values.
    
    Also, let's explain why we do it (to avoid polluting the agent log with DEBUG container
related information, because DEBUG containers can be started in a periodic manner).



src/slave/containerizer/mesos/containerizer.cpp
Line 2335 (original), 2338-2339 (patched)
<https://reviews.apache.org/r/63953/#comment272565>

    Ups


- Alexander Rukletsov


On Dec. 13, 2017, 7:47 p.m., Armand Grillet wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/63953/
> -----------------------------------------------------------
> 
> (Updated Dec. 13, 2017, 7: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 965e183bb5c54f31d90e910edd35313ab380cea9

>   src/slave/containerizer/mesos/containerizer.cpp 7ab0b07f689f872573ca458ae47cd6426ebc0365

> 
> 
> Diff: https://reviews.apache.org/r/63953/diff/8/
> 
> 
> 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
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message