Skip to content

fix(ci): 이미 착지한 시험 제거의 정당화 행은 오류 대신 경고로 (#7483) - #7485

Merged
itismyfield merged 3 commits into
mainfrom
fix/i7483-stale-justification-rows
Oct 28, 2026
Merged

itismyfield merged 3 commits into
mainfrom
fix/i7483-stale-justification-rows

Conversation

@itismyfield

Copy link
Copy Markdown
Owner

요약

시험을 지운 PR이 머지되면 그 PR의 정당화 행(scripts/ci/test-removal-justifications.txt)이 main에 남습니다. 그 뒤로 오는 모든 PR의 Rust tests build and plan이 justification row matches no loss로 실패했습니다(10-27~28 정리 PR #7457·#7472·#7482). 이 PR은 이미 착지한 제거 행을 오류 대신 경고 한 줄로 바꿉니다.

변경

  • scripts/ci/nextest_ci.py: 비일치(stale) 행 가운데, 그 시험이 base·head 계획(runnable·ignored)과 양쪽 native 소스(macOS·Windows compiled) 어디에도 없는 행만 landed_justifications로 분류합니다. 이 행은 행 전체를 WARN 한 줄로 출력하고 rc에는 반영하지 않습니다. 정당화 없는 손실과 나머지 stale 행은 지금처럼 rc 1입니다.
  • scripts/ci/rust_test_sources.py removed: present_tests(플랫폼별 base/head compiled 합집합)를 출력합니다. 이 키가 없거나 일부 플랫폼이 빠졌거나 타입이 틀리면 부재로 추측하지 않고 오류를 유지합니다(fail-closed).
  • 정당화 파일 머리말 한 줄과 docs/ci/release-gates.md 설명 한 문장을 새 규칙에 맞췄습니다.

검증

  • 수정 전 landed 시험 RED → 수정 후 GREEN. landed 조건을 되돌린 뮤턴트 RED.
  • tests.test_nextest_ci·test_nextest_required·test_known_flaky_retry 153개 실행, 실패 0, skip 0.
  • prod 3파일 순증 +16(pr_cap_prod.py), prepush_gates.sh --static-only PASS, native prepush PREPUSH_OK.
  • 독립 리뷰(WebGPT pro) CLEAN. P0·P1 0건이고, P2 1건(문서 문장)은 이 PR에 반영했습니다.

남은 확인

  • 머지 뒤 시험을 지우는 PR이 들어오면, 그다음 PR 두 개의 CI가 그 행 때문에 실패하지 않는지 봅니다.

Closes #7483

🤖 Generated with Claude Code

itismyfield and others added 3 commits October 28, 2026 05:42
Keep unmatched rows visible while allowing rows for tests absent from both
comparison trees. Require native source presence evidence before relaxing
platform rows, and keep unjustified losses and live mismatches failing.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: GPT-6.1 Sol (via Codex) <noreply@openai.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@itismyfield
itismyfield merged commit eabac85 into main Oct 28, 2026
23 checks passed
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.

[ci] 시험 제거 정당화 행이 착지 직후 모든 PR의 CI를 막는다 — 이미 지워진 시험의 행은 오류 대신 경고로

1 participant