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

APIのDocker化 #57

Merged
merged 12 commits into from
May 16, 2024
Merged

APIのDocker化 #57

merged 12 commits into from
May 16, 2024

Conversation

sushichan044
Copy link
Collaborator

@sushichan044 sushichan044 commented May 9, 2024

closes #54

@sushichan044
Copy link
Collaborator Author

sushichan044 commented May 13, 2024

開発時にDBへデータをSeedingする際にapi側に余分なコードを増やしたくなかったので、
migrateに開発時のDB Migrationに使うものを移してdocker compose upする際に実行するようにした。

convert_data_from_v1.pyは手動実行でも良いと考えているので開発環境起動時には実行していない。

@sushichan044 sushichan044 marked this pull request as ready for review May 13, 2024 05:30
@sushichan044 sushichan044 requested a review from yu23ki14 May 13, 2024 05:44
@sushichan044
Copy link
Collaborator Author

sushichan044 commented May 13, 2024

DBマイグレーションの部分は別のIssueでやったほうが良かったかもしれない

@sushichan044
Copy link
Collaborator Author

DBコンテナが立ち上がるまでマイグレーションスクリプトの実行を待つ挙動、docker composeのhealthcheckを使えば出来そうなので修正します

@osoken
Copy link
Collaborator

osoken commented May 16, 2024

よくできていて素晴らしいです。コントリビューションありがとうございました。
ENVIRONMENT=prod の場合も動くようにしました。
これで一旦マージします。

@osoken osoken merged commit 6640c6e into main May 16, 2024
3 checks passed
@osoken osoken deleted the chore/issue-54-dockerize-api branch May 16, 2024 14:21
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.

API部分のDocker化
2 participants