So, I played a bit with them and found a few things:

1. On the remote tests, most of the time is consumed on the
container/infinispan startup (~5 seconds).
2. On the remote cluster tests, most of the time is spent on the test
itself (> 13 secs per test on my desktop) and not on the
container/infinispan startup (~5 seconds). So it took about a bit more than
a minute to run all 3 clustered tests.
3. For the remote spring tests, it's the same scenario as #1.

There's a few things we could do to reduce the overall test time for these,
but I'd like feedback before going further.

For the remote cluster tests, I reworked them to use JUnit 5 nested classes
to organize related tests and use per class lifecycle. With this I could
reduce the execution time by 25 seconds for this test group:
-
https://github.com/orpiske/camel/blob/infinispan-perf/components/camel-infinispan/camel-infinispan/src/test/java/org/apache/camel/component/infinispan/remote/cluster/AbstractInfinispanRemoteClusteredTest.java

We could try the same strategy to group together other tests in the remote
and spring packages and squeeze a few more seconds/minutes of the test
time.

The downside is that the nested classes - IMHO - may make the code harder
to read / understand.

Kind regards

On Wed, Feb 3, 2021 at 6:58 PM Otavio Rodolfo Piske <[email protected]>
wrote:

> Thanks Claus,
>
> It is running slow here as well (~3 min on my desktop). I think it may be
> related to the test lifecycle used in the test-infra service for this one.
> I'll take a look.
>
> Kind regards
>
> On Wed, Feb 3, 2021 at 3:52 PM Claus Ibsen <[email protected]> wrote:
>
>> Hi Otavia
>>
>> The CI server seems to run those tests < 1 min so they may be quicker
>> there.
>> But locally it was taking 5+ min or longer on mine - I breaked the tests
>> So it may be slow on my end, but would like to hear from others
>>
>>
>>
>> https://ci-builds.apache.org/job/Camel/job/Camel%20JDK11/job/master/523/testReport/
>>
>>
>> On Wed, Feb 3, 2021 at 3:43 PM Otavio Rodolfo Piske
>> <[email protected]> wrote:
>> >
>> > Hi Claus,
>> >
>> > Is this still a problem? If yes, I can try to take a look at them after
>> > tomorrow.
>> >
>> > Kind regards
>> >
>> > On Sun, Jan 31, 2021 at 4:01 PM Claus Ibsen <[email protected]>
>> wrote:
>> >
>> > > Hi
>> > >
>> > > I wonder if these tests works correctly. On stop it seems there is a 1
>> > > min timeout failure from infispan server about adding client
>> > >
>> > > 2021-01-31 15:56:48,538 [-async-pool-1-2] WARN  HOTROD
>> > >         - ISPN004015: Failed adding new server 172.17.0.5:11222
>> > > io.netty.channel.ConnectTimeoutException: connection timed out:
>> > > /172.17.0.5:11222
>> > >
>> > >
>> > >
>> > >
>> > >
>> > > --
>> > > Claus Ibsen
>> > > -----------------
>> > > http://davsclaus.com @davsclaus
>> > > Camel in Action 2: https://www.manning.com/ibsen2
>> > >
>> >
>> >
>> > --
>> > Otavio R. Piske
>> > http://orpiske.net
>>
>>
>>
>> --
>> Claus Ibsen
>> -----------------
>> http://davsclaus.com @davsclaus
>> Camel in Action 2: https://www.manning.com/ibsen2
>>
>
>
> --
> Otavio R. Piske
> http://orpiske.net
>


-- 
Otavio R. Piske
http://orpiske.net

Reply via email to