KoriIku / luxirty-search

一个搜索引擎,基于 Google,屏蔽内容农场,无广告,无跟踪,干净,简洁,快。
https://search.luxirty.com
GNU Affero General Public License v3.0
1.31k stars 128 forks source link

Docker deployment #18

Closed luojiyin1987 closed 1 month ago

luojiyin1987 commented 1 month ago

vite build in dockerfile, all thing in dockerfile.

vercel[bot] commented 1 month ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
luxirty-search-2 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 10, 2024 9:15am
KoriIku commented 1 month ago

抱歉我不熟悉 dockerfile 和 action 相关,目前有一个 workflow 自动构建 docker image,此修改需要同时修改 workflow 吗?

希望 @shadowofmoo 帮忙看看,多谢。

luojiyin1987 commented 1 month ago

need to set VITE_GOOGLE_CSE_CX var, from this webpage get more info

luojiyin1987 commented 1 month ago

抱歉我不熟悉 dockerfile 和 action 相关,目前有一个 workflow 自动构建 docker image,此修改需要同时修改 workflow 吗?

希望 @shadowofmoo 帮忙看看,多谢。

按理不需要修改, 麻烦在 dockerhub 上的镜像名, 我进行测试。

shadowofmoo commented 1 month ago

抱歉我不熟悉 dockerfile 和 action 相关,目前有一个 workflow 自动构建 docker image,此修改需要同时修改 workflow 吗? 希望 @shadowofmoo 帮忙看看,多谢。

按理不需要修改, 麻烦在 dockerhub 上的镜像名, 我进行测试。

github action 有相关编译代码麻烦去掉一下 用的是ghcr 没有用docker hub 你在你仓库里测试一下 看看action能不能正常运行 看着应该没啥问题

luojiyin1987 commented 1 month ago

github action 有相关编译代码麻烦去掉一下 这是什么意思?

用的是ghcr 没有用docker hub 这我可以调整一下

你在你仓库里测试一下 看看action能不能正常运行 看着应该没啥问题 测试结果。会写在 pr 里。

@shadowofmoo

KoriIku commented 1 month ago

我的理解是这样:在 workflows/main.yml 里定义了一个自动构建 docker 镜像并推送到 ghcr 的 github action,你在 dockerfile 里新增的内容和这个 action 里定义的内容有一些重复的地方可能需要去掉,你可以检查一下 .github/workflows/main.yml 这个文件吗?

完成修改后,你可以在你的仓库中触发一次 action (触发方式是合并到 main 分支,当然,你也可以手动触发),它应该会构建一个 docker 镜像到你的仓库中 (记得在setting-actions-workflow permissions打开读写权限),然后可以对自动构建的镜像进行测试。

(如果有错误还请指正)

luojiyin1987 commented 1 month ago

workflows/main.yml 已经修改了, 具体见 https://github.com/KoriIku/luxirty-search/pull/18/commits/1ffee0b25f93f294939ee7fd31c4ad3b4c60ef9c

在自己的代码库上的 github action 运行日志 ,没有报错, 还需要别人的测试。

@shadowofmoo

shadowofmoo commented 1 month ago

@luojiyin1987 @KoriIku 我测试了一下没有问题,麻烦合并吧

KoriIku commented 1 month ago

感谢 @luojiyin1987 pr~ 感谢 @shadowofmoo 审阅~