Skip to content

Commit

Permalink
fix: eks tests broken due to incorrect addon versions
Browse files Browse the repository at this point in the history
This updates the addon versions used for the eks tests inline with
the updated k8s version for the created cluster.

Also update the IPv6 test to be more in line with the other tests.

Signed-off-by: Richard Case <[email protected]>
  • Loading branch information
richardcase committed Oct 25, 2024
1 parent f3c178f commit 1386136
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 29 deletions.
8 changes: 4 additions & 4 deletions test/e2e/data/e2e_eks_conf.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -126,11 +126,11 @@ variables:
AWS_SSH_KEY_NAME: "cluster-api-provider-aws-sigs-k8s-io"
EXP_EKS_IAM: "false"
EXP_EKS_ADD_ROLES: "false"
VPC_ADDON_VERSION: "v1.16.2-eksbuild.1"
COREDNS_ADDON_VERSION: "v1.11.1-eksbuild.6"
VPC_ADDON_VERSION: "v1.18.1-eksbuild.3"
COREDNS_ADDON_VERSION: "v1.11.1-eksbuild.8"
COREDNS_ADDON_CONFIGURATION: '{"replicaCount":3}'
KUBE_PROXY_ADDON_VERSION: "v1.29.0-eksbuild.2"
CONFORMANCE_CI_ARTIFACTS_KUBERNETES_VERSION: "1.29.1"
KUBE_PROXY_ADDON_VERSION: "v1.30.0-eksbuild.3"
CONFORMANCE_CI_ARTIFACTS_KUBERNETES_VERSION: "1.30.2"
IP_FAMILY: "IPv4"
CAPA_LOGLEVEL: "4"
EXP_EXTERNAL_RESOURCE_GC: "true"
Expand Down
23 changes: 0 additions & 23 deletions test/e2e/data/eks/cluster-template-eks-ipv6-cluster.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -54,26 +54,3 @@ spec:
identityRef:
kind: AWSClusterStaticIdentity
name: e2e-account
---
apiVersion: cluster.x-k8s.io/v1beta1
kind: MachinePool
metadata:
name: "${CLUSTER_NAME}-pool-0"
spec:
clusterName: "${CLUSTER_NAME}"
replicas: ${WORKER_MACHINE_COUNT}
template:
spec:
clusterName: "${CLUSTER_NAME}"
bootstrap:
dataSecretName: ""
infrastructureRef:
name: "${CLUSTER_NAME}-pool-0"
apiVersion: infrastructure.cluster.x-k8s.io/v1beta2
kind: AWSManagedMachinePool
---
apiVersion: infrastructure.cluster.x-k8s.io/v1beta2
kind: AWSManagedMachinePool
metadata:
name: "${CLUSTER_NAME}-pool-0"
spec: {}
7 changes: 5 additions & 2 deletions test/e2e/suites/managed/eks_ipv6_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ var _ = ginkgo.Describe("[managed] [general] [ipv6] EKS cluster tests", func() {
Namespace: namespace,
ClusterName: clusterName,
Flavour: EKSIPv6ClusterFlavor,
ControlPlaneMachineCount: 1, //NOTE: this cannot be zero as clusterctl returns an error
ControlPlaneMachineCount: 1, // NOTE: this cannot be zero as clusterctl returns an error
WorkerMachineCount: 0,
}
})
Expand All @@ -85,8 +85,11 @@ var _ = ginkgo.Describe("[managed] [general] [ipv6] EKS cluster tests", func() {
AWSSession: e2eCtx.BootstrapUserAWSSession,
Namespace: namespace,
ClusterName: clusterName,
IncludeScaling: false,
Cleanup: false,
ManagedMachinePool: true,
Flavor: EKSIPv6ClusterFlavor,
Flavor: EKSManagedMachinePoolOnlyFlavor,
UsesLaunchTemplate: false,
}
})

Expand Down

0 comments on commit 1386136

Please sign in to comment.