docs: Updates the Unraid documentation for alternative way to deploy Seerr using correct file system permissions and prevent access issues#2471
Conversation
📝 WalkthroughWalkthroughDocumentation updates for Unraid: clarified manual-install wording, added Unraid Community Apps note, provided two permission-handling options (chown 1000:1000 or 99:100 and optional Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Tip Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
🤖 Fix all issues with AI agents
Verify each finding against the current code and only fix it if needed.
In `@docs/getting-started/third-parties/unraid.mdx`:
- Around line 79-82: Bold the second occurrence of "Extra Parameters" to match
the first (make "Extra Parameters" bold wherever referenced) and insert a blank
line between the paragraph describing the `--init` flag and the paragraph
describing the `--user 99:100` flag so the `--init` and `--user` explanations
are separated for consistent formatting; update the block that mentions
`--init`, `--user 99:100`, and "Extra Parameters" accordingly.
🧹 Nitpick comments (1)
🤖 Fix all nitpicks with AI agents
Verify each finding against the current code and only fix it if needed. In `@docs/getting-started/third-parties/unraid.mdx`: - Around line 79-82: Bold the second occurrence of "Extra Parameters" to match the first (make "Extra Parameters" bold wherever referenced) and insert a blank line between the paragraph describing the `--init` flag and the paragraph describing the `--user 99:100` flag so the `--init` and `--user` explanations are separated for consistent formatting; update the block that mentions `--init`, `--user 99:100`, and "Extra Parameters" accordingly.docs/getting-started/third-parties/unraid.mdx (1)
79-82: Minor formatting inconsistency: bold styling differs between lines.Line 80 uses
**Extra Parameters**(bold), but line 81 uses plainExtra Parameters. For consistency, consider bolding both occurrences.Additionally, inserting a blank line between the
--initand--userexplanations would improve readability and ensure consistent paragraph separation across different Markdown renderers.📝 Suggested formatting fix
:::info -The `--init` flag in **Extra Parameters** is required. Seerr does not include its own init process, so `--init` ensures proper signal handling and clean container shutdowns. -The `--user 99:100` flag in Extra Parameters ensures the container runs with the correct permissions. In Unraid, the default user account is nobody (ID 99) and the default group is users (ID 100). Using this flag prevents "Permission Denied" errors when the container tries to read or write to your shares. +The `--init` flag in **Extra Parameters** is required. Seerr does not include its own init process, so `--init` ensures proper signal handling and clean container shutdowns. + +The `--user 99:100` flag in **Extra Parameters** ensures the container runs with the correct permissions. In Unraid, the default user account is nobody (ID 99) and the default group is users (ID 100). Using this flag prevents "Permission Denied" errors when the container tries to read or write to your shares. :::🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/getting-started/third-parties/unraid.mdx` around lines 79 - 82, Bold the second occurrence of "Extra Parameters" to match the first (make "Extra Parameters" bold wherever referenced) and insert a blank line between the paragraph describing the `--init` flag and the paragraph describing the `--user 99:100` flag so the `--init` and `--user` explanations are separated for consistent formatting; update the block that mentions `--init`, `--user 99:100`, and "Extra Parameters" accordingly.
|
#2470 literally explains the proper method to do this on unraid though. |
Well 1000:1000 isn't right. Should be 99:100 on the file folder |
It is correct. There are multiple ways to do this. Your way is an ALTERNATIVE way. But that pr is absolutely correct. |
|
Accidentally closed. Please rework this pr stating doing this as an alternative method. |
|
This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged. |
fallenbagel
left a comment
fallenbagel
left a comment
There was a problem hiding this comment.
Please make the requested changes.
will do. |
github-actions
bot
removed
the
merge conflict
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (1)
docs/getting-started/third-parties/unraid.mdx (1)
88-89: Good explanation, but adjust if presenting as alternative.The explanation of
--initand--user 99:100is clear and accurate. If the documentation is restructured per maintainer feedback to present both methods, consider updating this block to explain:
--initis always required--user 99:100is needed only if you chose the 99:100 ownership approach- Without
--user, the container runs as 1000:1000 (matching Method 1's chown)This would help users understand which flags apply to their chosen method.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/getting-started/third-parties/unraid.mdx` around lines 88 - 89, Update the paragraph to present the flags as conditional guidance: state that the `--init` flag is always required for proper signal handling and clean shutdown, explain that `--user 99:100` is only required if the reader chooses the "99:100 ownership" approach, and clarify that if `--user` is omitted the container will run as 1000:1000 (matching the chown behavior described in Method 1); keep references to `--init`, `--user 99:100`, and "Method 1" so readers can map flags to the chosen ownership method.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/getting-started/third-parties/unraid.mdx`:
- Line 50: Update the Extra Parameters row to present both supported methods:
show Method 1 as "`--init` only (use when your host uses chown 1000:1000)" and
Method 2 as "`--init --user 99:100` (use when your host uses chown 99:100)`", so
readers can pick based on their Unraid setup; mention the corresponding chown
examples (`chown 1000:1000` and `chown 99:100`) alongside the flags for clarity.
- Line 36: Update the documentation to present the two chown approaches as
alternatives rather than replacing one with the other: keep the existing "chown
-R 1000:1000 /mnt/user/appdata/seerr" as Method 1 (container internal UID) and
add Method 2 showing "chown -R 99:100 /mnt/user/appdata/seerr" with an
explanatory note to use "--user 99:100" for Unraid's nobody:users mapping;
ensure both commands are shown with brief one-line context explaining when to
use each.
---
Nitpick comments:
In `@docs/getting-started/third-parties/unraid.mdx`:
- Around line 88-89: Update the paragraph to present the flags as conditional
guidance: state that the `--init` flag is always required for proper signal
handling and clean shutdown, explain that `--user 99:100` is only required if
the reader chooses the "99:100 ownership" approach, and clarify that if `--user`
is omitted the container will run as 1000:1000 (matching the chown behavior
described in Method 1); keep references to `--init`, `--user 99:100`, and
"Method 1" so readers can map flags to the chosen ownership method.
milapointe
changed the title
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/getting-started/third-parties/unraid.mdx`:
- Line 54: Update the misspelled word "Explaination" to "Explanation" in the
docs text shown in unraid.mdx (the line that currently reads "Explaination : Do
not change host ownership..."); search the file for any other occurrences of
"Explaination" and correct them to "Explanation" to keep the document
consistent.
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
docs/getting-started/third-parties/unraid.mdx (1)
56-59: Add blank line after closing the warning block.A blank line after
:::and before the### 2.heading improves readability and ensures consistent rendering across MDX parsers.📝 Proposed fix
:::warning Do not use both approaches at the same time, either change the host folder ownership to `1000:1000` or run the container with `--user 99:100`. Mixing both can create confusing ownership and permission states. ::: + ### 2. Add the Docker container🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/getting-started/third-parties/unraid.mdx` around lines 56 - 59, Add a single blank line after the closing warning block (the line with ":::") so there is an empty line between that block and the subsequent heading "### 2. Add the Docker container"; update the MDX near the ':::warning' block to ensure a blank line exists immediately before the "### 2. Add the Docker container" heading to improve readability and parser consistency.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/getting-started/third-parties/unraid.mdx`:
- Around line 48-54: Fix the contradictory wording and punctuation in the Unraid
note: remove the extra space before the colon so "Explanation:" is correctly
formatted, and reword the sentence that follows the chown example to clarify
intent — state that after adjusting host ownership with "chown -R 99:100
/mnt/user/appdata/seerr" you should then run the container with Unraid's
nobody:users via the Docker param "--user 99:100" to ensure the container
process uses Unraid's default UID/GID and keeps files accessible via SMB and the
Unraid WebGUI (replace the current "Do not change host ownership" phrasing with
this clearer description).
---
Nitpick comments:
In `@docs/getting-started/third-parties/unraid.mdx`:
- Around line 56-59: Add a single blank line after the closing warning block
(the line with ":::") so there is an empty line between that block and the
subsequent heading "### 2. Add the Docker container"; update the MDX near the
':::warning' block to ensure a blank line exists immediately before the "### 2.
Add the Docker container" heading to improve readability and parser consistency.
| - Alternative: | ||
|
|
||
| ```bash | ||
| chown -R 99:100 /mnt/user/appdata/seerr | ||
| ``` | ||
|
|
||
| Explanation : Do not change host ownership and instead run the container as Unraid's `nobody:users` by adding the extra Docker parameter `--user 99:100` when creating the container. This runs the container process with Unraid's default UID/GID and avoids permission errors accessing your shares. |
There was a problem hiding this comment.
Fix formatting and clarify the explanation text.
Two issues:
- "Explanation : " has a space before the colon — should be "Explanation:" (no space).
- The text says "Do not change host ownership" but this option explicitly runs
chown -R 99:100, which does change ownership. The wording is contradictory.
Consider rewording to clarify the intent — that this approach keeps files accessible by Unraid's default user rather than Seerr's internal user.
📝 Proposed fix
- Alternative:
+ Alternative: Keep files accessible to Unraid by setting ownership to `nobody:users`:
```bash
chown -R 99:100 /mnt/user/appdata/seerr- Explanation : Do not change host ownership and instead run the container as Unraid's
nobody:usersby adding the extra Docker parameter--user 99:100when creating the container. This runs the container process with Unraid's default UID/GID and avoids permission errors accessing your shares.
- Then run the container as Unraid's
nobody:usersby adding the extra Docker parameter--user 99:100when creating the container. This ensures the container process uses Unraid's default UID/GID, keeping files accessible via SMB and the Unraid WebGUI.
</details>
<details>
<summary>🤖 Prompt for AI Agents</summary>
Verify each finding against the current code and only fix it if needed.
In @docs/getting-started/third-parties/unraid.mdx around lines 48 - 54, Fix the
contradictory wording and punctuation in the Unraid note: remove the extra space
before the colon so "Explanation:" is correctly formatted, and reword the
sentence that follows the chown example to clarify intent — state that after
adjusting host ownership with "chown -R 99:100 /mnt/user/appdata/seerr" you
should then run the container with Unraid's nobody:users via the Docker param
"--user 99:100" to ensure the container process uses Unraid's default UID/GID
and keeps files accessible via SMB and the Unraid WebGUI (replace the current
"Do not change host ownership" phrasing with this clearer description).
</details>
<!-- fingerprinting:phantom:poseidon:ocelot -->
<!-- This is an auto-generated comment by CodeRabbit -->
M0NsTeRRR
left a comment
M0NsTeRRR
left a comment
There was a problem hiding this comment.
Hello,
@fallenbagel asked to update the doc by adding an alternative method not removing an existing one.
You can do it by using tabs like here https://github.com/seerr-team/seerr/blob/develop/docs/migration-guide.mdx?plain=1#L66
i didnt remove, I added after his comment. |
I would suggest using tabs. So it's better ux |
milapointe
changed the title
3 participants
Description
This PR updates the Unraid documentation for Seerr to clarify supported permission-handling options and avoid access issues.
chown -R 1000:1000 /mnt/user/appdata/seerrnobody:usersaccount by adding the extra Docker parameter--user 99:100(avoids changing host ownership)chown -R 1000:1000on the host with--user 99:100in the container to prevent confusing ownership and permission states.This change ensures files created by the container are handled predictably and prevents "Permission Denied" errors when managing files via SMB or the Unraid WebGUI.
How Has This Been Tested?
--user 99:100allows the container to write to/mnt/user/appdata/seerrand that the resulting files are accessible and manageable by the Unraid host system without permission conflicts.Screenshots / Logs (if applicable)
Refer to the "Files changed" tab for the updated documentation layout.
Checklist:
pnpm buildpnpm i18n:extractSummary by CodeRabbit