mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Armand Grillet <agril...@mesosphere.io>
Subject Re: Review Request 63953: Added logging based on container class.
Date Wed, 13 Dec 2017 15:10:57 GMT


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


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