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

feat(server): better mount checks #13092

Merged
merged 1 commit into from
Oct 1, 2024
Merged

feat(server): better mount checks #13092

merged 1 commit into from
Oct 1, 2024

Conversation

jrasm91
Copy link
Contributor

@jrasm91 jrasm91 commented Oct 1, 2024

  • Ignore errors when .immich already exists, but mount checks aren't enabled
  • Include a link in the error message to the docs
  • Add a docs page about system integrity
  • Add an env option to skip mount checks

@github-actions github-actions bot added documentation Improvements or additions to documentation 🗄️server labels Oct 1, 2024
@jrasm91 jrasm91 requested a review from bo0tzz October 1, 2024 14:31
@jrasm91 jrasm91 force-pushed the feat/better-mount-checks branch from 4f37297 to 7dffc95 Compare October 1, 2024 14:36
Comment on lines +41 to +47
### Ignoring the checks

The checks are designed to catch common problems that we have seen users have in the past, but if you want to disable them you can set the following environment variable:

```
IMMICH_IGNORE_MOUNT_CHECK_ERRORS=true
```
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want to add a disclaimer here along the lines of "if you've used this env var then unfortunately we can't help you with mountpoint issues"?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it is implied that when you disable default protections you are assuming the associated risk, but I can add a explicit note if you think that would be beneficial.

server/src/repositories/config.repository.ts Outdated Show resolved Hide resolved
@mmomjian
Copy link
Contributor

mmomjian commented Oct 1, 2024

Can we add an entry to the env vars page for this?

@jrasm91
Copy link
Contributor Author

jrasm91 commented Oct 1, 2024

Can we add an entry to the env vars page for this?

Sure.

@jrasm91 jrasm91 force-pushed the feat/better-mount-checks branch from 7dffc95 to eace7ed Compare October 1, 2024 15:19
@jrasm91 jrasm91 merged commit 305fc77 into main Oct 1, 2024
35 checks passed
@jrasm91 jrasm91 deleted the feat/better-mount-checks branch October 1, 2024 17:04
yosit pushed a commit to yosit/immich that referenced this pull request Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog:enhancement documentation Improvements or additions to documentation 🗄️server
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants