Skip to content

Commit

Permalink
fix: Make docker volume script respect compose project name (#3039)
Browse files Browse the repository at this point in the history
make the script a bit more robust in finding compose peojct name
  • Loading branch information
hubertdeng123 committed May 13, 2024
1 parent 40eed10 commit 9de4b70
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion install/update-docker-volume-permissions.sh
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ echo "${_group}Ensuring Kafka and Zookeeper volumes have correct permissions ...
if [[ "$DOCKER_PLATFORM" = "linux/amd64" && -n "$(docker volume ls -q -f name=sentry-zookeeper)" && -n "$(docker volume ls -q -f name=sentry-kafka)" ]]; then
zookeeper_data_dir="/var/lib/docker/volumes/sentry-zookeeper/_data"
kafka_data_dir="/var/lib/docker/volumes/sentry-kafka/_data"
zookeeper_log_data_dir="/var/lib/docker/volumes/sentry-self-hosted_sentry-zookeeper-log/_data"
zookeeper_log_data_dir="/var/lib/docker/volumes/${COMPOSE_PROJECT_NAME}_sentry-zookeeper-log/_data"
chmod -R a+w $zookeeper_data_dir $kafka_data_dir $zookeeper_log_data_dir && returncode=$? || returncode=$?
if [[ $returncode == "1" ]]; then
echo "WARNING: Error when setting appropriate permissions for zookeeper, kafka, and zookeeper log docker volumes. This may corrupt your self-hosted install. See https://github.com/confluentinc/kafka-images/issues/127 for context on why this was added."
Expand Down

0 comments on commit 9de4b70

Please sign in to comment.