Skip to content

fix: pod status를 확인 하기 위한 namespace 변경#364

Merged
dongmin0204 merged 5 commits into
mainfrom
develop
Jul 13, 2026
Merged

fix: pod status를 확인 하기 위한 namespace 변경#364
dongmin0204 merged 5 commits into
mainfrom
develop

Conversation

@dongmin0204

Copy link
Copy Markdown
Collaborator

🌱 관련 이슈

  • close : #[Issue number]

🌱 작업 사항

해당 Pull Request에서 수행한 작업 목록을 제시해야 합니다.

🌱 참고 사항

기능을 만들 때 생긴 이슈에 대해서 다른사람들이 참고해야 할 사항을 적습니다.

yoon6yo and others added 5 commits July 13, 2026 05:33
config-server가 파드 생성 시 USER_PW를 주입할 수 있도록
/api/requests/config/{username} 응답에 passwd_base64 필드를 포함시킴
fix: AcceptInfoResponseDTO passwd_base64 develop 브랜치 반영 (backport)
Pod 목록/상세 조회 시 inNamespace("default") 가 실제 운영 namespace인
ailab-infra 와 불일치하여 KubernetesClient 가 500을 반환하던 버그 수정.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@NotNull 제거 → null이면 사용자 신청 시 입력한 volumeSizeGiB 유지.
Request.approve()에서 null-safe 처리 추가.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
hotfix: PodController namespace 'default' → 'ailab-infra' 수정 + passwd_base64 backport
@dongmin0204
dongmin0204 requested a review from Copilot July 13, 2026 07:24
@coderabbitai

coderabbitai Bot commented Jul 13, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 1257151b-6c49-4884-9e87-0e0d6515b869

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch develop

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.

@dongmin0204
dongmin0204 marked this pull request as ready for review July 13, 2026 07:25

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Kubernetes Pod 조회 대상 namespace를 변경하고, 관리자 승인 시 볼륨 크기(volumeSizeGiB)를 선택 입력으로 바꿔서 null이면 기존(신청 시) 값을 유지하도록 승인 로직을 조정합니다.

Changes:

  • Pod 조회 API가 default 대신 ailab-infra namespace를 조회하도록 변경
  • 관리자 승인 DTO에서 volumeSizeGiB를 선택값으로 전환 (null 허용)
  • 승인 도메인 로직에서 volumeSizeGiB가 null이면 기존 값을 유지하도록 변경

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
src/main/java/DGU_AI_LAB/admin_be/domain/requests/entity/Request.java 승인 시 volumeSizeGiB가 null이면 기존 값을 유지하도록 approve 로직 변경
src/main/java/DGU_AI_LAB/admin_be/domain/requests/dto/request/ApproveRequestDTO.java 승인 요청에서 volumeSizeGiB를 null 허용으로 변경하고 스키마 설명 업데이트
src/main/java/DGU_AI_LAB/admin_be/domain/pod/controller/PodController.java Pod 목록/상세 조회 시 namespace를 ailab-infra로 변경

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 28 to 30
return client.pods()
.inNamespace("default")
.inNamespace("ailab-infra")
.list()
Comment on lines +22 to 24
@Schema(description = "할당할 볼륨 크기 (GiB, null이면 신청 시 값 유지)", example = "20")
@Positive(message = "볼륨 크기는 양수여야 합니다.")
Long volumeSizeGiB,
Comment on lines 145 to +150
public void approve(ContainerImage image, ResourceGroup resourceGroup, Long volumeSizeGiB, String adminComment) {
this.containerImage = image;
this.resourceGroup = resourceGroup;
this.volumeSizeGiB = volumeSizeGiB;
if (volumeSizeGiB != null) {
this.volumeSizeGiB = volumeSizeGiB;
}
@dongmin0204
dongmin0204 merged commit 660a74c into main Jul 13, 2026
2 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.

3 participants