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

[Fix] docker compose 스크립트 수정 #156

Merged
merged 2 commits into from
Nov 24, 2023

Conversation

ccxz84
Copy link
Collaborator

@ccxz84 ccxz84 commented Nov 23, 2023

설명

  • 백엔드 서버에 prisma 마이그레이션 정보 볼륨 마운트 되게 수정

@ccxz84 ccxz84 requested a review from ldhbenecia as a code owner November 23, 2023 08:48
@ccxz84 ccxz84 force-pushed the fix/docker-compose branch from 3be4ec2 to ba2a726 Compare November 23, 2023 12:58
@ccxz84 ccxz84 force-pushed the fix/docker-compose branch from ba2a726 to eb805bf Compare November 23, 2023 13:00
@ttaerrim ttaerrim changed the title 🐛 docker compose 스크립트 수정 [Fix] docker compose 스크립트 수정 Nov 23, 2023
@ldhbenecia ldhbenecia added ✨ enhancement 개선 또는 피드백 반영 💻 Back-end 🐛 bug labels Nov 24, 2023
Comment on lines +18 to +19
volumes:
- prisma-migrations:/app/prisma/migrations
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

해당 부분 스크립트 조절로 인해서 마이그레이션에서 어떠한 점이 바뀐 걸까요 ??

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

프리즈마에서 마이그레이션 정보가 prisma/migrations에 저장 되잖아요? 그걸 Docker 컨테이너에서 host os의 드라이브에 마운트 했다고 생각하면 됩니다.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아하!! 이번에 일어난 오류를 해결하는 스크립트 수정이 이 부분이겠네요.
고생하셨습니다!

@ldhbenecia ldhbenecia merged commit c23562f into boostcampwm2023:develop Nov 24, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💻 Back-end 🐛 bug ✨ enhancement 개선 또는 피드백 반영
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants