You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mesos.apache.org by Yan Xu <ya...@jxu.me> on 2013/05/31 07:55:00 UTC
Re: Reaper related changes

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 <ya...@jxu.me> @xujyan <http://twitter.com/xujyan>


On Tue, Apr 30, 2013 at 12:09 PM, Yan Xu <ya...@jxu.me> 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 <ya...@twitter.com> @xujyan <http://twitter.com/xujyan>
>
>
> On Tue, Apr 30, 2013 at 11:44 AM, Yan Xu <ya...@jxu.me> 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 <ya...@twitter.com> @xujyan <http://twitter.com/xujyan>
>>
>>
>> On Tue, Apr 30, 2013 at 9:26 AM, Vinod Kone <vi...@twitter.com> 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 <ya...@jxu.me> wrote:
>>>
>>>> Rebased off the trunk. All commits have "Ship it"s. Should be good to
>>>> go.
>>>>
>>>> Yan
>>>> --
>>>> Jiang Yan Xu <ya...@twitter.com> @xujyan <http://twitter.com/xujyan>
>>>>
>>>>
>>>> On Mon, Apr 29, 2013 at 12:38 AM, Yan Xu <ya...@jxu.me> 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 <ya...@twitter.com> @xujyan <http://twitter.com/xujyan>
>>>>>
>>>>>
>>>>> On Tue, Apr 23, 2013 at 6:29 PM, Yan Xu <ya...@jxu.me> 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 <ya...@twitter.com> @xujyan <http://twitter.com/xujyan>
>>>>>>
>>>>>
>>>>>
>>>>
>>>
>>
>