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

配置文件热重载(不知道是否本身就有) #54

Open
Marcus-zh opened this issue Apr 3, 2024 · 5 comments · May be fixed by #71
Open

配置文件热重载(不知道是否本身就有) #54

Marcus-zh opened this issue Apr 3, 2024 · 5 comments · May be fixed by #71
Assignees
Labels
enhancement New feature or request mid-priority 不影响主要功能的一些问题

Comments

@Marcus-zh
Copy link

每次改完配置似乎都要重启一遍服务,检查文件又检查半天才上线
希望配置能自动热重载(或者网页提供个按钮),输出一下重载了,才放心
(如果有当我没说,当也希望加个提示)

@zyxkad zyxkad self-assigned this Apr 3, 2024
@zyxkad zyxkad added enhancement New feature or request high-priority 会尽可能的整好 labels Apr 3, 2024
@zyxkad
Copy link
Member

zyxkad commented Apr 3, 2024

自动重载是不现实的,目前可以发SIGQUIT重载配置文件,但其实本质上也是重启

如果用了systemd那么可以直接调用systemctl reload go-openbmclapi

@8MiYile
Copy link
Contributor

8MiYile commented Apr 4, 2024

塞网页内 登录后才能点重载的
@zyxkad

@Marcus-zh
Copy link
Author

自动重载是不现实的,目前可以发SIGQUIT重载配置文件,但其实本质上也是重启

如果用了systemd那么可以直接调用systemctl reload go-openbmclapi

反正不是需要重新检查一遍文件就好了

@zyxkad zyxkad added mid-priority 不影响主要功能的一些问题 and removed high-priority 会尽可能的整好 labels Apr 20, 2024
@unify-z
Copy link

unify-z commented Apr 26, 2024

自动重载是不现实的,目前可以发SIGQUIT重载配置文件,但其实本质上也是重启
如果用了systemd,那么可以直接调用systemctl reload go-openbmclapi

反正不是需要重新检查一遍文件就好了

重新检查文件不是有缓存吗,我这里webdav储存13s就完了

@Marcus-zh
Copy link
Author

自动重载是不现实的,目前可以发SIGQUIT重载配置文件,但其实本质上也是重启
如果用了systemd,那么可以直接调用systemctl reload go-openbmclapi

反正不是需要重新检查一遍文件就好了

重新检查文件不是有缓存吗,我这里webdav储存13s就完了

啊,真的吗,我再试试

@zyxkad zyxkad linked a pull request Jun 27, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request mid-priority 不影响主要功能的一些问题
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants