Migrate integration tests to KRaft broker

[loicgreffier]

No description provided.

[sonarqubecloud]

Quality Gate passed

Issues
3 New issues
0 Accepted issues

Measures
0 Security Hotspots
88.9% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[loicgreffier]

@ThomasCAI-mlv

This PR migrates current integration tests to KRaft:

• Version is fixed to 1.20.0 for this fix: [Bug]: Running multiple instances of static Kraft container will fail on listeners config testcontainers/testcontainers-java#8619.
• TopicIntegrationTest use Thread.sleep to fix unstable tests

[ThomasCAI-mlv]

This sleep patches KRaft-related unstability or general unstability?

[loicgreffier]

Well it happens after migrating, in TopicIntegrationTest#shouldUpdateTopic, where it takes some time for the broker to update the topic configuration despite Ns4Kafka topic executor ended the synchronization successfully.

It precisely happens when checking the configuration update here:

• ns4kafka/src/test/java/com/michelin/ns4kafka/integration/TopicIntegrationTest.java

  Line 342 in 948322f

  "retention.ms", "70000")) //This line was changed

• ns4kafka/src/test/java/com/michelin/ns4kafka/integration/TopicIntegrationTest.java

  Line 385 in 948322f

  valueToVerify.forEach(entry -> assertEquals(config.get(entry.name()), entry.value()));

Without sleeping, retention.ms can be still equal to 60000 (like 1 time over 10...)

I'm still wondering how it can be related to KRaft

[ThomasCAI-mlv]

Ok for me then