From 24ec382673c6c4dee3a0f3e39ae1dcd5e95aceca Mon Sep 17 00:00:00 2001
From: Bitnami Bot <bitnami-bot@vmware.com>
Date: Mon, 19 Feb 2024 17:13:31 +0100
Subject: [PATCH] [bitnami/clickhouse] Release 23.8.9-debian-11-r24 (#62110)

Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
---
 bitnami/clickhouse/23.8/debian-11/Dockerfile                | 4 ++--
 .../debian-11/rootfs/opt/bitnami/scripts/clickhouse-env.sh  | 1 +
 .../rootfs/opt/bitnami/scripts/clickhouse/entrypoint.sh     | 6 ++++++
 .../rootfs/opt/bitnami/scripts/clickhouse/postunpack.sh     | 6 +++++-
 4 files changed, 14 insertions(+), 3 deletions(-)

diff --git a/bitnami/clickhouse/23.8/debian-11/Dockerfile b/bitnami/clickhouse/23.8/debian-11/Dockerfile
index 84f533ded20e..c94cdb8882a2 100644
--- a/bitnami/clickhouse/23.8/debian-11/Dockerfile
+++ b/bitnami/clickhouse/23.8/debian-11/Dockerfile
@@ -7,10 +7,10 @@ ARG TARGETARCH
 
 LABEL com.vmware.cp.artifact.flavor="sha256:1e1b4657a77f0d47e9220f0c37b9bf7802581b93214fff7d1bd2364c8bf22e8e" \
       org.opencontainers.image.base.name="docker.io/bitnami/minideb:bullseye" \
-      org.opencontainers.image.created="2024-02-16T16:24:39Z" \
+      org.opencontainers.image.created="2024-02-19T14:57:35Z" \
       org.opencontainers.image.description="Application packaged by VMware, Inc" \
       org.opencontainers.image.licenses="Apache-2.0" \
-      org.opencontainers.image.ref.name="23.8.9-debian-11-r23" \
+      org.opencontainers.image.ref.name="23.8.9-debian-11-r24" \
       org.opencontainers.image.title="clickhouse" \
       org.opencontainers.image.vendor="VMware, Inc." \
       org.opencontainers.image.version="23.8.9"
diff --git a/bitnami/clickhouse/23.8/debian-11/rootfs/opt/bitnami/scripts/clickhouse-env.sh b/bitnami/clickhouse/23.8/debian-11/rootfs/opt/bitnami/scripts/clickhouse-env.sh
index 114d66894372..4d8154d88637 100644
--- a/bitnami/clickhouse/23.8/debian-11/rootfs/opt/bitnami/scripts/clickhouse-env.sh
+++ b/bitnami/clickhouse/23.8/debian-11/rootfs/opt/bitnami/scripts/clickhouse-env.sh
@@ -52,6 +52,7 @@ unset clickhouse_env_vars
 export CLICKHOUSE_BASE_DIR="${BITNAMI_ROOT_DIR}/clickhouse"
 export CLICKHOUSE_VOLUME_DIR="/bitnami/clickhouse"
 export CLICKHOUSE_CONF_DIR="${CLICKHOUSE_BASE_DIR}/etc"
+export CLICKHOUSE_DEFAULT_CONF_DIR="${CLICKHOUSE_BASE_DIR}/etc.default"
 export CLICKHOUSE_MOUNTED_CONF_DIR="${CLICKHOUSE_VOLUME_DIR}/etc"
 export CLICKHOUSE_DATA_DIR="${CLICKHOUSE_VOLUME_DIR}/data"
 export CLICKHOUSE_LOG_DIR="${CLICKHOUSE_BASE_DIR}/logs"
diff --git a/bitnami/clickhouse/23.8/debian-11/rootfs/opt/bitnami/scripts/clickhouse/entrypoint.sh b/bitnami/clickhouse/23.8/debian-11/rootfs/opt/bitnami/scripts/clickhouse/entrypoint.sh
index 096951d7fa5b..bdfeae779b2b 100755
--- a/bitnami/clickhouse/23.8/debian-11/rootfs/opt/bitnami/scripts/clickhouse/entrypoint.sh
+++ b/bitnami/clickhouse/23.8/debian-11/rootfs/opt/bitnami/scripts/clickhouse/entrypoint.sh
@@ -18,6 +18,12 @@ set -o pipefail
 
 print_welcome_page
 
+# We add the copy from default config in the entrypoint to not break users 
+# bypassing the setup.sh logic. If the file already exists do not overwrite (in
+# case someone mounts a configuration file in /opt/bitnami/clickhouse/etc)
+debug "Copying files from $CLICKHOUSE_DEFAULT_CONF_DIR to $CLICKHOUSE_CONF_DIR"
+cp -nr "$CLICKHOUSE_DEFAULT_CONF_DIR"/. "$CLICKHOUSE_CONF_DIR"
+
 if [[ "$1" = "/opt/bitnami/scripts/clickhouse/run.sh" ]]; then
     info "** Starting ClickHouse setup **"
     /opt/bitnami/scripts/clickhouse/setup.sh
diff --git a/bitnami/clickhouse/23.8/debian-11/rootfs/opt/bitnami/scripts/clickhouse/postunpack.sh b/bitnami/clickhouse/23.8/debian-11/rootfs/opt/bitnami/scripts/clickhouse/postunpack.sh
index b8fd48f8919c..c097a84005ea 100755
--- a/bitnami/clickhouse/23.8/debian-11/rootfs/opt/bitnami/scripts/clickhouse/postunpack.sh
+++ b/bitnami/clickhouse/23.8/debian-11/rootfs/opt/bitnami/scripts/clickhouse/postunpack.sh
@@ -20,7 +20,7 @@ set -o pipefail
 ensure_user_exists "$CLICKHOUSE_DAEMON_USER" --group "$CLICKHOUSE_DAEMON_GROUP" --system
 
 # Create directories
-for dir in "$CLICKHOUSE_DATA_DIR" "$CLICKHOUSE_CONF_DIR" "${CLICKHOUSE_CONF_DIR}/conf.d" "${CLICKHOUSE_CONF_DIR}/users.d" "$CLICKHOUSE_LOG_DIR" "$CLICKHOUSE_TMP_DIR" "$CLICKHOUSE_MOUNTED_CONF_DIR" "/docker-entrypoint-startdb.d" "/docker-entrypoint-initdb.d"; do
+for dir in "$CLICKHOUSE_DATA_DIR" "$CLICKHOUSE_CONF_DIR" "${CLICKHOUSE_CONF_DIR}/conf.d" "${CLICKHOUSE_CONF_DIR}/users.d" "$CLICKHOUSE_DEFAULT_CONF_DIR" "$CLICKHOUSE_LOG_DIR" "$CLICKHOUSE_TMP_DIR" "$CLICKHOUSE_MOUNTED_CONF_DIR" "/docker-entrypoint-startdb.d" "/docker-entrypoint-initdb.d"; do
     ensure_dir_exists "$dir"
     configure_permissions_ownership "$dir" -d "775" -f "664" -u "$CLICKHOUSE_DAEMON_USER" -g "root"
 done
@@ -78,3 +78,7 @@ chmod g+rw /.clickhouse-client-history
 xmlstarlet ed -L -d "/clickhouse/logger/log" "$CLICKHOUSE_CONF_FILE"
 xmlstarlet ed -L -d "/clickhouse/logger/errorlog" "$CLICKHOUSE_CONF_FILE"
 clickhouse_conf_set "/clickhouse/logger/console" "1"
+
+# Copy all initially generated configuration files to the default directory
+# (this is to avoid breaking when entrypoint is being overridden)
+cp -r "${CLICKHOUSE_CONF_DIR}/"* "$CLICKHOUSE_DEFAULT_CONF_DIR"
\ No newline at end of file