Skip to content

test(actions): pin org-sweep queue-hygiene owner boundary - #1821

Merged
seonghobae merged 10 commits into
mainfrom
test/1796-org-sweep-queue-owner
Sep 4, 2026
Merged

test(actions): pin org-sweep queue-hygiene owner boundary#1821
seonghobae merged 10 commits into
mainfrom
test/1796-org-sweep-queue-owner

Conversation

@seonghobae

@seonghobae seonghobae commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

문제

조직 단위 sweep가 모든 저장소의 실행 목록을 순회하고 취소까지 담당해 API 호출량과 취소 소유권이 겹쳤습니다. 같은 PR의 이전 실행 취소는 각 워크플로의 concurrency와 current-head coalescer가 이미 담당합니다.

변경

  • 조직 sweep에서 저장소 전체 실행 목록 조회와 취소 로직을 제거했습니다.
  • 더 이상 쓰이지 않는 취소 재검증 스크립트와 전용 테스트를 삭제했습니다.
  • 조직 sweep가 큐 위생을 다시 소유하지 못하게 계약 테스트를 남겼습니다.
  • 새 품질 워크플로를 만들지 않고 기존 Agent Review Runtime Quality CI에 해당 계약 검증을 합쳤습니다.
  • scheduler의 병합 복구, 토큰 회전, 실행 예산 관리는 유지했습니다.

검증

  • uv run pytest -q tests/test_org_sweep_queue_hygiene_owner.py tests/test_required_workflow_queue_contract.py tests/test_agent_review_runtime_quality_consolidation.py — 78 passed
  • actionlint -shellcheck= .github/workflows/pr-review-merge-scheduler.yml .github/workflows/agent-review-runtime-quality-ci.yml
  • git diff --check

Chicken-and-egg 상태의 Actions 적체를 줄이는 구조 변경이므로 승인된 우회 병합 대상입니다.

Summary by CodeRabbit

  • 변경 사항

    • 조직 단위 Actions 실행 인벤토리 및 오래된 대기 실행 취소 기능을 제거했습니다.
    • PR별 동시성과 정확한 헤드 병합기가 실행 대체 작업을 담당하도록 조정했습니다.
    • 조직 스윕은 열린 작업이 있는 저장소의 스케줄러 재실행에 집중하며, 실행 주기가 일일로 변경되었습니다.
  • 검증 및 문서

    • 대기열 소유권 계약 검증을 CI에 추가했습니다.
    • 관련 운영 문서와 변경 로그를 현재 동작에 맞게 갱신했습니다.
    • 더 이상 사용하지 않는 취소 로직과 관련 테스트를 제거했습니다.

@coderabbitai

coderabbitai Bot commented Sep 4, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: dd2cca0a-a939-423c-b7d0-2224caf9f967

📥 Commits

Reviewing files that changed from the base of the PR and between 0b4a6d9 and 1cfacad.

📒 Files selected for processing (14)
  • .github/workflows/agent-review-runtime-quality-ci.yml
  • .github/workflows/pr-review-merge-scheduler.yml
  • CHANGELOG.md
  • docs/doctoring/actions-queue-saturation-hourly-sweep.md
  • docs/doctoring/queue-hygiene-live-ref-race.md
  • docs/doctoring/scheduler-stale-headrefoid-cancellation.md
  • docs/org-required-workflow-rollout.md
  • scripts/ci/revalidate_queue_cancellation.sh
  • tests/test_agent_review_runtime_quality_consolidation.py
  • tests/test_org_sweep_queue_hygiene_owner.py
  • tests/test_queue_cancellation_open_pr_revalidation.py
  • tests/test_queue_cancellation_revalidation.py
  • tests/test_queue_cancellation_scheduler_contract.py
  • tests/test_required_workflow_queue_contract.py

📝 Walkthrough

Walkthrough

조직 PR 복구 스윕에서 Actions 큐 인벤토리와 취소 로직을 제거했습니다. 네이티브 PR별 동시성과 로컬 exact-head coalescer가 supersession을 담당합니다. 큐 소유권 계약 테스트와 조건부 품질 검사를 추가했습니다.

Changes

큐 취소 소유권 정리

Layer / File(s) Summary
조직 스윕 큐 위생 제거
.github/workflows/pr-review-merge-scheduler.yml, scripts/ci/revalidate_queue_cancellation.sh, tests/test_queue_cancellation_*.py, tests/test_required_workflow_queue_contract.py
org-queue-sweep에서 Actions 실행 조회, stale 실행 취소, 취소 재검증 로직을 제거했습니다. 관련 셸 스크립트와 회귀 계약 테스트도 삭제했습니다.
큐 소유권 계약 검증
tests/test_org_sweep_queue_hygiene_owner.py
조직 스윕이 저장소 전체 실행을 조회하지 않는지 검증합니다. current-head-run coalescer가 대상 저장소와 정확한 PR 헤드를 사용하고 라이브 PR 상태를 확인하는지 검증합니다.
품질 워크플로우 연결
.github/workflows/agent-review-runtime-quality-ci.yml, tests/test_agent_review_runtime_quality_consolidation.py
큐 관련 경로를 감지하고 queue_suite를 활성화합니다. 조건에 따라 큐 소유권 계약 테스트와 compileall 검사를 실행합니다.
운영 문서 및 변경 로그 갱신
CHANGELOG.md, docs/doctoring/*.md, docs/org-required-workflow-rollout.md
조직 스윕의 큐 취소 제거와 native per-PR concurrency 및 local exact-head coalescer의 소유권을 문서에 반영했습니다.

Estimated code review effort: 3 (Moderate) | ~25 minutes

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch test/1796-org-sweep-queue-owner

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@seonghobae
seonghobae marked this pull request as ready for review September 4, 2026 06:14
@seonghobae
seonghobae merged commit 11bb6a7 into main Sep 4, 2026
4 of 21 checks passed
@seonghobae
seonghobae deleted the test/1796-org-sweep-queue-owner branch September 4, 2026 06:17
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