Skip to content

[Fix] 로컬 서비스 설정과 기능별 준비 점검 통일 - #554

Merged
Gimini-3 merged 5 commits into
developfrom
fix/553
Oct 18, 2026
Merged

Gimini-3 merged 5 commits into
developfrom
fix/553

Conversation

@Gimini-3

Copy link
Copy Markdown
Contributor

목적

명시적인 로컬 시연에서 앱과 점검 스크립트가 같은 설정을 사용하고, search 범위가 Ollama를 요구하지 않도록 수정합니다. 기존 인자 없는 명령은 유지합니다. Closes #553

테스트 환경

테스트 방법

  1. .env를 source/eval하지 않고 예제 KEY=VALUE 부분집합만 해석합니다. 환경변수가 파일보다 우선합니다.
  2. local 프로필·Local Filesystem·확인된 Unix Docker endpoint로 제한합니다. 원격/모호한 설정은 자동 수정하지 않고 거부합니다.
  3. Compose up 전 컨테이너 출처를 확인하고 --no-recreate를 사용합니다. 컨테이너·포트·DB 설정 대조와 호스트 TCP 연결, 같은 DB 계정/SSL/search_path의 읽기 전용 SELECT를 검사합니다.
  4. search에서는 Ollama를 시작하거나 호출하지 않습니다. 명시적 rag는 사전 기동된 Native Ollama를 재사용하고 없으면 Compose up 전에 실패합니다.
  5. 기존 시연 회귀와 새 계약을 python3 -m unittest discover -s scripts/demo -p 'test_*.py' -v, Bash 문법을 bash -n scripts/local-services.sh로 확인했습니다.

관측 결과

실행 결과
demo-readiness-contract-01 51/51 통과
demo-readiness-shell-syntax-01 종료 코드 0
demo-readiness-contract-02 53/53 통과
demo-readiness-contract-03 55/55 통과

실행 파일 대체물을 사용한 search 기동은 PostgreSQL·Valkey·BGE-M3만 요청했고 Ollama 주소 호출 0건이었습니다. 기존 인자 없는 status는 여전히 Ollama가 없으면 실패했습니다.

검증 과정에서 확인된 사항

  • localhost 자체는 SSH 터널과 로컬 DB를 구분하지 못합니다. Compose 출처·포트·DB 설정을 함께 대조합니다.
  • DOCKER_CONTEXT 우선순위를 반영하고 확인한 endpoint를 자식 명령에 고정합니다. 전역 Docker context는 변경하지 않습니다.
  • 컨테이너 실행과 앱 대상 포트 연결은 별개여서 TCP 점검을 추가했습니다.
  • DB 비밀번호는 CLI 인자가 아닌 환경으로 전달하고 SQL은 SELECT만 실행합니다. pg_trgm은 이후 Flyway 단계입니다.
  • Native Ollama의 서비스 관리자·앱은 통합 시연의 소유 자원으로 취급하지 않습니다. 명시적 rag는 사전 기동이 필요하며, 실제 생성 답변은 이 회차에서 미검증입니다.
  • 원본 개인 환경 파일과 기존 Docker 자원을 수정하지 않았습니다. 새 환경의 실제 start/verify/stop은 후속 통합 명령에서 검증합니다.

@Gimini-3
Gimini-3 requested a review from kangcheolung October 18, 2026 05:40
@coderabbitai

coderabbitai Bot commented Oct 18, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Repository: DocGrid/docgrid/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 5d27e2f9-9d9d-4914-ac2c-cb647bbcf13f
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@Gimini-3

Copy link
Copy Markdown
Contributor Author

변경 코드 검토

  • .env는 읽기만 하며 치환·인용·중복을 거부하고 source/eval 경로가 없는지 확인했습니다.
  • remote 프로필·저장소·DB와 Spring/JVM 설정 우회를 거부하는 분기를 확인했습니다.
  • Docker context 우선순위와 endpoint 고정, up 전 출처 대조, --no-recreate, host TCP·DB 인증·vector SELECT를 검토했습니다.
  • search의 Ollama 호출 0, 기존 status의 Ollama 필수 판정, Native 미준비 rag의 up 전 중단을 실행 파일 대체물로 확인했습니다.
  • Python 회귀 55/55, Bash 문법, backend/frontend/demo-scripts CI가 모두 통과했습니다.
  • 이 PR의 CodeRabbit 성공 표시는 자동 리뷰 생략 상태이며, 팀원의 승인이나 실제 모델 E2E가 수행됐다는 뜻이 아닙니다. 위 직접 검토와 회귀·CI 결과를 병합 근거로 사용합니다.

@Gimini-3
Gimini-3 merged commit 5a583a4 into develop Oct 18, 2026
4 checks passed
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.

[Fix] 로컬 서비스 설정과 기능별 준비 점검 통일

1 participant