From reviews-return-27120-apmail-mesos-reviews-archive=mesos.apache.org@mesos.apache.org Mon Mar 14 19:56:25 2016 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 37E0718F5F for ; Mon, 14 Mar 2016 19:56:25 +0000 (UTC) Received: (qmail 82271 invoked by uid 500); 14 Mar 2016 19:56:25 -0000 Delivered-To: apmail-mesos-reviews-archive@mesos.apache.org Received: (qmail 82224 invoked by uid 500); 14 Mar 2016 19:56:25 -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 82206 invoked by uid 99); 14 Mar 2016 19:56:24 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 14 Mar 2016 19:56:24 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id E9EC72ABBA9; Mon, 14 Mar 2016 19:56:22 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============7316199987535006462==" MIME-Version: 1.0 Subject: Re: Review Request 44439: Added device support in cgroups abstraction. From: Abhishek Dasgupta To: Felix Abecassis , Niklas Nielsen , Kevin Klues , Ben Mahler Cc: Mesos ReviewBot , Abhishek Dasgupta , mesos Date: Mon, 14 Mar 2016 19:56:22 -0000 Message-ID: <20160314195622.26509.34518@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org/ Auto-Submitted: auto-generated Sender: Abhishek Dasgupta X-ReviewGroup: mesos X-Auto-Response-Suppress: DR, RN, OOF, AutoReply X-ReviewRequest-URL: https://reviews.apache.org/r/44439/ X-Sender: Abhishek Dasgupta References: <20160309015833.8638.2118@reviews.apache.org> In-Reply-To: <20160309015833.8638.2118@reviews.apache.org> Reply-To: Abhishek Dasgupta X-ReviewRequest-Repository: mesos --===============7316199987535006462== MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit > On March 9, 2016, 1:58 a.m., Ben Mahler wrote: > > src/tests/containerizer/cgroups_tests.cpp, lines 1278-1309 > > > > > > Does this test pass? Did you make sure this test ran? Note that you have to run the tests under 'sudo' on Linux for this test to run because it has the ROOT and CGROUPS strings in the test name. Hi, I actually ran these tests. And here is the output: [==========] Running 1 test from 1 test case. [----------] Global test environment set-up. [----------] 1 test from CgroupsAnyHierarchyDevicesTest [ RUN ] CgroupsAnyHierarchyDevicesTest.ROOT_CGROUPS_Devices [ OK ] CgroupsAnyHierarchyDevicesTest.ROOT_CGROUPS_Devices (10 ms) [----------] 1 test from CgroupsAnyHierarchyDevicesTest (10 ms total) [----------] Global test environment tear-down [==========] 1 test from 1 test case ran. (31 ms total) [ PASSED ] 1 test. make[3]: Leaving directory '/home/abhishek/test/mesos/build/src' make[2]: Leaving directory '/home/abhishek/test/mesos/build/src' devices.list can't be changed by writing on it. It is to be changed by writing in devices.deny or device.allow. I followed this logic developing the code. Please, contradict me if I am wrong in any assumption/deduction. - Abhishek ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44439/#review122646 ----------------------------------------------------------- On March 14, 2016, 6:25 p.m., Abhishek Dasgupta wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44439/ > ----------------------------------------------------------- > > (Updated March 14, 2016, 6:25 p.m.) > > > Review request for mesos, Ben Mahler, Felix Abecassis, Kevin Klues, and Niklas Nielsen. > > > Bugs: MESOS-3368 > https://issues.apache.org/jira/browse/MESOS-3368 > > > Repository: mesos > > > Description > ------- > > There are some helper methods added to support device cgroups in cgroups abstraction to aid isolators controlling access to devices. > > > Diffs > ----- > > src/linux/cgroups.hpp 51ccefd1fd2d0989b9bd31342d3d1e3701f88ed2 > src/linux/cgroups.cpp df18ed46a2a96871f67c7eb4233c3b4c27b7aa1c > src/tests/containerizer/cgroups_tests.cpp acaed9b3f8a04964092cef413133834d0cf5a145 > > Diff: https://reviews.apache.org/r/44439/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Abhishek Dasgupta > > --===============7316199987535006462==--