Skip to content

resource/alicloud_eflo_node: reset payment_type default to Subscription and fixes the extending error when happening the internal throlling error#9693

Merged
ChenHanZhang merged 1 commit into
masterfrom
xiaozhu
Apr 26, 2026
Merged

resource/alicloud_eflo_node: reset payment_type default to Subscription and fixes the extending error when happening the internal throlling error#9693
ChenHanZhang merged 1 commit into
masterfrom
xiaozhu

Conversation

@xiaozhu36
Copy link
Copy Markdown
Member

Comment thread alicloud/resource_alicloud_eflo_node.go Outdated
},
"data_disk": {
Type: schema.TypeList,
Type: schema.TypeSet,
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

TypeSet 那个会有两个问题,一是用户以前如果使用下标索引,改完后会失效报错;二是 TypeSet里的元素不能重复,用户如果定义多个值一样的item,会仅生效一个。需要确认这两个风险可接受

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

已修改

if v, ok := d.GetOk("zone"); ok {
nodeGroupsDataList["ZoneId"] = v
}
//if v, ok := d.GetOk("zone"); ok {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

在 payasyougo 的时候, zone 这个参数不需要了吗?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

没必要指定,节点分组中已经包含zoneId了

…on and fixes the extending error when happening the internal throlling error
@ChenHanZhang ChenHanZhang merged commit 396561a into master Apr 26, 2026
23 of 24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants