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.

Sign up for GitHub

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

Jump to bottom

CI: Update tested k8s version for aks #27457

Merged
merged 1 commit into from Aug 16, 2023
Merged

CI: Update tested k8s version for aks #27457

merged 1 commit into from Aug 16, 2023

Conversation

brlbil
Copy link
Contributor

@brlbil brlbil commented Aug 11, 2023 •
edited

AKS-supported k8s versions changed.
This PR update tested k8s versions for AKS according to the latest support matrix

Successful run

Fixes: #27443

@brlbil
ci: update tested k8s version for aks
ccceb20 
AKS supported k8s versions changed.
This commit updates tested k8s versions for AKS

Signed-off-by: Birol Bilgin <birol@cilium.io>
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Aug 11, 2023
@brlbil brlbil added area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI. labels Aug 11, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Aug 11, 2023
@brlbil
Copy link
Contributor Author

brlbil commented Aug 11, 2023

/ci-aks

@brlbil brlbil added needs-backport/1.12 needs-backport/1.13 This PR / issue needs backporting to the v1.13 branch needs-backport/1.14 This PR / issue needs backporting to the v1.14 branch labels Aug 11, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from main in 1.12.13 Aug 11, 2023
@brlbil brlbil force-pushed the pr/brlbil/ci-update-aks-k8s-versions branch from cc519f7 to ccceb20 Compare August 11, 2023 13:25
@brlbil brlbil marked this pull request as ready for review August 11, 2023 13:26
@brlbil brlbil requested a review from a team as a code owner August 11, 2023 13:26
@brlbil brlbil requested a review from bimmlerd August 11, 2023 13:26
@asauber asauber added this to Needs backport from main in 1.12.14 Aug 13, 2023
@asauber asauber removed this from Needs backport from main in 1.12.13 Aug 13, 2023
@brlbil brlbil requested review from aanm and removed request for bimmlerd August 15, 2023 09:28
@brlbil
Copy link
Contributor Author

brlbil commented Aug 15, 2023

Since failed test is not related, setting it as ready-to-merge

@brlbil brlbil added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Aug 15, 2023
@ti-mo ti-mo removed the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Aug 16, 2023
@ti-mo
Copy link
Contributor

ti-mo commented Aug 16, 2023

@brlbil Looks like the AKS test(s) haven't run? At least I can't find them under Checks. Removed the label for now.

@brlbil
Copy link
Contributor Author

brlbil commented Aug 16, 2023

@ti-mo Since we are testing the scheduled run that runs all the versions of k8s, I have tested with a later deleted commit.
So you can see the run test in the PR comment, for simplicity here is the link to the successful test run

@ti-mo
Copy link
Contributor

ti-mo commented Aug 16, 2023

Interesting, thanks!

@ti-mo ti-mo merged commit 4cb2a4b into main Aug 16, 2023
64 of 85 checks passed
@ti-mo ti-mo deleted the pr/brlbil/ci-update-aks-k8s-versions branch August 16, 2023 08:59
@tklauser tklauser mentioned this pull request Aug 22, 2023
Merged
22 tasks
@tklauser tklauser added the backport-pending/1.14 The backport for Cilium 1.14.x for this PR is in progress. label Aug 22, 2023
@tklauser tklauser removed the needs-backport/1.14 This PR / issue needs backporting to the v1.14 branch label Aug 22, 2023
@tklauser tklauser mentioned this pull request Aug 23, 2023
Merged
8 tasks
@tklauser tklauser added backport-pending/1.13 The backport for Cilium 1.13.x for this PR is in progress. and removed needs-backport/1.13 This PR / issue needs backporting to the v1.13 branch labels Aug 23, 2023
@tklauser tklauser mentioned this pull request Aug 23, 2023
Merged
7 tasks
@joestringer joestringer added backport-done/1.13 The backport for Cilium 1.13.x for this PR is done. and removed backport-pending/1.13 The backport for Cilium 1.13.x for this PR is in progress. labels Aug 25, 2023
@tklauser tklauser added backport-done/1.12 The backport for Cilium 1.12.x for this PR is done. and removed backport-pending/1.12 labels Aug 25, 2023
@joestringer joestringer added backport-done/1.14 The backport for Cilium 1.14.x for this PR is done. and removed backport-pending/1.14 The backport for Cilium 1.14.x for this PR is in progress. labels Aug 25, 2023
@joestringer joestringer moved this from Needs backport from main to Backport done to v1.12 in 1.12.14 Aug 25, 2023
This was referenced Sep 9, 2023
Merged
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aanm aanm aanm approved these changes

Assignees
No one assigned
Labels
area/CI Continuous Integration testing issue or flake backport-done/1.12 The backport for Cilium 1.12.x for this PR is done. backport-done/1.13 The backport for Cilium 1.13.x for this PR is done. backport-done/1.14 The backport for Cilium 1.14.x for this PR is done. release-note/ci This PR makes changes to the CI.
Projects
No open projects
1.12.14
Backport done to v1.12
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

CI: v1.12: Conformance AKS fails during provisioning: UnsupportedAliasMinorVersion
5 participants
@brlbil @ti-mo @aanm @tklauser @joestringer