Skip to content

chore: Docker Image 설정 및 docker-compose 설정 #81

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nonegom
Copy link
Contributor

@nonegom nonegom commented Apr 29, 2025

#️⃣ Issue Number

📝 요약(Summary)

  • pip install 이외에 docker-compose를 통해서 streamlit을 사용할 수 있게 Dockerfile과 docker-compose 파일을 설정했습니다.
  • Streamlit뿐만 아니라 이전 회의에 나온 내부 DB로 Postgres를 추가적으로 이미지 설정해서 사용할 수 있게 연결했습니다.
  • Postgres를 기반으로 추후 벡터 DB를 교체할 수 있을 것으로 생각됩니다

💬 To Reviewers (선택)

  • Docker 및 Docker Compose가 설치된 환경에서 테스트를 해보실 수 있습니다 (추후 가이드 문서 작성)
  • docker-compsose up으로 API를 올려보실 수 있습니다.
  • 참고) docekr-compose down으로 이미지를 내릴 수 있습니다

PR Checklist

  • Vector DB 선정 및 Docker 이미지 생성
  • Streamlit Web 서비스용 Docker 이미지 생성
  • Docker Compose 파일 작성 및 통합 구성

reference) How to Code Review

  • 따봉(👍): 리뷰어가 리뷰이의 코드에서 칭찬의 의견을 남기고 싶을 때 사용합니다.
  • 느낌표(❗): 리뷰어가 리뷰이에게 필수적으로 코드 수정을 요청할 때 사용합니다.
  • 물음표 (❓): 리뷰어가 리뷰이에게 의견을 물어보고 싶을 때 사용합니다.
  • 알약 (💊): 리뷰어가 리뷰이의 코드에서 개선된 방법을 제안하지만 그것의 반영이 필수까지는 아닐 때 사용합니다.

@nonegom nonegom added the enhancement New feature or request label Apr 29, 2025
@nonegom nonegom self-assigned this Apr 29, 2025
Copy link
Contributor

@ParkGyeongTae ParkGyeongTae left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Collaborator

@ehddnr301 ehddnr301 left a comment

Choose a reason for hiding this comment

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

👍 docker화가 되면서 기존에 파이썬 프로세스로 뜨던 프로젝트가 container로도 진행가능해지겠네요! cli와의 연동도 고려해보면 좋을것 같습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants