Re: Review Request 36620: WIP Added Non-Freezeer Task Killer.

2015-08-17 Thread Timothy Chen

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

Ship it!


Ship It!


src/linux/cgroups.cpp (line 1686)
https://reviews.apache.org/r/36620/#comment150750

More like a code suggestion, looks like reap is always called after kill, 
and just waiting on all the statuses you gathered.

I wonder why not just move collect inside of  kill(processes) in the end 
and return that instead of another callback?


- Timothy Chen


On Aug. 13, 2015, 1:52 p.m., Joerg Schad wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/36620/
 ---
 
 (Updated Aug. 13, 2015, 1:52 p.m.)
 
 
 Review request for mesos, Benjamin Hindman and Timothy Chen.
 
 
 Bugs: MESOS-3086
 https://issues.apache.org/jira/browse/MESOS-3086
 
 
 Repository: mesos
 
 
 Description
 ---
 
 WIP Added Non-Freezeer Task Killer.
 
 
 Diffs
 -
 
   src/linux/cgroups.cpp 6ef42ed1bc719f334d1ac6e90919a1bc1840d31f 
 
 Diff: https://reviews.apache.org/r/36620/diff/
 
 
 Testing
 ---
 
 sudo make check
 + manual tests
 
 
 Thanks,
 
 Joerg Schad
 




Re: Review Request 36620: WIP Added Non-Freezeer Task Killer.

2015-08-13 Thread Joerg Schad

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

(Updated Aug. 13, 2015, 1:52 p.m.)


Review request for mesos, Benjamin Hindman and Timothy Chen.


Changes
---

Addressed comment and rebased.


Bugs: MESOS-3086
https://issues.apache.org/jira/browse/MESOS-3086


Repository: mesos


Description
---

WIP Added Non-Freezeer Task Killer.


Diffs (updated)
-

  src/linux/cgroups.cpp 6ef42ed1bc719f334d1ac6e90919a1bc1840d31f 

Diff: https://reviews.apache.org/r/36620/diff/


Testing (updated)
---

sudo make check
+ manual tests


Thanks,

Joerg Schad



Re: Review Request 36620: WIP Added Non-Freezeer Task Killer.

2015-08-13 Thread Joerg Schad


 On July 28, 2015, 7:20 p.m., Timothy Chen wrote:
  I notice there are no new tests added for this, can you add a test to 
  verify the new change works?
 
 Timothy Chen wrote:
 Are you able to add this? Otherwise let's add a TODO and get this in.

Created MESOS-3255 to follow up with tests.


- Joerg


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


On July 24, 2015, 1:07 p.m., Joerg Schad wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/36620/
 ---
 
 (Updated July 24, 2015, 1:07 p.m.)
 
 
 Review request for mesos, Benjamin Hindman and Timothy Chen.
 
 
 Bugs: MESOS-3086
 https://issues.apache.org/jira/browse/MESOS-3086
 
 
 Repository: mesos
 
 
 Description
 ---
 
 WIP Added Non-Freezeer Task Killer.
 
 
 Diffs
 -
 
   src/linux/cgroups.cpp e062fcbd56315f11882fe0ccb615c490dd719934 
 
 Diff: https://reviews.apache.org/r/36620/diff/
 
 
 Testing
 ---
 
 sudo make check
 
 
 Thanks,
 
 Joerg Schad
 




Re: Review Request 36620: WIP Added Non-Freezeer Task Killer.

2015-08-13 Thread Mesos ReviewBot

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


Patch looks great!

Reviews applied: [36612, 36620]

All tests passed.

- Mesos ReviewBot


On Aug. 13, 2015, 1:52 p.m., Joerg Schad wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/36620/
 ---
 
 (Updated Aug. 13, 2015, 1:52 p.m.)
 
 
 Review request for mesos, Benjamin Hindman and Timothy Chen.
 
 
 Bugs: MESOS-3086
 https://issues.apache.org/jira/browse/MESOS-3086
 
 
 Repository: mesos
 
 
 Description
 ---
 
 WIP Added Non-Freezeer Task Killer.
 
 
 Diffs
 -
 
   src/linux/cgroups.cpp 6ef42ed1bc719f334d1ac6e90919a1bc1840d31f 
 
 Diff: https://reviews.apache.org/r/36620/diff/
 
 
 Testing
 ---
 
 sudo make check
 + manual tests
 
 
 Thanks,
 
 Joerg Schad
 




Re: Review Request 36620: WIP Added Non-Freezeer Task Killer.

2015-08-12 Thread Timothy Chen


 On July 28, 2015, 7:20 p.m., Timothy Chen wrote:
  I notice there are no new tests added for this, can you add a test to 
  verify the new change works?

Are you able to add this? Otherwise let's add a TODO and get this in.


- Timothy


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


On July 24, 2015, 1:07 p.m., Joerg Schad wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/36620/
 ---
 
 (Updated July 24, 2015, 1:07 p.m.)
 
 
 Review request for mesos, Benjamin Hindman and Timothy Chen.
 
 
 Bugs: MESOS-3086
 https://issues.apache.org/jira/browse/MESOS-3086
 
 
 Repository: mesos
 
 
 Description
 ---
 
 WIP Added Non-Freezeer Task Killer.
 
 
 Diffs
 -
 
   src/linux/cgroups.cpp e062fcbd56315f11882fe0ccb615c490dd719934 
 
 Diff: https://reviews.apache.org/r/36620/diff/
 
 
 Testing
 ---
 
 sudo make check
 
 
 Thanks,
 
 Joerg Schad
 




Re: Review Request 36620: WIP Added Non-Freezeer Task Killer.

2015-07-28 Thread Timothy Chen

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


I notice there are no new tests added for this, can you add a test to verify 
the new change works?


src/linux/cgroups.cpp (line 1689)
https://reviews.apache.org/r/36620/#comment147707

I'll move the onFailed to next line so we line up the indentation.


- Timothy Chen


On July 24, 2015, 1:07 p.m., Joerg Schad wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/36620/
 ---
 
 (Updated July 24, 2015, 1:07 p.m.)
 
 
 Review request for mesos, Benjamin Hindman and Timothy Chen.
 
 
 Bugs: MESOS-3086
 https://issues.apache.org/jira/browse/MESOS-3086
 
 
 Repository: mesos
 
 
 Description
 ---
 
 WIP Added Non-Freezeer Task Killer.
 
 
 Diffs
 -
 
   src/linux/cgroups.cpp e062fcbd56315f11882fe0ccb615c490dd719934 
 
 Diff: https://reviews.apache.org/r/36620/diff/
 
 
 Testing
 ---
 
 sudo make check
 
 
 Thanks,
 
 Joerg Schad
 




Re: Review Request 36620: WIP Added Non-Freezeer Task Killer.

2015-07-27 Thread Joerg Schad


 On July 23, 2015, 5:42 p.m., Timothy Chen wrote:
  src/linux/cgroups.cpp, line 1696
  https://reviews.apache.org/r/36620/diff/10/?file=1019693#file1019693line1696
 
  Seems like we can just a lambda instead of a new fail method, then we 
  don't even need to store chain variable right?

We need to store the chain variable for the finalize call, e.g. when the 
timeout kills the process.


- Joerg


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


On July 24, 2015, 1:07 p.m., Joerg Schad wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/36620/
 ---
 
 (Updated July 24, 2015, 1:07 p.m.)
 
 
 Review request for mesos, Benjamin Hindman and Timothy Chen.
 
 
 Bugs: MESOS-3086
 https://issues.apache.org/jira/browse/MESOS-3086
 
 
 Repository: mesos
 
 
 Description
 ---
 
 WIP Added Non-Freezeer Task Killer.
 
 
 Diffs
 -
 
   src/linux/cgroups.cpp e062fcbd56315f11882fe0ccb615c490dd719934 
 
 Diff: https://reviews.apache.org/r/36620/diff/
 
 
 Testing
 ---
 
 sudo make check
 
 
 Thanks,
 
 Joerg Schad
 




Re: Review Request 36620: WIP Added Non-Freezeer Task Killer.

2015-07-24 Thread Mesos ReviewBot

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


Bad patch!

Reviews applied: [36612, 36620]

Failed command: ./support/apply-review.sh -n -r 36620

Error:
 2015-07-24 12:57:59 URL:https://reviews.apache.org/r/36620/diff/raw/ 
[234469/234469] - 36620.patch [1]
Successfully applied: WIP Added Non-Freezeer Task Killer.

WIP Added Non-Freezeer Task Killer.


Review: https://reviews.apache.org/r/36620
Checking 54 files using filter 
--filter=-,+build/class,+build/deprecated,+build/endif_comment,+readability/todo,+readability/namespace,+runtime/vlog,+whitespace/blank_line,+whitespace/comma,+whitespace/end_of_line,+whitespace/ending_newline,+whitespace/forcolon,+whitespace/indent,+whitespace/line_length,+whitespace/operators,+whitespace/semicolon,+whitespace/tab,+whitespace/todo
Total errors found: 0
ERROR: Commit spanning multiple projects.

Please use separate commits for mesos, libprocess and stout.

Paths grouped by project:
mesos:
  cmake/MesosConfigure.cmake
  docs/app-framework-development-guide.md
  docs/fetcher-cache-internals.md
  docs/release-guide.md
  include/mesos/mesos.proto
  include/mesos/scheduler/scheduler.proto
  include/mesos/slave/isolator.hpp
  include/mesos/slave/isolator.proto
  src/Makefile.am
  src/common/protobuf_utils.cpp
  src/common/protobuf_utils.hpp
  src/exec/exec.cpp
  src/linux/cgroups.cpp
  src/master/master.cpp
  src/master/master.hpp
  src/sched/sched.cpp
  src/scheduler/scheduler.cpp
  src/slave/containerizer/isolator.cpp
  src/slave/containerizer/isolators/cgroups/cpushare.cpp
  src/slave/containerizer/isolators/cgroups/cpushare.hpp
  src/slave/containerizer/isolators/cgroups/mem.cpp
  src/slave/containerizer/isolators/cgroups/mem.hpp
  src/slave/containerizer/isolators/cgroups/perf_event.cpp
  src/slave/containerizer/isolators/cgroups/perf_event.hpp
  src/slave/containerizer/isolators/filesystem/posix.cpp
  src/slave/containerizer/isolators/filesystem/posix.hpp
  src/slave/containerizer/isolators/filesystem/shared.cpp
  src/slave/containerizer/isolators/filesystem/shared.hpp
  src/slave/containerizer/isolators/namespaces/pid.cpp
  src/slave/containerizer/isolators/namespaces/pid.hpp
  src/slave/containerizer/isolators/network/port_mapping.cpp
  src/slave/containerizer/isolators/network/port_mapping.hpp
  src/slave/containerizer/isolators/posix.hpp
  src/slave/containerizer/isolators/posix/disk.cpp
  src/slave/containerizer/isolators/posix/disk.hpp
  src/slave/containerizer/launcher.cpp
  src/slave/containerizer/linux_launcher.cpp
  src/slave/containerizer/mesos/containerizer.cpp
  src/slave/containerizer/mesos/containerizer.hpp
  src/tests/containerizer_tests.cpp
  src/tests/disk_quota_tests.cpp
  src/tests/docker_containerizer_tests.cpp
  src/tests/external_containerizer_test.cpp
  src/tests/fault_tolerance_tests.cpp
  src/tests/hook_tests.cpp
  src/tests/isolator.hpp
  src/tests/master_authorization_tests.cpp
  src/tests/master_slave_reconciliation_tests.cpp
  src/tests/master_tests.cpp
  src/tests/master_validation_tests.cpp
  src/tests/memory_pressure_tests.cpp
  src/tests/partition_tests.cpp
  src/tests/persistent_volume_tests.cpp
  src/tests/port_mapping_tests.cpp
  src/tests/rate_limiting_tests.cpp
  src/tests/reconciliation_tests.cpp
  src/tests/registrar_zookeeper_tests.cpp
  src/tests/scheduler_tests.cpp
  src/tests/slave_recovery_tests.cpp
  src/tests/slave_tests.cpp
  support/tag.sh
stout:
  3rdparty/libprocess/3rdparty/stout/cmake/FindApr.cmake
  3rdparty/libprocess/3rdparty/stout/cmake/FindSvn.cmake
  3rdparty/libprocess/3rdparty/stout/cmake/StoutTestsConfigure.cmake
  3rdparty/libprocess/3rdparty/stout/include/stout/path.hpp
  3rdparty/libprocess/3rdparty/stout/tests/CMakeLists.txt
libprocess:
  3rdparty/libprocess/3rdparty/CMakeLists.txt
  3rdparty/libprocess/cmake/ProcessConfigure.cmake
  3rdparty/libprocess/cmake/ProcessTestsConfigure.cmake
  3rdparty/libprocess/include/process/http.hpp
  3rdparty/libprocess/src/tests/CMakeLists.txt
Failed to commit patch

- Mesos ReviewBot


On July 24, 2015, 12:15 p.m., Joerg Schad wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/36620/
 ---
 
 (Updated July 24, 2015, 12:15 p.m.)
 
 
 Review request for mesos, Benjamin Hindman and Timothy Chen.
 
 
 Bugs: MESOS-3086
 https://issues.apache.org/jira/browse/MESOS-3086
 
 
 Repository: mesos
 
 
 Description
 ---
 
 WIP Added Non-Freezeer Task Killer.
 
 
 Diffs
 -
 
   3rdparty/libprocess/3rdparty/CMakeLists.txt 
 d7c6dfb04120a7fd28e1a4682f39015bd149eb40 
   3rdparty/libprocess/3rdparty/stout/cmake/FindApr.cmake 
 4b28aa170f48d37ae9096bc28a64d8a32e8d35dd 
   3rdparty/libprocess/3rdparty/stout/cmake/FindSvn.cmake 
 

Re: Review Request 36620: WIP Added Non-Freezeer Task Killer.

2015-07-23 Thread Joerg Schad

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

(Updated July 23, 2015, 12:21 p.m.)


Review request for mesos and Timothy Chen.


Changes
---

Improved comments.


Bugs: MESOS-3086
https://issues.apache.org/jira/browse/MESOS-3086


Repository: mesos


Description
---

WIP Added Non-Freezeer Task Killer.


Diffs (updated)
-

  src/linux/cgroups.cpp e062fcbd56315f11882fe0ccb615c490dd719934 

Diff: https://reviews.apache.org/r/36620/diff/


Testing
---

sudo make check


Thanks,

Joerg Schad



Re: Review Request 36620: WIP Added Non-Freezeer Task Killer.

2015-07-23 Thread Mesos ReviewBot

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


Patch looks great!

Reviews applied: [36612, 36620]

All tests passed.

- Mesos ReviewBot


On July 23, 2015, 12:21 p.m., Joerg Schad wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/36620/
 ---
 
 (Updated July 23, 2015, 12:21 p.m.)
 
 
 Review request for mesos and Timothy Chen.
 
 
 Bugs: MESOS-3086
 https://issues.apache.org/jira/browse/MESOS-3086
 
 
 Repository: mesos
 
 
 Description
 ---
 
 WIP Added Non-Freezeer Task Killer.
 
 
 Diffs
 -
 
   src/linux/cgroups.cpp e062fcbd56315f11882fe0ccb615c490dd719934 
 
 Diff: https://reviews.apache.org/r/36620/diff/
 
 
 Testing
 ---
 
 sudo make check
 
 
 Thanks,
 
 Joerg Schad
 




Re: Review Request 36620: WIP Added Non-Freezeer Task Killer.

2015-07-23 Thread Joerg Schad

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

(Updated July 23, 2015, 12:19 p.m.)


Review request for mesos and Timothy Chen.


Changes
---

Improved comments.


Bugs: MESOS-3086
https://issues.apache.org/jira/browse/MESOS-3086


Repository: mesos


Description
---

WIP Added Non-Freezeer Task Killer.


Diffs (updated)
-

  src/linux/cgroups.cpp e062fcbd56315f11882fe0ccb615c490dd719934 

Diff: https://reviews.apache.org/r/36620/diff/


Testing
---

sudo make check


Thanks,

Joerg Schad



Re: Review Request 36620: WIP Added Non-Freezeer Task Killer.

2015-07-22 Thread Joerg Schad

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

(Updated July 22, 2015, 10:04 a.m.)


Review request for mesos and Timothy Chen.


Bugs: MESOS-3086
https://issues.apache.org/jira/browse/MESOS-3086


Repository: mesos


Description
---

WIP Added Non-Freezeer Task Killer.


Diffs (updated)
-

  src/linux/cgroups.cpp e062fcbd56315f11882fe0ccb615c490dd719934 

Diff: https://reviews.apache.org/r/36620/diff/


Testing
---

sudo make check


Thanks,

Joerg Schad



Re: Review Request 36620: WIP Added Non-Freezeer Task Killer.

2015-07-22 Thread Mesos ReviewBot

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


Patch looks great!

Reviews applied: [36612, 36620]

All tests passed.

- Mesos ReviewBot


On July 22, 2015, 10:04 a.m., Joerg Schad wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/36620/
 ---
 
 (Updated July 22, 2015, 10:04 a.m.)
 
 
 Review request for mesos and Timothy Chen.
 
 
 Bugs: MESOS-3086
 https://issues.apache.org/jira/browse/MESOS-3086
 
 
 Repository: mesos
 
 
 Description
 ---
 
 WIP Added Non-Freezeer Task Killer.
 
 
 Diffs
 -
 
   src/linux/cgroups.cpp e062fcbd56315f11882fe0ccb615c490dd719934 
 
 Diff: https://reviews.apache.org/r/36620/diff/
 
 
 Testing
 ---
 
 sudo make check
 
 
 Thanks,
 
 Joerg Schad
 




Re: Review Request 36620: WIP Added Non-Freezeer Task Killer.

2015-07-21 Thread Joerg Schad

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

(Updated July 21, 2015, 3:59 p.m.)


Review request for mesos and Timothy Chen.


Changes
---

Style fixes


Repository: mesos


Description
---

WIP Added Non-Freezeer Task Killer.


Diffs (updated)
-

  src/linux/cgroups.cpp b7d11ac4eb8b78664c8c7bd7a5b0d566b5bf2f0e 

Diff: https://reviews.apache.org/r/36620/diff/


Testing
---


Thanks,

Joerg Schad



Re: Review Request 36620: WIP Added Non-Freezeer Task Killer.

2015-07-21 Thread Joerg Schad

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

(Updated July 21, 2015, 4:42 p.m.)


Review request for mesos and Timothy Chen.


Repository: mesos


Description
---

WIP Added Non-Freezeer Task Killer.


Diffs (updated)
-

  src/linux/cgroups.cpp b7d11ac4eb8b78664c8c7bd7a5b0d566b5bf2f0e 

Diff: https://reviews.apache.org/r/36620/diff/


Testing
---


Thanks,

Joerg Schad



Re: Review Request 36620: WIP Added Non-Freezeer Task Killer.

2015-07-21 Thread Joerg Schad

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

(Updated July 22, 2015, 4:37 a.m.)


Review request for mesos and Timothy Chen.


Bugs: MESOS-3086
https://issues.apache.org/jira/browse/MESOS-3086


Repository: mesos


Description
---

WIP Added Non-Freezeer Task Killer.


Diffs (updated)
-

  src/linux/cgroups.cpp b7d11ac4eb8b78664c8c7bd7a5b0d566b5bf2f0e 

Diff: https://reviews.apache.org/r/36620/diff/


Testing
---

sudo make check


Thanks,

Joerg Schad



Re: Review Request 36620: WIP Added Non-Freezeer Task Killer.

2015-07-21 Thread Joerg Schad

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

(Updated July 21, 2015, 5:18 p.m.)


Review request for mesos and Timothy Chen.


Bugs: MESOS-3086
https://issues.apache.org/jira/browse/MESOS-3086


Repository: mesos


Description
---

WIP Added Non-Freezeer Task Killer.


Diffs
-

  src/linux/cgroups.cpp b7d11ac4eb8b78664c8c7bd7a5b0d566b5bf2f0e 

Diff: https://reviews.apache.org/r/36620/diff/


Testing (updated)
---

sudo make check


Thanks,

Joerg Schad



Re: Review Request 36620: WIP Added Non-Freezeer Task Killer.

2015-07-21 Thread Jie Yu

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


Could you please attach a ticket or explain in the description about the 
motivation?

- Jie Yu


On July 21, 2015, 4:42 p.m., Joerg Schad wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/36620/
 ---
 
 (Updated July 21, 2015, 4:42 p.m.)
 
 
 Review request for mesos and Timothy Chen.
 
 
 Repository: mesos
 
 
 Description
 ---
 
 WIP Added Non-Freezeer Task Killer.
 
 
 Diffs
 -
 
   src/linux/cgroups.cpp b7d11ac4eb8b78664c8c7bd7a5b0d566b5bf2f0e 
 
 Diff: https://reviews.apache.org/r/36620/diff/
 
 
 Testing
 ---
 
 
 Thanks,
 
 Joerg Schad
 




Re: Review Request 36620: WIP Added Non-Freezeer Task Killer.

2015-07-21 Thread Mesos ReviewBot

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


Bad patch!

Reviews applied: [36612]

Failed command: ./support/apply-review.sh -n -r 36612

Error:
 2015-07-22 05:28:09 URL:https://reviews.apache.org/r/36612/diff/raw/ 
[12620/12620] - 36612.patch [1]
error: patch failed: src/linux/cgroups.hpp:92
error: src/linux/cgroups.hpp: patch does not apply
error: patch failed: src/linux/cgroups.cpp:110
error: src/linux/cgroups.cpp: patch does not apply
Failed to apply patch

- Mesos ReviewBot


On July 22, 2015, 4:37 a.m., Joerg Schad wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/36620/
 ---
 
 (Updated July 22, 2015, 4:37 a.m.)
 
 
 Review request for mesos and Timothy Chen.
 
 
 Bugs: MESOS-3086
 https://issues.apache.org/jira/browse/MESOS-3086
 
 
 Repository: mesos
 
 
 Description
 ---
 
 WIP Added Non-Freezeer Task Killer.
 
 
 Diffs
 -
 
   src/linux/cgroups.cpp b7d11ac4eb8b78664c8c7bd7a5b0d566b5bf2f0e 
 
 Diff: https://reviews.apache.org/r/36620/diff/
 
 
 Testing
 ---
 
 sudo make check
 
 
 Thanks,
 
 Joerg Schad
 




Re: Review Request 36620: WIP Added Non-Freezeer Task Killer.

2015-07-21 Thread Timothy Chen

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



src/linux/cgroups.cpp (line 1504)
https://reviews.apache.org/r/36620/#comment146628

This is actually the Freezer right? 
And we should make this VLOG(1)



src/linux/cgroups.cpp (line 1627)
https://reviews.apache.org/r/36620/#comment146629

To me this doesn't killer doesn't atomically kill all of them, as that 
implies either all or none.

I think the TaskKiller actually best effort try to kill by signal and 
reintrospect all tasks in a cgroup within the timeout



src/linux/cgroups.cpp (line 1650)
https://reviews.apache.org/r/36620/#comment146630

100 ms seems way too fast?
And why not use the default timeout that's already there?



src/linux/cgroups.cpp (line 1668)
https://reviews.apache.org/r/36620/#comment146632

This should go to line 1674?



src/linux/cgroups.cpp (line 1676)
https://reviews.apache.org/r/36620/#comment146631

Move to previous line.



src/linux/cgroups.cpp (line 1684)
https://reviews.apache.org/r/36620/#comment146633

This fits on to previous line?


- Timothy Chen


On July 21, 2015, 5:18 p.m., Joerg Schad wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/36620/
 ---
 
 (Updated July 21, 2015, 5:18 p.m.)
 
 
 Review request for mesos and Timothy Chen.
 
 
 Bugs: MESOS-3086
 https://issues.apache.org/jira/browse/MESOS-3086
 
 
 Repository: mesos
 
 
 Description
 ---
 
 WIP Added Non-Freezeer Task Killer.
 
 
 Diffs
 -
 
   src/linux/cgroups.cpp b7d11ac4eb8b78664c8c7bd7a5b0d566b5bf2f0e 
 
 Diff: https://reviews.apache.org/r/36620/diff/
 
 
 Testing
 ---
 
 sudo make check
 
 
 Thanks,
 
 Joerg Schad
 




Re: Review Request 36620: WIP Added Non-Freezeer Task Killer.

2015-07-21 Thread Timothy Chen

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



src/linux/cgroups.cpp (line 1671)
https://reviews.apache.org/r/36620/#comment146638

It's very odd to see the failure message set here but then a lot more 
attempts to try to set it in the finished callback.

How about having a failed method that sets the failure and another finished 
callback that is what you have?



src/linux/cgroups.cpp (line 1683)
https://reviews.apache.org/r/36620/#comment146637

Very wierd identation?



src/linux/cgroups.cpp (line 1686)
https://reviews.apache.org/r/36620/#comment146634

//check - // Check



src/linux/cgroups.cpp (line 1688)
https://reviews.apache.org/r/36620/#comment146636

one more space to the right



src/linux/cgroups.cpp (line 1708)
https://reviews.apache.org/r/36620/#comment146635

extra space between the and pids


- Timothy Chen


On July 21, 2015, 5:18 p.m., Joerg Schad wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/36620/
 ---
 
 (Updated July 21, 2015, 5:18 p.m.)
 
 
 Review request for mesos and Timothy Chen.
 
 
 Bugs: MESOS-3086
 https://issues.apache.org/jira/browse/MESOS-3086
 
 
 Repository: mesos
 
 
 Description
 ---
 
 WIP Added Non-Freezeer Task Killer.
 
 
 Diffs
 -
 
   src/linux/cgroups.cpp b7d11ac4eb8b78664c8c7bd7a5b0d566b5bf2f0e 
 
 Diff: https://reviews.apache.org/r/36620/diff/
 
 
 Testing
 ---
 
 sudo make check
 
 
 Thanks,
 
 Joerg Schad