Fix/94 gatewayidentity test fail - #102
Merged
3 commits merged intoSep 8, 2026
Merged
Conversation
|
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: Advanced 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 |
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
routeId,downstreamKey,pathPrefix로 분리해 evaluation·schedule을 포함한 모든 라우트를 충돌 없이 등록합니다..github/workflows/ci.yml변경은 이 PR에 포함하지 않습니다.Related Issue
Root Cause
GatewayService.pathPrefix대신/api/{service}를 조립해/api/document라우트를 404로 요청했습니다.Fix Description
GatewayDownstream을 추가해 하나의 downstream URI를 여러 개의 고유한 Gateway route가 공유하도록 변경했습니다.GatewayService.entries와 각 항목의pathPrefix를 단일 진실 공급원으로 사용하도록 수정했습니다.devprofile, in-memory resilience store, random server port를 적용했습니다.GATEWAY_INTEGRATION_REQUIRED=true인 경우 Redis 연결 실패를 테스트 실패로 처리하고, 기본 로컬 실행에서는 Redis 의존 테스트만 명시적으로 skip하도록 integration gate를 추가했습니다.src/test/resources를 포함하고application-test.yml을 추가했습니다.allowPublicKeyRetrieval=true를 추가했습니다.IDENTITY_INTEGRATION_REQUIRED=true인 경우 Docker 미가용을 실패로 처리하도록 기존 Identity integration gate를 연결했습니다.Testing
bazel test //systems/gateway/gateway-adapter-in:test //systems/gateway/gateway-domain:test //systems/gateway/gateway-bootstrap:test //systems/identity/identity-bootstrap:test //systems/identity/identity-adapter-out:test --test_output=errorsRisk & Impact
Checklist