labring / FastGPT

FastGPT is a knowledge-based platform built on the LLMs, offers a comprehensive suite of out-of-the-box capabilities such as data processing, RAG retrieval, and visual AI workflow orchestration, letting you easily develop and deploy complex question-answering systems without the need for extensive setup or configuration.
https://fastgpt.in
Other
15.58k stars 4.09k forks source link

v4.6.8之后的版本修改config.json文件后不自动加载,必须重启容器才能生效 #1861

Closed i-square closed 3 weeks ago

i-square commented 3 weeks ago

例行检查

你的版本

问题描述, 日志截图 之前我使用了v4.6.8版本的本地部署,修改 config.json 文件之后过一会儿会自动重新加载,这样可以做到不停服更新配置

但是最近更新到了最新版本 v4.8.4,发现无法自动加载配置,修改之后必须restart容器才能生效

请问这是符合预期的变化吗?为什么去掉这个功能呢?

复现步骤 v4.6.8 版本和 v4.8.4-fix 版本修改 config.json 文件之后看是否生效

预期结果 修改 config.json 文件之后不停服即可生效

相关截图

c121914yu commented 3 weeks ago

没哪个版本不需要重启

i-square commented 3 weeks ago

没哪个版本不需要重启

我翻了翻提交记录,是从 4.6.9-alpha 开始去掉了刷新config的定时任务 setUpdateSystemConfigCronhttps://github.com/labring/FastGPT/pull/918/commits/f0fe8fba9c915e27db7a0bc4611677e1e5fe02c7#diff-0a87f3b3657de120dd17f8c74a681455b88097e2f0ca6cb7955d49b34b0f715cL10

这是出于什么原因?

c121914yu commented 3 weeks ago

没哪个版本不需要重启

我翻了翻提交记录,是从 4.6.9-alpha 开始去掉了刷新config的定时任务 setUpdateSystemConfigCronf0fe8fb#diff-0a87f3b3657de120dd17f8c74a681455b88097e2f0ca6cb7955d49b34b0f715cL10

这是出于什么原因?

o记起来了,因为商业版有后台了,可以实时刷新了就不需要定时刷新了。

c121914yu commented 3 weeks ago

补充一下,这个需求背景是有个 admin 管理端,在上面改了配置后需要同步到 fastgpt,一开始没想到方法,就搞了个定时器。 后面发现可以通过监听数据库方式,于是就删除了。😂所以没考虑过改配置文件等定时器的问题。