Update overseerr to develop branch to fix issue with Plex watchlist sync#19
Conversation
WalkthroughThe docker-compose configuration for the Overseerr service was updated to use the image tag lscr.io/linuxserver/overseerr:develop instead of lscr.io/linuxserver/overseerr:latest. No other services or fields in docker-compose.yml were changed. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests
Tip 👮 Agentic pre-merge checks are now available in preview!Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.
Please see the documentation for more information. Example: reviews:
pre_merge_checks:
custom_checks:
- name: "Undocumented Breaking Changes"
mode: "warning"
instructions: |
Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).Please share your feedback with us on this Discord post. Comment |
There was a problem hiding this comment.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
docker-compose.yml (1)
248-248: Config volume path likely typo:overseervsoverseerr.This will mount a different directory than intended, causing Overseerr to lose state or re‑initialize on each run.
Apply:
- - ${USERDIR}/docker/overseer/config:/config + - ${USERDIR}/docker/overseerr/config:/config
🧹 Nitpick comments (1)
docker-compose.yml (1)
247-251: Optional: add a simple healthcheck for safer restarts and dependency ordering.Helps Compose and tooling detect when Overseerr is actually ready.
ports: - "5055:5055" restart: unless-stopped + healthcheck: + test: ["CMD-SHELL", "wget -qO- http://localhost:5055 > /dev/null || exit 1"] + interval: 30s + timeout: 5s + retries: 5If Overseerr exposes a better status endpoint in your build, we can tighten this probe.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
docker-compose.yml(1 hunks)
🔇 Additional comments (1)
docker-compose.yml (1)
242-255: Switch todeveloptag: acceptable for the hotfix, but mitigate rollout risk.Given the PR goal (fix Plex watchlist sync), moving Overseerr to
lscr.io/linuxserver/overseerr:developis reasonable. However,developcan introduce breaking changes and you have Watchtower auto‑updating nightly, which may pull untested builds.Recommend one of:
- Pin the current digest to freeze the exact build, and bump intentionally later.
- Or exclude this one service from Watchtower until the fix lands in stable.
Example (choose one approach):
- image: lscr.io/linuxserver/overseerr:develop + image: lscr.io/linuxserver/overseerr:develop@sha256:<digest_here>or
overseerr: image: lscr.io/linuxserver/overseerr:develop + labels: + - "com.centurylinklabs.watchtower.enable=false"If you want, I can fetch the current image digest and propose an exact pin. Also, please confirm that this
developbuild indeed contains the Plex watchlist sync fix you need.
Summary by CodeRabbit