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

MSTR-376 : [AI] FastAPI로 마이그레이션 #13

Merged
merged 16 commits into from
Dec 17, 2022
Merged

MSTR-376 : [AI] FastAPI로 마이그레이션 #13

merged 16 commits into from
Dec 17, 2022

Conversation

ekzm8523
Copy link
Member

Issue Number

close: MSTR-376

작업 내역

구현 내용 및 작업 했던 내역

  • FastAPI로 마이그레이션
  • BentoML의 Runnable이 Controller로 변경되었습니다.
  • documentation을 더 자세하게 하기 위해서 example들을 추가하였습니다. (이제 API 테스팅이 가능한 Swagger와 Redoc 둘 다 지원합니다.)
  • custom exception을 위해 middleware 에 exception handler 를 추가하였습니다.
  • 모든 test가 async test로 변경
  • 부하 테스트를 위한 스크립트 locust.py 생성

변경사항

  • 의존성 목록

작업 유형

  • 신규 기능 추가
  • 버그 수정
  • 리펙토링
  • 문서 업데이트

PR 특이 사항

PR을 볼 때 주의깊게 봐야하거나 말하고 싶은 점

  • 기능이 변경된 건 없으나 Framework를 변경하였기 때문에 구조 변경과 많은 file change가 있습니다.
  • 아직 github action file은 변경하지 않아서 action은 전부 실패가 뜰 것 입니다. (Docker file과 CI 셋팅을 좀 더 구성해서 합치겠습니다.)
  • 주요 변경 점
    • app/api/dependency.py 는 FastAPI에서 의존성을 관리하는 파일로 모델과 컨트롤러를 싱글톤으로 주입하도록 구성
    • 모든 Controller는 BaseController라는 추상 클래스를 상속받도록 구성

ekzm8523 and others added 15 commits November 16, 2022 17:12
add locust test
default fastapi config setting
migration to fastAPI
add gitignore
add model cache
add gitignore
change bentoml test code to fastapi test code
add api example in documentation
add api example in documentation
add custom validation exception handler
remove dead code
rename pr template
add custom validation exception handler
add test ai server host env
Copy link
Member

@kshired kshired left a comment

Choose a reason for hiding this comment

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

27개? 화이팅! 잘했을거라고믿어요!

@ekzm8523 ekzm8523 merged commit 2223702 into dev Dec 17, 2022
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