Skip to content

Commit

Permalink
adapt to 3.9.1
Browse files Browse the repository at this point in the history
  • Loading branch information
maltesander committed Feb 8, 2024
1 parent 01ed63f commit 68d7760
Show file tree
Hide file tree
Showing 16 changed files with 24 additions and 26 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ zkCli_ls() {
# tag::zkcli-ls[]
kubectl run my-pod \
--stdin --tty --quiet --restart=Never \
--image docker.stackable.tech/stackable/zookeeper:3.8.3-stackable0.0.0-dev -- \
--image docker.stackable.tech/stackable/zookeeper:3.9.1-stackable0.0.0-dev -- \
bin/zkCli.sh -server simple-zk-server-default:2282 ls / > /dev/null && \
kubectl logs my-pod && \
kubectl delete pods my-pod
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ zkCli_ls() {
# tag::zkcli-ls[]
kubectl run my-pod \
--stdin --tty --quiet --restart=Never \
--image docker.stackable.tech/stackable/zookeeper:3.8.3-stackable{{ versions.zookeeper }} -- \
--image docker.stackable.tech/stackable/zookeeper:3.9.1-stackable{{ versions.zookeeper }} -- \
bin/zkCli.sh -server simple-zk-server-default:2282 ls / > /dev/null && \
kubectl logs my-pod && \
kubectl delete pods my-pod
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ metadata:
name: simple-zk
spec:
image:
productVersion: 3.8.3
productVersion: 3.9.1
servers:
roleGroups:
default:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ spec:
clusterConfig:
listenerClass: external-unstable
image:
productVersion: 3.8.3
productVersion: 3.9.1
servers:
roleGroups:
default:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ metadata:
name: simple-zk
spec:
image:
productVersion: 3.8.3
productVersion: 3.9.1
clusterConfig:
authentication:
- authenticationClass: zk-client-tls # <1>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ metadata:
name: simple-zk
spec:
image:
productVersion: 3.8.3
productVersion: 3.9.1
clusterConfig:
tls:
serverSecretClass: tls # <1>
Expand Down
2 changes: 0 additions & 2 deletions docs/modules/zookeeper/pages/index.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,4 @@ components will also deploy a ZooKeeper ensemble. Here is the list of the demos

The Stackable Operator for Apache ZooKeeper currently supports the following versions of ZooKeeper:

WARNING: The use of 3.8.1 is strongly discouraged due to https://zookeeper.apache.org/security.html#CVE-2023-44981[Authorization bypass in SASL Quorum Peer Authentication]. Several other versions are affected but not offered by Stackable. Please take this into account when using custom images.

include::partial$supported-versions.adoc[]
2 changes: 1 addition & 1 deletion docs/modules/zookeeper/pages/reference/discovery.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ The name of the ConfigMap created for this discovery profile is `$BASENAME-nodep

Each discovery profile contains the following fields:

`ZOOKEEPER`:: A connection string, as accepted by https://zookeeper.apache.org/doc/r3.8.3/apidocs/zookeeper-server/org/apache/zookeeper/ZooKeeper.html#ZooKeeper-java.lang.String-int-org.apache.zookeeper.Watcher-[the official Java client], e.g. `test-zk-server-default-0.test-zk-server-default.kuttl-test-proper-spaniel.svc.cluster.local:2282,test-zk-server-default-1.test-zk-server-default.kuttl-test-proper-spaniel.svc.cluster.local:2282/znode-4e169890-d2eb-4d62-9515-e4786f0ac58e`
`ZOOKEEPER`:: A connection string, as accepted by https://zookeeper.apache.org/doc/r3.9.1/apidocs/zookeeper-server/org/apache/zookeeper/ZooKeeper.html#ZooKeeper-java.lang.String-int-org.apache.zookeeper.Watcher-[the official Java client], e.g. `test-zk-server-default-0.test-zk-server-default.kuttl-test-proper-spaniel.svc.cluster.local:2282,test-zk-server-default-1.test-zk-server-default.kuttl-test-proper-spaniel.svc.cluster.local:2282/znode-4e169890-d2eb-4d62-9515-e4786f0ac58e`
`ZOOKEEPER_HOSTS`:: A comma-separated list of `node1:port1,node2:port2,...`, e.g. `test-zk-server-default-0.test-zk-server-default.kuttl-test-proper-spaniel.svc.cluster.local:2282,test-zk-server-default-1.test-zk-server-default.kuttl-test-proper-spaniel.svc.cluster.local:2282`
`ZOOKEEPER_CHROOT`:: The name of the root ZNode associated with the discovery profile, should be used if (and only if) connecting using `ZOOKEEPER_HOSTS` (rather than `ZOOKEEPER`), e.g. `/znode-4e169890-d2eb-4d62-9515-e4786f0ac58e` in case of a ZNode discovery or `/` in case of a ZookeeperServer discovery
`ZOOKEEPER_CLIENT_PORT`:: The port clients should use when connecting, e.g. `2282`
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ servers:

All property values must be strings.

For a full list of configuration options we refer to the Apache ZooKeeper https://zookeeper.apache.org/doc/r3.8.3/zookeeperAdmin.html#sc_configuration[Configuration Reference].
For a full list of configuration options we refer to the Apache ZooKeeper https://zookeeper.apache.org/doc/r3.9.1/zookeeperAdmin.html#sc_configuration[Configuration Reference].

=== Overriding entries in security.properties

Expand Down
2 changes: 1 addition & 1 deletion docs/modules/zookeeper/pages/znodes.adoc
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
= ZNodes

Apache ZooKeeper organizes all data into a hierarchical system of https://zookeeper.apache.org/doc/r3.8.3/zookeeperProgrammers.html#ch_zkDataModel[ZNodes],
Apache ZooKeeper organizes all data into a hierarchical system of https://zookeeper.apache.org/doc/r3.9.1/zookeeperProgrammers.html#ch_zkDataModel[ZNodes],
which act as both files (they can have data associated with them) and folders (they can contain other ZNodes) when compared to a traditional (POSIX-like) file system.

In order to isolate different clients using the same ZooKeeper cluster, each client application should be assigned a unique root ZNode, which it can then organize
Expand Down
4 changes: 2 additions & 2 deletions docs/modules/zookeeper/partials/supported-versions.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -2,5 +2,5 @@
// This is a separate file, since it is used by both the direct ZooKeeper documentation, and the overarching
// Stackable Platform documentation.

- 3.8.3
- 3.8.1 (deprecated)
- 3.9.1
- 3.8.3 (LTS)
2 changes: 1 addition & 1 deletion examples/simple-zookeeper-tls-cluster.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ metadata:
name: simple-zk
spec:
image:
productVersion: 3.8.3
productVersion: 3.9.1
clusterConfig:
authentication:
- authenticationClass: zk-client-tls
Expand Down
2 changes: 1 addition & 1 deletion rust/crd/src/affinity.rs
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ mod tests {
name: simple-zk
spec:
image:
productVersion: 3.8.3
productVersion: 3.9.1
clusterConfig:
authentication:
- authenticationClass: zk-client-tls
Expand Down
14 changes: 7 additions & 7 deletions rust/crd/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -740,7 +740,7 @@ mod tests {
name: simple-zookeeper
spec:
image:
productVersion: "3.8.3"
productVersion: "3.9.1"
"#;
let zookeeper: ZookeeperCluster = serde_yaml::from_str(input).expect("illegal test input");
assert_eq!(
Expand All @@ -759,7 +759,7 @@ mod tests {
name: simple-zookeeper
spec:
image:
productVersion: "3.8.3"
productVersion: "3.9.1"
clusterConfig:
tls:
serverSecretClass: simple-zookeeper-client-tls
Expand All @@ -782,7 +782,7 @@ mod tests {
name: simple-zookeeper
spec:
image:
productVersion: "3.8.3"
productVersion: "3.9.1"
clusterConfig:
tls:
serverSecretClass: null
Expand All @@ -801,7 +801,7 @@ mod tests {
name: simple-zookeeper
spec:
image:
productVersion: "3.8.3"
productVersion: "3.9.1"
clusterConfig:
tls:
quorumSecretClass: simple-zookeeper-quorum-tls
Expand All @@ -826,7 +826,7 @@ mod tests {
name: simple-zookeeper
spec:
image:
productVersion: "3.8.3"
productVersion: "3.9.1"
"#;
let zookeeper: ZookeeperCluster = serde_yaml::from_str(input).expect("illegal test input");

Expand All @@ -846,7 +846,7 @@ mod tests {
name: simple-zookeeper
spec:
image:
productVersion: "3.8.3"
productVersion: "3.9.1"
clusterConfig:
tls:
quorumSecretClass: simple-zookeeper-quorum-tls
Expand All @@ -868,7 +868,7 @@ mod tests {
name: simple-zookeeper
spec:
image:
productVersion: "3.8.3"
productVersion: "3.9.1"
clusterConfig:
tls:
serverSecretClass: simple-zookeeper-server-tls
Expand Down
4 changes: 2 additions & 2 deletions rust/operator-binary/src/zk_controller.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1021,7 +1021,7 @@ mod tests {
name: simple-zookeeper
spec:
image:
productVersion: "3.8.3"
productVersion: "3.9.1"
servers:
roleGroups:
default:
Expand All @@ -1048,7 +1048,7 @@ mod tests {
name: simple-zookeeper
spec:
image:
productVersion: "3.8.3"
productVersion: "3.9.1"
servers:
configOverrides:
zoo.cfg:
Expand Down
4 changes: 2 additions & 2 deletions tests/test-definition.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,11 @@
dimensions:
- name: zookeeper
values:
- 3.8.1
- 3.8.3
- 3.9.1
- name: zookeeper-latest
values:
- 3.8.3
- 3.9.1
- name: use-server-tls
values:
- "true"
Expand Down

0 comments on commit 68d7760

Please sign in to comment.