chore(ci): PR 단위 Bazel 빌드/테스트 자동화 #93 - #95
Open
tlgms wants to merge 2 commits into
Open
Conversation
main/develop 대상 PR과 push에서 bazel build //... 와 bazel test //... 를 실행하는 CI 워크플로를 추가한다. - bazel-contrib/setup-bazel로 bazelisk 설치 및 캐시 구성 (.bazelversion 8.5.1을 그대로 사용해 로컬과 동일한 버전으로 실행) - concurrency로 동일 ref의 이전 실행을 취소해 러너 사용량 절감 - 기존에 실패하던 3개 타깃은 제외하고 #94로 추적 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (1)
🚫 Excluded labels (none allowed) (2)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Team Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
identity-adapter-out:test가 CI에서 실패하여 제외 목록에 추가한다. Testcontainers로 mysql:8.4를 띄우는 테스트인데, macOS의 Bazel 샌드박스에서는 Docker 소켓이 보이지 않아 assumeTrue로 조용히 skip 되어 그동안 드러나지 않았다. Docker가 있는 CI에서는 실제로 실행되면서 JDBC URL에 allowPublicKeyRetrieval이 없어 caching_sha2_password 인증에 실패한다. 원인은 #94에 기록했다. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This was referenced Sep 7, 2026
5 tasks
kusuri12-09
approved these changes
Sep 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
main/develop대상 PR과 push에서bazel build //...와bazel test //...를 실행하는 CI 워크플로를 추가합니다.commitlint.yaml)와 수동 이미지 빌드(deploy.yml,workflow_dispatch전용)만 있고, 코드가 빌드되고 테스트가 통과하는지 검증하는 워크플로가 없었습니다. 7개 서비스에 45개 테스트 타깃이 있지만 PR에서 아무도 실행하지 않는 상태였습니다.Related Issue
Scope
.github/workflows/ci.yml신규 파일 1개Change Type
Risk
검증
로컬에서
origin/develop기준으로 CI가 실행할 명령을 그대로 돌려 확인했습니다.짚어둘 점
1. 실패 중인 테스트 3개를 제외했습니다
bazel test //...전체 실행 시 아래 3개 타깃이 실패합니다. 이 PR 이전부터 실패하던 것이며, CI를 처음부터 red 상태로 두면 아무도 보지 않게 되므로 일단 제외하고 #94 로 추적합니다. 수정되면ci.yml의 제외 목록 3줄을 지우면 됩니다.gateway-bootstrap:testapplication.yaml이 기본값 없이${GATEWAY_RESILIENCE_STATE_STORE}참조 (application-dev.yaml에만 기본값 존재)identity-bootstrap:test${AUTH_PASSWORD_RESET_ENABLED}gateway-adapter-in:test앞의 2개는 "운영에서 환경변수 누락 시 fail-fast" 의도일 수 있어 기본값 추가 대신 테스트를 고치는 게 맞을 수도 있습니다. 설계 의도를 아는 분의 판단이 필요해 이 PR 범위에서는 건드리지 않았습니다.
2. 브랜치 보호 설정이 별도로 필요합니다
CI가 실제 머지 게이트로 동작하려면 저장소 Settings에서
build-test를 required status check로 등록해야 합니다. 코드로는 반영할 수 없는 부분이라 별도로 설정 부탁드립니다.3. CD는 이 PR 범위 밖입니다
배포 자동화는 #79 에서 다루고 있어
deploy.yml은 건드리지 않았습니다. 다만 현재:latest태그만 푸시하고 있어 롤백과 추적이 불가능하니, #79 진행 시 커밋 SHA 태그 추가를 함께 고려하면 좋겠습니다.Checklist
🤖 Generated with Claude Code