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

headscale crashes with OOM when sqlite path is ommited #1761

Closed
2 tasks
SuperSandro2000 opened this issue Feb 15, 2024 · 2 comments · Fixed by #1765
Closed
2 tasks

headscale crashes with OOM when sqlite path is ommited #1761

SuperSandro2000 opened this issue Feb 15, 2024 · 2 comments · Fixed by #1765
Labels
bug Something isn't working
Milestone

Comments

@SuperSandro2000
Copy link
Contributor

Bug description

When ommiting database.sqlite.path from the config, headscale doesn't properly fallback to /var/lib/headscale/db.sqlite and instead crash loops with the following error:

Feb 15 23:54:38 hydrogen headscale-start[2491519]: 2024-02-15T23:54:38+01:00 FTL github.com/juanfont/headscale/cmd/headscale/cli/server.go:21 > Error initializing error="unable to open database file: out of memory (14)"

Environment

  • OS: NixOS unstable
  • Headscale version: 0.23.0-alpha4
  • Tailscale version: 1.58.2
  • Headscale is behind a (reverse) proxy
  • Headscale runs in a container

To Reproduce

Logs and attachments

@SuperSandro2000 SuperSandro2000 added the bug Something isn't working label Feb 15, 2024
@kradalby kradalby added this to the v0.23.0 milestone Feb 16, 2024
kradalby added a commit to kradalby/headscale that referenced this issue Feb 16, 2024
kradalby added a commit to kradalby/headscale that referenced this issue Feb 16, 2024
@kradalby kradalby linked a pull request Feb 16, 2024 that will close this issue
kradalby added a commit to kradalby/headscale that referenced this issue Feb 17, 2024
kradalby added a commit that referenced this issue Feb 17, 2024
* improve errors for missing directories

Fixes #1761
Updates #1760

Signed-off-by: Kristoffer Dalby <[email protected]>

* update container docs

Signed-off-by: Kristoffer Dalby <[email protected]>

* update changelog with /var changes

Signed-off-by: Kristoffer Dalby <[email protected]>

---------

Signed-off-by: Kristoffer Dalby <[email protected]>
@Nickiel12
Copy link

This issue appeared again in alpha-8. I was getting a "out of memory" error, until I specified the full path as shown by OP.

@aqxa1
Copy link

aqxa1 commented Jul 13, 2024

I've still got the issue with alpha-12. Setting sqlite instead of sqlite3 worked around the problem, although I'm not sure how they differ.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants