[bitnami/clickhouse] fix startup issue when using json logs

Signed-off-by: Jesus Navas Orozco <[email protected]>
bitnami · Dec 2, 2024 · ef23b54 · ef23b54
1 parent 9315f67
commit ef23b54
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 3 deletions.
diff --git a/bitnami/clickhouse/24.3/debian-12/rootfs/opt/bitnami/scripts/libclickhouse.sh b/bitnami/clickhouse/24.3/debian-12/rootfs/opt/bitnami/scripts/libclickhouse.sh
@@ -243,7 +243,7 @@ clickhouse_start_bg() {
         error "ClickHouse failed to start"
         exit 1
     fi
-    wait_for_log_entry "Application: Ready for connections" "$log_file"
+    wait_for_log_entry "Ready for connections" "$log_file"
     info "ClickHouse started successfully"
 }
 

diff --git a/bitnami/clickhouse/24.8/debian-12/rootfs/opt/bitnami/scripts/libclickhouse.sh b/bitnami/clickhouse/24.8/debian-12/rootfs/opt/bitnami/scripts/libclickhouse.sh
@@ -243,7 +243,7 @@ clickhouse_start_bg() {
         error "ClickHouse failed to start"
         exit 1
     fi
-    wait_for_log_entry "Application: Ready for connections" "$log_file"
+    wait_for_log_entry "Ready for connections" "$log_file"
     info "ClickHouse started successfully"
 }
 

diff --git a/bitnami/clickhouse/24/debian-12/rootfs/opt/bitnami/scripts/libclickhouse.sh b/bitnami/clickhouse/24/debian-12/rootfs/opt/bitnami/scripts/libclickhouse.sh
@@ -243,7 +243,7 @@ clickhouse_start_bg() {
         error "ClickHouse failed to start"
         exit 1
     fi
-    wait_for_log_entry "Application: Ready for connections" "$log_file"
+    wait_for_log_entry "Ready for connections" "$log_file"
     info "ClickHouse started successfully"
 }
 

diff --git a/bitnami/clickhouse/README.md b/bitnami/clickhouse/README.md
@@ -189,6 +189,8 @@ When the container is executed for the first time, it will execute the files wit
 
 In order to have your custom files inside the docker image you can mount them as a volume.
 
+> NOTE: If you use JSON format for clickhouse logs and remove the message field of the logs, the application will fail to start if there are init or start scripts in any of those 2 folders.
+
 ### Environment variables
 
 #### Customizable environment variables