Skip to content

Configure NCP subnets to use single zone for cluster creation#2137

Merged
cb-github-robot merged 2 commits intocloud-barista:mainfrom
hanizang77:branch-for-ncp
Sep 4, 2025
Merged

Configure NCP subnets to use single zone for cluster creation#2137
cb-github-robot merged 2 commits intocloud-barista:mainfrom
hanizang77:branch-for-ncp

Conversation

@hanizang77
Copy link
Copy Markdown
Contributor

@hanizang77 hanizang77 commented Sep 4, 2025

Problem

NCP K8s cluster creation requires all subnets (including LB subnets) to be within the same availability zone, unlike other CSPs that support multi-AZ deployments. Current logic was assigning subnets to different zones, causing K8s cluster creation failures.

Solution

Testing

  • Verified NCP K8s cluster creation works with single-zone subnets

Testing Details

API Used: k8sClusterDynamic
Request Body:

{
  "imageId": "default",
  "specId": "ncp+kr+ci2-g3",
  "connectionName": "ncp-kr",
  "name": "k8scluster02",
  "nodeGroupName": "k8sng02"
}

Fixes #2136

- Modify NCP subnet creation logic to use zones[0] instead of zones[1] to meet single-AZ requirement for NCP K8s clusters (ref: cloud-barista#2136)
- Add NCP-specific settings including versions, node group rules, and disk configurations
@github-actions github-actions bot added the asset label Sep 4, 2025
Copy link
Copy Markdown
Member

@yunkon-kim yunkon-kim left a comment

Choose a reason for hiding this comment

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

@hanizang77 LGGGGTM !!

@yunkon-kim
Copy link
Copy Markdown
Member

/approve

@github-actions github-actions bot added the approved This PR is approved and will be merged soon. label Sep 4, 2025
@cb-github-robot cb-github-robot merged commit 3b8f81e into cloud-barista:main Sep 4, 2025
5 checks passed
@seokho-son
Copy link
Copy Markdown
Member

/lgtm

@github-actions github-actions bot added the lgtm This PR is acceptable by at least one reviewer label Sep 4, 2025
@hanizang77 hanizang77 deleted the branch-for-ncp branch September 5, 2025 01:40
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 lgtm This PR is acceptable by at least one reviewer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[NCP] Cluster creation fails due to cross-AZ subnets in k8sClusterDynamic

4 participants