Skip to content

add mod_autoreload for automatically reloading ejabberd when config and/of certificates changes#368

Merged
badlop merged 1 commit intoprocessone:masterfrom
blieb:master
Nov 26, 2025
Merged

add mod_autoreload for automatically reloading ejabberd when config and/of certificates changes#368
badlop merged 1 commit intoprocessone:masterfrom
blieb:master

Conversation

@blieb
Copy link
Copy Markdown
Contributor

@blieb blieb commented Nov 24, 2025

This module makes it possible to let ejabberd autoreload when certificates and/or config files changes.
For example when running inside kubernetes cluster and certificates are updated by external progress or config file is inside a configmap, and when you update all nodes automatically load new config.

@badlop badlop merged commit b6b36bc into processone:master Nov 26, 2025
6 checks passed
@badlop
Copy link
Copy Markdown
Member

badlop commented Nov 26, 2025

It is possible to enable the module with some default options when installing the module. That feature makes sense when the module has a very simple configuration; for example mod_shcommands and mod_webadmin_config.

I wonder if it would make sense to add a file conf/mod_autoreload.yml that enables the module with its default configuration when the module is installed.

I would say that it makes sense (but I am not a potential user of this module). What do you think?

@blieb
Copy link
Copy Markdown
Contributor Author

blieb commented Nov 26, 2025

It is possible to enable the module with some default options when installing the module. That feature makes sense when the module has a very simple configuration; for example mod_shcommands and mod_webadmin_config.

I wonder if it would make sense to add a file conf/mod_autoreload.yml that enables the module with its default configuration when the module is installed.

I would say that it makes sense (but I am not a potential user of this module). What do you think?

Hmm stange I thought I did that, but I see the config file is missing, let me add that

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.

2 participants