nekomeowww / insights-bot

A bot works with OpenAI GPT models to provide insights for your info flows.
MIT License
254 stars 25 forks source link

fix(heatlhcheck): add curl to Dockerfile, supports to set password for Redis from envs #159

Closed PA733 closed 1 year ago

PA733 commented 1 year ago

This commit fixes the error of /bin/sh: 1: curl: not found when running healthcheck for insights_bot service in docker-compose. 需要在insights-bot容器内装上curl,否则 这里的 healthcheck会报/bin/sh: 1: curl: not found

PA733 commented 1 year ago

还发现了一些小问题,我可以多交几个pr吗? 或者您喜欢的话,我可以合并成一个pr交(但那样可读性就比较差)。

nekomeowww commented 1 year ago

可以合并的,如果都是改的同一个资源的话,辛苦了

PA733 commented 1 year ago

好的,我接下来合并成一个pr交吧。 感谢您做了这么好的项目!💕

nekomeowww commented 1 year ago

@PA733 谢谢你认可,能帮到大家就好!

很抱歉我在官方实例上没有使用 docker-compose 而是使用的 k8s 去部署,所以一直以来没有发现这么多问题,谢谢你愿意花时间优化。

PA733 commented 1 year ago

已准备好合并。 目前我做了以下修复或改进:

请您忽略我pr的分支名(pr提交后就改不了了),并建议您只看Files changed板块(因为commit message太长)。 目前我遵守的是angular commit规范,如果您不喜欢,我可以修改后再推送。 另有一处拼写错误,由于改动范围不同,将在新的pr中修复。