Skip to content

워크 플로 분리 Spring / FastAPI#125

Merged
rll2641 merged 6 commits intodevelopfrom
refactor/separate-environment
Sep 19, 2025
Merged

워크 플로 분리 Spring / FastAPI#125
rll2641 merged 6 commits intodevelopfrom
refactor/separate-environment

Conversation

@rll2641
Copy link
Collaborator

@rll2641 rll2641 commented Sep 19, 2025

📝 작업 내용

  1. Fast API EC2에 Docker 설치
  2. production-fastapi 폴더 생성 -> docker-compose.yml 작성 -> promtail-config.yml 작성
  3. production 폴더 내 docker-compose에서 파이썬 관련 삭제
  4. ci-python.yml에서 test 관련 주석 처리 (현재는 분할이 목적)
  5. deploy-fastapi.yml에 환경 변수 추가 및 Github Secret에 변수 추가
    1. OPENAI_API_KEY, FASTAPI_SERVER_HOST(public IPv4)
  6. deploy-fastapi.yml에 onnx 모델을 FastAPI Server에 복사하는 step 생성
  7. deploy-fastapi.yml에 onnx모델이 기존 FastAPI Server에 존재하면 복사 X (모델 이름으로 조건)
  8. 기본적으로 옮겨야할 step 생성
  9. deploy-java에서 deploy-fastapi분리

🔗 관련 이슈

  • Closes #이슈번호
  • Related to #이슈번호

💬 추가 요청사항


✅ 체크리스트

코드 품질

  • 커밋 컨벤션 준수 (feat/fix/docs/refactor 등)
  • 불필요한 코드/주석 제거

테스트

  • 로컬 환경에서 동작 확인 완료
  • 기존 기능에 영향 없음 확인

배포 준비

  • 환경변수 추가/변경사항 문서화
  • DB 마이그레이션 필요 여부 확인
  • 배포 시 주의사항 없음

@rll2641 rll2641 changed the base branch from main to develop September 19, 2025 03:47
@@ -0,0 +1,37 @@
version: "3.9"
Copy link
Collaborator

Choose a reason for hiding this comment

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

이렇게 했을 때 Spring에서 http 요청 시 8000으로 쏘는건가요?

Comment on lines 26 to 32
echo "DB_USER=${{ secrets.DB_USER }}" >> .env.prod
echo "DB_PASS=${{ secrets.DB_PASS }}" >> .env.prod
echo "DB_NAME=${{ secrets.DB_NAME }}" >> .env.prod
echo "ENV_NAME=${{ secrets.LOKI_URL }}" >> .env.prod
echo "ENV_NAME=${{ secrets.LOKI_USERNAME }}" >> .env.prod
echo "ENV_NAME=${{ secrets.LOKI_PASSWORD }}" >> .env.prod
echo "LOKI_HOST=${{ secrets.LOKI_HOST }}" >> .env.prod
echo "LOKI_USERNAME=${{ secrets.LOKI_USERNAME }}" >> .env.prod
echo "LOKI_PASSWORD=${{ secrets.LOKI_PASSWORD }}" >> .env.prod
echo "ENV_NAME=${{ secrets.ENV_NAME }}" >> .env.prod
Copy link
Collaborator

Choose a reason for hiding this comment

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

java 쪽 fast api url을 넣어줘야 spring에서 호출 할 수 있을 것 같습니다

Copy link
Collaborator

@can019 can019 left a comment

Choose a reason for hiding this comment

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

@can019 can019 marked this pull request as ready for review September 19, 2025 03:57
@rll2641 rll2641 merged commit ee534c8 into develop Sep 19, 2025
4 checks passed
@rll2641 rll2641 deleted the refactor/separate-environment branch September 19, 2025 05:02
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.

2 participants