Skip to content

fix(pmk): Recommendation 팝업 개선 - #94

Merged
MZC-CSC merged 4 commits into
m-cmp:mainfrom
MZC-CSC:develop
Jun 30, 2026
Merged

fix(pmk): Recommendation 팝업 개선#94
MZC-CSC merged 4 commits into
m-cmp:mainfrom
MZC-CSC:develop

Conversation

@MZC-CSC

@MZC-CSC MZC-CSC commented Jun 30, 2026

Copy link
Copy Markdown
Member

Summary

  • Add Node 시 Spec Recommendation 팝업에서 선택된 클러스터의 provider 자동 필터링
  • Image Recommendation 팝업에 is Kubernetes Image 토글 추가 (기본 ON, 결과 없을 시 OFF 재조회 가능)

Spec Recommendation provider 필터 (pmk_serverrecommendation.js, clustercreate.js)

  • shown.bs.modal 에서 cluster_provider_dynamic 없을 경우 selectedPmkObj[0].provider fallback

Image Recommendation K8s 필터 (pmk_imagerecommendation.js, _pmk_imagerecommendation.html)

  • is Kubernetes Image 토글 추가 (기본 ON)
  • isKubernetesImage 하드코딩 제거 → 체크박스 값 반영

yh-noh and others added 4 commits June 30, 2026 10:50
- NodeGroup 없는 경우: GetAvailableK8sVersion으로 K8s 버전 동적 조회 후 version 필드 설정
- NodeGroup 없는 경우: 불필요한 pre-validation(checkK8sClusterDynamic) 제거
- NodeGroup 없는 경우: GetAvailableK8sNodeImage 호출 제거 (UNDER DEVELOPMENT API)
- NodeGroup 없는 경우: CSP별 하드코딩 specId/imageId 제거 → imageId는 "default" 사용
- NodeGroup 있는 경우: pre-validation 유지
- pmk_api.js: getAvailablek8sClusterNodeImage await 누락 수정 및 반환값 통일
- desiredNodeSize/minNodeSize/maxNodeSize/rootDiskSize를 int로 변환 (tumblebug 타입 불일치 해소)
- createNode 검증을 name/specId/imageId 필수로 완화 (autoScaling OFF 시 min/maxNodeSize 없음)
- clustercreate.js에서 createNode await 추가 및 검증 실패 시 early return 처리
- RecommendK8sNode 실패 시 CSP별 default specId fallback 적용 (NodeGroup-less 배포용)
- Add Node 시 Spec Recommendation 팝업에서 선택된 클러스터의 provider 자동 필터링
  (selectedPmkObj fallback 추가 — cluster_provider_dynamic이 없는 경우 대응)
- Image Recommendation 팝업에 is Kubernetes Image 토글 추가 (기본 ON)
  K8s 이미지가 없을 때 OFF로 전환해 전체 이미지 재조회 가능
fix(pmk): PMK Recommendation 팝업 개선
@MZC-CSC MZC-CSC changed the title fix(pmk): PMK NodeGroup 선택적 배포 및 Recommendation 팝업 개선 fix(pmk): Recommendation 팝업 개선 Jun 30, 2026
@MZC-CSC
MZC-CSC merged commit a767020 into m-cmp:main Jun 30, 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.

2 participants