Utilize the 'Enabled' parameter to toggle the bootstrap mode on or off for Redis nodes.

CHANGELOG.md

@@ -10,6 +10,7 @@ Also check this project's [releases](https://github.com/powerhome/redis-operator
 ## Unreleased
 
 ### Changed
+- [Utilize the 'Enabled' parameter to toggle the bootstrap mode on or off for Redis nodes.](https://github.com/powerhome/redis-operator/pull/32).
 - [Use the new docker bake tooling to build the developer tools image and remove vestigial development targets from the Makefile](https://github.com/powerhome/redis-operator/pull/31).
 
 ## [v1.8.0-rc2] - 2023-12-20

api/redisfailover/v1/bootstrapping.go

@@ -2,7 +2,7 @@ package v1
 
 // Bootstrapping returns true when a BootstrapNode is provided to the RedisFailover spec. Otherwise, it returns false.
 func (r *RedisFailover) Bootstrapping() bool {
-	return r.Spec.BootstrapNode != nil
+	return r.Spec.BootstrapNode != nil && r.Spec.BootstrapNode.Enabled
 }
 
 // SentinelsAllowed returns true if not Bootstrapping orif BootstrapNode settings allow sentinels to exist

api/redisfailover/v1/bootstrapping_test.go

@@ -33,8 +33,9 @@ func TestBootstrapping(t *testing.T) {
 			name:        "with BootstrapSettings",
 			expectation: true,
 			bootstrapSettings: &BootstrapSettings{
-				Host: "127.0.0.1",
-				Port: "6379",
+				Host:    "127.0.0.1",
+				Port:    "6379",
+				Enabled: true,
 			},
 		},
 	}
@@ -61,8 +62,9 @@ func TestSentinelsAllowed(t *testing.T) {
 			name:        "with BootstrapSettings",
 			expectation: false,
 			bootstrapSettings: &BootstrapSettings{
-				Host: "127.0.0.1",
-				Port: "6379",
+				Host:    "127.0.0.1",
+				Port:    "6379",
+				Enabled: true,
 			},
 		},
 		{
@@ -72,6 +74,7 @@ func TestSentinelsAllowed(t *testing.T) {
 				Host:           "127.0.0.1",
 				Port:           "6379",
 				AllowSentinels: true,
+				Enabled:        true,
 			},
 		},
 	}

api/redisfailover/v1/types.go

@@ -175,6 +175,8 @@ type BootstrapSettings struct {
 	Host           string `json:"host,omitempty"`
 	Port           string `json:"port,omitempty"`
 	AllowSentinels bool   `json:"allowSentinels,omitempty"`
+	// +kubebuilder:default=true
+	Enabled        bool   `json:"enabled,omitempty"`
 }
 
 // Exporter defines the specification for the redis/sentinel exporter

api/redisfailover/v1/validate_test.go

@@ -34,7 +34,7 @@ func TestValidate(t *testing.T) {
 		{
 			name:            "BootstrapNode provided without a host",
 			rfName:          "test",
-			rfBootstrapNode: &BootstrapSettings{},
+			rfBootstrapNode: &BootstrapSettings{Enabled: true},
 			expectedError:   "BootstrapNode must include a host when provided",
 		},
 		{
@@ -44,14 +44,14 @@ func TestValidate(t *testing.T) {
 		{
 			name:                  "Populates default bootstrap port when valid",
 			rfName:                "test",
-			rfBootstrapNode:       &BootstrapSettings{Host: "127.0.0.1"},
-			expectedBootstrapNode: &BootstrapSettings{Host: "127.0.0.1", Port: "6379"},
+			rfBootstrapNode:       &BootstrapSettings{Host: "127.0.0.1", Enabled: true},
+			expectedBootstrapNode: &BootstrapSettings{Host: "127.0.0.1", Port: "6379", Enabled: true},
 		},
 		{
 			name:                  "Allows for specifying boostrap port",
 			rfName:                "test",
-			rfBootstrapNode:       &BootstrapSettings{Host: "127.0.0.1", Port: "6380"},
-			expectedBootstrapNode: &BootstrapSettings{Host: "127.0.0.1", Port: "6380"},
+			rfBootstrapNode:       &BootstrapSettings{Host: "127.0.0.1", Port: "6380", Enabled: true},
+			expectedBootstrapNode: &BootstrapSettings{Host: "127.0.0.1", Port: "6380", Enabled: true},
 		},
 		{
 			name:                "Appends applied custom config to default initial values",
@@ -62,8 +62,8 @@ func TestValidate(t *testing.T) {
 			name:                  "Appends applied custom config to default initial values when bootstrapping",
 			rfName:                "test",
 			rfRedisCustomConfig:   []string{"tcp-keepalive 60"},
-			rfBootstrapNode:       &BootstrapSettings{Host: "127.0.0.1"},
-			expectedBootstrapNode: &BootstrapSettings{Host: "127.0.0.1", Port: "6379"},
+			rfBootstrapNode:       &BootstrapSettings{Host: "127.0.0.1", Enabled: true},
+			expectedBootstrapNode: &BootstrapSettings{Host: "127.0.0.1", Port: "6379", Enabled: true},
 		},
 	}
 
@@ -83,7 +83,7 @@ func TestValidate(t *testing.T) {
 					"replica-priority 100",
 				}
 
-				if test.rfBootstrapNode != nil {
+				if test.rfBootstrapNode != nil && test.rfBootstrapNode.Enabled {
 					expectedRedisCustomConfig = []string{
 						"replica-priority 0",
 					}

docs/README.md

@@ -318,6 +318,7 @@ This `bootstrapNode` can be configured as follows:
 | host           | **required** | The IP of the target Redis address or the ClusterIP of a pre-existing Kubernetes Service targeting Redis pods                                                                             | [bootstrapping.yaml](example/redisfailover/bootstrapping.yaml)                               |
 | port           | _optional_   | The Port that the target Redis address is listening to. Defaults to `6379`.                                                                                                               | [bootstrapping-with-port.yaml](example/redisfailover/bootstrapping-with-port.yaml)           |
 | allowSentinels | _optional_   | Allow the Operator to also create the specified Sentinel resources and point them to the target Node/Port. By default, the Sentinel resources will **not** be created when bootstrapping. | [bootstrapping-with-sentinels.yaml](example/redisfailover/bootstrapping-with-sentinels.yaml) |
+| enabled        | _optional_   | By default, Bootstrap mode is activated, prompting Redis nodes to attempt a connection with the source host. However, if this parameter is set to `false', Redis will instead establish a cluster among the local nodes. In this scenario, a master node will be designated, to which the replica nodes will connect | [bootstrapping-disabled.yaml](example/redisfailover/bootstrapping-disabled.yaml) |
 
 #### What is Bootstrapping?
 When a `bootstrapNode` is provided, the Operator will always set all of the defined Redis instances to replicate from the provided `bootstrapNode` host value.

example/redisfailover/bootstrapping-disabled.yaml

@@ -0,0 +1,12 @@
+apiVersion: databases.spotahome.com/v1
+kind: RedisFailover
+metadata:
+  name: redisfailover
+spec:
+  bootstrapNode:
+    host: "127.0.0.1"
+    enabled: false
+  sentinel:
+    replicas: 3
+  redis:
+    replicas: 3

manifests/databases.spotahome.com_redisfailovers.yaml

@@ -65,6 +65,9 @@ spec:
                 properties:
                   allowSentinels:
                     type: boolean
+                  enabled:
+                    default: true
+                    type: boolean
                   host:
                     type: string
                   port:

manifests/kustomize/base/databases.spotahome.com_redisfailovers.yaml

@@ -65,6 +65,9 @@ spec:
                 properties:
                   allowSentinels:
                     type: boolean
+                  enabled:
+                    default: true
+                    type: boolean
                   host:
                     type: string
                   port:

operator/redisfailover/ensurer_test.go

@@ -50,13 +50,12 @@ func generateRF(enableExporter bool, bootstrapping bool) *redisfailoverv1.RedisF
 }
 
 func generateRFBootstrappingNode(bootstrapping bool) *redisfailoverv1.BootstrapSettings {
-	if bootstrapping {
-		return &redisfailoverv1.BootstrapSettings{
-			Host: "127.0.0.1",
-			Port: "6379",
-		}
+
+	return &redisfailoverv1.BootstrapSettings{
+		Host:    "127.0.0.1",
+		Port:    "6379",
+		Enabled: bootstrapping,
 	}
-	return nil
 }
 
 func TestEnsure(t *testing.T) {

operator/redisfailover/service/check_test.go

@@ -1034,6 +1034,7 @@ func TestClusterRunningWithBootstrap(t *testing.T) {
 	rf.Spec.BootstrapNode = &redisfailoverv1.BootstrapSettings{
 		Host:           "fake-host",
 		AllowSentinels: false,
+		Enabled:        true,
 	}
 	ms.On("GetDeploymentPods", namespace, rfservice.GetSentinelName(rf)).Once().Return(notAllRunning, nil)
 	ms.On("GetStatefulSetPods", namespace, rfservice.GetRedisName(rf)).Once().Return(notAllRunning, nil)
@@ -1124,6 +1125,7 @@ func TestClusterRunningWithBootstrapSentinels(t *testing.T) {
 	rf.Spec.BootstrapNode = &redisfailoverv1.BootstrapSettings{
 		Host:           "fake-host",
 		AllowSentinels: true,
+		Enabled:        true,
 	}
 	ms.On("GetDeploymentPods", namespace, rfservice.GetSentinelName(rf)).Once().Return(allRunning, nil)
 	ms.On("GetStatefulSetPods", namespace, rfservice.GetRedisName(rf)).Once().Return(allRunning, nil)