chore: mysql reconfigure by shell trigger in 0.9

Author: kizuna-lek

addons/mysql/config/mysql-reload.tpl

@@ -1096,9 +1096,6 @@
 	// Server current time zone
 	default_time_zone?: string
 
-	// This variable is used to specify which events should be logged.
-	audit_log_policy?: string & "ALL" | "LOGINS" | "QUERIES" | "NONE" | *"ALL"
-
 	// slave_exec_mode controls how a replication thread resolves conflicts and errors during replication.
 	slave_exec_mode?: string & "IDEMPOTENT" | "STRICT"
 
@@ -1141,28 +1138,28 @@
 	relay_log_info_repository?: string & "FILE" | "TABLE"
 
 	// Specifies the audit log plugin to use.
-	audit_log_handler?: string & "FILE" | "SYSLOG" | *"FILE"
+	loose_audit_log_handler?: string & "FILE" | "SYSLOG" | *"FILE"
 
 	// The file name of the audit log.
-	audit_log_file?: string
+	loose_audit_log_file?: string
 
 	// The size at which the audit log file is rotated.
-	audit_log_buffer_size?: string
+	loose_audit_log_buffer_size?: string
 
 	// This variable is used to specify which events should be logged.
-	audit_log_policy?: string & "ALL" | "LOGINS" | "QUERIES" | "NONE" | *"ALL"
+	loose_audit_log_policy?: string & "ALL" | "LOGINS" | "QUERIES" | "NONE" | *"ALL"
 
 	// The strategy used to write audit log records.
-	audit_log_strategy?: string & "ASYNCHRONOUS" | "PERFORMANCE" | "SEMISYNCHRONOUS" | "SYNCHRONOUS" | *"ASYNCHRONOUS"
+	loose_audit_log_strategy?: string & "ASYNCHRONOUS" | "PERFORMANCE" | "SEMISYNCHRONOUS" | "SYNCHRONOUS" | *"ASYNCHRONOUS"
 
 	// The size at which the audit log file is rotated.
-	audit_log_rotate_on_size?: int | *0
+	loose_audit_log_rotate_on_size?: int | *0
 
 	// The number of rotated audit log files to keep.
-	audit_log_rotations?: int | *0
+	loose_audit_log_rotations?: int | *0
 
 	// A comma-separated list of accounts that are not subject to audit logging.
-	audit_log_exclude_accounts?: string
+	loose_audit_log_exclude_accounts?: string
 
 	// other parameters
 	// reference mysql parameters

addons/mysql/config/mysql5.7-config-constraint.cue

@@ -251,7 +251,7 @@ dynamicParameters:
   - transaction_write_set_extraction
   - unique_checks
   - updatable_views_with_limit
-  - audit_log_policy
+  - loose_audit_log_policy
   - slave_exec_mode
   - slave_net_timeout
   - sort_buffer_size
@@ -263,9 +263,9 @@ dynamicParameters:
   - performance_schema_events_stages_history_long_size
   - performance_schema_events_transactions_history_long_size
   - relay_log_info_repository
-  - audit_log_exclude_accounts
-  - audit_log_rotations
-  - audit_log_rotate_on_size
+  - loose_audit_log_exclude_accounts
+  - loose_audit_log_rotations
+  - loose_audit_log_rotate_on_size
 immutableParameters:
   - auto_generate_certs
   - basedir
@@ -386,8 +386,7 @@ staticParameters:
   - thread_stack
   - version_comment
   - version_compile_os
-  - audit_log_handler
-  - audit_log_file
-  - audit_log_buffer_size
-  - audit_log_policy
-  - audit_log_strategy
+  - loose_audit_log_handler
+  - loose_audit_log_file
+  - loose_audit_log_buffer_size
+  - loose_audit_log_strategy

addons/mysql/config/mysql5.7-config-effect-scope.yaml

@@ -1532,31 +1532,28 @@
 	windowing_use_high_precision: string & "OFF" | "ON" | *"ON"
 
 	// This variable is used to specify which events should be logged.
-	audit_log_policy?: string & "ALL" | "LOGINS" | "QUERIES" | "NONE" | *"ALL"
+	loose_audit_log_policy?: string & "ALL" | "LOGINS" | "QUERIES" | "NONE" | *"ALL"
 
 	// Specifies the audit log plugin to use.
-	audit_log_handler?: string & "FILE" | "SYSLOG" | *"FILE"
+	loose_audit_log_handler?: string & "FILE" | "SYSLOG" | *"FILE"
 
 	// The file name of the audit log.
-	audit_log_file?: string
+	loose_audit_log_file?: string
 
 	// The size at which the audit log file is rotated.
-	audit_log_buffer_size?: string
-
-	// This variable is used to specify which events should be logged.
-	audit_log_policy?: string & "ALL" | "LOGINS" | "QUERIES" | "NONE" | *"ALL"
+	loose_audit_log_buffer_size?: string
 
 	// The strategy used to write audit log records.
-	audit_log_strategy?: string & "ASYNCHRONOUS" | "PERFORMANCE" | "SEMISYNCHRONOUS" | "SYNCHRONOUS" | *"ASYNCHRONOUS"
+	loose_audit_log_strategy?: string & "ASYNCHRONOUS" | "PERFORMANCE" | "SEMISYNCHRONOUS" | "SYNCHRONOUS" | *"ASYNCHRONOUS"
 
 	// The size at which the audit log file is rotated.
-	audit_log_rotate_on_size?: int | *0
+	loose_audit_log_rotate_on_size?: int | *0
 
 	// The number of rotated audit log files to keep.
-	audit_log_rotations?: int | *0
+	loose_audit_log_rotations?: int | *0
 
 	// A comma-separated list of accounts that are not subject to audit logging.
-	audit_log_exclude_accounts?: string
+	loose_audit_log_exclude_accounts?: string
 
 	// other parameters
 	// reference mysql parameters

addons/mysql/config/mysql8-config-constraint.cue

@@ -187,11 +187,10 @@ staticParameters:
   - version_compile_zlib
   - log_replica_updates
   - relay_log_recovery
-  - audit_log_handler
-  - audit_log_file
-  - audit_log_buffer_size
-  - audit_log_policy
-  - audit_log_strategy
+  - loose_audit_log_handler
+  - loose_audit_log_file
+  - loose_audit_log_buffer_size
+  - loose_audit_log_strategy
 
 dynamicParameters:
   - activate_all_roles_on_login
@@ -538,11 +537,11 @@ dynamicParameters:
   - wait_timeout
   - windowing_use_high_precision
   - innodb_redo_log_capacity
-  - audit_log_policy
+  - loose_audit_log_policy
   - relay_log_info_repository
-  - audit_log_exclude_accounts
-  - audit_log_rotations
-  - audit_log_rotate_on_size
+  - loose_audit_log_exclude_accounts
+  - loose_audit_log_rotations
+  - loose_audit_log_rotate_on_size
 
 immutableParameters:
   - allow-suspicious-udfs

addons/mysql/config/mysql8-config-effect-scope.yaml

@@ -0,0 +1,16 @@
+#!/bin/sh
+set -e
+
+function mysql_exec() {
+    local query="$1"
+    mysql --user=${MYSQL_ADMIN_USER} --password=${MYSQL_ADMIN_PASSWORD} --host=127.0.0.1 -P 3306 -NBe "${query}"
+}
+
+paramName="${1:?missing param name}"
+paramValue="${2:?missing value}"
+
+if echo "${paramName}" | grep -q "^loose_"; then
+    paramName=$(echo "${paramName}" | sed 's/^loose_//')
+fi
+
+mysql_exec "SET GLOBAL ${paramName}=${paramValue};"

addons/mysql/reloader/update-parameter.sh

@@ -689,3 +689,13 @@ volumeMounts:
   - name: scripts
     mountPath: /scripts
 {{- end -}}
+
+{{/*
+Generate reloader scripts configmap
+*/}}
+{{- define "mysql.extend.reload.scripts" -}}
+{{- range $path, $_ :=  $.Files.Glob "reloader/**" }}
+{{ $path | base }}: |-
+{{- $.Files.Get $path | nindent 2 }}
+{{- end }}
+{{- end }}

addons/mysql/templates/_helpers.tpl

@@ -9,11 +9,19 @@ spec:
   # configmap reference
   # tplRef: mysql-3node-tpl-8.0
   reloadAction:
-    tplScriptTrigger:
+    shellTrigger:
       sync: true
-      # still share same reload script with 8.0
-      scriptConfigMapRef: mysql8.0-reload-script
-      namespace: {{ .Release.Namespace }}
+      command:
+        - "update-parameter.sh"
+      scriptConfig:
+        scriptConfigMapRef: mysql-reload-script
+        namespace: {{ .Release.Namespace }}
+      toolsSetup:
+        mountPoint: /kb_tools
+        toolConfigs:
+          - name: kb-tools
+            image: {{ .Values.image.registry | default "docker.io" }}/{{ .Values.image.repository }}:5.7.44
+            asContainerImage: true
 
 
   # ConfigurationSchema that impose restrictions on engine parameter's rule

addons/mysql/templates/config-constraint-57.yaml

@@ -9,10 +9,19 @@ spec:
   # configmap reference
   # tplRef: mysql-3node-tpl-8.0
   reloadAction:
-    tplScriptTrigger:
+    shellTrigger:
       sync: true
-      scriptConfigMapRef: mysql8.0-reload-script
-      namespace: {{ .Release.Namespace }}
+      command:
+        - "update-parameter.sh"
+      scriptConfig:
+        scriptConfigMapRef: mysql-reload-script
+        namespace: {{ .Release.Namespace }}
+      toolsSetup:
+        mountPoint: /kb_tools
+        toolConfigs:
+          - name: kb-tools
+            image: {{ .Values.image.registry | default "docker.io" }}/{{ .Values.image.repository }}:8.0.30
+            asContainerImage: true
 
 
   # ConfigurationSchema that impose restrictions on engine parameter's rule

addons/mysql/templates/config-constraint-80.yaml

@@ -28,25 +28,12 @@ data:
 apiVersion: v1
 kind: ConfigMap
 metadata:
-  name: mysql8.0-reload-script
+  name: mysql-reload-script
   labels:
     {{- include "mysql.labels" . | nindent 4 }}
 data:
-  reload.tpl: |-
-    {{- .Files.Get "config/mysql-reload.tpl" | nindent 4 }}
-  reload.yaml: |-
-    scripts: reload.tpl
-    fileRegex: my.cnf
-    formatterConfig:
-      format: ini
-      iniConfig:
-        sectionName: mysqld
+  {{- include "mysql.extend.reload.scripts" . | nindent 2 }}
 
-    dataType: mysql
-    dsn: |-
-      {%- $user := env "MYSQL_ADMIN_USER" | default ( env "MYSQL_ROOT_USER" ) | default ( env "MYSQL_USER" ) %}
-      {%- $pass := env "MYSQL_ADMIN_PASSWORD" | default ( env "MYSQL_ROOT_PASSWORD" ) | default ( env "MYSQL_PASSWORD" ) %}
-      {%- printf "%s:%s@(localhost:3306)/" $user $pass | trim %}
 ---
 apiVersion: v1
 kind: ConfigMap