Skip to content

Fix registration flow error#2270

Merged
cb-github-robot merged 2 commits intocloud-barista:mainfrom
leehyeoklee:improve-registration
Jan 12, 2026
Merged

Fix registration flow error#2270
cb-github-robot merged 2 commits intocloud-barista:mainfrom
leehyeoklee:improve-registration

Conversation

@leehyeoklee
Copy link
Copy Markdown
Contributor

🛠️ Fixes and Improvements

1. Fix Type Assertion Failed for Cache Item

🛑 Issue
A runtime error (Type assertion failed for cache item) occurred when retrieving resource status (e.g., SSHKey) from CB-Spider. The variable callResult was declared as interface{}, causing a mismatch when the generic cache system expected the concrete type model.SpiderAllListWrapper.

Log Screenshot:
cacheTypeAssertion

✅ Fix
Updated the variable declaration to use the explicit type model.SpiderAllListWrapper to ensure type safety and correct assertion.

// AS-IS
var callResult interface{}

// TO-BE
var callResult model.SpiderAllListWrapper

2. Fix Subnet Labeling Resource ID Mismatch

🛑 Issue
When applying tags (labels) to a Subnet, an error occurred (Subnet does not exist) because the internal Tumblebug Uid was being passed to CB-Spider. CB-Spider requires the CSP-native resource name (CspResourceName) to identify the target resource.

Log Screenshot:
vnetLabelingLog

✅ Fix
Updated the CreateOrUpdateLabel function call to pass subnetInfo.CspResourceName instead of subnetInfo.Uid.

// AS-IS
err = label.CreateOrUpdateLabel(model.StrSubnet, subnetInfo.Uid, subnetKey, labels)

// TO-BE
err = label.CreateOrUpdateLabel(model.StrSubnet, subnetInfo.CspResourceName, subnetKey, labels)

Copy link
Copy Markdown
Member

@seokho-son seokho-son left a comment

Choose a reason for hiding this comment

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

Thanks @leehyeoklee
Very clear!

LGTM.

@seokho-son
Copy link
Copy Markdown
Member

/approve

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

@all-contributors please add @leehyeoklee for Code

@allcontributors
Copy link
Copy Markdown
Contributor

@seokho-son

I've put up a pull request to add @leehyeoklee! 🎉

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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants