merge: #4044

4044: chore: update userdata example to include dockerhub login r=sprutton1 a=sprutton1 Dockerhub is a curse on the land. SDF failed to start correctly because we use docker to build the config file and the pull was rate limited. I'm also going to change the healthcheck so the LB will kill the instance if it is unresponsive. We need better healthchecks for the other services, though. Co-authored-by: Scott Prutton <[email protected]>
systeminit · Jun 26, 2024 · 35be650 · 35be650
2 parents a6acbd6 + 1aadca3
commit 35be650
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/bin/veritech/scripts/userdata.sh b/bin/veritech/scripts/userdata.sh
@@ -73,6 +73,9 @@ wget https://artifacts.systeminit.com/${SI_SERVICE}/${SI_VERSION}/omnibus/linux/
 
 # prep system
 mkdir -p /run/app
+
+DOCKER_CREDS=$(aws secretsmanager get-secret-value --region us-east-1 --secret-id ${SI_HOSTENV}-dockerhub-creds | jq -r '.SecretString')
+docker login --username $(echo $DOCKER_CREDS | jq -r '.username') --password-stdin <<< $(echo $DOCKER_CREDS | jq -r '.password')
 wget https://raw.githubusercontent.com/systeminit/si/${BRANCH:-main}/component/deploy/docker-compose.yaml -O /run/app/docker-compose.yaml
 
 docker-compose -f /run/app/docker-compose.yaml --profile $SI_SERVICE up --wait

diff --git a/component/deploy/userdata b/component/deploy/userdata
@@ -22,6 +22,9 @@ wget https://artifacts.systeminit.com/${SI_SERVICE}/${SI_VERSION}/omnibus/linux/
 
 # prep system
 mkdir -p /run/app
+
+DOCKER_CREDS=$(aws secretsmanager get-secret-value --region us-east-1 --secret-id ${SI_HOSTENV}-dockerhub-creds | jq -r '.SecretString')
+docker login --username $(echo $DOCKER_CREDS | jq -r '.username') --password-stdin <<< $(echo $DOCKER_CREDS | jq -r '.password')
 wget https://raw.githubusercontent.com/systeminit/si/${BRANCH:-main}/component/deploy/docker-compose.yaml -O /run/app/docker-compose.yaml
 
 docker-compose -f /run/app/docker-compose.yaml --profile $SI_SERVICE up --wait