Skip to content

Commit

Permalink
Fix deprecated configs in sample files (#870)
Browse files Browse the repository at this point in the history
  • Loading branch information
bartam1 authored Oct 5, 2022
1 parent 2317b2d commit b22016f
Show file tree
Hide file tree
Showing 10 changed files with 7 additions and 101 deletions.
4 changes: 1 addition & 3 deletions config/samples/banzaicloud_v1beta1_kafkacluster.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -331,9 +331,7 @@ spec:
# The server certificate is used by listeners using SSL.
# The data field in the secret must contain a caCert entry and a caKey entry.
# The caCert also can contain certificate chain in this order: intermediate(s) -> root.
tlsSecretName: "test-kafka-operator"
# jksPasswordName should contain a password field which contains the jks password
jksPasswordName: "test-kafka-operator-pass"
tlsSecretName: "kafka-ca-certs"
# create tells the installed cert manager to create the required certs keys
create: true
# disruptionBudget defines the configuration for PodDisruptionBudget
Expand Down
29 changes: 0 additions & 29 deletions config/samples/cfssl/cfssl-ca.json

This file was deleted.

14 changes: 0 additions & 14 deletions config/samples/cfssl/cfssl-server-csr.json

This file was deleted.

6 changes: 0 additions & 6 deletions config/samples/cfssl/gencert.sh

This file was deleted.

16 changes: 0 additions & 16 deletions config/samples/example-secret.sh

This file was deleted.

27 changes: 2 additions & 25 deletions config/samples/example-secret.yaml

Large diffs are not rendered by default.

3 changes: 1 addition & 2 deletions config/samples/kafkacluster_with_external_ssl.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -67,8 +67,7 @@ spec:
usedForInnerBrokerCommunication: false
usedForControllerCommunication: true
sslSecrets:
tlsSecretName: "test-kafka-operator"
jksPasswordName: "test-kafka-operator-pass"
tlsSecretName: "kafka-ca-certs"
create: true
cruiseControlConfig:
config: |
Expand Down
3 changes: 1 addition & 2 deletions config/samples/kafkacluster_with_ssl_groups.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -57,8 +57,7 @@ spec:
# sslClientAuth defaults to be "required" for two-way SSL authentication, possible values are: "required", "requested", and "none"
# sslClientAuth: "requested"
sslSecrets:
tlsSecretName: "test-kafka-operator"
jksPasswordName: "test-kafka-operator-pass"
tlsSecretName: "kafka-ca-certs"
create: true
cruiseControlConfig:
config: |
Expand Down
3 changes: 1 addition & 2 deletions config/samples/kafkacluster_with_ssl_hybrid_customcert.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -66,8 +66,7 @@ spec:
usedForInnerBrokerCommunication: false
usedForControllerCommunication: true
sslSecrets:
tlsSecretName: "test-kafka-operator"
jksPasswordName: "test-kafka-operator-pass"
tlsSecretName: "kafka-ca-certs"
create: true
cruiseControlConfig:
config: |
Expand Down
3 changes: 1 addition & 2 deletions config/samples/simplekafkacluster_ssl.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -58,8 +58,7 @@ spec:
# sslClientAuth defaults to be "required" for two-way SSL authentication, possible values are: "required", "requested", and "none"
# sslClientAuth: "requested"
sslSecrets:
tlsSecretName: "test-kafka-operator"
jksPasswordName: "test-kafka-operator-pass"
tlsSecretName: "kafka-ca-certs"
create: true
cruiseControlConfig:
# podSecurityContext:
Expand Down

0 comments on commit b22016f

Please sign in to comment.