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

[0.9.4.1] runtime/cgo: out of memory in thread_start #4624

Closed
brettdh opened this issue Oct 30, 2015 · 4 comments
Closed

[0.9.4.1] runtime/cgo: out of memory in thread_start #4624

brettdh opened this issue Oct 30, 2015 · 4 comments
Labels

Comments

@brettdh
Copy link

brettdh commented Oct 30, 2015

Crash log file (raw)

Search for "out of memory" or "cannot allocate memory"; the log contains several such failures, after about a day's worth of writes every 10 seconds, with successful queries interspersed. Some of the panics occurred immediately after a query; some of them occurred in server restart loops; some occurred during WAL flushes.

There was also a random "unexpected fault address" panic in this log, but I have no idea if it's related.

@brettdh
Copy link
Author

brettdh commented Oct 30, 2015

Note: in our setup, we impose a 1GB memory limit on our daemons by default, including influxd. It's currently hovering at 956m virtual, 35m resident, under the same write volume and occasional queries, so I see no obvious leak there.

@jwilder
Copy link
Contributor

jwilder commented Oct 30, 2015

1GB may not be enough RAM for your dataset plus working room for queries and other tasks. I would try increasing it.

It's also possible that you are hitting a Go 1.5 bug that seems to be causing #4548 too. Go 1.5.2 might fix this.

@brettdh
Copy link
Author

brettdh commented Oct 30, 2015

Bumped to 4GB and restarted influxd. Got stuck in the same startup loop for a while; realized new rlimit was not in place; did it right the second time. No restart loop with 4GB limit, and now it's running again. Will check it periodically for more query-related panics.

@jwilder jwilder added the panic label Nov 5, 2015
@nathanielc
Copy link
Contributor

we have upgraded to go1.5.2, closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants