Skip to content

testcase: fixed cases for ecs.#9669

Open
ChenHanZhang wants to merge 1 commit into
aliyun:masterfrom
ChenHanZhang:fix/ecs-regression-20260420
Open

testcase: fixed cases for ecs.#9669
ChenHanZhang wants to merge 1 commit into
aliyun:masterfrom
ChenHanZhang:fix/ecs-regression-20260420

Conversation

@ChenHanZhang
Copy link
Copy Markdown
Collaborator

No description provided.

- TestAccAliCloudECSSnapshot_basic0/_twin: remove available_disk_category filter from zones, use image_id + system_disk_category=cloud_essd in instance_types, force cn-hangzhou via PreCheckWithRegions
- TestAccAliCloudECSSnapshot_basic1_twin: same as above, remove available_disk_category, add system_disk_category=cloud_essd filter
- TestAccAliCloudECSSnapshotGroup_basic0/1: remove available_disk_category, change cloud_efficiency→cloud_essd for disks/instance, add system_disk_category=cloud_essd to instance_types, force cn-hangzhou
- TestAccAliCloudECSNetworkInterfaceAttachment_basic0_twin/Multi: replace instance_type_family+availability_zone with image_id+system_disk_category=cloud_essd in instance_types, add PreCheckWithRegions for cn-hangzhou
- TestAccAliCloudECSImageBasic1/2/7009: change instance_type_family ecs.sn1ne→ecs.g6, add system_disk_category=cloud_essd filter, change cloud_efficiency→cloud_essd for disks, add PreCheckWithRegions for cn-hangzhou

Root cause: data.alicloud_instance_types returns empty when filtering by instance_type_family or availability_zone unavailable in the test region. DescribeAvailableResource API returns zero instance type stock in regions like eu-central-1, causing the data source to filter out ALL results. Adding system_disk_category=cloud_essd ensures only ESSD-compatible instance types are returned, and PreCheckWithRegions forces tests to run in cn-hangzhou where resources have stock.

Also: add provider-fix-regression skill with QUOTA_EXCEEDED sweep guidance (make sweep REGION=cn-hangzhou RESOURCE=alicloud_vpc)
Copy link
Copy Markdown
Collaborator Author

@ChenHanZhang ChenHanZhang left a comment

Choose a reason for hiding this comment

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

approved

Copy link
Copy Markdown
Collaborator Author

@ChenHanZhang ChenHanZhang left a comment

Choose a reason for hiding this comment

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

approved

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