QMAws::startTask | ECS failures - job_id: 1254229, task_definition: cupix-pix-genie-preprocessor-age
Validation: 2c538c9d-e625-4efe-beca-8333136706f7
Verdict: APPROVED#
Completeness#
| Plan Item | Status | Evidence |
|---|---|---|
| runTask 인스턴스 선택 루프에 GPU 잔여 리소스 체크 추가 | PASS | qmAws.ts: 151-154행에서 registeredResources/remainingResources의 name === 'GPU' STRINGSET을 조회하고 isGpuInstance/availableGpu를 계산 |
GPU 등록 O + 잔여 GPU 0 인스턴스는 startTask 미호출, continue |
PASS | qmAws.ts: 169-172행 if (isGpuInstance && availableGpu < 1) { logger.info(...); continue; } — startTask 호출부(179행) 이전에서 skip |
| 비GPU 인스턴스는 기존 동작 유지 (MEMORY/CPU/idle 체크만) | PASS | qmAws.ts: 153행 isGpuInstance가 false면 169행 조건이 short-circuit되어 173-184행 기존 ACTIVE/agentConnected/idle 분기로 진입 (회귀 없음) |
| startTask data.failures 로깅/심각도 경로 미변경 | PASS | 제공된 diff는 runTask 루프(145-185행)에만 국한; startTask(195-241행)는 diff에 포함되지 않음 |
| SQS 재시도 로직 미변경 | PASS | diff 범위에 SQS/visibility-timeout 관련 변경 없음 (runTask 선택 루프만 수정) |
Acceptance Criteria#
| Criterion | Status | Evidence |
|---|---|---|
선택 루프가 name === 'GPU'에 대해 remainingResources 조회 (grep >=1) |
PASS | qmAws.ts: 152행 remainingResources?.find(r => r.name === 'GPU'); grep으로 151·152·153·154행 확인 |
GPU 등록했으나 잔여 0인 인스턴스는 startTask 미호출, continue |
PASS | qmAws.ts: 169-172행 continue; 스펙 qmAws.spec.ts: 162-163행 registered ['0'] + remaining [] 케이스가 이를 커버 |
| 비GPU 인스턴스(GPU 미등록)는 기존대로 처리 | PASS | qmAws.ts: 153행 registeredGpuResource?.stringSetValue?.length ?? 0) > 0 → 미등록 시 false로 skip 미적용; 스펙에 비GPU 케이스 존재 |
remainingResources undefined 또는 GPU 리소스 없을 때 안전 (optional chaining) |
PASS | qmAws.ts: 151-154행 ?.find, ?.stringSetValue?.length, ?? 0 널 안전 처리 |
| startTask data.failures 로깅 / SQS visibility-timeout 재시도 경로 미변경 | PASS | 해당 코드가 diff에 부재; runTask 루프 외 변경 없음 |
| eslint 통과, TypeScript 컴파일 오류 없음 | UNVERIFIABLE | diff만으로 검증 불가 — 런타임 확인 필요 (worktree pnpm install가 GitHub Packages 401로 실패, 코드 무관). 규칙상 reject 사유 아님 |
Issues Found#
차단 이슈 없음.
Observations#
- 로그 라인(155-164행)에
available_gpu: %d필드와availableGpu인자를 추가했다. 계획의 명시적 항목은 아니나 GPU 체크의 관측성 향상을 위한 합리적 부수 변경으로, 기존 MEMORY/CPU 로깅 스타일과 일치. reject 사유 아님. isGpuInstance판정을remainingResources가 아니라registeredResources의 GPU STRINGSET 길이로 하는 점이 정확하다: 잔여가 0(빈 set)이어도 등록된 GPU 인스턴스를 올바르게 GPU 인스턴스로 인식해 skip한다.remainingResources로만 판정했다면 잔여 0 GPU 인스턴스를 비GPU로 오인해 skip이 누락될 위험이 있었다.- 스펙 파일(
qmAws.spec.ts) 두 케이스가 skip 경로와 비GPU 경로를 모두 커버한다. 다만 테스트 실행은 위 환경 제약으로 미확인. - startTask의 DRAINING/warn 심각도 분기(217-231행)가 코드에 존재하나 제공된 diff 범위 밖이므로 이번 변경 대상이 아님 — 계획의 "startTask 로깅 경로 미변경" 요구와 일치.