[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-openjfx-dev
Subject:    Re: [Integrated] RFR: 8246099: Intermittent test failures in SandboxAppTest
From:       Kevin Rushforth <kcr () openjdk ! java ! net>
Date:       2020-05-29 18:34:42
Message-ID: j8tm43-Z3yLhtcFHJgUDhH1-oZkdc6GzA7VDRW_phP0=.e4dbe5b3-bfdc-46af-90a1-7eefb48d0c32 () github ! com
[Download RAW message or body]

On Thu, 28 May 2020 21:02:08 GMT, Kevin Rushforth <kcr@openjdk.org> wrote:

> This is a fix for an intermittent test failure, due to a timeout, in \
> `SandboxAppTest` that we are starting to see on some of our test machines.
> The failure is happening because of a 10 second timeout in the launched \
> application. This doesn't seem to be enough to run the test to completion in all \
> cases. I note that the similar launched apps, such as those launched by \
> `MainLauncherTest`, `ModuleLauncherTest`, and `JarLauncherTest` don't have a \
> timeout (they just rely on the timeout value of the launching test itself).  The \
> app timeout was just added to prevent the test suite from hanging if the \
> application were to get stuck, so there is no need for it to be such a small value \
> (nor to have it be less than the test timeout).  The proposed solution is to \
> increase the app timeout to 30 seconds (up from 10) and the test timeout to 25 (up \
> from 15).

This pull request has now been integrated.

Changeset: 168b7f79
Author:    Kevin Rushforth <kcr@openjdk.org>
URL:       https://git.openjdk.java.net/jfx/commit/168b7f79
Stats:     7 lines in 2 files changed: 0 ins; 0 del; 7 mod

8246099: Intermittent test failures in SandboxAppTest

Reviewed-by: arapte, jvos

-------------

PR: https://git.openjdk.java.net/jfx/pull/241


[prev in list] [next in list] [prev in thread] [next in thread]