Skip to content

Commit 0032fbd

Browse files
committed
Update e2e tests to use Adobe images (#148)
1 parent 0bb02b2 commit 0032fbd

39 files changed

+205
-196
lines changed

.github/workflows/e2e-test.yaml

Lines changed: 16 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -30,16 +30,27 @@ jobs:
3030
uses: actions/setup-go@v6
3131
with:
3232
go-version: '1.25'
33+
34+
- name: Setup Helm
35+
uses: azure/setup-helm@v4
36+
with:
37+
version: '3.19.0'
3338

3439
- name: Cache Go modules
3540
uses: actions/cache@v4
3641
with:
37-
path: |
38-
~/.cache/go-build
39-
~/go/pkg/mod
40-
key: ${{ runner.os }}-go-${{ hashFiles('go.sum', 'api/go.sum', 'properties/go.sum', 'tests/e2e/go.sum') }}
42+
path: ~/go/pkg/mod
43+
key: ${{ runner.os }}-gomod-${{ hashFiles('**/go.sum') }}
44+
restore-keys: |
45+
${{ runner.os }}-gomod-
46+
47+
- name: Cache Go build cache
48+
uses: actions/cache@v4
49+
with:
50+
path: ~/.cache/go-build
51+
key: ${{ runner.os }}-gobuild-${{ github.sha }}
4152
restore-keys: |
42-
${{ runner.os }}-go-
53+
${{ runner.os }}-gobuild-
4354
4455
# Enable Tmate Session if you'd like to Debut the E2E Kind Cluster
4556
# - name: Setup tmate session

.github/workflows/helm.yml

Lines changed: 0 additions & 102 deletions
This file was deleted.

CONTRIBUTING.md

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ any real-time space e.g., Slack, Discord, etc.
1414
## Reporting Issues
1515

1616
Before reporting a new issue, please ensure that the issue was not already reported or fixed by searching through our
17-
[issues list](https://github.com/banzaicloud/koperator/issues).
17+
[issues list](https://github.com/adobe/koperator/issues).
1818

1919
When creating a new issue, please be sure to include a **title and clear description**, as much relevant information as
2020
possible, and, if possible, a test case.
@@ -37,8 +37,6 @@ and fix existing bugs. Here is what you can do:
3737

3838
- Help ensure that existing issues follows the recommendations from the _[Reporting Issues](#reporting-issues)_ section,
3939
providing feedback to the issue's author on what might be missing.
40-
- Review and update the existing content of our [Wiki](https://github.com/banzaicloud/koperator/wiki) with up-to-date
41-
instructions and code samples.
4240
- Review existing pull requests, and testing patches against real existing applications that use Koperator.
4341
- Write a test, or add a missing test case to an existing test.
4442

api/assets/kafka/jmx-exporter.yml

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,20 @@
11
lowercaseOutputName: true
22
rules:
33
# Special cases and very specific rules
4-
# Export kraft current state metrics
5-
- pattern: 'kafka.server<type=raft-metrics><>current-state: (.+)'
6-
name: kafka_server_raft_metrics_current_state_$1
4+
# KRaft current state info metric rule
5+
- pattern: "kafka.server<type=raft-metrics><>current-state: ([a-z]+)"
6+
name: kafka_server_raft_metrics_current_state_info
77
type: GAUGE
88
value: 1
9+
labels:
10+
"state": "$1"
11+
# KRaft specific rules for raft-metrics, raft-channel-metrics, broker-metadata-metrics
12+
- pattern: kafka.server<type=(.+)><>([a-z-]+)-total
13+
name: kafka_server_$1_$2_total
14+
type: COUNTER
15+
- pattern: kafka.server<type=(.+)><>([a-z-]+)
16+
name: kafka_server_$1_$2
17+
type: GAUGE
918
- pattern: 'kafka.server<type=(app-info), id=(\d+)><>(Version): ([-.~+\w\d]+)'
1019
name: kafka_server_$1_$3
1120
type: COUNTER

api/assets/kafka/kraft-controller-healthcheck.sh

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -25,16 +25,16 @@ if [ "$skip_check" = "true" ]; then
2525
fi
2626

2727
JMX_ENDPOINT="http://localhost:9020/metrics"
28-
METRIC_PREFIX="kafka_server_raft_metrics_current_state_"
28+
METRIC_NAME="kafka_server_raft_metrics_current_state_info"
2929

30-
# Fetch the matching current-state metric with value of 1.0 from the JMX endpoint
31-
MATCHING_METRIC=$(curl -s "$JMX_ENDPOINT" | grep "^${METRIC_PREFIX}" | awk '$2 == 1.0 {print $1}')
30+
# Fetch the current-state metric with value of 1.0 from the JMX endpoint
31+
MATCHING_METRIC=$(curl -s "$JMX_ENDPOINT" | grep "^${METRIC_NAME}" | awk '$2 == 1.0 {print $0}')
3232

3333
# If it's not empty, it means we found a metric with a value of 1.0.
3434
if [ -n "$MATCHING_METRIC" ]; then
35-
# Determine the state of the controller using the last field name of the metric
36-
# Possible values are leader, candidate, voted, follower, unattached, observer
37-
STATE=$(echo "$MATCHING_METRIC" | rev | cut -d'_' -f1 | rev)
35+
# Extract the state from the label state="value" in the metric line
36+
# Possible values are leader, candidate, voted, follower, unattached, observer
37+
STATE=$(echo "$MATCHING_METRIC" | sed -n 's/.*state="\([^"]*\)".*/\1/p')
3838

3939
# Check if the extracted state is 'leader' or 'follower'
4040
if [ "$STATE" == "leader" ] || [ "$STATE" == "follower" ]; then
@@ -46,6 +46,6 @@ if [ -n "$MATCHING_METRIC" ]; then
4646
exit 1
4747
fi
4848
else
49-
echo "JMX Exporter endpoint is not avaiable or kafka_server_raft_metrics_current_state_ was not found."
49+
echo "JMX Exporter endpoint is not avaiable or kafka_server_raft_metrics_current_state_info was not found."
5050
exit 0
5151
fi

api/v1beta1/kafkacluster_types.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -53,10 +53,10 @@ const (
5353
IsControllerNodeKey = "isControllerNode"
5454

5555
// DefaultCruiseControlImage is the default CC image used when users don't specify it in CruiseControlConfig.Image
56-
DefaultCruiseControlImage = "ghcr.io/banzaicloud/cruise-control:2.5.123"
56+
DefaultCruiseControlImage = "adobe/cruise-control:3.0.3-adbe-20250804"
5757

5858
// DefaultKafkaImage is the default Kafka image used when users don't specify it in KafkaClusterSpec.ClusterImage
59-
DefaultKafkaImage = "ghcr.io/adobe/kafka:2.13-3.9.1"
59+
DefaultKafkaImage = "ghcr.io/adobe/koperator/kafka:2.13-3.9.1"
6060

6161
// ControllerNodeProcessRole represents the node is a controller node
6262
ControllerNodeProcessRole = "controller"
@@ -110,7 +110,7 @@ const (
110110
/* Cruise Control Config */
111111

112112
// CruiseControlDeployment.spec.template.spec.container["%s-cruisecontrol"].image
113-
defaultCruiseControlImage = "ghcr.io/banzaicloud/cruise-control:2.5.123"
113+
defaultCruiseControlImage = "adobe/cruise-control:3.0.3-adbe-20250804"
114114

115115
// CruiseControlDeployment.spec.template.spec.container["%s-cruisecontrol"].resources
116116
defaultCruiseControlRequestResourceCpu = "200m"
@@ -126,7 +126,7 @@ const (
126126
defaultKafkaClusterK8sClusterDomain = "cluster.local"
127127

128128
// KafkaBroker.spec.container["kafka"].image
129-
defaultKafkaImage = "ghcr.io/adobe/kafka:2.13-3.9.1"
129+
defaultKafkaImage = "ghcr.io/adobe/koperator/kafka:2.13-3.9.1"
130130

131131
/* Istio Ingress Config */
132132

@@ -145,7 +145,7 @@ const (
145145

146146
// KafkaBrokerPod.spec.initContainer[jmx-exporter].image
147147
// kafkaClusterDeployment.spec.template.spec.initContainer["jmx-exporter"].image
148-
defaultMonitorImage = "ghcr.io/banzaicloud/jmx-javaagent:0.16.1"
148+
defaultMonitorImage = "ghcr.io/amuraru/jmx-javaagent:0.19.2"
149149

150150
// KafkaBrokerPod.spec.initContainer["jmx-exporter"].command
151151
// kafkaClusterDeployment.spec.template.spec.initContainer["jmx-exporter"].command

config/samples/banzaicloud_v1beta1_kafkacluster.yaml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ spec:
3939

4040
# Specify the Kafka Broker related settings
4141
# clusterImage can specify the whole kafkacluster image in one place
42-
#clusterImage: "ghcr.io/adobe/kafka:2.13-3.9.1
42+
#clusterImage: "ghcr.io/adobe/koperator/kafka:2.13-3.9.1
4343

4444
#clusterWideConfig specifies the cluster-wide kafka config cluster wide, all these can be overridden per-broker
4545
#clusterWideConfig: |
@@ -181,7 +181,7 @@ spec:
181181
cruise.control.metrics.topic.replication.factor=2
182182
brokerConfig:
183183
# Docker image used by the operator to create the Broker with id 0
184-
#image: "ghcr.io/adobe/kafka:2.13-3.9.1
184+
#image: "ghcr.io/adobe/koperator/kafka:2.13-3.9.1
185185
# resourceRequirements works exactly like Container resources, the user can specify the limit and the requests
186186
# through this property
187187
#resourceRequirements:
@@ -252,7 +252,7 @@ spec:
252252
#volumeMounts:
253253
# - mountPath: "/opt/kafka/libs/extra-jars"
254254
# name: "jars"
255-
- image: "ghcr.io/adobe/kafka:2.13-3.9.1"
255+
- image: "ghcr.io/adobe/koperator/kafka:2.13-3.9.1"
256256
id: 1
257257
config: |
258258
auto.create.topics.enable=false
@@ -274,7 +274,7 @@ spec:
274274
readOnlyConfig: |
275275
auto.create.topics.enable=false
276276
brokerConfig:
277-
image: "ghcr.io/adobe/kafka:2.13-3.9.1"
277+
image: "ghcr.io/adobe/koperator/kafka:2.13-3.9.1"
278278
storageConfigs:
279279
- mountPath: "/kafka-logs"
280280
pvcSpec:
@@ -708,9 +708,9 @@ spec:
708708
# monitoringConfig describes the monitoring related configs
709709
#monitoringConfig:
710710
# jmxImage describes the used prometheus jmx exporter agent container
711-
# jmxImage: "ghcr.io/banzaicloud/jmx-javaagent:0.16.1"
711+
# jmxImage: "ghcr.io/amuraru/jmx-javaagent:0.19.2"
712712
# pathToJar describes the path to the jar file in the given image
713-
# pathToJar: "/opt/jmx_exporter/jmx_prometheus_javaagent-0.16.1.jar"
713+
# pathToJar: "/jmx_prometheus_javaagent.jar"
714714
# kafkaJMXExporterConfig describes jmx exporter config for Kafka
715715
# kafkaJMXExporterConfig: |
716716
# lowercaseOutputName: true

config/samples/kafkacluster-prometheus.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,7 @@ spec:
8585
storageClass: 'gp2'
8686
mountPath: '/kafkalog'
8787
diskSize: '10G'
88-
image: 'ghcr.io/adobe/kafka:2.13-3.9.1'
88+
image: 'ghcr.io/adobe/koperator/kafka:2.13-3.9.1'
8989
# annotations to be applied onto the broker pod
9090
# brokerAnnotations: '{ "sidecar.istio.io/logLevel": "trace", "sidecar.istio.io/proxyCPULimit": "50m" }'
9191
command: 'upScale'
@@ -101,7 +101,7 @@ spec:
101101
storageClass: 'gp2'
102102
mountPath: '/kafkalog'
103103
diskSize: '10G'
104-
image: 'ghcr.io/adobe/kafka:2.13-3.9.1'
104+
image: 'ghcr.io/adobe/koperator/kafka:2.13-3.9.1'
105105
# annotations to be applied onto the broker pod
106106
# brokerAnnotations: '{ "sidecar.istio.io/logLevel": "trace", "sidecar.istio.io/proxyCPULimit": "50m" }'
107107
command: 'upScale'
@@ -117,7 +117,7 @@ spec:
117117
storageClass: 'gp2'
118118
mountPath: '/kafkalog'
119119
diskSize: '10G'
120-
image: 'ghcr.io/adobe/kafka:2.13-3.9.1'
120+
image: 'ghcr.io/adobe/koperator/kafka:2.13-3.9.1'
121121
# annotations to be applied onto the broker pod
122122
# brokerAnnotations: '{ "sidecar.istio.io/logLevel": "trace", "sidecar.istio.io/proxyCPULimit": "50m" }'
123123
command: 'upScale'

config/samples/kafkacluster-with-istio.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ spec:
1616
zkAddresses:
1717
- "zookeeper-server-client.zookeeper:2181"
1818
oneBrokerPerNode: false
19-
clusterImage: "ghcr.io/adobe/kafka:2.13-3.9.1"
19+
clusterImage: "ghcr.io/adobe/koperator/kafka:2.13-3.9.1"
2020
readOnlyConfig: |
2121
auto.create.topics.enable=false
2222
cruise.control.metrics.topic.auto.create=true

config/samples/kafkacluster_with_external_ssl.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ spec:
1010
zkAddresses:
1111
- "zookeeper-server-client.zookeeper:2181"
1212
oneBrokerPerNode: false
13-
clusterImage: "ghcr.io/adobe/kafka:2.13-3.9.1"
13+
clusterImage: "ghcr.io/adobe/koperator/kafka:2.13-3.9.1"
1414
readOnlyConfig: |
1515
auto.create.topics.enable=false
1616
cruise.control.metrics.topic.auto.create=true

0 commit comments

Comments
 (0)