Skip to content

Commit cfff89c

Browse files
authored
improve the enable primary ipv6 address and TEST_ID in prow script (#3524)
* improve the enable primary ipv6 address and TEST_ID in prow script * bump up IngressDNSAvailableWaitTimeout to tolerate the timeout in prow * bump up go version to 1.21.5
1 parent fd22cde commit cfff89c

File tree

4 files changed

+9
-6
lines changed

4 files changed

+9
-6
lines changed

.github/workflows/deps.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,5 +22,5 @@ jobs:
2222
- id: govulncheck
2323
uses: golang/govulncheck-action@v1
2424
with:
25-
go-version-input: 1.21.4
25+
go-version-input: 1.21.5
2626
go-version-file: go.mod

Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ MAKEFILE_PATH = $(dir $(realpath -s $(firstword $(MAKEFILE_LIST))))
44
# Image URL to use all building/pushing image targets
55
IMG ?= public.ecr.aws/eks/aws-load-balancer-controller:v2.6.1
66
# Image URL to use for builder stage in Docker build
7-
BUILD_IMAGE ?= public.ecr.aws/docker/library/golang:1.21.4
7+
BUILD_IMAGE ?= public.ecr.aws/docker/library/golang:1.21.5
88
# Image URL to use for base layer in Docker build
99
BASE_IMAGE ?= public.ecr.aws/eks-distro-build-tooling/eks-distro-minimal-base-nonroot:2023-09-06-1694026927.2
1010
IMG_PLATFORM ?= linux/amd64,linux/arm64

scripts/run-e2e-test.sh

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ function toggle_windows_scheduling(){
2828
done
2929
}
3030

31-
TEST_ID=$(date +%s)
31+
TEST_ID=$(date +%s)-$((RANDOM % 1000))
3232
echo "TEST_ID: $TEST_ID"
3333
ROLE_NAME="aws-load-balancer-controller-$TEST_ID"
3434
POLICY_NAME="AWSLoadBalancerControllerIAMPolicy-$TEST_ID"
@@ -159,9 +159,12 @@ function install_controller_for_adc_regions() {
159159

160160
function enable_primary_ipv6_address() {
161161
echo "enable primary ipv6 address for the ec2 instance"
162-
ENI_IDS=$(aws ec2 describe-instances --filters "Name=tag:aws:eks:cluster-name,Values=$CLUSTER_NAME" --query "Reservations[].Instances[].NetworkInterfaces[].NetworkInterfaceId" --output text)
162+
ENI_IDS=$(aws ec2 describe-instances --region $REGION --filters "Name=tag:aws:eks:cluster-name,Values=$CLUSTER_NAME" --query "Reservations[].Instances[].NetworkInterfaces[].NetworkInterfaceId" --output text)
163+
ENI_COUNT=$(echo "$ENI_IDS" | wc -w)
164+
echo "found $ENI_COUNT ENIs: $ENI_IDS"
163165
for ENI_ID in $ENI_IDS; do
164-
aws ec2 modify-network-interface-attribute --network-interface-id $ENI_ID --enable-primary-ipv6 || true
166+
echo "enable primary ipv6 address for ENI $ENI_ID"
167+
aws ec2 modify-network-interface-attribute --region $REGION --network-interface-id $ENI_ID --enable-primary-ipv6
165168
done
166169
}
167170

test/framework/utils/poll.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,5 +13,5 @@ const (
1313
// IngressReconcileTimeout is the timeout we expected the controller finishes reconcile for Ingresses.
1414
IngressReconcileTimeout = 1 * time.Minute
1515
// IngressDNSAvailableWaitTimeout is the timeout we expect the DNS records of ALB to be propagated.
16-
IngressDNSAvailableWaitTimeout = 3 * time.Minute
16+
IngressDNSAvailableWaitTimeout = 5 * time.Minute
1717
)

0 commit comments

Comments
 (0)