Skip to content

[VM/PublicIP] Fix: Resolve default-PublicIP assign/unassign bugs across AWS/NCP/KT/OpenStack/NHN - #1829

Merged
cb-spider merged 2 commits into
cloud-barista:masterfrom
powerkimhub:fix/vm-default-publicip-assign-unassign
Sep 4, 2026
Merged

[VM/PublicIP] Fix: Resolve default-PublicIP assign/unassign bugs across AWS/NCP/KT/OpenStack/NHN#1829
cb-spider merged 2 commits into
cloud-barista:masterfrom
powerkimhub:fix/vm-default-publicip-assign-unassign

Conversation

@powerkimhub

Copy link
Copy Markdown
Member
  • Add assign-publicip-true test suite (true→false→true lifecycle), mirroring assign-publicip-false, across all 10 CSPs
  • Fix AWS RemoveDefaultPublicIP AuthFailure caused by an unnormalized "Not set" StsToken sentinel
  • Fix NCP RemoveDefaultPublicIP 400 error: match by ServerInstanceNo instead of a bogus ServerName, and wait for the PublicIP's async state to settle before Disassociate/Delete
  • Fix KT resolveVMPrivateIP: replace the unreliable ports.List call with the existing GetVM()-based lookup
  • Fix OpenStack/NHN GetVM: clear PublicIP when the live floating-IP lookup finds nothing, instead of leaving a stale value
  • Add read-after-write wait in common-runtime's AssignVMDefaultPublicIP so a just-assigned PublicIP is reliably visible on the next GetVM
  • Fix test script to preserve already-captured PrivateIP/PublicIP/SSH fields on later-step failures instead of blanking them

@powerkimhub powerkimhub added enhancement New feature or request test labels Sep 4, 2026
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

Could you please check and revise Korean texts?
Note - All output of print and log statements should be in English. 😉

cloud-control-manager/cloud-driver/drivers/ncp/resources/PublicIPHandler.go

12:+// PublicIpInstanceOperation ("공인IP인스턴스OP") - e.g. a just-completed

@powerkimhub

Copy link
Copy Markdown
Member Author

[Test Results]

./run-all-csp-assign-publicip-true-tests.sh 전체 실행 (10개 CSP 병렬) — 10/10 PASS.

CSP         | Result | VMStatus   | PrivateIP       | PubIP(init)     | SSH    | PubIP(unassign) | PubIP(reassign) | SSH2   | Elapsed  | Reason
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
AWS         | PASS   | Running    | 192.168.1.154   | 3.25.115.56     | OK     | (none)          | 3.24.171.164    | OK     | 1m55s    | -
AZURE       | PASS   | Running    | 192.168.0.4     | 40.82.134.16    | OK     | (none)          | 52.141.56.178   | OK     | 1m36s    | -
GCP         | PASS   | Running    | 192.168.1.9     | 136.113.32.98   | OK     | (none)          | 34.136.155.127  | OK     | 2m17s    | -
ALIBABA     | PASS   | Running    | 192.168.1.127   | 123.56.91.87    | OK     | (none)          | 39.96.59.58     | OK     | 1m4s     | -
TENCENT     | PASS   | Running    | 192.168.1.9     | 101.43.140.61   | OK     | (none)          | 82.157.148.206  | OK     | 1m37s    | -
IBM         | PASS   | Running    | 192.168.1.6     | 150.239.80.67   | OK     | (none)          | 169.63.102.8    | OK     | 2m10s    | -
OPENSTACK   | PASS   | Running    | 192.168.1.39    | 183.111.177.156 | OK     | (none)          | 183.111.177.137 | OK     | 3m26s    | -
NCP         | PASS   | Running    | 192.168.1.6     | 49.50.143.235   | OK     | (none)          | 49.50.143.236   | OK     | 7m29s    | -
NHN         | PASS   | Running    | 192.168.1.101   | 103.218.159.89  | OK     | (none)          | 133.186.228.111 | OK     | 3m22s    | -
KT          | PASS   | Running    | 10.29.102.141   | 210.104.76.98   | OK     | (none)          | 210.104.76.98   | OK     | 4m53s    | -
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Total: 10  PASS: 10  FAIL: 0

@cb-spider
cb-spider merged commit 44cc63a into cloud-barista:master Sep 4, 2026
3 checks passed
@powerkimhub
powerkimhub deleted the fix/vm-default-publicip-assign-unassign branch September 4, 2026 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants