Support SIGHUP to re-read and update running config

Question

hloeung opened this issue 4 years ago · 2 comments

Hi,

This is more of a feature request, any chance of supporting SIGHUP to trigger re-reading the matterircd.toml and apply any config changes?

Would be nice to allow enabling or disabling configs without having to do a full restart (plus mattermost login and then wait for sync).

Per 0a871bd, it already supports SIGHUP for reloading certs. So maybe another signal.Notify() to call config.LoadConfig()?

Thanks!

Answer 1 · 2020-12-12T14:02:14.000Z

This shouldn't be necessary?
We already do a automatic reload on config changes (except when your on illumos)
What changes are you not picking up?

Answer 2 · 2020-12-12T15:06:06.000Z

Ah yes, confirmed. I've been spawning new matterircd instances to test between matterircd generated and mattermost message IDs.