Merge pull request #673 from l1b0k/release-1.8

backport the error handle
AliyunContainerService · Aug 29, 2024 · a33d624 · a33d624
2 parents 252fbca + bc32481
commit a33d624
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/pkg/aliyun/client/ecs.go b/pkg/aliyun/client/ecs.go
@@ -77,6 +77,7 @@ func (a *OpenAPI) CreateNetworkInterface(ctx context.Context, opts ...CreateNetw
 
 	if err != nil {
 		rollBackFunc()
+		return nil, err
 	}
 
 	return FromCreateResp(resp), err
@@ -281,6 +282,7 @@ func (a *OpenAPI) AssignPrivateIPAddress(ctx context.Context, opts ...AssignPriv
 	})
 	if err != nil {
 		rollBackFunc()
+		return nil, err
 	}
 
 	ips, err := ip.ToIPAddrs(resp.AssignedPrivateIpAddressesSet.PrivateIpSet.PrivateIpAddress)
@@ -362,6 +364,7 @@ func (a *OpenAPI) AssignIpv6Addresses(ctx context.Context, opts ...AssignIPv6Add
 	})
 	if err != nil {
 		rollBackFunc()
+		return nil, err
 	}
 
 	ips, err := ip.ToIPAddrs(resp.Ipv6Sets.Ipv6Address)