Skip to content

ci: clear the test-removal rows consumed by #7477 - #7482

Merged
itismyfield merged 1 commit into
mainfrom
ci/clear-rows-7477
Oct 27, 2026
Merged

itismyfield merged 1 commit into
mainfrom
ci/clear-rows-7477

Conversation

@itismyfield

Copy link
Copy Markdown
Owner

무엇

#7477(시스템 프롬프트에서 [Agent Performance] 블록 제거)이 머지되면서, 시험 제거 사유 목록의 bucket_cadence_tests 행 3개가 오래된 행이 됐습니다. 남겨 두면 다음 PR들의 Rust tests build and plan이 "justification row matches no loss"로 실패합니다. #7481이 이 원인으로 실패했습니다(비교 기준 main acd2e489a5, 3행 모두 해당). 머리말 4줄만 남깁니다.

  • 바뀐 파일: scripts/ci/test-removal-justifications.txt, 3줄 삭제
  • 코드 변경 없음
  • 확인: CI 오류로 나온 3행과 파일의 데이터 3행이 같은 집합이다.

같은 정리: #7457(#7446 뒤), #7472(#7463 뒤).

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@itismyfield
itismyfield merged commit 2803ee7 into main Oct 27, 2026
23 checks passed
itismyfield added a commit that referenced this pull request Oct 28, 2026
## 요약
시험을 지운 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](https://claude.com/claude-code)

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Co-authored-by: GPT-6.1 Sol (via Codex) <noreply@openai.com>
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.

1 participant