Server does not shut down cleanly when rules file has syntax errors.

[beorn7]

Should be easy to fix... either load rules before starting the storage, or shut down storage cleanly if rules don't parse.

[beorn7]

Other errors (e.g. if the web.listen address cannot be bound to) may have the same effect.

[fabxc]

The rule issue should be fixed by #694 as we no longer shutdown if rules don't parse.

[juliusv]

@fabxc But this previously only affected startups - and we still want to die when the configs/rules can't be parsed upon startup, no?

[fabxc]

Yes, that's what I was talking about with @beorn7 before. We won't get around restructuring the storage startup.

You can find an attempt in this branch.

No background goroutines are started until after Storage.Start() knows that it can return without an error.
Chances are I missed something essential, so the general approach has to be verified by @beorn7 :)

[fabxc]

Fixed by #704

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.