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.

Sign up for GitHub

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

Jump to bottom

考虑升级下dify charts中的minio、redis等的版本否 #76

Closed
jeessy2 opened this issue Apr 18, 2024 · 4 comments · Fixed by #77
Closed

考虑升级下dify charts中的minio、redis等的版本否 #76

jeessy2 opened this issue Apr 18, 2024 · 4 comments · Fixed by #77

Comments

@jeessy2
Copy link
Contributor

jeessy2 commented Apr 18, 2024

No description provided.

@LeoQuote
Copy link
Contributor

欢迎 pr, chart 中自带的 minio, redis pg 等都不建议使用, 只是用于实验, 默认配置是无持久卷的, 在 readme 中也是建议用外部资源

@jeessy2
Copy link
Contributor Author

jeessy2 commented Apr 18, 2024

考虑默认配置为持久不呢？用于实验也要把数据保存下来吧

@LeoQuote
Copy link
Contributor

我建议不加, 因为需要加持久卷的, 又有一些说明要写, storageclass 之类的, 包括你用本地卷什么的都是一堆事, 这样你默认配置反而就不能一键启动了.

你想做的话, 我建议只写在Readme 里, 或者说你提供一个 values-with-persistence.yaml 也可以. 我个人观点是这个配置绝对够实验, 你就不要重启或者删除 pod 就没事了, 我承认这可能会有一些风险, 比如有的人就忘了这事, 但是这也没办法...

@jeessy2 jeessy2 mentioned this issue Apr 18, 2024
Merged
@jeessy2
Copy link
Contributor Author

jeessy2 commented Apr 18, 2024

确实容易忘了，如果不仔细看文档的话。bitnami默认就是true,结果这边默认false了...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Updating dify's dependencies
2 participants
@LeoQuote @jeessy2