Repository navigation
fix: VM 생성 작업 완료 후 사용자 결과 알림 전송 - #172
Open
junjuny0227 wants to merge 3 commits into
Open
junjuny0227 wants to merge 3 commits into
junjuny0227 wants to merge 3 commits into
Conversation
성공 알림은 작업 상태 저장 이후 기록하고 실패 알림에는 공개 가능한 오류만 포함합니다. 알림 저장 실패는 작업 상태를 변경하지 않습니다. Co-Authored-By: Hermes Agent <noreply@nousresearch.com>
Co-Authored-By: Hermes Agent <hermes@nousresearch.com>
알림 삽입 오류만 SAVEPOINT에서 되돌리고 완료 상태는 유지합니다. 완료 기록 이후 예외가 작업을 실패로 뒤집지 않도록 분리합니다. Co-Authored-By: Hermes Agent <hermes@nousresearch.com>
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.
Summary
vm_creation_queue가 완료 상태를 커밋한 뒤 요청자에게 VM 이름·노드·VMID·접속 IP·SSH 사용자 정보를success알림으로 저장합니다. 비밀번호는 포함하지 않습니다.failed상태를 커밋한 뒤 사용자에게 기존에 정제된error_message를error알림으로 전달합니다. 내부 SQL/예외 원문은 알림에 노출하지 않습니다.create_vm()의 선행 성공 알림을 큐의 후속 알림으로 이동해 중복을 방지하고, 알림 저장 실패가 이미 저장된 작업 상태를 덮어쓰지 않게 했습니다. 이미 처리된 작업은 재알림하지 않습니다.Closes #161
Test plan
SECRET_KEY=unit-test-only-not-a-secret uv run --no-project --with-requirements requirements.txt --with 'ruff==0.15.22' pytest tests/test_vm_lifecycle.py tests/test_vm_policy.py -q— 65 passed (기존 의존성 경고 11개)uv run --no-project --with-requirements requirements.txt --with 'ruff==0.15.22' ruff check services/vm_creation_queue.py services/vm_service.py tests/test_vm_lifecycle.py— All checks passedgit diff --check— 통과관리자용 Discord 실패 웹훅은 선택 제안이므로 이번 PR에는 포함하지 않았습니다.