Skip to content

chore: 요청 리소스 임시 주석 처리#45

Merged
doup2001 merged 1 commit intomainfrom
chore/#40
Apr 9, 2026
Merged

chore: 요청 리소스 임시 주석 처리#45
doup2001 merged 1 commit intomainfrom
chore/#40

Conversation

@doup2001
Copy link
Copy Markdown
Member

@doup2001 doup2001 commented Apr 9, 2026

📌 작업한 내용

  • 모니터링 Pod의 requests 리소스 설정을 임시로 주석 처리.
  • 클러스터 리소스 부족으로 인한 배포 실패를 방지.

🔍 참석 사항

  • requests 설정이 너무 높아 스케줄러가 Pod를 배치하지 못하는 문제를 임시 우회.
  • 운영 안정화 후 적절한 requests 값으로 복구 계획 필요.

🖼️ 스크린샷

(해당 사항 없음)

🔗 관련 이슈

#40

✅ 체크리스트

  • 로컬에서 빌드 및 테스트 완료
  • 코드 리뷰 반영 완료
  • 문서화 필요 여부 확인

Summary by CodeRabbit

Chores

  • 모니터링 컴포넌트의 명시적 쿠버네티스 리소스 설정을 제거했습니다.
  • 해당 컴포넌트들은 이제 Helm 차트의 기본 리소스 값을 사용하도록 변경되었습니다.

@doup2001 doup2001 self-assigned this Apr 9, 2026
@doup2001 doup2001 added the chore 빌드, 배포, CI/CD 등 인프라 관련 작업 label Apr 9, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 9, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 0a7c06a4-0198-4b4b-9771-fe4e4a9852dd

📥 Commits

Reviewing files that changed from the base of the PR and between 9295870 and ed6d94d.

📒 Files selected for processing (2)
  • k8s-helm/releases/monitoring-core/values-prod-gitops.yaml
  • k8s-helm/releases/monitoring-core/values.yaml

워크스루(Walkthrough)

k8s-helm 모니터링 코어 릴리스의 두 값 파일에서 Grafana, Prometheus, Alertmanager, Prometheus Operator, Kube-State-Metrics, Prometheus Node Exporter 등 6개 컴포넌트의 명시적 Kubernetes 리소스 설정(CPU/메모리 요청 및 제한)이 제거되고 주석 처리되었습니다. 변경으로 인해 이러한 컴포넌트들은 Helm 차트의 기본 리소스 값으로 돌아갑니다.

변경 사항

응집도 / 파일 요약
Helm 모니터링 코어 값 설정
k8s-helm/releases/monitoring-core/values.yaml, k8s-helm/releases/monitoring-core/values-prod-gitops.yaml
6개 컴포넌트(Grafana, Prometheus, Alertmanager, Prometheus Operator, Kube-State-Metrics, Prometheus Node Exporter)의 명시적 리소스 설정(CPU/메모리 요청 및 제한) 블록을 제거하고 주석 처리. Helm 차트 기본값 사용으로 전환.

예상 코드 리뷰 노력

🎯 2 (Simple) | ⏱️ ~10분

관련 가능성이 있는 PR

  • chore: 모니터링 ArgoCD 및 Helm 설정 정리 #26: 동일한 모니터링 코어 Helm 값 파일들을 수정하며, 특히 Grafana/Prometheus/Alertmanager 등의 리소스 블록에 대해 이번 PR의 리소스 제거와 직접적으로 연관된 변경 사항 포함.

🐰 리소스 설정을 정리하니
차트의 기본값이 반짝거려
마법처럼 간결해진 설정
모니터링 스택이 더 가벼워졌네
혼란 없이 아름답게 ✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/#40

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.

@doup2001 doup2001 merged commit 80b7735 into main Apr 9, 2026
1 check was pending
@doup2001 doup2001 deleted the chore/#40 branch April 9, 2026 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore 빌드, 배포, CI/CD 등 인프라 관련 작업

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant