Merge pull request #38 from tmirks/fix-sts-scale-down

Fix StatefulSet scale down
ScientaNL · Apr 12, 2019 · 2001e4a · 2001e4a
2 parents 7ece10c + 064b1d2
commit 2001e4a
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/deploy/charts/proxysql-cluster/templates/statefulset.yaml b/deploy/charts/proxysql-cluster/templates/statefulset.yaml
@@ -53,6 +53,13 @@ spec:
             - mountPath: /etc/proxysql.cnf
               name: proxysql
               subPath: proxysql.cnf
+          lifecycle:
+            preStop:
+              exec:
+                command:
+                  - /bin/bash
+                  - -c
+                  - "/usr/bin/proxysql-cli remove"
           {{ if .Values.resources }}
           resources:
 {{ .Values.resources | indent 12 }}

diff --git a/docker/files/cli/commands/cluster.sh b/docker/files/cli/commands/cluster.sh
@@ -53,11 +53,20 @@ command_query:rules() {
     proxysql_execute_query_hr "SELECT * FROM mysql_query_rules"
 }
 
-commands_add "query:nodes" "Show al the nodes of the cluster"
+commands_add "query:nodes" "Show all the nodes of the cluster"
 command_query:nodes() {
     proxysql_execute_query_hr "SELECT * FROM proxysql_servers;"
 }
 
+commands_add "remove" "Remove this node from the cluster"
+command_remove() {
+    proxysql_execute_query_hr "
+        DELETE FROM proxysql_servers
+        WHERE hostname = '${IP}';
+        LOAD PROXYSQL SERVERS TO RUN;
+    " ${PROXYSQL_SERVICE}
+}
+
 commands_add "sync" "Synchronize from backends"
 command_sync() {
     local joinExistingCluster=${1};