Skip to content

ci(metadata): 시간당 중복 전체 테스트 실행 제거 - #1971

Draft
seonghobae wants to merge 3 commits into
mainfrom
codex/metadata-focused-regressions
Draft

ci(metadata): 시간당 중복 전체 테스트 실행 제거#1971
seonghobae wants to merge 3 commits into
mainfrom
codex/metadata-focused-regressions

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

변경

시간당 Repository Metadata Reconcile 검증 단계에서 무관한 전체 pytest 호출 하나를 제거합니다. 기존 담당 테스트 11개 파일은 모두 두 coverage 실행에 명시적으로 포함합니다. 새 job·step·의존성·권한은 없으며 privileged apply job과 전용 credential 경계는 그대로 유지합니다.

검증

정확한 head: 8a3c2bb

  • RED: 기존 전체-suite 호출에서 계약 테스트 1 failed / 1 passed.
  • 수정된 실제 workflow 테스트 단계: 43 + 17 = 60 passed.
  • metadata 284 statements/142 branches, labels 167 statements/74 branches: 각각 coverage 100%; docstring 100%.
  • 명시적 대상 없는 전체-suite 호출에 옵션을 덧붙이는 회귀도 검출합니다. 기존 셸 추출 helper를 재사용했습니다.
  • main 별도 Actions 환경 계약 검사: 3 passed (-W error).
  • actionlint / diff-check exit 0, 검증 전후 head 동일·tracked clean·stderr 0 bytes.

효과와 한계

기존 실제 run34015563450/job101438583906에서는 이미 담당 테스트를 실행한 뒤 전체 2920개 테스트가 88.32초 더 실행됐습니다. 이번 수정은 그 중복 호출을 없애지만 해당 측정 한 건을 조직 전체 절감률로 환산하지 않습니다. 전체 저장소 CI·보안 gate를 제거하지 않으며 실제 배포 후 실행시간과 queue 감소는 후속 검증 대상입니다. 기존 concurrency 표준화는 별도 소유 PR #1899와 조정합니다. 최신 Visual Inspection과 hosted checks는 아직 완료되지 않았습니다.

Signed-off-by: Seongho Bae <me@seonghobae.me>
Signed-off-by: Seongho Bae <me@seonghobae.me>
@coderabbitai

coderabbitai Bot commented Sep 6, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

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

Copy link
Copy Markdown
Contributor Author

추가 head bf5dfef: 기존 대기열 진단 문서의 인과 단정을 정정했습니다. 과거3repo 관측·미보존스크린샷 한계는 유지하며, workflow run과job수를 구분하고 사후 primary quota 잔여량만으로 실패 당시 rate limit을 배제하지 않도록 고쳤습니다. 용량 확대와 중복실행·점유시간 감소는 함께 검토할 수 있습니다. GitHub 공식 jobs/rate-limit 문서를 확인해 출처를 추가했습니다. 현재 커밋의 인접 metadata 계약3tests -Werror 통과 및 diff-check 이상 없음이며, 이는 문서의 모든 사실을 증명하는 테스트가 아닙니다. 기존 workflow 구현은8a3과동일하고, 그 커밋의60tests/coverage100증거는 이전수정의 근거로만 유지합니다. 새 head의 hosted/VI/보호merge는 별도미완입니다.

@seonghobae seonghobae added area: ci-cd CI, GitHub Actions, checks, release, or supply chain priority: medium Normal-priority or P2 work status: draft Draft pull request type: maintenance Maintenance, build, dependency, or operational upkeep maintenance labels Sep 6, 2026 — with ChatGPT Codex Connector
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ci-cd CI, GitHub Actions, checks, release, or supply chain maintenance priority: medium Normal-priority or P2 work status: draft Draft pull request type: maintenance Maintenance, build, dependency, or operational upkeep

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant