Skip to content

Commit

Permalink
Merge pull request #163 from cloudpilot-ai/cherry-pick-157-release-0.1
Browse files Browse the repository at this point in the history
Cherry pick PR(157)/chore: use SecurityGroupIds instad of SecurityGroupId for ProvisionGroup
  • Loading branch information
jwcesign authored Dec 12, 2024
2 parents 2d30829 + 8c8f246 commit b700bf6
Showing 1 changed file with 3 additions and 7 deletions.
10 changes: 3 additions & 7 deletions pkg/providers/instance/instance.go
Original file line number Diff line number Diff line change
Expand Up @@ -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,
Expand Down

0 comments on commit b700bf6

Please sign in to comment.