Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[k8scluster] support k8sclusterDynamic like mciDynamic #1911

Merged

Conversation

sykim-etri
Copy link
Member

본 PR은 K8s를 대상으로 mciDynamic과 유사한 k8sclusterDynamic API를 제공하며, 일부 API 이름이 변경되었습니다.

o API changes

  • (rename) /tumblebug/availableK8sClusterVersion to availableK8sVersion
  • (rename) /tumblebug/availableK8sClusterNodeImage to availableK8sNodeImage
  • (rename) /checkNodeGroupsOnK8sCreation to checkK8sNodeGroupsOnK8sCreation
  • (add) /requiredK8sSubnetCount
    K8s 생성시 CSP별 필요한 서브넷 수의 차이를 파악하기 위해 assets/k8sclusterinfo.yaml에 이와 관련된 값을 기록하고 K8s 생성시 참조하고 있습니다.
  • (add) /k8sclusterDynamicCheckRequest
    /mciDynamicCheckRequest와 유사한 기능을 수행하며, 하나의 CommonSpec만을 허용합니다.
  • (add) /ns/{nsId}/k8sclusterDynamic
    /mciDynamic과 유사하게 K8sCluster 생성 요청시 shared resource를 자동으로 생성한 후 K8sCluster를 생성하며, hold 옵션을 사용하는 경우 K8sCluster 생성을 대기합니다.
  • (add) /ns/{nsId}/control/k8scluster/{k8sClusterId}
    생성 대기 중인 K8sCluster에 대해 생성을 시작할지(continue), 생성을 포기할지(withdraw)를 결정합니다.
  • (add) /k8sclusterRecommendNode
    /mciRecommendVm과 유사한 기능으로 K8sCluster를 지원할 수 있는 Spec들에 대해 조건 검색을 수행합니다.

@github-actions github-actions bot added the asset label Oct 31, 2024
@sykim-etri sykim-etri added the hold Need to hold merge label Oct 31, 2024
o API changes
  - (rename) /tumblebug/availableK8sClusterVersion to availableK8sVersion
  - (rename) /tumblebug/availableK8sClusterNodeImage to
    availableK8sNodeImage
  - (rename) /checkNodeGroupsOnK8sCreation to checkK8sNodeGroupsOnK8sCreation
  - (add) /requiredK8sSubnetCount
  - (add) /k8sclusterDynamicCheckRequest
  - (add) /ns/{nsId}/k8sclusterDynamic
  - (add) /ns/{nsId}/control/k8scluster/{k8sClusterId}
  - (add) /k8sclusterRecommendNode
@sykim-etri sykim-etri force-pushed the support-k8scluster-dynamic branch from 2c88a17 to 051f6ab Compare October 31, 2024 12:18
@sykim-etri sykim-etri removed the hold Need to hold merge label Oct 31, 2024
@seokho-son
Copy link
Member

@sykim-etri 너무 고생 많으셨습니다. 감사합니다.

코드는 전반적으로 살펴보았습니다. 마이너한 개선 포인트들이 보이긴 합니다만,

특별히 기존 기능에 영향이 없을 것 같고, 일단 이번에 변경 코드량이 많은 바,

우선 승인을 진행하고, 향후에 점차 개선되는 것으로 생각하도록 하겠습니다. :)

@seokho-son
Copy link
Member

/approve

@github-actions github-actions bot added the approved This PR is approved and will be merged soon. label Oct 31, 2024
@cb-github-robot cb-github-robot merged commit 10091e2 into cloud-barista:main Oct 31, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved This PR is approved and will be merged soon. asset
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants