You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Joris Van Remoortere <jo...@gmail.com> on 2017/03/01 18:07:58 UTC
Re: Review Request 46182: Adding app runtime processing to appc store
and provisioner.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46182/#review167325
-----------------------------------------------------------
Closing this review due to inactivity. Please see our [guidelines](https://github.com/apache/mesos/blob/master/docs/reopening-reviews.md) for reopening reviews.
- Joris Van Remoortere
On June 21, 2016, 6:11 p.m., Srinivas Brahmaroutu wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46182/
> -----------------------------------------------------------
>
> (Updated June 21, 2016, 6:11 p.m.)
>
>
> Review request for mesos, Gilbert Song and Jie Yu.
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Adding app runtime processing to appc store and provisioner.
>
>
> Diffs
> -----
>
> include/mesos/appc/spec.proto 4ef0473748c2f47aff1729c1d969185803fe72d4
> include/mesos/slave/isolator.proto 08cd9a0383bcfe2814388d7d9b1feae812b112da
> src/slave/containerizer/mesos/provisioner/appc/store.cpp aaa0efe63e587b9e604082b52a3cb8c11545fbb9
> src/slave/containerizer/mesos/provisioner/provisioner.hpp 48a05059969e068a0ee0d38b61be9e7104e3188d
> src/slave/containerizer/mesos/provisioner/provisioner.cpp 249acad49122d988e44744384bcf840b941c0997
> src/slave/containerizer/mesos/provisioner/store.hpp 1d477ef13ddd24fd8badae0decaa4a2271ecc746
>
>
> Diff: https://reviews.apache.org/r/46182/diff/3/
>
>
> Testing
> -------
>
> make check on linux passes AppcProvisionerIntegrationTest.ROOT_SimpleLinuxImageTest
>
>
> Thanks,
>
> Srinivas Brahmaroutu
>
>