Merge pull request #1326 from ioito/hotfix/qx-zstack-secgroup

fix delete rules failure
This commit is contained in:
yunion-ci-robot
2019-06-20 15:39:48 +08:00
committed by GitHub
+9 -2
View File
@@ -247,8 +247,15 @@ func (region *SRegion) AddSecurityGroupRule(secgroupId string, rules []secrules.
}
func (region *SRegion) DeleteSecurityGroupRules(ruleIds []string) error {
resource := fmt.Sprintf("security-groups/rules?ruleUuids=%s", strings.Join(ruleIds, ","))
return region.client.delete(resource, "", "")
if len(ruleIds) > 0 {
ids := []string{}
for _, ruleId := range ruleIds {
ids = append(ids, fmt.Sprintf("ruleUuids=%s", ruleId))
}
resource := fmt.Sprintf("security-groups/rules?%s", strings.Join(ids, "&"))
return region.client.delete(resource, "", "")
}
return nil
}
func (region *SRegion) CreateSecurityGroup(name, desc string) (*SSecurityGroup, error) {