You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@geode.apache.org by Anthony Baker <ab...@apache.org> on 2016/06/14 16:58:58 UTC

Re: Review Request 47157: GEODE-1374: Add a flakyTest build task

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

(Updated June 14, 2016, 4:58 p.m.)


Review request for geode, Dick Cavender, Jens Deppe, Kirk Lund, Mark Bretl, and Dan Smith.


Repository: geode


Description
-------

Add a new test task specifically for running tests annotated with
@FlakyTest.  This allows us to isolate tests that exhibit
intermittent failures.  Other changes include:

- Exclude flaky tests from test, integrationTest, uiTest, and
  distributedTest tasks
- Cleanup how tests are found using the minimum set of
  include/exclude categories


Diffs (updated)
-----

  gradle/test.gradle eb10cad78272a54f84e17ca6b984da224dccfbba 

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


Testing
-------

Tested by running test, flakyTest, integrationTest.  Will run further tests before pushing.


Thanks,

Anthony Baker

Re: Review Request 47157: GEODE-1374: Add a flakyTest build task

Posted by Kirk Lund <ki...@gmail.com>.

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


Ship it!




Ship It!

- Kirk Lund


On June 14, 2016, 4:58 p.m., Anthony Baker wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/47157/
> -----------------------------------------------------------
> 
> (Updated June 14, 2016, 4:58 p.m.)
> 
> 
> Review request for geode, Dick Cavender, Jens Deppe, Kirk Lund, Mark Bretl, and Dan Smith.
> 
> 
> Repository: geode
> 
> 
> Description
> -------
> 
> Add a new test task specifically for running tests annotated with
> @FlakyTest.  This allows us to isolate tests that exhibit
> intermittent failures.  Other changes include:
> 
> - Exclude flaky tests from test, integrationTest, uiTest, and
>   distributedTest tasks
> - Cleanup how tests are found using the minimum set of
>   include/exclude categories
> 
> 
> Diffs
> -----
> 
>   gradle/test.gradle eb10cad78272a54f84e17ca6b984da224dccfbba 
> 
> Diff: https://reviews.apache.org/r/47157/diff/
> 
> 
> Testing
> -------
> 
> Tested by running test, flakyTest, integrationTest.  Will run further tests before pushing.
> 
> 
> Thanks,
> 
> Anthony Baker
> 
>

Re: Review Request 47157: GEODE-1374: Add a flakyTest build task

Posted by Mark Bretl <mb...@apache.org>.

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


Ship it!




Ship It!

- Mark Bretl


On June 14, 2016, 9:58 a.m., Anthony Baker wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/47157/
> -----------------------------------------------------------
> 
> (Updated June 14, 2016, 9:58 a.m.)
> 
> 
> Review request for geode, Dick Cavender, Jens Deppe, Kirk Lund, Mark Bretl, and Dan Smith.
> 
> 
> Repository: geode
> 
> 
> Description
> -------
> 
> Add a new test task specifically for running tests annotated with
> @FlakyTest.  This allows us to isolate tests that exhibit
> intermittent failures.  Other changes include:
> 
> - Exclude flaky tests from test, integrationTest, uiTest, and
>   distributedTest tasks
> - Cleanup how tests are found using the minimum set of
>   include/exclude categories
> 
> 
> Diffs
> -----
> 
>   gradle/test.gradle eb10cad78272a54f84e17ca6b984da224dccfbba 
> 
> Diff: https://reviews.apache.org/r/47157/diff/
> 
> 
> Testing
> -------
> 
> Tested by running test, flakyTest, integrationTest.  Will run further tests before pushing.
> 
> 
> Thanks,
> 
> Anthony Baker
> 
>

Re: Review Request 47157: GEODE-1374: Add a flakyTest build task

Posted by Dan Smith <ds...@pivotal.io>.

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


Ship it!




Ship It!

- Dan Smith


On June 14, 2016, 4:58 p.m., Anthony Baker wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/47157/
> -----------------------------------------------------------
> 
> (Updated June 14, 2016, 4:58 p.m.)
> 
> 
> Review request for geode, Dick Cavender, Jens Deppe, Kirk Lund, Mark Bretl, and Dan Smith.
> 
> 
> Repository: geode
> 
> 
> Description
> -------
> 
> Add a new test task specifically for running tests annotated with
> @FlakyTest.  This allows us to isolate tests that exhibit
> intermittent failures.  Other changes include:
> 
> - Exclude flaky tests from test, integrationTest, uiTest, and
>   distributedTest tasks
> - Cleanup how tests are found using the minimum set of
>   include/exclude categories
> 
> 
> Diffs
> -----
> 
>   gradle/test.gradle eb10cad78272a54f84e17ca6b984da224dccfbba 
> 
> Diff: https://reviews.apache.org/r/47157/diff/
> 
> 
> Testing
> -------
> 
> Tested by running test, flakyTest, integrationTest.  Will run further tests before pushing.
> 
> 
> Thanks,
> 
> Anthony Baker
> 
>

Re: Review Request 47157: GEODE-1374: Add a flakyTest build task

Posted by Dick Cavender <dc...@pivotal.io>.

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


Ship it!




Ship It!

- Dick Cavender


On June 14, 2016, 4:58 p.m., Anthony Baker wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/47157/
> -----------------------------------------------------------
> 
> (Updated June 14, 2016, 4:58 p.m.)
> 
> 
> Review request for geode, Dick Cavender, Jens Deppe, Kirk Lund, Mark Bretl, and Dan Smith.
> 
> 
> Repository: geode
> 
> 
> Description
> -------
> 
> Add a new test task specifically for running tests annotated with
> @FlakyTest.  This allows us to isolate tests that exhibit
> intermittent failures.  Other changes include:
> 
> - Exclude flaky tests from test, integrationTest, uiTest, and
>   distributedTest tasks
> - Cleanup how tests are found using the minimum set of
>   include/exclude categories
> 
> 
> Diffs
> -----
> 
>   gradle/test.gradle eb10cad78272a54f84e17ca6b984da224dccfbba 
> 
> Diff: https://reviews.apache.org/r/47157/diff/
> 
> 
> Testing
> -------
> 
> Tested by running test, flakyTest, integrationTest.  Will run further tests before pushing.
> 
> 
> Thanks,
> 
> Anthony Baker
> 
>