Merge pull request #615 from l1b0k/fix/err

fix: default option
AliyunContainerService · Apr 24, 2024 · ada08d8 · ada08d8
2 parents 52f0eaf + 6d7699b
commit ada08d8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/pkg/aliyun/client/ecs_options.go b/pkg/aliyun/client/ecs_options.go
@@ -83,7 +83,7 @@ func (c *CreateNetworkInterfaceOptions) Finish(idempotentKeyGen IdempotentKeyGen
 	argsHash := md5Hash(req)
 	req.ClientToken = idempotentKeyGen.GenerateKey(argsHash)
 
-	if c.Backoff != nil {
+	if c.Backoff == nil {
 		c.Backoff = &wait.Backoff{
 			Steps: 1,
 		}
@@ -124,7 +124,7 @@ func (c *AssignPrivateIPAddressOptions) Finish(idempotentKeyGen IdempotentKeyGen
 	argsHash := md5Hash(req)
 	req.ClientToken = idempotentKeyGen.GenerateKey(argsHash)
 
-	if c.Backoff != nil {
+	if c.Backoff == nil {
 		c.Backoff = &wait.Backoff{
 			Steps: 1,
 		}
@@ -165,7 +165,7 @@ func (c *AssignIPv6AddressesOptions) Finish(idempotentKeyGen IdempotentKeyGen) (
 	argsHash := md5Hash(req)
 	req.ClientToken = idempotentKeyGen.GenerateKey(argsHash)
 
-	if c.Backoff != nil {
+	if c.Backoff == nil {
 		c.Backoff = &wait.Backoff{
 			Steps: 1,
 		}

diff --git a/pkg/aliyun/client/ecs_options_test.go b/pkg/aliyun/client/ecs_options_test.go
@@ -55,6 +55,7 @@ func TestCreateNetworkInterfaceOptions_Finish(t *testing.T) {
 	assert.Equal(t, niOptions.ResourceGroupID, req.ResourceGroupId)
 	assert.Equal(t, eniDescription, req.Description)
 	assert.Equal(t, "mockToken", req.ClientToken)
+	assert.NotNil(t, c.Backoff)
 
 	// Cleanup
 	cleanup()