cluster-api-provider-aws icon indicating copy to clipboard operation
cluster-api-provider-aws copied to clipboard

[release-1.5] fix: use v1beta1 version in EKS user kubeconfig

Open k8s-infra-cherrypick-robot opened this issue 2 years ago • 7 comments

This is an automated cherry-pick of #3740

/assign dlipovetsky

Looking into the failure.

dlipovetsky avatar Oct 11 '22 17:10 dlipovetsky

There's a failure in the should create cluster with nested assumed role test. I see the same failure for an older backport PR, #3632.

I don't see any changes in this PR, or between release-1.5 and main, that appear to be related to the test failure.

I'll re-run the test.

dlipovetsky avatar Oct 11 '22 19:10 dlipovetsky

/test pull-cluster-api-provider-aws-e2e-blocking-release-1-5

dlipovetsky avatar Oct 11 '22 19:10 dlipovetsky

A different test, "Should successfully create a cluster with machine pool machines" failed. It passed the first run.

I'll re-run again.

dlipovetsky avatar Oct 11 '22 22:10 dlipovetsky

/test pull-cluster-api-provider-aws-e2e-blocking-release-1-5

dlipovetsky avatar Oct 11 '22 22:10 dlipovetsky

This PR only affects managed (EKS) clusters, but the failing tests in pull-cluster-api-provider-aws-e2e-blocking-release-1-5 have all been for an unmanaged cluster.

dlipovetsky avatar Oct 17 '22 15:10 dlipovetsky

/test pull-cluster-api-provider-aws-e2e-blocking-release-1-5

dlipovetsky avatar Oct 17 '22 15:10 dlipovetsky

/lgtm

Skarlso avatar Oct 17 '22 15:10 Skarlso

@sedefsavas I think this is worth merging into release-1.5

We can decide separately whether or not to release 1.5.1

dlipovetsky avatar Oct 18 '22 16:10 dlipovetsky

/approve

Ankitasw avatar Oct 25 '22 09:10 Ankitasw

/approve

This should now work, shouldn't it? 🤔

Skarlso avatar Oct 26 '22 06:10 Skarlso

It looks like the maintainer approval has no effect because the https://github.com/orgs/kubernetes-sigs/teams/cluster-api-provider-aws-maintainers is not up to date. I'll raise this on slack.

Update: We'll be able to approve once we backport the maintainer metadata to release-1.5 (https://github.com/kubernetes-sigs/cluster-api-provider-aws/pull/3807)

dlipovetsky avatar Oct 31 '22 17:10 dlipovetsky

/approve

richardcase avatar Oct 31 '22 18:10 richardcase

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Ankitasw, richardcase, Skarlso

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

k8s-ci-robot avatar Oct 31 '22 18:10 k8s-ci-robot