Skip to content

Commit

Permalink
Fix Strimzi Helm values ZK indent bug + stale broker service name (#492)
Browse files Browse the repository at this point in the history
* Fix strimzi helm values bug for zookeeper persistent storage

* fix kafka bootstrap location to be internal SVC
  • Loading branch information
ukclivecox committed Oct 11, 2022
1 parent e801c3d commit 8927119
Show file tree
Hide file tree
Showing 7 changed files with 8 additions and 8 deletions.
2 changes: 1 addition & 1 deletion k8s/helm-charts/seldon-core-v2-setup/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ hodometer:

kafka:
debug:
bootstrap: seldon-kafka-plain-bootstrap.seldon-mesh:9092
bootstrap: seldon-kafka-bootstrap.seldon-mesh:9092
consumer:
sessionTimeoutMs: 6000
autoOffsetReset: earliest
Expand Down
2 changes: 1 addition & 1 deletion k8s/helm-charts/seldon-core-v2-setup/values.yaml.template
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ hodometer:

kafka:
debug:
bootstrap: seldon-kafka-plain-bootstrap.seldon-mesh:9092
bootstrap: seldon-kafka-bootstrap.seldon-mesh:9092
consumer:
sessionTimeoutMs: 6000
autoOffsetReset: earliest
Expand Down
2 changes: 1 addition & 1 deletion k8s/samples/values-strimzi-kafka-mtls.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
kafka:
bootstrap: seldon-kafka-tls-bootstrap.seldon-mesh.svc.cluster.local:9093
bootstrap: seldon-kafka-bootstrap.seldon-mesh.svc.cluster.local:9093

security:
kafka:
Expand Down
4 changes: 2 additions & 2 deletions k8s/yaml/seldon-v2-components.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -409,7 +409,7 @@ apiVersion: v1
data:
kafka.json: |-
{
"bootstrap.servers": "seldon-kafka-plain-bootstrap.seldon-mesh:9092",
"bootstrap.servers": "seldon-kafka-bootstrap.seldon-mesh:9092",
"consumer":{
"session.timeout.ms":6000,
"auto.offset.reset":"earliest",
Expand Down Expand Up @@ -633,7 +633,7 @@ spec:
containers:
- env:
- name: SELDON_KAFKA_BOOTSTRAP_SERVERS
value: seldon-kafka-plain-bootstrap.seldon-mesh:9092
value: seldon-kafka-bootstrap.seldon-mesh:9092
- name: SELDON_UPSTREAM_HOST
value: seldon-scheduler
- name: SELDON_UPSTREAM_PORT
Expand Down
2 changes: 1 addition & 1 deletion kafka/strimzi/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -51,4 +51,4 @@ zookeeper:
initialDelaySeconds: 15
timeoutSeconds: 5

pvcSize: 100Gi
pvcSize: 100Gi
2 changes: 1 addition & 1 deletion scheduler/k8s/config/kafka.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ metadata:
data:
kafka.json: |-
{
"bootstrap.servers": "seldon-kafka-plain-bootstrap.seldon-mesh:9092",
"bootstrap.servers": "seldon-kafka-bootstrap.seldon-mesh:9092",
"consumer":{
"session.timeout.ms":6000,
"auto.offset.reset":"earliest",
Expand Down
2 changes: 1 addition & 1 deletion scheduler/k8s/dataflow-engine/dataflow-engine.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ spec:
name: dataflow-engine
env:
- name: SELDON_KAFKA_BOOTSTRAP_SERVERS
value: "seldon-kafka-plain-bootstrap.seldon-mesh:9092"
value: "seldon-kafka-bootstrap.seldon-mesh:9092"
- name: SELDON_UPSTREAM_HOST
value: "seldon-scheduler"
- name: SELDON_UPSTREAM_PORT
Expand Down

0 comments on commit 8927119

Please sign in to comment.