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 64575: Added containerClass() to MesosContainerizerProcess::Container.
Date Mon, 18 Dec 2017 10:03:56 GMT

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

(Updated Dec. 18, 2017, 10:03 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 function always returns a ContainerClass, `DEFAULT` being the
default value returned. Also simplifies the conditional statements
in mesos/containerizer.cpp to use the function added.


Diffs (updated)
-----

  src/slave/containerizer/mesos/containerizer.hpp 85cb325fba2e49b78a1f381183a61f03ccd5194a

  src/slave/containerizer/mesos/containerizer.cpp 1a398a8ea7f3e49273459b318ae2e8a3eaae2d9c



Diff: https://reviews.apache.org/r/64575/diff/3/

Changes: https://reviews.apache.org/r/64575/diff/2-3/


Testing
-------

```
GTEST_FILTER="" nice make check -j16 V=0
```


Thanks,

Armand Grillet


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