mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Yan Xu <...@jxu.me>
Subject Re: Reaper related changes
Date Fri, 31 May 2013 05:55:00 GMT
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 <yan@jxu.me> @xujyan <http://twitter.com/xujyan>


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

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