Hi,

On Fri, May 31, 2013 at 9:14 PM, Robert Munteanu <romb...@apache.org> wrote:
> ...It seems that the ErrorHandlingTest fails sporadically when run inside a
> full maven build...

I fixed a related issue in SLING-2818 a few weeks ago, is it something else now?

> ....For this test, and for future flaky/failing tests, I
> suggest that we
>
> 1. Create an issue for the failing test
> 2. Disable the test and mark it with the issue key
> 3. Re-enable the test when it is stable/passing ( which may be
> considerably later than step 2)
> 4. Close the issue after the test is re-enabled...

I agree with that, and I would make the next release of the module in
question dependent on that issue, to make sure we look at it before
releasing.

>
> ...This has the advantage of keeping the build green and making it easier
> to find regressions since a failing or unstable build will actually mean
> something....

I agree - Jenkins flakyness and some semi-randomly failing tests lead
us to neglect the "jenkins build must be green" rule and that's not
good.

-Bertrand

Reply via email to