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.

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

Docker deployment #18

Merged
merged 5 commits into from
Oct 10, 2024
Merged

Docker deployment #18

merged 5 commits into from
Oct 10, 2024

Conversation

luojiyin1987
Copy link
Contributor

vite build in dockerfile, all thing in dockerfile.

Copy link

vercel bot commented Oct 10, 2024

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
Copy link
Owner

KoriIku commented Oct 10, 2024

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

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

@luojiyin1987
Copy link
Contributor Author

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

@luojiyin1987
Copy link
Contributor Author

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

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

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

@shadowofmoo
Copy link
Contributor

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

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

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

@luojiyin1987
Copy link
Contributor Author

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

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

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

@shadowofmoo

@KoriIku
Copy link
Owner

KoriIku commented Oct 10, 2024

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

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

(如果有错误还请指正)

@luojiyin1987
Copy link
Contributor Author

workflows/main.yml 已经修改了, 具体见 1ffee0b

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

@shadowofmoo

@shadowofmoo
Copy link
Contributor

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

@KoriIku
Copy link
Owner

KoriIku commented Oct 10, 2024

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

@KoriIku KoriIku merged commit 6962a33 into KoriIku:main Oct 10, 2024
2 checks passed
@luojiyin1987 luojiyin1987 deleted the docker-deployment branch October 11, 2024 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants