From 18bbdddc996b4ab5395d5266faae13d1694db765 Mon Sep 17 00:00:00 2001
From: Andrew Spiers <andrew@andrewspiers.net>
Date: Fri, 2 Jul 2021 17:23:00 +0930
Subject: [PATCH] Default to using the number of brokers we create.

By default, the operator creates a 3 node cluster, but default replication factor is currently set to 1, ie no replication at all.
This creates a dangerous footgun, where creating a default cluster can give the impression that having multiple brokers means there is some protection from hardware failure, where there is none.
A safer default is to match the number of brokers.
---
 operator/params.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/operator/params.yaml b/operator/params.yaml
index ed91706..89c1339 100644
--- a/operator/params.yaml
+++ b/operator/params.yaml
@@ -535,7 +535,7 @@ parameters:
   - name: DEFAULT_REPLICATION_FACTOR
     displayName: "default.replication.factor"
     description: "Default replication factors for automatically created topics"
-    default: "1"
+    default: "3"
     trigger: "update-instance"
   - name: EXTERNAL_ADVERTISED_LISTENER
     displayName: "Enable external advertised listeners"