-
Notifications
You must be signed in to change notification settings - Fork 388
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
Bump K8s version to 1.31 in EKS CI script #7005
Conversation
@wenyingd was the issue specific to EKS 1.27, or is it that vanilla K8s 1.27 does not support this field selector? I am asking because if it is the latter (field selector was introduced after K8s 1.27), then we may need some additional changes. In theory Antrea is supposed to work on all K8s versions >= 1.19 out-of-the-box. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, EKS test passed in http://34.172.220.207:8080/job/cloud-antrea-eks-conformance-net-policy/88/console.
I tried on K8s v1.27.13, it looks that the field selector is not supported either. So it looks it is a general limitation for k8s versions.
|
The field selector was introduced in 1.28 and was not backported: kubernetes/kubernetes@7fb88ee. |
We should still merge this PR as it is good to upgrade the EKS version we use for CI, but we need another one to remove both usages of the field selector. |
/test-all |
Please change the commit message and description as well |
This change is to bump up the EKS version to 1.31 in the CI script. Signed-off-by: Wenying Dong <wenyingd@vmware.com>
/test-all |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - did the Jenkins CI job for EKS succeed?
Hi @wenyingd @XinShuYang I know the job was passed before, since the fix #7012 was merged, could you rerun the this change based on the latest main? Thanks. |
CI with EKS 1.27 also recovered after the merge of #7012. cc @antoninbas @luolanzone |
This change is to bump up the EKS version to 1.31 in the CI script.