feat: backend-native timeout 계약 도입 - #8
Merged
Merged
Conversation
|
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
요약
사용되지 않던 전역
TIMEOUT설정을 제거하고, 각 backend가 실제로 강제할 수 있는 timeout만 지원하도록 동작 계약을 명확히 했습니다. timeout으로 판별된 실패는 안전한 상세 응답에서reason="timeout"으로 구분됩니다.변경 배경
기존 전역
DEPLOY_PROBES["TIMEOUT"]은 정의만 되어 있고 실행에 적용되지 않았습니다. Celery result backend 검사도 timeout 인자를 받았지만 실제 backend 호출에는 전달할 수 없어 오해를 만들었습니다. DB, storage, migration, custom check는 공통 방식으로 안전하게 중단할 수 없으므로 가짜 범용 timeout을 제공하지 않고 실제 client/backend 설정을 계약으로 삼습니다.변경 내용
TIMEOUT설정을 제거하고 설정 시deploy_probes.E019오류 제공TIMEOUT을 연결 및 소켓 timeout으로 명시TIMEOUT을 broker 연결 및 worker ping timeout으로 명시reason="timeout"제공E020~E022시스템 검사 추가timeoutSeconds등 외부 호출자가 관리하도록 문서화영향 범위
DEPLOY_PROBES["TIMEOUT"]을 사용하던 설정은 제거해야 합니다.DETAIL_LEVEL="safe"에서 timeout 실패 사유가 명확해집니다.검증
리뷰 포인트