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

Added Docker Support #74

Merged
3 commits merged into from
Jan 20, 2024
Merged

Added Docker Support #74

3 commits merged into from
Jan 20, 2024

Conversation

eric2788
Copy link
Contributor

@eric2788 eric2788 commented Jan 16, 2024

此PR有幾點需要注意:

  • Lagrange.OneBot-docker-push.yml 需要 ${{ secrets.DOCKER_USERNAME }}${{ secrets.DOCKER_PASSWORD }} 方可生效 已轉用 github docker registry
  • 目前的 Dockerfile 有 30% 概率會出現 failed unexpectedly, 再跑一次通常就能成功運行,目前原因未知 應該修好了
  • appsettings.json 采用 array 的原因是因爲容易讓 docker 透過 環境參數 進行操作
  • Lagrange.OneBot/Core/Network/LagrangeWebSvcCollection.cs 新增 section["Enable"] 是爲了方便透過 docker 設置 環境參數 選擇使用哪個服務。注意:此改動是很久以前改的,新改動之後尚未知道有無兼容問題

@eric2788 eric2788 marked this pull request as ready for review January 16, 2024 13:24
@eric2788
Copy link
Contributor Author

等等,有東西要更新

@eric2788
Copy link
Contributor Author

ok 了,但我建議你先開一個 別的branch做實驗 看看能否真的 push 上去 (我自己那邊是連續兩次沒問題)

@ghost ghost merged commit 9c5b61b into LagrangeDev:master Jan 20, 2024
0 of 8 checks passed
This pull request was closed.
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.

1 participant