You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Joseph Wu <jo...@mesosphere.io> on 2016/02/25 01:26:58 UTC

Review Request 43971: Fixed flakiness in MasterMaintenanceTest.InverseOffers.

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

Review request for mesos, Artem Harutyunyan and Joris Van Remoortere.


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


Repository: mesos


Description
-------

The accept/decline of the inverse offers in the test could arrive after the clock is paused and advanced in the test.  This meant that the test would trigger an allocation and then idle (because the clock is paused).  This is the same problem we fixed in https://issues.apache.org/jira/browse/MESOS-4059 .


Diffs
-----

  src/tests/master_maintenance_tests.cpp c9e3828933d16ec65298e4ad4b1a3e0ddf99032d 

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


Testing
-------

(Not done yet) make check (OSX)


Thanks,

Joseph Wu


Re: Review Request 43971: Fixed flakiness in MasterMaintenanceTest.InverseOffers.

Posted by Joris Van Remoortere <jo...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43971/#review120671
-----------------------------------------------------------


Ship it!




Ship It!

- Joris Van Remoortere


On Feb. 25, 2016, 1:01 a.m., Joseph Wu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/43971/
> -----------------------------------------------------------
> 
> (Updated Feb. 25, 2016, 1:01 a.m.)
> 
> 
> Review request for mesos, Artem Harutyunyan and Joris Van Remoortere.
> 
> 
> Bugs: MESOS-4768
>     https://issues.apache.org/jira/browse/MESOS-4768
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> The accept/decline of the inverse offers in the test could arrive after the clock is paused and advanced in the test.  This meant that the test would trigger an allocation and then idle (because the clock is paused).  This is the same problem we fixed in https://issues.apache.org/jira/browse/MESOS-4059 .
> 
> 
> Diffs
> -----
> 
>   src/tests/master_maintenance_tests.cpp c9e3828933d16ec65298e4ad4b1a3e0ddf99032d 
> 
> Diff: https://reviews.apache.org/r/43971/diff/
> 
> 
> Testing
> -------
> 
> make check (OSX)
> 
> bin/mesos-tests.sh --gtest_filter="MasterMaintenanceTest.InverseOffers" --gtest_repeat=5000 --gtest_break_on_failure
> 
> 
> Thanks,
> 
> Joseph Wu
> 
>


Re: Review Request 43971: Fixed flakiness in MasterMaintenanceTest.InverseOffers.

Posted by haosdent huang <ha...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43971/#review120652
-----------------------------------------------------------


Ship it!




Ship It!

- haosdent huang


On Feb. 25, 2016, 1:01 a.m., Joseph Wu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/43971/
> -----------------------------------------------------------
> 
> (Updated Feb. 25, 2016, 1:01 a.m.)
> 
> 
> Review request for mesos, Artem Harutyunyan and Joris Van Remoortere.
> 
> 
> Bugs: MESOS-4768
>     https://issues.apache.org/jira/browse/MESOS-4768
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> The accept/decline of the inverse offers in the test could arrive after the clock is paused and advanced in the test.  This meant that the test would trigger an allocation and then idle (because the clock is paused).  This is the same problem we fixed in https://issues.apache.org/jira/browse/MESOS-4059 .
> 
> 
> Diffs
> -----
> 
>   src/tests/master_maintenance_tests.cpp c9e3828933d16ec65298e4ad4b1a3e0ddf99032d 
> 
> Diff: https://reviews.apache.org/r/43971/diff/
> 
> 
> Testing
> -------
> 
> make check (OSX)
> 
> bin/mesos-tests.sh --gtest_filter="MasterMaintenanceTest.InverseOffers" --gtest_repeat=5000 --gtest_break_on_failure
> 
> 
> Thanks,
> 
> Joseph Wu
> 
>


Re: Review Request 43971: Fixed flakiness in MasterMaintenanceTest.InverseOffers.

Posted by Joseph Wu <jo...@mesosphere.io>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43971/
-----------------------------------------------------------

(Updated Feb. 24, 2016, 5:01 p.m.)


Review request for mesos, Artem Harutyunyan and Joris Van Remoortere.


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


Repository: mesos


Description
-------

The accept/decline of the inverse offers in the test could arrive after the clock is paused and advanced in the test.  This meant that the test would trigger an allocation and then idle (because the clock is paused).  This is the same problem we fixed in https://issues.apache.org/jira/browse/MESOS-4059 .


Diffs
-----

  src/tests/master_maintenance_tests.cpp c9e3828933d16ec65298e4ad4b1a3e0ddf99032d 

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


Testing
-------

make check (OSX)

bin/mesos-tests.sh --gtest_filter="MasterMaintenanceTest.InverseOffers" --gtest_repeat=5000 --gtest_break_on_failure


Thanks,

Joseph Wu


Re: Review Request 43971: Fixed flakiness in MasterMaintenanceTest.InverseOffers.

Posted by Joseph Wu <jo...@mesosphere.io>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43971/
-----------------------------------------------------------

(Updated Feb. 24, 2016, 4:51 p.m.)


Review request for mesos, Artem Harutyunyan and Joris Van Remoortere.


Changes
-------

Finished testing the patch.


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


Repository: mesos


Description
-------

The accept/decline of the inverse offers in the test could arrive after the clock is paused and advanced in the test.  This meant that the test would trigger an allocation and then idle (because the clock is paused).  This is the same problem we fixed in https://issues.apache.org/jira/browse/MESOS-4059 .


Diffs
-----

  src/tests/master_maintenance_tests.cpp c9e3828933d16ec65298e4ad4b1a3e0ddf99032d 

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


Testing (updated)
-------

make check (OSX)

bin/mesos-tests.sh --gtest_filter="MasterMaintenanceTest.InverseOffers" --gtest_repeat=5000 --gtest_break_on_failure


Thanks,

Joseph Wu