From 880d6384119368d89056a782d87070e62dfb21f2 Mon Sep 17 00:00:00 2001
From: Gavin <gaprice@lbl.gov>
Date: Sun, 16 Jun 2024 10:52:06 -0700
Subject: [PATCH 1/2] Add sleep option to FE entrypoint

Useful for debugging configs in Rancher if the service doesn't start
---
 docker-compose.yml       | 1 +
 scripts/cn-entrypoint.sh | 1 +
 scripts/fe-entrypoint.sh | 5 +++++
 3 files changed, 7 insertions(+)

diff --git a/docker-compose.yml b/docker-compose.yml
index 3643d9e..df06d85 100644
--- a/docker-compose.yml
+++ b/docker-compose.yml
@@ -58,6 +58,7 @@ services:
       - MINIO_BUCKET=starrocks
       - MINIO_ACCESS_KEY=AAAAAAAAAAAAAAAAAAAA
       - MINIO_SECRET_KEY=BBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBBB
+      - SLEEP_FOR=0.1s
     ports:
       - 8031:8031
       - 9021:9021
diff --git a/scripts/cn-entrypoint.sh b/scripts/cn-entrypoint.sh
index d2a85b7..400d317 100644
--- a/scripts/cn-entrypoint.sh
+++ b/scripts/cn-entrypoint.sh
@@ -16,6 +16,7 @@ echo "BRPC_PORT=${BRPC_PORT}"
 echo "STARLET_PORT=${STARLET_PORT}"
 
 if [ -n "$SLEEP_FOR" ]; then
+	echo "Sleeping for $SLEEP_FOR"
 	sleep $SLEEP_FOR;
 fi
 
diff --git a/scripts/fe-entrypoint.sh b/scripts/fe-entrypoint.sh
index bea25fa..6408238 100644
--- a/scripts/fe-entrypoint.sh
+++ b/scripts/fe-entrypoint.sh
@@ -17,6 +17,11 @@ echo "SQL_QUERY_PORT=${SQL_QUERY_PORT}"
 echo "EDIT_LOG_PORT=${EDIT_LOG_PORT}"
 echo "START_FOLLOWER_W_LEADER_EDIT_LOG_URL=${START_FOLLOWER_W_LEADER_EDIT_LOG_URL}"
 
+if [ -n "$SLEEP_FOR" ]; then
+	echo "Sleeping for $SLEEP_FOR"
+	sleep $SLEEP_FOR;
+fi
+
 # NOTE: this env var is only required the first time the follower is added to the cluster
 # See https://docs.starrocks.io/docs/deployment/deploy_manually/
 if [ -n "$START_FOLLOWER_W_LEADER_EDIT_LOG_URL" ]; then

From 282d9664e41c49c62a6b48237592c7f0080315cf Mon Sep 17 00:00:00 2001
From: Gavin <gaprice@lbl.gov>
Date: Sun, 16 Jun 2024 18:27:09 -0700
Subject: [PATCH 2/2] Note that AVX2 is required

Starrocks docs don't make this as clear as it could be
---
 README.md | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/README.md b/README.md
index 527b692..d344557 100644
--- a/README.md
+++ b/README.md
@@ -16,6 +16,8 @@ https://docs.starrocks.io/docs/deployment/environment_configurations/
 
 These are NOT taken into account in these Dockerfiles.
 
+However, the AVX2 instruction set is required for these images to run.
+
 ## Variables
 
 Variables that need to be set are documented in the scripts in the ./scripts directory.