kingmo888 / rustdesk-api-server

基于Django的RustDesk Api&Web Server,除了支持api所有功能,还支持web注册、管理、展示等。已支持到最新1.2.7版本。
774 stars 177 forks source link

优化docker构建 #9

Closed Cp0204 closed 10 months ago

Cp0204 commented 10 months ago
  1. 优化 Dockerfile,删除与合并不必要的RUN步骤
  2. ./db.sqlite3./db./db.sqlite3,方便docker目录映射

综上,降低镜像50%体积,./db为docker映射数据目录,首次运行时如不存在./db/db.sqlite3就从./db_bak/db.sqlite3复制。更适合以docker容器运行。

鉴于大部分人都是在服务器上部署的,用docker最方便了,建议作者使用Github Actions发布docker镜像到仓库

kingmo888 commented 10 months ago

不行,还是要撤回合并。你这样直接把python启动的给ban了(比如我)。

Cp0204 commented 10 months ago

不行,还是要撤回合并。你这样直接把python启动的给ban了(比如我)。

主要是删掉这句是不? https://github.com/kingmo888/rustdesk-api-server/pull/9/commits/73708128e0db0417b9caa807b30ec0d8e341420f#diff-d31ce0453051853c17ba2a5225b3d1bfab548e095bab0967d6acfd1b3ce1b35dL3

因为DockerfileWORKDIR /rustdesk-api-server,工作目录已经在程序目录了,就没必要重复执行。加上也不影响的。

kingmo888 commented 10 months ago

不行,还是要撤回合并。你这样直接把python启动的给ban了(比如我)。

主要是删掉这句是不? 7370812#diff-d31ce0453051853c17ba2a5225b3d1bfab548e095bab0967d6acfd1b3ce1b35dL3

因为DockerfileWORKDIR /rustdesk-api-server,工作目录已经在程序目录了,就没必要重复执行。加上也不影响的。

对啊,原则上来说得保持目录结构不变。因为可能存在一些未纳入的开发内容依赖着结构和路径。 当然,按照你的描述,整体体积降低不小,还是很优秀。

我对docker不了解,也不是专业ITer,还望谅解。

Cp0204 commented 10 months ago

不行,还是要撤回合并。你这样直接把python启动的给ban了(比如我)。

主要是删掉这句是不? 7370812#diff-d31ce0453051853c17ba2a5225b3d1bfab548e095bab0967d6acfd1b3ce1b35dL3 因为DockerfileWORKDIR /rustdesk-api-server,工作目录已经在程序目录了,就没必要重复执行。加上也不影响的。

对啊,原则上来说得保持目录结构不变。因为可能存在一些未纳入的开发内容依赖着结构和路径。 当然,按照你的描述,整体体积降低不小,还是很优秀。

我对docker不了解,也不是专业ITer,还望谅解。

我也不是专业ITer哈~ 那你很牛了,写出这个项目! 我跟你相反,我python只会些皮毛,docker用得多 真心推荐入坑,会发现新大陆。