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

add docker compose deploy #108

Merged
merged 1 commit into from
Mar 22, 2023
Merged

add docker compose deploy #108

merged 1 commit into from
Mar 22, 2023

Conversation

yokaimeow
Copy link
Contributor

描述

添加docker compose部署方式。

@Leizhenpeng
Copy link
Member

可以增加一下有关 config.yaml 的描述吗

@yokaimeow
Copy link
Contributor Author

调用原来的 Dockerfile 脚本,config.yaml 用的是 /code/config.example.yaml

@Leizhenpeng
Copy link
Member

参见entrypoint.sh, docker部署时,会调用entrypoint.sh将环境变量中的同名变量重写到config.yaml当中

因此docker-compose也需要补充一些环境变量的配置引导·~

🥰

@yokaimeow
Copy link
Contributor Author

image

只是会有警告

image

配置在 /code/config.example.yaml 会被直接替换成容器内 /dist/config.yaml

@yokaimeow
Copy link
Contributor Author

同理可得 docker 部署方式也只需要修改了 /code/config.example.yaml 然后进行构建就可以运行了,不需要带环境变量

@Leizhenpeng
Copy link
Member

参考 #32

@yokaimeow
Copy link
Contributor Author

我知道entrypoint.sh会替换,但是为空是只是输出Warning。

#32 也说了在Dockerfile直接从config.example.yaml 复制文件,不需要减少了目前需要前置mv动作。所以只需要修改config.example.yaml,就可以完成配置不需要再写一遍环境变量。

@Leizhenpeng Leizhenpeng merged commit 06299ef into ConnectAI-E:master Mar 22, 2023
@Leizhenpeng
Copy link
Member

thx~

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.

2 participants