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

InfluxDB crashes if unable to load configuration file #674

Closed
falzm opened this issue Jun 22, 2014 · 1 comment
Closed

InfluxDB crashes if unable to load configuration file #674

falzm opened this issue Jun 22, 2014 · 1 comment

Comments

@falzm
Copy link

falzm commented Jun 22, 2014

When I first tried to run the influxdb binary without options (version 0.7.3):

marc@basalt:/opt/influxdb/current% ./influxdb
panic: open config.sample.toml: no such file or directory

goroutine 1 [running]:
runtime.panic(0x87abc0, 0xc210079780)
        /home/vagrant/bin/go/src/pkg/runtime/panic.c:266 +0xb6
configuration.LoadConfiguration(0x9734b0, 0x12, 0x7f7f9da5cf00)
        /home/vagrant/influxdb/src/configuration/configuration.go:264 +0x1e7
main.main()
        /home/vagrant/influxdb/src/daemon/influxd.go:74 +0x4ea

goroutine 3 [syscall]:
os/signal.loop()
        /home/vagrant/bin/go/src/pkg/os/signal/signal_unix.go:21 +0x1e
created by os/signal.init·1
        /home/vagrant/bin/go/src/pkg/os/signal/signal_unix.go:27 +0x31

goroutine 4 [runnable]:
code.google.com/p/log4go.ConsoleLogWriter.run(0xc210069000, 0x7f7f9dbfc110, 0xc210000008)
        /home/vagrant/influxdb/src/code.google.com/p/log4go/termlog.go:27 +0x60
created by code.google.com/p/log4go.NewConsoleLogWriter
        /home/vagrant/influxdb/src/code.google.com/p/log4go/termlog.go:19 +0x67

goroutine 5 [syscall]:
runtime.goexit()
        /home/vagrant/bin/go/src/pkg/runtime/proc.c:1394

There must be a more graceful way to tell the user that InfluxDB cannot run without a configuration file ;)

m.

jvshahid added a commit that referenced this issue Aug 29, 2014
Fix #674: graceful exit when having config file issues
@falzm
Copy link
Author

falzm commented Aug 29, 2014

Thanks!

@jvshahid jvshahid added this to the Next release milestone Aug 29, 2014
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

No branches or pull requests

2 participants