From 8c8f246dd6db5f49349557e539a2c7038a2f5e86 Mon Sep 17 00:00:00 2001 From: Vacant2333 Date: Mon, 2 Dec 2024 19:05:44 +0800 Subject: [PATCH] fix security group ids Signed-off-by: Vacant2333 --- pkg/providers/instance/instance.go | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/pkg/providers/instance/instance.go b/pkg/providers/instance/instance.go index 13437fd..e8e8d18 100644 --- a/pkg/providers/instance/instance.go +++ b/pkg/providers/instance/instance.go @@ -533,14 +533,10 @@ func (p *DefaultProvider) getProvisioningGroup(ctx context.Context, nodeClass *v AutoProvisioningGroupType: tea.String("instant"), LaunchConfiguration: &ecsclient.CreateAutoProvisioningGroupRequestLaunchConfiguration{ // TODO: we should set image id for each instance types after alibabacloud supports - ImageId: tea.String(imageID), - SecurityGroupIds: securityGroupIDs, - UserData: tea.String(userData), - - // TODO: AutoProvisioningGroup is not compatible with SecurityGroupIds, waiting for Aliyun developers to fix it, - // so here we only take the first one. + ImageId: tea.String(imageID), + UserData: tea.String(userData), ResourceGroupId: tea.String(nodeClass.Spec.ResourceGroupID), - SecurityGroupId: securityGroupIDs[0], + SecurityGroupIds: securityGroupIDs, SystemDiskSize: systemDisk.Size, SystemDiskPerformanceLevel: systemDisk.PerformanceLevel, Tag: reqTags,