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 対応 #425

Merged
merged 12 commits into from
Feb 16, 2021
Merged

docker-compose 対応 #425

merged 12 commits into from
Feb 16, 2021

Conversation

nanasess
Copy link
Contributor

@nanasess nanasess commented Sep 25, 2020

  • docker-compose.*.yml を追加
  • docker-compose.<DB_TYPE>.yml を指定した場合で data/config/config.phpが存在しない場合は自動インストールされるよう修正
  • docker-compose.dev.yml を指定した場合はローカル環境をマウントするよう修正
  • ベースイメージとして https://github.com/ec-cube2/docker を使用

close #424
refs EC-CUBE/ec-cube#4694

使用方法

DB を別途用意する場合

php:7.4-apache のみ起動します

docker-compose up

PostgreSQL を使用する場合

docker-compose.pgsql.yml を指定します。 data/config/config.php が存在しない場合は、 EC-CUBE のインストールまで実行します。

docker-compose -f docker-compose.yml -f docker-compose.pgsql.yml up

MySQL を使用する場合

docker-compose.mysql.yml を指定します。 data/config/config.php が存在しない場合は、 EC-CUBE のインストールまで実行します。

docker-compose -f docker-compose.yml -f docker-compose.mysql.yml up

ローカル環境をマウントする場合

docker-compose.dev.yml を指定します。

## MySQL を使用する例
docker-compose -f docker-compose.yml -f docker-compose.mysql.yml -f docker-compose.dev.yml up

OWASP ZAP を使用する場合

docker-compose -f docker-compose.yml -f docker-compose.mysql.yml -f docker-compose.dev.yml -f -f docker-compose.owaspzap.yml up

TODO

  • 環境変数やハードコーディングの見直し
  • PostgreSQL/MySQL の docker-compose.yml を分割
  • EC-CUBEのインストール自動化(https://packagist.org/packages/ec-cube2/cli)
  • data/config/config.php の内容を環境変数で渡せるように修正

@okazy okazy added this to the 2.17.2 milestone Nov 10, 2020
- docker-compose.*.yml を追加
- docker-compose.<DB_TYPE>.yml を指定した場合で data/config/config.phpが存在しない場合は自動インストールされるよう修正
- docker-compose.dev.yml を指定した場合はローカル環境をマウントするよう修正
@nanasess nanasess changed the title [WIP] docker-compose 対応 docker-compose 対応 Jan 5, 2021
@nanasess nanasess requested a review from nobuhiko January 5, 2021 07:57
@nanasess
Copy link
Contributor Author

nanasess commented Jan 6, 2021

OWASP ZAP と連携できることも確認

@nobuhiko
Copy link
Contributor

@okazy お時間ある時に、テスト落ちる原因になってるやつだけでもmergeお願いします〜

@okazy
Copy link
Contributor

okazy commented Feb 16, 2021

Windowsのテストは他のプルリクで解決済みです。
travisが以下のエラーで落ちていましたので再実行しました。
The command "docker pull schickling/mailcatcher" failed and exited with 1 during .

@okazy
Copy link
Contributor

okazy commented Feb 16, 2021

travis通りましたのでマージします。対応ありがとうございました。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

docker-compose の対応
3 participants