-
Notifications
You must be signed in to change notification settings - Fork 325
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
Fix database reading issues and support new deployment methods #628
Conversation
感谢贡献
|
1.在vercel中程序创建的collection默认名称为test,我这么做是为了让私有部署和vercel数据库能够相互兼容。 |
vercel的数据库名称不是固定的test,是MONGODB_URI结尾的
如果你设置 所以你应该检查,多半是你的 vercel 环境和 huggingface 的MONGODB_URI不一样导致的
这一点,不太相信,我下周亲自测试一下看看 |
大多数用户都不会去自定义vercel的collection名称,则默认创建的数据库名称为test,而私有部署中默认创建collection为twikoo所以添加一个能够读取test collection的判断条件是必要的 |
关于huggingface space无法读取数据库的问题,后面发现一个盲点:私有部署默认创建的collection为twikoo,而vercel默认为test,在我对代码进行修改后,twikoo docker镜像并没有更新,这可能是无法读取数据库的原因所在。这一点有待测试。虽然如此,对huggingface space镜像进行体积优化是不必要的,因为huggingface space中的镜像是程序通过dockerfile即时构建的,并不需要体积优化,整个操作流程也不需要任何的人工干预,所以即时构建就好了 |
私有部署数据库名的读取逻辑和vercel是一样的,直接把vercel上的MONGODB_URI拿过来用根本不会有这样的问题,所以“大多数用户都不会……”是不成立的。
不用再纠结数据库名的问题,看wordpress,看typecho等项目,就没有哪个是把数据库名字写死在代码里的,最多提供一个默认值。 dockerfile即使不需要体积优化,也需要考虑版本控制问题。 此功能我会在下周优化、完善相关部署教程后合并。 |
这需要你来控制huggingface和vercel的环境变量一致来解决,而不是改twikoo的代码,你把默认库改成test,虽然解决了你的问题,但是会造成其他人的问题,这个改动即使合理,也会是一个breaking change |
好的 |
#627