Skip to content

Commit

Permalink
Update to the newest Kafka + some minor changes (#53)
Browse files Browse the repository at this point in the history
* Update to the newest Kafka + some minor changes

Signed-off-by: see-quick <maros.orsak159@gmail.com>

* tom commend

Signed-off-by: see-quick <maros.orsak159@gmail.com>

* back to contains but let startWith method

Signed-off-by: see-quick <maros.orsak159@gmail.com>

---------

Signed-off-by: see-quick <maros.orsak159@gmail.com>
  • Loading branch information
see-quick authored Feb 28, 2023
1 parent c5f9854 commit a4fa1ce
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,7 @@ protected void doStart() {
this.imageNameProvider.complete(KafkaVersionService.strimziTestContainerImageName(this.kafkaVersion));
}
try {
if (this.useKraft && ((this.kafkaVersion != null && this.kafkaVersion.equals("2.8.1")) || this.imageNameProvider.get().contains("2.8.1"))) {
if (this.useKraft && ((this.kafkaVersion != null && this.kafkaVersion.startsWith("2.")) || this.imageNameProvider.get().contains("2.8.2"))) {
throw new UnsupportedKraftKafkaVersionException("Specified Kafka version " + this.kafkaVersion + " is not supported in KRaft mode.");
}
} catch (InterruptedException | ExecutionException e) {
Expand Down Expand Up @@ -163,7 +163,7 @@ public StrimziKafkaContainer waitForRunning() {
if (this.useKraft) {
super.waitingFor(Wait.forLogMessage(".*Transitioning from RECOVERY to RUNNING.*", 1));
} else {
super.waitingFor(Wait.forLogMessage(".*Recorded new controller, from now on will use broker.*", 1));
super.waitingFor(Wait.forLogMessage(".*Recorded new controller, from now on will use [node|broker].*", 1));
}
return this;
}
Expand Down
12 changes: 6 additions & 6 deletions src/main/resources/kafka_versions.json
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
{
"kafkaVersions": {
"2.8.1": "quay.io/strimzi-test-container/test-container:latest-kafka-2.8.1",
"3.0.1": "quay.io/strimzi-test-container/test-container:latest-kafka-3.0.1",
"3.1.0": "quay.io/strimzi-test-container/test-container:latest-kafka-3.1.0",
"3.1.1": "quay.io/strimzi-test-container/test-container:latest-kafka-3.1.1",
"3.2.0": "quay.io/strimzi-test-container/test-container:latest-kafka-3.2.0",
"3.2.1": "quay.io/strimzi-test-container/test-container:latest-kafka-3.2.1"
"2.8.2": "quay.io/strimzi-test-container/test-container:latest-kafka-2.8.2",
"3.0.2": "quay.io/strimzi-test-container/test-container:latest-kafka-3.0.2",
"3.1.2": "quay.io/strimzi-test-container/test-container:latest-kafka-3.1.2",
"3.2.3": "quay.io/strimzi-test-container/test-container:latest-kafka-3.2.3",
"3.3.2": "quay.io/strimzi-test-container/test-container:latest-kafka-3.3.2",
"3.4.0": "quay.io/strimzi-test-container/test-container:latest-kafka-3.4.0"
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,7 @@ void testUnsupportedKRaftUsingKafkaVersion() {

try {
systemUnderTest = new StrimziKafkaContainer()
.withKafkaVersion("2.8.1")
.withKafkaVersion("2.8.2")
.withBrokerId(1)
.withKraft()
.waitForRunning();
Expand All @@ -127,7 +127,7 @@ void testUnsupportedKRaftUsingImageName() {
assumeDocker();

try {
systemUnderTest = new StrimziKafkaContainer("quay.io/strimzi-test-container/test-container:latest-kafka-2.8.1")
systemUnderTest = new StrimziKafkaContainer("quay.io/strimzi-test-container/test-container:latest-kafka-2.8.2")
.withBrokerId(1)
.withKraft()
.waitForRunning();
Expand Down

0 comments on commit a4fa1ce

Please sign in to comment.