Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added support for Kafka 3.1.0 #774

Merged
merged 3 commits into from
Feb 25, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions api/v1beta1/kafkacluster_types.go
Original file line number Diff line number Diff line change
Expand Up @@ -627,7 +627,7 @@ func (kSpec *KafkaClusterSpec) GetClusterImage() string {
if kSpec.ClusterImage != "" {
return kSpec.ClusterImage
}
return "ghcr.io/banzaicloud/kafka:2.13-2.8.1"
return "ghcr.io/banzaicloud/kafka:2.13-3.1.0"
panyuenlau marked this conversation as resolved.
Show resolved Hide resolved
}

// GetClusterMetricsReporterImage returns the default container image for Kafka Cluster
Expand Down Expand Up @@ -878,7 +878,7 @@ func (cConfig *CruiseControlConfig) GetCCImage() string {
if cConfig.Image != "" {
return cConfig.Image
}
return "ghcr.io/banzaicloud/cruise-control:2.5.80"
return "ghcr.io/banzaicloud/cruise-control:2.5.85"
}

// GetCCLog4jConfig returns the used Cruise Control log4j configuration
Expand Down
8 changes: 4 additions & 4 deletions config/samples/banzaicloud_v1beta1_kafkacluster.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ spec:
oneBrokerPerNode: false
# Specify the Kafka Broker related settings
# clusterImage can specify the whole kafkacluster image in one place
#clusterImage: "ghcr.io/banzaicloud/kafka:2.13-2.8.1
#clusterImage: "ghcr.io/banzaicloud/kafka:2.13-3.1.0
# readOnlyConfig specifies the read-only type kafka config cluster wide, all these will be merged with broker specified
# readOnly configurations, so it can be overwritten per broker.
#clusterWideConfig specifies the cluster-wide kafka config cluster wide, all these can be overridden per-broker
Expand Down Expand Up @@ -71,7 +71,7 @@ spec:
cruise.control.metrics.topic.replication.factor=2
brokerConfig:
# Docker image used by the operator to create the Broker with id 0
#image: "ghcr.io/banzaicloud/kafka:2.13-2.8.1
#image: "ghcr.io/banzaicloud/kafka:2.13-3.1.0
# resourceRequirements works exactly like Container resources, the user can specify the limit and the requests
# through this property
#resourceRequirements:
Expand Down Expand Up @@ -134,7 +134,7 @@ spec:
#volumeMounts:
# - mountPath: "/opt/kafka/libs/extra-jars"
# name: "jars"
- image: "ghcr.io/banzaicloud/kafka:2.13-2.8.1"
- image: "ghcr.io/banzaicloud/kafka:2.13-3.1.0"
id: 1
config: |
auto.create.topics.enable=false
Expand All @@ -156,7 +156,7 @@ spec:
readOnlyConfig: |
auto.create.topics.enable=false
brokerConfig:
image: "ghcr.io/banzaicloud/kafka:2.13-2.8.1"
image: "ghcr.io/banzaicloud/kafka:2.13-3.1.0"
storageConfigs:
- mountPath: "/kafka-logs"
pvcSpec:
Expand Down
6 changes: 3 additions & 3 deletions config/samples/kafkacluster-prometheus.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ spec:
storageClass: 'gp2'
mountPath: '/kafkalog'
diskSize: '10G'
image: 'ghcr.io/banzaicloud/kafka:2.13-2.8.1'
image: 'ghcr.io/banzaicloud/kafka:2.13-3.1.0'
# annotations to be applied onto the broker pod
# brokerAnnotations: '{ "sidecar.istio.io/logLevel": "trace", "sidecar.istio.io/proxyCPULimit": "50m" }'
command: 'upScale'
Expand All @@ -101,7 +101,7 @@ spec:
storageClass: 'gp2'
mountPath: '/kafkalog'
diskSize: '10G'
image: 'ghcr.io/banzaicloud/kafka:2.13-2.8.1'
image: 'ghcr.io/banzaicloud/kafka:2.13-3.1.0'
# annotations to be applied onto the broker pod
# brokerAnnotations: '{ "sidecar.istio.io/logLevel": "trace", "sidecar.istio.io/proxyCPULimit": "50m" }'
command: 'upScale'
Expand All @@ -117,7 +117,7 @@ spec:
storageClass: 'gp2'
mountPath: '/kafkalog'
diskSize: '10G'
image: 'ghcr.io/banzaicloud/kafka:2.13-2.8.1'
image: 'ghcr.io/banzaicloud/kafka:2.13-3.1.0'
# annotations to be applied onto the broker pod
# brokerAnnotations: '{ "sidecar.istio.io/logLevel": "trace", "sidecar.istio.io/proxyCPULimit": "50m" }'
command: 'upScale'
Expand Down
2 changes: 1 addition & 1 deletion config/samples/kafkacluster-with-istio.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ spec:
zkAddresses:
- "zookeeper-client.zookeeper:2181"
oneBrokerPerNode: false
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-2.8.1"
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-3.1.0"
readOnlyConfig: |
auto.create.topics.enable=false
cruise.control.metrics.topic.auto.create=true
Expand Down
2 changes: 1 addition & 1 deletion config/samples/kafkacluster_with_external_ssl.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ spec:
zkAddresses:
- "zookeeper-client.zookeeper:2181"
oneBrokerPerNode: false
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-2.8.1"
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-3.1.0"
readOnlyConfig: |
auto.create.topics.enable=false
cruise.control.metrics.topic.auto.create=true
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ spec:
zkAddresses:
- "zookeeper-client.zookeeper:2181"
oneBrokerPerNode: false
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-2.8.1"
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-3.1.0"
readOnlyConfig: |
auto.create.topics.enable=false
cruise.control.metrics.topic.auto.create=true
Expand Down
2 changes: 1 addition & 1 deletion config/samples/kafkacluster_with_ssl_groups.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ spec:
- "failure-domain.beta.kubernetes.io/region"
- "failure-domain.beta.kubernetes.io/zone"
oneBrokerPerNode: false
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-2.8.1"
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-3.1.0"
readOnlyConfig: |
auto.create.topics.enable=false
cruise.control.metrics.topic.auto.create=true
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ spec:
- "failure-domain.beta.kubernetes.io/region"
- "failure-domain.beta.kubernetes.io/zone"
oneBrokerPerNode: false
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-2.8.1"
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-3.1.0"
readOnlyConfig: |
auto.create.topics.enable=false
cruise.control.metrics.topic.auto.create=true
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ spec:
- "failure-domain.beta.kubernetes.io/region"
- "failure-domain.beta.kubernetes.io/zone"
oneBrokerPerNode: false
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-2.8.1"
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-3.1.0"
readOnlyConfig: |
auto.create.topics.enable=false
cruise.control.metrics.topic.auto.create=true
Expand Down
2 changes: 1 addition & 1 deletion config/samples/kafkacluster_without_ssl.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ spec:
zkAddresses:
- "zookeeper-client.zookeeper:2181"
oneBrokerPerNode: false
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-2.8.1"
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-3.1.0"
readOnlyConfig: |
auto.create.topics.enable=false
cruise.control.metrics.topic.auto.create=true
Expand Down
2 changes: 1 addition & 1 deletion config/samples/kafkacluster_without_ssl_groups.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ spec:
zkAddresses:
- "zookeeper-client.zookeeper:2181"
oneBrokerPerNode: false
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-2.8.1"
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-3.1.0"
readOnlyConfig: |
auto.create.topics.enable=false
cruise.control.metrics.topic.auto.create=true
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ spec:
- "zookeeper-client.zookeeper:2181"
propagateLabels: false
oneBrokerPerNode: false
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-2.8.1"
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-3.1.0"
readOnlyConfig: |
auto.create.topics.enable=false
cruise.control.metrics.topic.auto.create=true
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ spec:
- "zookeeper-client.zookeeper:2181"
propagateLabels: false
oneBrokerPerNode: false
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-2.8.1"
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-3.1.0"
readOnlyConfig: |
auto.create.topics.enable=false
cruise.control.metrics.topic.auto.create=true
Expand Down
4 changes: 2 additions & 2 deletions config/samples/simplekafkacluster.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ spec:
- "zookeeper-client.zookeeper:2181"
propagateLabels: false
oneBrokerPerNode: false
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-2.8.1"
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-3.1.0"
readOnlyConfig: |
auto.create.topics.enable=false
cruise.control.metrics.topic.auto.create=true
Expand Down Expand Up @@ -78,7 +78,7 @@ spec:
# limits:
# cpu: 500m
# memory: 1Gi
# image: "ghcr.io/banzaicloud/cruise-control:2.5.80"
# image: "ghcr.io/banzaicloud/cruise-control:2.5.85"
config: |
# Copyright 2017 LinkedIn Corp. Licensed under the BSD 2-Clause License (the "License"). See License in the project root for license information.
#
Expand Down
2 changes: 1 addition & 1 deletion config/samples/simplekafkacluster_affinity.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ spec:
# If true OneBrokerPerNode ensures that each kafka broker will be placed on a different node unless a custom
# Affinity definition overrides this behavior either set in brokerConfigGroups or in one specific brokerConfig
oneBrokerPerNode: false
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-2.8.1"
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-3.1.0"
readOnlyConfig: |
auto.create.topics.enable=false
cruise.control.metrics.topic.auto.create=true
Expand Down
2 changes: 1 addition & 1 deletion config/samples/simplekafkacluster_ebs_csi.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ spec:
zkPath: "/kafka"
propagateLabels: true
oneBrokerPerNode: false
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-2.8.1"
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-3.1.0"
readOnlyConfig: |
auto.create.topics.enable=false
cruise.control.metrics.topic.auto.create=true
Expand Down
2 changes: 1 addition & 1 deletion config/samples/simplekafkacluster_ssl.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ spec:
- "zookeeper-client.zookeeper:2181"
propagateLabels: false
oneBrokerPerNode: false
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-2.6.0-bzc.1"
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-3.1.0"
readOnlyConfig: |
auto.create.topics.enable=false
cruise.control.metrics.topic.auto.create=true
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ spec:
- "zookeeper-client.zookeeper:2181"
propagateLabels: false
oneBrokerPerNode: false
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-2.8.1"
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-3.1.0"
readOnlyConfig: |
auto.create.topics.enable=false
cruise.control.metrics.topic.auto.create=true
Expand Down
2 changes: 1 addition & 1 deletion config/samples/simplekafkacluster_with_sasl.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ spec:
- "zookeeper-client.zookeeper:2181"
propagateLabels: false
oneBrokerPerNode: false
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-2.8.1"
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-3.1.0"
readOnlyConfig: |
auto.create.topics.enable=false
cruise.control.metrics.topic.auto.create=true
Expand Down
2 changes: 1 addition & 1 deletion controllers/tests/clusterregistry/common_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@ func createMinimalKafkaClusterCR(name, namespace string) *v1beta1.KafkaCluster {
BrokerConfigGroup: defaultBrokerConfigGroup,
},
},
ClusterImage: "ghcr.io/banzaicloud/kafka:2.13-2.8.1",
ClusterImage: "ghcr.io/banzaicloud/kafka:2.13-3.1.0",
ZKAddresses: []string{},
MonitoringConfig: v1beta1.MonitoringConfig{
CCJMXExporterConfig: "custom_property: custom_value",
Expand Down
2 changes: 1 addition & 1 deletion controllers/tests/common_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ func createMinimalKafkaClusterCR(name, namespace string) *v1beta1.KafkaCluster {
BrokerConfigGroup: defaultBrokerConfigGroup,
},
},
ClusterImage: "ghcr.io/banzaicloud/kafka:2.13-2.8.1",
ClusterImage: "ghcr.io/banzaicloud/kafka:2.13-3.1.0",
ZKAddresses: []string{},
MonitoringConfig: v1beta1.MonitoringConfig{
CCJMXExporterConfig: "custom_property: custom_value",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -259,7 +259,7 @@ func expectCruiseControlDeployment(kafkaCluster *v1beta1.KafkaCluster) {
Expect(container.Lifecycle).NotTo(BeNil())
Expect(container.Lifecycle.PreStop).NotTo(BeNil())
Expect(container.Lifecycle.PreStop.Exec).NotTo(BeNil())
Expect(container.Image).To(Equal("ghcr.io/banzaicloud/cruise-control:2.5.80"))
Expect(container.Image).To(Equal("ghcr.io/banzaicloud/cruise-control:2.5.85"))
Expect(container.Ports).To(ConsistOf(
corev1.ContainerPort{
ContainerPort: 8090,
Expand Down
2 changes: 1 addition & 1 deletion controllers/tests/kafkacluster_controller_kafka_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -251,7 +251,7 @@ func expectKafkaBrokerPod(kafkaCluster *v1beta1.KafkaCluster, broker v1beta1.Bro
Expect(pod.Spec.Containers[1]).To(WithTransform(getContainerName, Equal("test-container")))
container := pod.Spec.Containers[0]
Expect(container.Name).To(Equal("kafka"))
Expect(container.Image).To(Equal("ghcr.io/banzaicloud/kafka:2.13-2.8.1"))
Expect(container.Image).To(Equal("ghcr.io/banzaicloud/kafka:2.13-3.1.0"))
Expect(container.Lifecycle).NotTo(BeNil())
Expect(container.Lifecycle.PreStop).NotTo(BeNil())
getEnvName := func(c corev1.EnvVar) string { return c.Name }
Expand Down
2 changes: 1 addition & 1 deletion docs/benchmarks/infrastructure/kafka.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ spec:
zkAddresses:
- "zookeeper-client.zookeeper:2181"
oneBrokerPerNode: true
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-2.8.1"
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-3.1.0"
readOnlyConfig: |
auto.create.topics.enable=false
rackAwareness:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ spec:
zkAddresses:
- "zookeeper-client.zookeeper:2181"
oneBrokerPerNode: true
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-2.8.1"
clusterImage: "ghcr.io/banzaicloud/kafka:2.13-3.1.0"
readOnlyConfig: |
auto.create.topics.enable=false
min.insync.replicas=3
Expand Down