ci: add K8s 1.28 platform testing #29004
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Cilium itself officially supports K8s 1.28 since 6a70af2, however platforms did not have a compatible 1.28 version available for managed clusters until recently. Signed-off-by: Nicolas Busseneau <nicolas@isovalent.com>
|
/ci-aks |
|
/ci-eks |
|
/ci-gke |
|
/ci-awscni |
|
It seems AKS still does not support 1.28 |
Correct, I got misled by my local environment yesterday, it's still not GA. |
|
1.28 is now GA on AKS and test passed: https://github.com/cilium/cilium/actions/runs/6772112977 We can mark ready to merge as soon as reviews are in :) |
|
Please run the full suite in the future so that this can go into the merge queue automatically. |
|
/test |
|
@lmb This PR does not need to run the full test suite, it only modifies a restricted set of files, and does not need to go through the merge queue. My bad for not properly mentioning it when saying it was ready to merge. |
aanm
approved these changes
Nov 27, 2023
mhofstetter
added a commit
to mhofstetter/cilium
that referenced
this pull request
Nov 28, 2023
Currently, the tests `ci-eks` & `ci-aws-cni` fail due to issues deploying multiple clusters with the same name in the same region. Therefore, this commit uses a different region for k8s version v1.24 to ensure that only one cluster gets deployed per region. Fixes: cilium#29004 Signed-off-by: Marco Hofstetter <marco.hofstetter@isovalent.com>
github-merge-queue bot
pushed a commit
that referenced
this pull request
Nov 28, 2023
Currently, the tests `ci-eks` & `ci-aws-cni` fail due to issues deploying multiple clusters with the same name in the same region. Therefore, this commit uses a different region for k8s version v1.24 to ensure that only one cluster gets deployed per region. Fixes: #29004 Signed-off-by: Marco Hofstetter <marco.hofstetter@isovalent.com>
pjablonski123
pushed a commit
to pjablonski123/cilium
that referenced
this pull request
Dec 15, 2023
Currently, the tests `ci-eks` & `ci-aws-cni` fail due to issues deploying multiple clusters with the same name in the same region. Therefore, this commit uses a different region for k8s version v1.24 to ensure that only one cluster gets deployed per region. Fixes: cilium#29004 Signed-off-by: Marco Hofstetter <marco.hofstetter@isovalent.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Cilium itself officially supports K8s 1.28 since
6a70af2, however platforms did not have a compatible 1.28 version available for managed clusters until recently.