Return-Path: X-Original-To: apmail-mesos-reviews-archive@minotaur.apache.org Delivered-To: apmail-mesos-reviews-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 4112718986 for ; Wed, 21 Oct 2015 03:48:20 +0000 (UTC) Received: (qmail 85319 invoked by uid 500); 21 Oct 2015 03:48:20 -0000 Delivered-To: apmail-mesos-reviews-archive@mesos.apache.org Received: (qmail 85292 invoked by uid 500); 21 Oct 2015 03:48:20 -0000 Mailing-List: contact reviews-help@mesos.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: reviews@mesos.apache.org Delivered-To: mailing list reviews@mesos.apache.org Received: (qmail 85270 invoked by uid 99); 21 Oct 2015 03:48:19 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 21 Oct 2015 03:48:19 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id 5EE152783CC; Wed, 21 Oct 2015 03:48:19 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============1389869407383747394==" MIME-Version: 1.0 Subject: Re: Review Request 39360: Relocate MesosContainerizer specific files to the correct location From: "Mesos ReviewBot" To: "Ben Mahler" , "Jie Yu" , "Jojy Varghese" , "Timothy Chen" Cc: "Gilbert Song" , "mesos" , "Mesos ReviewBot" Date: Wed, 21 Oct 2015 03:48:19 -0000 Message-ID: <20151021034819.1667.4447@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org/ Auto-Submitted: auto-generated Sender: "Mesos ReviewBot" X-ReviewGroup: mesos X-Auto-Response-Suppress: DR, RN, OOF, AutoReply X-ReviewRequest-URL: https://reviews.apache.org/r/39360/ X-Sender: "Mesos ReviewBot" References: <20151020220021.1666.73945@reviews.apache.org> In-Reply-To: <20151020220021.1666.73945@reviews.apache.org> Reply-To: "Mesos ReviewBot" X-ReviewRequest-Repository: mesos --===============1389869407383747394== MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39360/#review103351 ----------------------------------------------------------- Patch looks great! Reviews applied: [39360] All tests passed. - Mesos ReviewBot On Oct. 20, 2015, 10 p.m., Gilbert Song wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39360/ > ----------------------------------------------------------- > > (Updated Oct. 20, 2015, 10 p.m.) > > > Review request for mesos, Ben Mahler, Jie Yu, Jojy Varghese, and Timothy Chen. > > > Bugs: MESOS-3129 > https://issues.apache.org/jira/browse/MESOS-3129 > > > Repository: mesos > > > Description > ------- > > Relocate MesosContainerizer specific files to the correct location > > > Diffs > ----- > > src/Makefile.am 98cbafc134ec388a176d50172912fbfdf9f5bfa3 > src/docker/docker.cpp 56d63dc75637c9f89a239af371f476a85a570696 > src/examples/test_isolator_module.cpp 577dfcac260b4f5df7ab4e9599e4caac46ccd1e1 > src/slave/containerizer/docker.cpp 702295808475c092dff66417f42af89b90e6d50d > src/slave/containerizer/isolators/cgroups/constants.hpp > src/slave/containerizer/isolators/cgroups/cpushare.hpp 54b83a7d67f9cacbca4f9dd9b9b72a3dbc2e5263 > src/slave/containerizer/isolators/cgroups/cpushare.cpp ba748c6caec7253b42167e8a4f9b4535da858259 > src/slave/containerizer/isolators/cgroups/mem.hpp > src/slave/containerizer/isolators/cgroups/mem.cpp 55fa6f4019e1521dd816138e82db110d573ae6b8 > src/slave/containerizer/isolators/cgroups/perf_event.hpp > src/slave/containerizer/isolators/cgroups/perf_event.cpp 03035dfbfb84470ba39ed9ecfd1eb73890e3f784 > src/slave/containerizer/isolators/filesystem/linux.hpp 93e85f2aa7bfceb7e55ff33bdc2e0e0a5cb8f880 > src/slave/containerizer/isolators/filesystem/linux.cpp 8823b7850a1ac17fc4f4868aadf1b04428d2381b > src/slave/containerizer/isolators/filesystem/posix.hpp > src/slave/containerizer/isolators/filesystem/posix.cpp eec510c4f7655d67b33ad90210eeb57fcc910684 > src/slave/containerizer/isolators/filesystem/shared.hpp > src/slave/containerizer/isolators/filesystem/shared.cpp 73804ca5a8a3bf03e13c74a247b5c21e9af5f040 > src/slave/containerizer/isolators/namespaces/pid.hpp > src/slave/containerizer/isolators/namespaces/pid.cpp a9823e08b195b8df82de2a7b410a4e6ef99f8853 > src/slave/containerizer/isolators/network/helper.cpp e5fb99e87ac16150b85b1c6f6965681f7fe77ce0 > src/slave/containerizer/isolators/network/port_mapping.hpp > src/slave/containerizer/isolators/network/port_mapping.cpp e6bb75e6f5ba48a0c4cf6dd8f353e5f5843d0eef > src/slave/containerizer/isolators/posix.hpp > src/slave/containerizer/isolators/posix/disk.hpp > src/slave/containerizer/isolators/posix/disk.cpp 73e62a225da062733557287afa2273d8183d76fd > src/slave/containerizer/linux_launcher.cpp c03b89eb0678825b03a052874d6262f377a39e13 > src/slave/containerizer/mesos/containerizer.cpp d1fc5a460e7313828014eea999cf4e63dde01921 > src/slave/containerizer/provisioner/appc/paths.hpp > src/slave/containerizer/provisioner/appc/paths.cpp 8817c0ff4b6806f08afd322e250a9a53b7b0a5d6 > src/slave/containerizer/provisioner/appc/spec.hpp > src/slave/containerizer/provisioner/appc/spec.cpp bbe523d2ee1dd558cc5007e578cbf23abac8e1de > src/slave/containerizer/provisioner/appc/store.hpp e8455197dcc3f4c9856db20605f6862b8755a946 > src/slave/containerizer/provisioner/appc/store.cpp a5ef4ea7cd08423360120430833c5881053637f5 > src/slave/containerizer/provisioner/backend.hpp > src/slave/containerizer/provisioner/backend.cpp b5d96701ae6bd49365b169f4e5150b8c4dae1870 > src/slave/containerizer/provisioner/backends/bind.hpp 1685938fb4349e790b9595cc4c67584c7f31a392 > src/slave/containerizer/provisioner/backends/bind.cpp 1fe1746c0bc1c9c12e1378e6438122a91b58316b > src/slave/containerizer/provisioner/backends/copy.hpp 7a5aaa41d8f6842ef437ed7a34235d8baac4bfff > src/slave/containerizer/provisioner/backends/copy.cpp 92fb0988da0bdd5a2b5a5f53ab61b7bb19c61cda > src/slave/containerizer/provisioner/docker/local_puller.hpp 4574e8a04663482625d7b54f765741f221ec13e0 > src/slave/containerizer/provisioner/docker/local_puller.cpp 74d0e1ead7d630e65a7e75cb6123139b9197efef > src/slave/containerizer/provisioner/docker/message.hpp 466e6f838d143917fa7eebb13b0a670a6b80117c > src/slave/containerizer/provisioner/docker/message.proto > src/slave/containerizer/provisioner/docker/metadata_manager.hpp 885080dbd3603f8c71ac867b88edcfd22276567f > src/slave/containerizer/provisioner/docker/metadata_manager.cpp 2b2de5245bccbd01a856b214ac6525278d794537 > src/slave/containerizer/provisioner/docker/paths.hpp > src/slave/containerizer/provisioner/docker/paths.cpp 5733fb7137b1ecb8a904cc5354425c60c9e065f5 > src/slave/containerizer/provisioner/docker/puller.hpp 105b4e75439c2ad4c08e2fd364f288f1d39b9b59 > src/slave/containerizer/provisioner/docker/puller.cpp cb05324689ffa26ce830b513e2d71b55517da3cb > src/slave/containerizer/provisioner/docker/registry_client.hpp > src/slave/containerizer/provisioner/docker/registry_client.cpp 24aa95c5800ff4dfb37465b71421b014b5dd5998 > src/slave/containerizer/provisioner/docker/spec.hpp 199db540e44581a411ce63082b917821d29360a8 > src/slave/containerizer/provisioner/docker/spec.cpp 5d6893dc6875eb67e3724f184c5ad171f94d88fa > src/slave/containerizer/provisioner/docker/store.hpp b5e6c87d38bea531c1b504915a84625fc9777e0d > src/slave/containerizer/provisioner/docker/store.cpp 50340131d10222ac06cf21e87ad46943f171d1f6 > src/slave/containerizer/provisioner/docker/token_manager.hpp > src/slave/containerizer/provisioner/docker/token_manager.cpp 18b29c3d0ddf25fd3f4713c5de10aeec3b381df0 > src/slave/containerizer/provisioner/paths.hpp > src/slave/containerizer/provisioner/paths.cpp 8d0f7d3cd6af132112f22d8709bc401745c5f43a > src/slave/containerizer/provisioner/provisioner.hpp > src/slave/containerizer/provisioner/provisioner.cpp 30a85a52ee5986c6e1652edfd08ae881280b23ab > src/slave/containerizer/provisioner/store.hpp > src/slave/containerizer/provisioner/store.cpp a8bc30264c58aa46a1eb8b6b4466114d9ed1acdc > src/tests/containerizer/filesystem_isolator_tests.cpp 1e332e0d36ac2812456514aa030f995b3a07dca1 > src/tests/containerizer/isolator_tests.cpp 237f3f27722b01ff92d0dcbaba7910613542a1a7 > src/tests/containerizer/port_mapping_tests.cpp feca2043503436ac9abac6017ae9059b3fcbed21 > src/tests/containerizer/provisioner.hpp 54aab5f279c66d57f9054ef8d9b37bd58f727bb8 > src/tests/containerizer/provisioner_appc_tests.cpp 3da759a5e6dd305c7c5455d26b5136065ed64c32 > src/tests/containerizer/provisioner_backend_tests.cpp b72edc6696955409e101e036182092dff4c64c10 > src/tests/containerizer/provisioner_docker_tests.cpp 01d3025f59d4a2714a856fe0f3a57192be023990 > src/tests/disk_quota_tests.cpp 1ed1342a10a4d59b84ef13a3d66eef95150c4e53 > > Diff: https://reviews.apache.org/r/39360/diff/ > > > Testing > ------- > > make check (Ubuntu14.04 + clang-3.6) > > Important Note: > Currently some patches pending to be committed rely on the old directory, which means that it will result in corresponding changes(fix conflicts during rebasing) in these patches on reviewboard if this post was merged. Please make a note below if you currently have patches working on MesosContainerizer and want this patch to be discarded and redo in the future. > > > Thanks, > > Gilbert Song > > --===============1389869407383747394==--