search

Byzanteam / skylark-docker

0 stars 5 forks source link

Optimize configs and README #1

Closed CptBreeza closed 7 years ago

CptBreeza commented 7 years ago

已解决的问题

  1. 删除了 docker-compose.yml 中多余的 expose
  2. 修正了 ruby-build 的仓库地址，现在 ruby-build 已经迁移到了 rbenv 组织下面
  3. 将脚本的 shell 声明修正为#!/usr/bin/env bash
  4. 删除了 puma.rb 中多余的端口声明
  5. 修改了 bundle 的源，使得bundle install可以更快执行
  6. 修改了 README

    待解决的问题

  7. secret_key_base 应该在构建/部署的过程中生成，而不是直接写入到 app.default.env 中
  8. ruby 的 ruby-china 源不可以直接使用，现在的 ruby-build 只支持mirror_url/checksum格式的镜像，所以现在还是直接从 ruby 官方源下载 ruby 源码的，速度非常慢
  9. README 中是不是忘记写 skylark/elasticsearch 镜像的构建了？
  10. 获取 Skylark 的源码，应该模仿 mina 使用 git 完成；./script/install.sh 中没有 precompile assets
fahchen commented 7 years ago
  1. commit 再细一点
  2. 把冲突解决一下
  3. 用有序列表，这样其他人好回复一些
CptBreeza commented 7 years ago

@fahchen 冲突解决了，你看一下 PR 的改动吧

CptBreeza commented 7 years ago

@fahchen 我才发现，skylark 的 nginx 和 puma 不是通过 sock 来 bind 的吗，而是直接通过 3000 端口

fahchen commented 7 years ago

不是，他们是在两台服务器里面

CptBreeza commented 7 years ago

@fahchen 一个 docker-compose 出来的 containers 应该是跑在一台机器上面的吧

fahchen commented 7 years ago

待解决的问题

  1. secret_key_base 不应该更改，所以直接存在 env 文件里面，在什么时候生成我觉得这个都可以
  2. 这个倒是没注意到
  3. 忘了写了，但是这个不应该写在 README 里面，README 里面写的是部署时方法
    1. elasticsearch 包括其他几个 image 我们需要自动去构建，不需要人工构建 image
    2. 下一步是自动构建，以及自动部署
  4. 这个应该由上面那个自动构建自动完成后就解决了？
CptBreeza commented 7 years ago

@fahchen 是的，自动构建写了之后就会舒服很多