Skip to content

Commit

Permalink
cherry-pick merged wrong variable
Browse files Browse the repository at this point in the history
  • Loading branch information
jayanthvn committed Feb 6, 2023
1 parent 2a39bdd commit bebd556
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pkg/awsutils/awsutils_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -618,7 +618,7 @@ func TestAllocIPAddressesAlreadyFull(t *testing.T) {
retErr := awserr.New("PrivateIpAddressLimitExceeded", "Too many IPs already allocated", nil)
mockEC2.EXPECT().AssignPrivateIpAddressesWithContext(gomock.Any(), input, gomock.Any()).Return(nil, retErr)
// If EC2 says that all IPs are already attached, then DS is out of sync so alloc will fail
_, err := cache.AllocIPAddresses(eniID, 14)
_, err := ins.AllocIPAddresses(eniID, 14)
assert.Error(t, err)
}

Expand Down Expand Up @@ -655,7 +655,7 @@ func TestAllocPrefixesAlreadyFull(t *testing.T) {
retErr := awserr.New("PrivateIpAddressLimitExceeded", "Too many IPs already allocated", nil)
mockEC2.EXPECT().AssignPrivateIpAddressesWithContext(gomock.Any(), input, gomock.Any()).Return(nil, retErr)
// If EC2 says that all IPs are already attached, then DS is out of sync so alloc will fail
_, err := cache.AllocIPAddresses(eniID, 1)
_, err := ins.AllocIPAddresses(eniID, 1)
assert.Error(t, err)
}

Expand Down

0 comments on commit bebd556

Please sign in to comment.