Make Litestream retention configurable (#447)

Details in #446 One problem though: due to current code organization (few "entrypoints"), I did not find a good **single** place for a default value. --------- Co-authored-by: Michael Lynch <[email protected]>
mtlynch · Jul 19, 2023 · e3ba3a7 · e3ba3a7
1 parent c67d347
commit e3ba3a7
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 2 deletions.
diff --git a/Dockerfile b/Dockerfile
@@ -67,6 +67,8 @@ COPY ./litestream.yml /etc/litestream.yml
 COPY ./static /app/static
 COPY ./LICENSE /app/LICENSE
 
+ENV LITESTREAM_RETENTION=72h
+
 WORKDIR /app
 
 ENTRYPOINT ["/app/docker-entrypoint"]

diff --git a/README.md b/README.md
@@ -123,6 +123,7 @@ You can adjust behavior of the Docker container by specifying these Docker-speci
 | `LITESTREAM_ENDPOINT`          | Litestream-compatible cloud storage endpoint where Litestream should replicate data.                  |
 | `LITESTREAM_ACCESS_KEY_ID`     | Litestream-compatible cloud storage access key ID to the bucket where you want to replicate data.     |
 | `LITESTREAM_SECRET_ACCESS_KEY` | Litestream-compatible cloud storage secret access key to the bucket where you want to replicate data. |
+| `LITESTREAM_RETENTION`         | The amount of time Litestream snapshots & WAL files will be kept (defaults to 72h).                   |
 
 ### Docker build args
 

diff --git a/docker-entrypoint b/docker-entrypoint
@@ -47,6 +47,7 @@ if [[ "${IS_LITESTREAM_ENABLED}" == 'true' ]]; then
   /app/litestream version
   echo "LITESTREAM_BUCKET=${LITESTREAM_BUCKET}"
   echo "LITESTREAM_ENDPOINT=${LITESTREAM_ENDPOINT}"
+  echo "LITESTREAM_RETENTION=${LITESTREAM_RETENTION}"
 
   if [[ -f "$DB_PATH" ]]; then
     echo "Existing database is $(stat -c %s "${DB_PATH}") bytes"

diff --git a/fly.toml b/fly.toml
@@ -9,6 +9,7 @@ processes = []
   PS_BEHIND_PROXY="yes"
   LITESTREAM_BUCKET="picoshare-litestream"
   LITESTREAM_ENDPOINT="s3.us-west-002.backblazeb2.com"
+  LITESTREAM_RETENTION="72h"
 
 [experimental]
   allowed_public_ports = []

diff --git a/litestream.yml b/litestream.yml
@@ -8,5 +8,5 @@ dbs:
         path: db
         endpoint: ${LITESTREAM_ENDPOINT}
         force-path-style: true
-        retention: 72h
+        retention: ${LITESTREAM_RETENTION}
         snapshot-interval: 24h
diff --git a/playwright.config.ts b/playwright.config.ts
@@ -3,7 +3,7 @@ import { devices } from "@playwright/test";
 
 const config: PlaywrightTestConfig = {
   testDir: "./e2e",
-  timeout: 30 * 1000,
+  timeout: 35 * 1000,
   expect: {
     timeout: 5 * 1000,
   },