Skip to content

[fix] #205 PodServiceStub 제거하여 실제 Pod 생성 API 호출 활성화#206

Open
uugaemi wants to merge 1 commit intodevelopfrom
fix/remove-pod-service-stub
Open

[fix] #205 PodServiceStub 제거하여 실제 Pod 생성 API 호출 활성화#206
uugaemi wants to merge 1 commit intodevelopfrom
fix/remove-pod-service-stub

Conversation

@uugaemi
Copy link
Copy Markdown
Contributor

@uugaemi uugaemi commented Apr 3, 2026

Summary

  • 인프라 연동 완료에 따라 임시 mock인 PodServiceStub.java 삭제
  • 모든 프로파일에서 실제 PodService.createPod()/create-pod API가 호출되도록 변경

관련 이슈

Test plan

  • prod 환경에서 request 승인 시 실제 Pod 생성 API 호출 확인
  • Pod 생성 후 podName, node, port 정보가 정상 저장되는지 확인

🤖 Generated with Claude Code

Summary by CodeRabbit

변경 사항

  • 개선 사항
    • 개발 환경에서 사용되던 Pod 생성 모의 기능이 제거되었습니다. 이에 따라 로컬 프로필에서 제공되던 기본 응답 및 포트 매핑 시뮬레이션은 더 이상 지원되지 않습니다.

인프라 연동 완료에 따라 임시로 사용하던 PodServiceStub을 삭제하여
request 승인 시 실제 create-pod API가 호출되도록 변경

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 3, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 61f2f65d-5eff-4027-92d5-a5b700597e82

📥 Commits

Reviewing files that changed from the base of the PR and between 393bbae and 5cb4e4c.

📒 Files selected for processing (1)
  • src/main/java/DGU_AI_LAB/admin_be/domain/requests/service/PodServiceStub.java
💤 Files with no reviewable changes (1)
  • src/main/java/DGU_AI_LAB/admin_be/domain/requests/service/PodServiceStub.java

📝 Walkthrough

Walkthrough

PodServiceStub 클래스가 삭제되었습니다. 이 클래스는 localstub 프로파일에서 Pod 생성을 모의 처리하던 Spring 서비스로, 삭제됨에 따라 실제 PodService가 모든 프로파일에서 자동 활성화됩니다.

Changes

Cohort / File(s) Summary
Pod Service Stub 제거
src/main/java/DGU_AI_LAB/admin_be/domain/requests/service/PodServiceStub.java
고정된 mock 응답(CreatePodResponseDTO 반환)을 제공하던 stub 클래스 완전 삭제. @Primary 어노테이션도 함께 제거되어 실제 Pod 생성 API 호출이 활성화됨.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

🐰 Mock의 세계를 떠나며
진짜 API가 깨어난다네
Stub은 이제 안녕, 바이바이
실제 pod이 자유롭게 춤춘다
진실의 코드만 남겨두고 ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 주요 변경사항(PodServiceStub 제거)을 명확하게 설명하고 있으며, 해당 변경의 영향(실제 Pod 생성 API 호출 활성화)까지 포함하고 있습니다.
Description check ✅ Passed PR 설명이 제공된 템플릿 구조를 따르고 있으며, 관련 이슈, 작업 사항, 테스트 계획을 모두 포함하고 있습니다.
Linked Issues check ✅ Passed 코드 변경이 연결된 이슈 #205의 요구사항(PodServiceStub.java 삭제, 실제 PodService 활성화)을 완벽하게 충족합니다.
Out of Scope Changes check ✅ Passed 변경사항은 PodServiceStub.java 파일 삭제로 제한되어 있으며, 모두 이슈 #205의 명시된 목표와 직접 관련이 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ 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 fix/remove-pod-service-stub

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 and usage tips.

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] PodServiceStub 제거 및 실제 Pod 생성 API 연동

1 participant