Re: RFR: 8341436: containers/docker/TestJcmdWithSideCar.java takes needlessly long to run [v3]

2024-11-25 Thread Kevin Walls
On Wed, 9 Oct 2024 19:45:24 GMT, Kevin Walls wrote: >> Sebastian Lövdahl has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Have EventGeneratorLoop end after a more predictable duration > > Yes OEL was Oracle Linux: > > Our CI tests with d

Re: RFR: 8341436: containers/docker/TestJcmdWithSideCar.java takes needlessly long to run [v3]

2024-11-25 Thread Sebastian Lövdahl
On Fri, 22 Nov 2024 21:56:40 GMT, Leonid Mesnik wrote: >> Sebastian Lövdahl has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Have EventGeneratorLoop end after a more predictable duration > > Marked as reviewed by lmesnik (Reviewer). Than

Re: RFR: 8341436: containers/docker/TestJcmdWithSideCar.java takes needlessly long to run [v3]

2024-11-22 Thread Leonid Mesnik
On Mon, 7 Oct 2024 19:37:51 GMT, Sebastian Lövdahl wrote: >> The fix is twofold. >> >> 1. Stop the main container after an iteration is done. The main container is >> started with its runtime defined as 120 seconds, which means that each >> iteration takes 120 seconds. In reality, one iteratio

Re: RFR: 8341436: containers/docker/TestJcmdWithSideCar.java takes needlessly long to run [v3]

2024-11-07 Thread Sebastian Lövdahl
On Wed, 9 Oct 2024 19:45:24 GMT, Kevin Walls wrote: >> Sebastian Lövdahl has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Have EventGeneratorLoop end after a more predictable duration > > Yes OEL was Oracle Linux: > > Our CI tests with d

Re: RFR: 8341436: containers/docker/TestJcmdWithSideCar.java takes needlessly long to run [v3]

2024-10-16 Thread Andrey Turbanov
On Mon, 7 Oct 2024 19:37:51 GMT, Sebastian Lövdahl wrote: >> The fix is twofold. >> >> 1. Stop the main container after an iteration is done. The main container is >> started with its runtime defined as 120 seconds, which means that each >> iteration takes 120 seconds. In reality, one iteratio

Re: RFR: 8341436: containers/docker/TestJcmdWithSideCar.java takes needlessly long to run [v3]

2024-10-11 Thread Sebastian Lövdahl
On Mon, 7 Oct 2024 19:37:51 GMT, Sebastian Lövdahl wrote: >> The fix is twofold. >> >> 1. Stop the main container after an iteration is done. The main container is >> started with its runtime defined as 120 seconds, which means that each >> iteration takes 120 seconds. In reality, one iteratio

Re: RFR: 8341436: containers/docker/TestJcmdWithSideCar.java takes needlessly long to run [v3]

2024-10-10 Thread Sebastian Lövdahl
On Wed, 9 Oct 2024 19:45:24 GMT, Kevin Walls wrote: >> Sebastian Lövdahl has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Have EventGeneratorLoop end after a more predictable duration > > Yes OEL was Oracle Linux: > > Our CI tests with d

Re: RFR: 8341436: containers/docker/TestJcmdWithSideCar.java takes needlessly long to run [v3]

2024-10-09 Thread Kevin Walls
On Mon, 7 Oct 2024 19:37:51 GMT, Sebastian Lövdahl wrote: >> The fix is twofold. >> >> 1. Stop the main container after an iteration is done. The main container is >> started with its runtime defined as 120 seconds, which means that each >> iteration takes 120 seconds. In reality, one iteratio

Re: RFR: 8341436: containers/docker/TestJcmdWithSideCar.java takes needlessly long to run [v3]

2024-10-09 Thread Sebastian Lövdahl
On Wed, 9 Oct 2024 11:11:24 GMT, Kevin Walls wrote: >> Sebastian Lövdahl has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Have EventGeneratorLoop end after a more predictable duration > > Hmm, actually no I _can_ still get a failure. > >

Re: RFR: 8341436: containers/docker/TestJcmdWithSideCar.java takes needlessly long to run [v3]

2024-10-09 Thread Kevin Walls
On Mon, 7 Oct 2024 19:37:51 GMT, Sebastian Lövdahl wrote: >> The fix is twofold. >> >> 1. Stop the main container after an iteration is done. The main container is >> started with its runtime defined as 120 seconds, which means that each >> iteration takes 120 seconds. In reality, one iteratio

Re: RFR: 8341436: containers/docker/TestJcmdWithSideCar.java takes needlessly long to run [v3]

2024-10-08 Thread Daniel D . Daugherty
On Mon, 7 Oct 2024 19:37:51 GMT, Sebastian Lövdahl wrote: >> The fix is twofold. >> >> 1. Stop the main container after an iteration is done. The main container is >> started with its runtime defined as 120 seconds, which means that each >> iteration takes 120 seconds. In reality, one iteratio

Re: RFR: 8341436: containers/docker/TestJcmdWithSideCar.java takes needlessly long to run [v3]

2024-10-08 Thread Daniel D . Daugherty
On Mon, 7 Oct 2024 19:50:46 GMT, Kevin Walls wrote: >> Sebastian Lövdahl has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Have EventGeneratorLoop end after a more predictable duration > > Don't worry I'll close 8341518 as a duplicate. @

Re: RFR: 8341436: containers/docker/TestJcmdWithSideCar.java takes needlessly long to run [v3]

2024-10-07 Thread duke
On Mon, 7 Oct 2024 19:37:51 GMT, Sebastian Lövdahl wrote: >> The fix is twofold. >> >> 1. Stop the main container after an iteration is done. The main container is >> started with its runtime defined as 120 seconds, which means that each >> iteration takes 120 seconds. In reality, one iteratio

Re: RFR: 8341436: containers/docker/TestJcmdWithSideCar.java takes needlessly long to run [v3]

2024-10-07 Thread Kevin Walls
On Mon, 7 Oct 2024 19:37:51 GMT, Sebastian Lövdahl wrote: >> The fix is twofold. >> >> 1. Stop the main container after an iteration is done. The main container is >> started with its runtime defined as 120 seconds, which means that each >> iteration takes 120 seconds. In reality, one iteratio

Re: RFR: 8341436: containers/docker/TestJcmdWithSideCar.java takes needlessly long to run [v3]

2024-10-07 Thread Sebastian Lövdahl
On Mon, 7 Oct 2024 19:37:51 GMT, Sebastian Lövdahl wrote: >> The fix is twofold. >> >> 1. Stop the main container after an iteration is done. The main container is >> started with its runtime defined as 120 seconds, which means that each >> iteration takes 120 seconds. In reality, one iteratio

Re: RFR: 8341436: containers/docker/TestJcmdWithSideCar.java takes needlessly long to run [v3]

2024-10-07 Thread Kevin Walls
On Mon, 7 Oct 2024 19:37:51 GMT, Sebastian Lövdahl wrote: >> The fix is twofold. >> >> 1. Stop the main container after an iteration is done. The main container is >> started with its runtime defined as 120 seconds, which means that each >> iteration takes 120 seconds. In reality, one iteratio

Re: RFR: 8341436: containers/docker/TestJcmdWithSideCar.java takes needlessly long to run [v3]

2024-10-07 Thread Sebastian Lövdahl
On Mon, 7 Oct 2024 19:37:51 GMT, Sebastian Lövdahl wrote: >> The fix is twofold. >> >> 1. Stop the main container after an iteration is done. The main container is >> started with its runtime defined as 120 seconds, which means that each >> iteration takes 120 seconds. In reality, one iteratio

Re: RFR: 8341436: containers/docker/TestJcmdWithSideCar.java takes needlessly long to run

2024-10-07 Thread Kevin Walls
On Mon, 7 Oct 2024 19:20:38 GMT, Sebastian Lövdahl wrote: > Interesting! What could possibly interrupt `EventGeneratorLoop`? It's a good question, but I don't know exactly... We do see interrupted exceptions in testing sometimes, and we might have various tests running on the same system at t

Re: RFR: 8341436: containers/docker/TestJcmdWithSideCar.java takes needlessly long to run [v3]

2024-10-07 Thread Sebastian Lövdahl
> The fix is twofold. > > 1. Stop the main container after an iteration is done. The main container is > started with its runtime defined as 120 seconds, which means that each > iteration takes 120 seconds. In reality, one iteration takes a few seconds > while 115 seconds is spent waiting on th

Re: RFR: 8341436: containers/docker/TestJcmdWithSideCar.java takes needlessly long to run

2024-10-07 Thread Sebastian Lövdahl
On Mon, 7 Oct 2024 11:57:39 GMT, Kevin Walls wrote: >> The fix is twofold. >> >> 1. Stop the main container after an iteration is done. The main container is >> started with its runtime defined as 120 seconds, which means that each >> iteration takes 120 seconds. In reality, one iteration take

Re: RFR: 8341436: containers/docker/TestJcmdWithSideCar.java takes needlessly long to run [v2]

2024-10-07 Thread Sebastian Lövdahl
> The fix is twofold. > > 1. Stop the main container after an iteration is done. The main container is > started with its runtime defined as 120 seconds, which means that each > iteration takes 120 seconds. In reality, one iteration takes a few seconds > while 115 seconds is spent waiting on th

Re: RFR: 8341436: containers/docker/TestJcmdWithSideCar.java takes needlessly long to run

2024-10-07 Thread Kevin Walls
On Thu, 3 Oct 2024 19:13:46 GMT, Sebastian Lövdahl wrote: > The fix is twofold. > > 1. Stop the main container after an iteration is done. The main container is > started with its runtime defined as 120 seconds, which means that each > iteration takes 120 seconds. In reality, one iteration tak

Re: RFR: 8341436: containers/docker/TestJcmdWithSideCar.java takes needlessly long to run

2024-10-03 Thread Sebastian Lövdahl
On Thu, 3 Oct 2024 19:13:46 GMT, Sebastian Lövdahl wrote: > The fix is twofold. > > 1. Stop the main container after an iteration is done. The main container is > started with its runtime defined as 120 seconds, which means that each > iteration takes 120 seconds. In reality, one iteration tak