Skip to content

[WIP] feat: add nvme_support to image import features block#9654

Open
imlk0 wants to merge 1 commit into
aliyun:masterfrom
inclavare-containers:feat/image-nvme-support
Open

[WIP] feat: add nvme_support to image import features block#9654
imlk0 wants to merge 1 commit into
aliyun:masterfrom
inclavare-containers:feat/image-nvme-support

Conversation

@imlk0
Copy link
Copy Markdown

@imlk0 imlk0 commented Apr 16, 2026

Summary

  • Migrate nvme_support parameter from disk_device_mapping block to the new features block in alicloud_image_import resource, aligned with the ImportImageFeatures struct in alibaba-cloud-sdk-go
  • Add input validation for nvme_support values (supported / unsupported)
  • Update documentation and add test cases for the features block
  • Bump alibaba-cloud-sdk-go to v1.63.28 (required for ImportImageFeatures API support)

Changes

  • alicloud/resource_alicloud_image_import.go — add features block with nvme_support field
  • alicloud/resource_alicloud_image_import_test.go — add acceptance tests
  • website/docs/r/image_import.html.markdown — update documentation
  • go.mod / go.sum — bump alibaba-cloud-sdk-go to v1.63.28

- Move nvme_support parameter from disk_device_mapping to features block
- Align with Alibaba Cloud SDK ImportImageFeatures struct
- Add proper validation for supported/unsupported values
- Update documentation and examples
- Add comprehensive test cases for features functionality
- Maintain backward compatibility with existing disk mappings
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 16, 2026

CLA assistant check
All committers have signed the CLA.

@imlk0 imlk0 force-pushed the feat/image-nvme-support branch from 3aacad6 to 8b38b22 Compare April 16, 2026 06:39
@imlk0 imlk0 changed the title feat: add nvme_support to image import features block [WIP] feat: add nvme_support to image import features block Apr 16, 2026
@imlk0 imlk0 force-pushed the feat/image-nvme-support branch from 8b38b22 to 51aa563 Compare April 17, 2026 07:09
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.

2 participants