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-compose環境の標準DBをSQLiteにし、MySQL/Postgresはオプションにする #4757

Closed
m-pyon23 opened this issue Oct 30, 2020 · 5 comments · Fixed by #4955
Closed

Comments

@m-pyon23
Copy link
Contributor

m-pyon23 commented Oct 30, 2020

概要(Overview)

docker-composeで環境を構築するとmysqlとpostgresの2つのDBが立ち上がるようになっているが、特に両方立ち上げる意味がなく、開発環境への無駄な負荷になる。

一方は記述を削除してもよいのではないか?

開発環境はsqliteでも十分なため、mysql/postgresはオプションにする(コメント欄参照)

期待する内容(Expect) or 要望 (Requirement)

  • docker-compose.ymlのservicesから、mysql/postgresのブロックを削除する

  • mysql/postgres向けはオプションとしてdocker-compose-xxxx.ymlのような形の環境を用意する

再現手順(Procedure)

環境 (environment)

  • EC-CUBE: 4.0.x
  • PHP: 7.x.x
  • DB:
    • PostgreSQL x.x.x
    • MySQL x.x.x

関連情報 (Ref)

#4391

@nanasess
Copy link
Contributor

標準は SQLite にして、両方オプションでよいかなと思います

@m-pyon23
Copy link
Contributor Author

#4391 時点では、なんとなくmysqlとpostgres両方の手順がドキュメントにあったので両方盛り込んでしまっていたのですが、使えば使うほど両方立ち上げる意味がないと感じました…

開発環境としてはsqliteでも充分ですし、mysqlやpostgresを使いたい方向けにはdocker-compose-mysql.yml、docker-compose-postgres.ymlのようなオプションを用意する形で提供するのはどうでしょうか

@m-pyon23
Copy link
Contributor Author

m-pyon23 commented Oct 30, 2020

標準は SQLite にして、両方オプションでよいかなと思います

コメント作成中にリプライいただいておりました…!

また、本体開発だと仕様の変更などはメジャーバージョンリリースにあわせるなど必要ですが、本件のように開発環境を立てる手順が少し変わる、というような内容については多少カジュアルに考えてしまってもよいでしょうか?
(もちろんその辺の最終的なジャッジは各種影響を考えた上でメンテナーの方にしていただけるとは思うのですが)

@m-pyon23 m-pyon23 changed the title docker-compose.ymlからmysqlかpostgresのどちらかを取り除くべきか docker-compose.ymlからmysqlかpostgresのどちらかを取り除きたい Oct 30, 2020
@m-pyon23 m-pyon23 changed the title docker-compose.ymlからmysqlかpostgresのどちらかを取り除きたい docker-compose環境の標準DBをSQLiteにし、MySQLかpostgresのどちらかを取り除きたい Oct 31, 2020
@m-pyon23 m-pyon23 changed the title docker-compose環境の標準DBをSQLiteにし、MySQLかpostgresのどちらかを取り除きたい docker-compose環境の標準DBをSQLiteにし、MySQL/Postgres Oct 31, 2020
@m-pyon23 m-pyon23 changed the title docker-compose環境の標準DBをSQLiteにし、MySQL/Postgres docker-compose環境の標準DBをSQLiteにし、MySQL/Postgresはオプションにする Oct 31, 2020
@kiy0taka
Copy link
Contributor

@m-pyon23 今回のような内容であれば、カジュアルに考えて頂いて大丈夫です。

docker-compose-(mysql| postgres).yml に +1 です。

@nanasess
Copy link
Contributor

nanasess commented Mar 9, 2021

#4955 で対応してみました

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 a pull request may close this issue.

3 participants