From 082f616668e1d62f3a6ae6084dabce5cb7e1c720 Mon Sep 17 00:00:00 2001 From: Rose Liverman Date: Wed, 30 Oct 2024 09:55:56 -0600 Subject: [PATCH] Remove duplicate instruction in ReadMe The docker-compose rebuilds the image, so it's a waste of time to build it first. --- README.md | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/README.md b/README.md index 61991657..e79753f6 100644 --- a/README.md +++ b/README.md @@ -44,10 +44,9 @@ One could put Elk behind popular reverse proxies with SSL Handling like Traefik, 1. checkout source ```git clone https://github.com/elk-zone/elk.git``` 1. got into new source dir: ```cd elk``` -1. build Docker image: ```docker build .``` 1. create local storage directory for settings: ```mkdir elk-storage``` 1. adjust permissions of storage dir: ```sudo chown 911:911 ./elk-storage``` -1. start container: ```docker-compose up -d``` +1. build Docker image & start container: ```docker-compose up -d``` > [!NOTE] > The provided Dockerfile creates a container which will eventually run Elk as non-root user and create a persistent named Docker volume upon first start (if that volume does not yet exist). This volume is always created with root permission. Failing to change the permissions of ```/elk/data``` inside this volume to UID:GID 911 (as specified for Elk in the Dockerfile) will prevent Elk from storing it's config for user accounts. You either have to fix the permission in the created named volume, or mount a directory with the correct permission to ```/elk/data``` into the container.