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.

Sign up for GitHub

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

Jump to bottom

Don't panic if getting current use fails #2039

Merged
merged 3 commits into from
Mar 20, 2015
Merged

Don't panic if getting current use fails #2039

merged 3 commits into from
Mar 20, 2015

Conversation

otoolep
Copy link
Contributor

@otoolep otoolep commented Mar 20, 2015

No description provided.

@otoolep
Copy link
Contributor Author

otoolep commented Mar 20, 2015 

[srvr] 2015/03/20 15:20:32 No config provided, using default settings
[srvr] 2015/03/20 15:20:32 failed to generate default config: failed to determine current user for storage. Please supply an explicit configuration file

@otoolep
Copy link
Contributor Author

otoolep commented Mar 20, 2015

@corylanou @dgnorton

@otoolep
Copy link
Contributor Author

otoolep commented Mar 20, 2015

This is the best we can do. In this corner case the user must supply an explicit config.

This was referenced Mar 20, 2015
Closed
Closed
@corylanou
Copy link
Contributor

+1

@otoolep
Copy link
Contributor Author

otoolep commented Mar 20, 2015

Thanks @corylanou -- I think this is a decent solution.

otoolep added a commit that referenced this pull request Mar 20, 2015
@otoolep
Merge pull request #2039 from influxdb/panic_home_dir
259f0aa 
Don't panic if getting current use fails
@otoolep otoolep merged commit 259f0aa into master Mar 20, 2015
@otoolep otoolep deleted the panic_home_dir branch March 20, 2015 22:39
mark-rushakoff pushed a commit that referenced this pull request Jan 11, 2019
@mavarius
Data loader button bar (#2039)
ef6321a 
* Add Skip to Verify on streaming data steps, style wizard button bar

* Add tests, update snapshots
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@otoolep @corylanou