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

ensure directory of pidfile exists after a restart #2297

Merged
merged 2 commits into from
Apr 15, 2015

Conversation

neonstalwart
Copy link
Contributor

/var/run on ubuntu is a tmpfs so /var/run/influxdb is lost after a reboot and the influxdb user does not have permission to create that directory. this ensures that the directory exists by creating it in the init script

fixes #2273

@neonstalwart
Copy link
Contributor Author

d'oh! missed rc25 😏

@neonstalwart
Copy link
Contributor Author

also, credit to http://serverfault.com/a/463743/234227

@otoolep
Copy link
Contributor

otoolep commented Apr 15, 2015

Looks good.

We should also remove creation of the directory from packaging.sh, since this covers everything.

@neonstalwart
Copy link
Contributor Author

hmm... i'm not sure how my changes relate to the test failure

@otoolep
Copy link
Contributor

otoolep commented Apr 15, 2015

@neonstalwart -- we are currently debugging some CI failures that are not related to your change. They are afflicting our builds every so often for an unknown reason. We will definitely merge your change once we are happy it's not the cause (which I am sure this one isn't).

@otoolep
Copy link
Contributor

otoolep commented Apr 15, 2015

This looks perfect -- thanks @neonstalwart

otoolep added a commit that referenced this pull request Apr 15, 2015
ensure directory of pidfile exists after a restart
@otoolep otoolep merged commit f6f4cc9 into influxdata:master Apr 15, 2015
@neonstalwart neonstalwart deleted the pidfile branch April 15, 2015 22:25
@timhallinflux timhallinflux added this to the 0.9.0 milestone Dec 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

/run/influxdb not persistent
3 participants