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.

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

add /var/lib/plausible #4275

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

add /var/lib/plausible #4275

wants to merge 2 commits into from

Conversation

ruslandoga
Copy link
Contributor

@ruslandoga ruslandoga commented Jun 25, 2024

Changes

This PR aims to simplify permissions in $DATA_DIR volume mounted into Plausible container. After adding automatic TLS we would be mounting a volume in docker-compose.yml by default to persist the certificates across restarts, so I thought we might as well allow writes for it by default too.

Before

User needs to create a directory on the host and set the correct permissions manually.
After that it can be mounted into the container.

console

$ cd hosting
$ mkdir ./data
$ chown -R 999:nogroup ./data

docker-compose.yml

plausible:
  volumes:
    - ./data:/data
  environment:
    - DATA_DIR=/data

After

A named volume can be mounted into the container without any extra steps.

docker-compose.yml

plausible:
  volumes:
    - plausible-data:/var/lib/plausible
    
volumes:
  plausible-data:

Tests

  • Automated tests have been added

Changelog

  • Entry has been added to changelog

Documentation

  • This change does not need a documentation update

Dark mode

  • This PR does not change the UI

@ruslandoga ruslandoga added the self-hosting Anything self-hosted label Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
self-hosting Anything self-hosted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant