It's changed to: https://reviews.apache.org/r/11552/ https://reviews.apache.org/r/11553/ *https://reviews.apache.org/r/11554/* *https://reviews.apache.org/r/11555/* https://reviews.apache.org/r/11556/
Rebased against the master. Ran the test and passed all but MESOS-487<https://issues.apache.org/jira/browse/MESOS-487> . Should be good to go. -- Jiang Yan Xu <[email protected]> @xujyan <http://twitter.com/xujyan> On Tue, Apr 30, 2013 at 12:09 PM, Yan Xu <[email protected]> wrote: > sudo ./bin/mesos-tests.sh --gtest_filter="SlaveRecoveryTest*" > --gtest_repeat=10 --gtest_break_on_failure passed on Linux. > On OS X it failed at a known issue > https://issues.apache.org/jira/browse/MESOS-393 > > -- > Jiang Yan Xu <[email protected]> @xujyan <http://twitter.com/xujyan> > > > On Tue, Apr 30, 2013 at 11:44 AM, Yan Xu <[email protected]> wrote: > >> Yeah. >> >> Mac & Linux >> >> make check >> ./bin/mesos-tests.sh --gtest_filter="ReaperTest.*" --gtest_repeat=200 >> --gtest_break_on_failure >> >> Linux: >> >> sudo ./bin/mesos-tests.sh --gtest_filter="CgroupsIsolatorTest*" >> --gtest_repeat=10 --gtest_break_on_failure >> sudo ./bin/mesos-tests.sh --gtest_filter="IsolatorTest*" >> --gtest_repeat=20 --gtest_break_on_failure >> >> -- >> Jiang Yan Xu <[email protected]> @xujyan <http://twitter.com/xujyan> >> >> >> On Tue, Apr 30, 2013 at 9:26 AM, Vinod Kone <[email protected]> wrote: >> >>> Yay! Can you confirm that you have tested this on both OSX and Linux w/ >>> multiple runs? >>> >>> >>> @vinodkone >>> >>> >>> On Mon, Apr 29, 2013 at 11:06 PM, Yan Xu <[email protected]> wrote: >>> >>>> Rebased off the trunk. All commits have "Ship it"s. Should be good to >>>> go. >>>> >>>> Yan >>>> -- >>>> Jiang Yan Xu <[email protected]> @xujyan <http://twitter.com/xujyan> >>>> >>>> >>>> On Mon, Apr 29, 2013 at 12:38 AM, Yan Xu <[email protected]> wrote: >>>> >>>>> The sequence is now: >>>>> https://reviews.apache.org/r/10744/ >>>>> https://reviews.apache.org/r/10745/ >>>>> https://reviews.apache.org/r/10746/ >>>>> https://reviews.apache.org/r/10803/ >>>>> https://reviews.apache.org/r/10747/ >>>>> >>>>> -- >>>>> Jiang Yan Xu <[email protected]> @xujyan <http://twitter.com/xujyan> >>>>> >>>>> >>>>> On Tue, Apr 23, 2013 at 6:29 PM, Yan Xu <[email protected]> wrote: >>>>> >>>>>> This batch of commits changed the reaper to use "Future" as the >>>>>> notification mechanism. >>>>>> >>>>>> Sequence: >>>>>> https://reviews.apache.org/r/10744/ >>>>>> https://reviews.apache.org/r/10745/ >>>>>> https://reviews.apache.org/r/10746/ >>>>>> https://reviews.apache.org/r/10747/ >>>>>> >>>>>> Best, >>>>>> Yan >>>>>> -- >>>>>> Jiang Yan Xu <[email protected]> @xujyan <http://twitter.com/xujyan> >>>>>> >>>>> >>>>> >>>> >>> >> >
