Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[v1.13] fix aws region being used twice #31740

Merged
merged 1 commit into from Apr 4, 2024

Conversation

brlbil
Copy link
Contributor

@brlbil brlbil commented Apr 3, 2024

When backporting ca-west-1 region used twice in tested aws k8s versions
Also, ca-west-1 region is not supported by the eksctl tool used in testing
This PR replaces ca-west-1 regions with ca-central-1 and us-east-1

Action run (cluster provisioning works): https://github.com/cilium/cilium/actions/runs/8537420316

@maintainer-s-little-helper maintainer-s-little-helper bot added backport/1.13 This PR represents a backport for Cilium 1.13.x of a PR that was merged to main. kind/backports This PR provides functionality previously merged into master. labels Apr 3, 2024
@brlbil brlbil force-pushed the pr/brlbil/fix-aws-test-region branch from 437575b to d421ec2 Compare April 3, 2024 09:34
@brlbil
Copy link
Contributor Author

brlbil commented Apr 3, 2024

/ci-awscni

when backporting ca-west-1 region used twice in tested aws k8s versions
also ca-west-1 region is not supported by the eksctl tool used in testing
this commit replaces ca-west-1 regions with ca-central-1 and us-east-1

Signed-off-by: Birol Bilgin <birol@cilium.io>
@brlbil brlbil force-pushed the pr/brlbil/fix-aws-test-region branch from d421ec2 to 559851c Compare April 3, 2024 10:29
@brlbil
Copy link
Contributor Author

brlbil commented Apr 3, 2024

/ci-awscni

@brlbil brlbil force-pushed the pr/brlbil/fix-aws-test-region branch from 559851c to af71c37 Compare April 3, 2024 11:25
@brlbil
Copy link
Contributor Author

brlbil commented Apr 3, 2024

/test-backport-1.13

@brlbil brlbil marked this pull request as ready for review April 3, 2024 11:51
@brlbil brlbil requested a review from a team as a code owner April 3, 2024 11:51
@julianwiedmann julianwiedmann changed the title fix aws region being used twice [v1.13] fix aws region being used twice Apr 3, 2024
@julianwiedmann
Copy link
Member

This is fallout from #30378, right?

@brlbil
Copy link
Contributor Author

brlbil commented Apr 3, 2024

Yes, it is

@julianwiedmann
Copy link
Member

/test-backport-1.13

@julianwiedmann julianwiedmann merged commit d501652 into v1.13 Apr 4, 2024
217 of 231 checks passed
@julianwiedmann julianwiedmann deleted the pr/brlbil/fix-aws-test-region branch April 4, 2024 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/1.13 This PR represents a backport for Cilium 1.13.x of a PR that was merged to main. kind/backports This PR provides functionality previously merged into master.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants