diff --git a/bitnami/redis-cluster/5.0/debian-10/Dockerfile b/bitnami/redis-cluster/5.0/debian-10/Dockerfile index 2a24c2fc1263..4c630f5ab549 100644 --- a/bitnami/redis-cluster/5.0/debian-10/Dockerfile +++ b/bitnami/redis-cluster/5.0/debian-10/Dockerfile @@ -18,7 +18,7 @@ RUN chmod g+rwX /opt/bitnami COPY rootfs / RUN /opt/bitnami/scripts/redis-cluster/postunpack.sh ENV BITNAMI_APP_NAME="redis-cluster" \ - BITNAMI_IMAGE_VERSION="5.0.10-debian-10-r68" \ + BITNAMI_IMAGE_VERSION="5.0.10-debian-10-r69" \ PATH="/opt/bitnami/redis/bin:/opt/bitnami/common/bin:$PATH" EXPOSE 6379 diff --git a/bitnami/redis-cluster/5.0/debian-10/prebuildfs/opt/bitnami/scripts/libos.sh b/bitnami/redis-cluster/5.0/debian-10/prebuildfs/opt/bitnami/scripts/libos.sh index cc45a19ce099..f38437a1d407 100644 --- a/bitnami/redis-cluster/5.0/debian-10/prebuildfs/opt/bitnami/scripts/libos.sh +++ b/bitnami/redis-cluster/5.0/debian-10/prebuildfs/opt/bitnami/scripts/libos.sh @@ -37,14 +37,34 @@ group_exists() { # Create a group in the system if it does not exist already # Arguments: # $1 - group +# Flags: +# -s|--system - Whether to create new user as system user (uid <= 999) # Returns: # None ######################### ensure_group_exists() { local group="${1:?group is missing}" + local is_system_user=false + + # Validate arguments + shift 1 + while [ "$#" -gt 0 ]; do + case "$1" in + -s|--system) + is_system_user=true + ;; + *) + echo "Invalid command line flag $1" >&2 + return 1 + ;; + esac + shift + done if ! group_exists "$group"; then - groupadd "$group" >/dev/null 2>&1 + local -a args=("$group") + $is_system_user && args+=("--system") + groupadd "${args[@]}" >/dev/null 2>&1 fi } @@ -52,22 +72,60 @@ ensure_group_exists() { # Create an user in the system if it does not exist already # Arguments: # $1 - user -# $2 - group +# Flags: +# -g|--group - the group the new user should belong to +# -h|--home - the home directory for the new user +# -s|--system - whether to create new user as system user (uid <= 999) # Returns: # None ######################### ensure_user_exists() { local user="${1:?user is missing}" - local group="${2:-}" + local group="" + local home="" + local is_system_user=false + + # Validate arguments + shift 1 + while [ "$#" -gt 0 ]; do + case "$1" in + -g|--group) + shift + group="${1:?missing group}" + ;; + -h|--home) + shift + home="${1:?missing home directory}" + ;; + -s|--system) + is_system_user=true + ;; + *) + echo "Invalid command line flag $1" >&2 + return 1 + ;; + esac + shift + done if ! user_exists "$user"; then - useradd "$user" >/dev/null 2>&1 + local -a user_args=("-N" "$user") + $is_system_user && user_args+=("--system") + useradd "${user_args[@]}" >/dev/null 2>&1 fi if [[ -n "$group" ]]; then - ensure_group_exists "$group" + local -a group_args=("$group") + $is_system_user && group_args+=("--system") + ensure_group_exists "${group_args[@]}" usermod -a -G "$group" "$user" >/dev/null 2>&1 fi + + if [[ -n "$home" ]]; then + mkdir -p "$home" + usermod -d "$home" "$user" >/dev/null 2>&1 + configure_permissions_ownership "$home" -d "775" -f "664" -u "$user" -g "$group" + fi } ######################## diff --git a/bitnami/redis-cluster/5.0/debian-10/rootfs/opt/bitnami/scripts/redis-cluster/setup.sh b/bitnami/redis-cluster/5.0/debian-10/rootfs/opt/bitnami/scripts/redis-cluster/setup.sh index 469895d11948..2b86ebc382f9 100755 --- a/bitnami/redis-cluster/5.0/debian-10/rootfs/opt/bitnami/scripts/redis-cluster/setup.sh +++ b/bitnami/redis-cluster/5.0/debian-10/rootfs/opt/bitnami/scripts/redis-cluster/setup.sh @@ -19,7 +19,7 @@ set -o pipefail redis_cluster_validate # Ensure Redis is stopped when this script ends trap "redis_stop" EXIT -am_i_root && ensure_user_exists "$REDIS_DAEMON_USER" "$REDIS_DAEMON_GROUP" +am_i_root && ensure_user_exists "$REDIS_DAEMON_USER" --group "$REDIS_DAEMON_GROUP" # Ensure Redis is initialized redis_cluster_initialize diff --git a/bitnami/redis-cluster/README.md b/bitnami/redis-cluster/README.md index c3db16424b97..fd3872982fd5 100644 --- a/bitnami/redis-cluster/README.md +++ b/bitnami/redis-cluster/README.md @@ -43,7 +43,7 @@ Learn more about the Bitnami tagging policy and the difference between rolling t * [`6.0`, `6.0-debian-10`, `6.0.10`, `6.0.10-debian-10-r1`, `latest` (6.0/debian-10/Dockerfile)](https://github.com/bitnami/bitnami-docker-redis-cluster/blob/6.0.10-debian-10-r1/6.0/debian-10/Dockerfile) -* [`5.0`, `5.0-debian-10`, `5.0.10`, `5.0.10-debian-10-r68` (5.0/debian-10/Dockerfile)](https://github.com/bitnami/bitnami-docker-redis-cluster/blob/5.0.10-debian-10-r68/5.0/debian-10/Dockerfile) +* [`5.0`, `5.0-debian-10`, `5.0.10`, `5.0.10-debian-10-r69` (5.0/debian-10/Dockerfile)](https://github.com/bitnami/bitnami-docker-redis-cluster/blob/5.0.10-debian-10-r69/5.0/debian-10/Dockerfile) Subscribe to project updates by watching the [bitnami/redis-cluster GitHub repo](https://github.com/bitnami/bitnami-docker-redis-cluster).