Skip to content

fix(import): add missing NCP resource importers#580

Open
1eedaegon wants to merge 1 commit into
NaverCloudPlatform:mainfrom
1eedaegon:fix/import-support-gaps
Open

fix(import): add missing NCP resource importers#580
1eedaegon wants to merge 1 commit into
NaverCloudPlatform:mainfrom
1eedaegon:fix/import-support-gaps

Conversation

@1eedaegon

Copy link
Copy Markdown

Summary

  • add import support for ncloud_network_acl_rule
  • add custom import support and stable IDs for ncloud_lb_target_group_attachment
  • document import formats and add import acceptance-test steps
  • add unit coverage for target group attachment import ID parsing

Test Plan

  • go test ./internal/service/vpc ./internal/service/loadbalancer
  • go vet ./...
  • go test ./...

Notes

  • ncloud_postgresql already implements Framework import, but some create-time fields are not recoverable from the instance detail response after import. That post-import drift issue is separate from the missing importer implementation fixed here.

@1eedaegon 1eedaegon force-pushed the fix/import-support-gaps branch from c349267 to af3980f Compare June 9, 2026 05:39
@1eedaegon 1eedaegon force-pushed the fix/import-support-gaps branch from af3980f to b172fe8 Compare June 9, 2026 05:56
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.

1 participant