mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jian Qiu" <qiuj...@cn.ibm.com>
Subject Re: Review Request 38287: Check if the futrue is failed before dispatch in freeze()
Date Mon, 23 Nov 2015 05:37:29 GMT

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

(Updated Nov. 23, 2015, 5:37 a.m.)


Review request for mesos, Ian Downes, Jie Yu, and Paul Brett.


Changes
-------

rebase


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


Repository: mesos


Description
-------

In CgroupsAnyHierarchyWithCpuMemoryTest.ROOT_CGROUPS_FreezeNonFreezer, the freezer process
is terminated in it initilize method, which causes the core dump when cgroups::freezer::freeze()
calls dispatch(freezer, &internal::Freezer::freeze). 

This check is added in cgroups::freezer::freeze() to avoid calling dispatch if the freezer
process is terminated.


Diffs (updated)
-----

  src/linux/cgroups.cpp f67633e31884b21e943a06100191d2228a637b1a 

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


Testing
-------

./mesos-tests.sh --gtest_filter="CgroupsAnyHierarchyWithCpuMemoryTest.ROOT_CGROUPS_FreezeNonFreezer"
--gtest_repeat=1000 --gtest_break_on_failure


Thanks,

Jian Qiu


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