Skip to content

Commit

Permalink
Connect by name resolution to individual zookeeper pods instead of
Browse files Browse the repository at this point in the history
through the service. This helps troubleshoot issues like #310 by pointing
out (by podIP) which actual zookeeper connection that failed.
Also I like the simplification.
  • Loading branch information
solsson committed Apr 23, 2020
1 parent bec515c commit f2a302e
Show file tree
Hide file tree
Showing 4 changed files with 22 additions and 0 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
[
{"op": "add", "path": "/spec/template/spec/containers/0/args/-", "value": "--override"},
{"op": "add", "path": "/spec/template/spec/containers/0/args/-", "value": "zookeeper.connect=zoo:2181"}
]
8 changes: 8 additions & 0 deletions variants/scale-3-5-nopzoo/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,3 +2,11 @@ bases:
- ../scale-3-5
patchesStrategicMerge:
- ./only-zoo-5.yaml
patchesJson6902:
- target:
group: apps
version: v1
kind: StatefulSet
name: kafka
namespace: kafka
path: kafka-zookeeper-connect-only-zoo.json
4 changes: 4 additions & 0 deletions variants/scale-6-9/kafka-zookeeper-connect-only-zoo.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
[
{"op": "add", "path": "/spec/template/spec/containers/0/args/-", "value": "--override"},
{"op": "add", "path": "/spec/template/spec/containers/0/args/-", "value": "zookeeper.connect=zoo:2181"}
]
6 changes: 6 additions & 0 deletions variants/scale-6-9/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,12 @@ patchesStrategicMerge:
- zoo-9.yaml
- kafka-6.yaml
patchesJson6902:
- target:
group: apps
version: v1
kind: StatefulSet
name: kafka
path: kafka-zookeeper-connect-only-zoo.json
# The nonroot image is distroless and doesn't support a shell that the prestop hook needs
- target:
group: apps
Expand Down

0 comments on commit f2a302e

Please sign in to comment.