Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Attempt to catch cases of incorrect datadir ownership #74

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Attempt to catch cases of incorrect datadir ownership #74

wants to merge 2 commits into from

Commits on Sep 15, 2020

  1. Attempt to catch cases of incorrect datadir ownership 

    If the Elasticsearch datadir is not owned by the user specified in
$DOCKER_USER, Elasticsearch will fail to start.

Since #55 the helper scripts
attempt to remedy this situation if the `pelias` script is run as root
(which is not recommended but is often done).

However, if the `pelias` script is not run as root and the permissions
are incorrect, this situation cannot be automatically fixed.

This code attempts to detect that case and recommend the proper command
to run (with sudo) and set proper directory permissions.

Connects #31
Connects #73
    @orangejulius
    orangejulius committed Sep 15, 2020
    Configuration menu
    Copy the full SHA
    fde416c View commit details
    Browse the repository at this point in the history

  2. Attempt to run command to fix permissions

    @orangejulius
    orangejulius committed Sep 15, 2020
    Configuration menu
    Copy the full SHA
    b9f4567 View commit details
    Browse the repository at this point in the history