Skip to content

fix(mci): update API field names and pathParam for infra#74

Merged
MZC-CSC merged 5 commits into
m-cmp:mainfrom
MZC-CSC:develop
Jun 8, 2026
Merged

fix(mci): update API field names and pathParam for infra#74
MZC-CSC merged 5 commits into
m-cmp:mainfrom
MZC-CSC:develop

Conversation

@dogfootman

@dogfootman dogfootman commented Jun 8, 2026

Copy link
Copy Markdown
Member

Summary

  • fix(mci): rename pathParam mciIdinfraId in API calls
  • fix(mci): update vmnode field names and nodeGroups for tumblebug API change
  • fix: MCI 동적 생성 시 subGroupsnodeGroups 필드명 수정

dogfootman and others added 5 commits June 6, 2026 07:12
mciDynamic, mciDynamicReview 함수에서 tumblebug /infraDynamic API에
전달하는 요청 본문의 필드명을 subGroups에서 nodeGroups로 수정한다.

tumblebug InfraReq 구조체는 nodeGroups 필드를 required로 선언하므로
subGroups로 보내면 nodeGroups: null로 인식되어 VM 프로비저닝이 수행되지
않고 MCI가 Prepared 상태에 머무르는 문제가 있었다.
… change

- Replace all mci.vm references with mci.node in mci.js (tumblebug renamed field)
- Add null guard in displayServerStatusList to prevent crash when node list is empty
- Fix mciDynamic/mciDynamicReview: subGroups→nodeGroups (tumblebug required field)
- Fix getMciInfoProviderNames: mciData.vm→mciData.node
- Fix providerFormatter/providerFormatterString: data.getData().vm→node
- Fix providerFilter: data.vm→data.node
tumblebug renamed path parameter {mciId} to {infraId} in all infra endpoints.
Update all pathParams keys in mci_api.js accordingly so the proxy can substitute
the path variable correctly (GetInfra, mciLifeCycle, mciDelete, vmDelete, vmLifeCycle, etc.)
fix: MCI 동적 생성 시 subGroups → nodeGroups 필드명 수정
fix(mci): update vm→node field names for tumblebug API change
@MZC-CSC MZC-CSC changed the title fix(mci): update API field names and pathParam for tumblebug changes fix(mci): update API field names and pathParam for infra Jun 8, 2026
@MZC-CSC MZC-CSC merged commit 25464c7 into m-cmp:main Jun 8, 2026
3 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