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

add kindest/node:1.31.0 for e2e #4089

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ty-dc
Copy link
Collaborator

@ty-dc ty-dc commented Sep 22, 2024

Thanks for contributing!

What type of PR is this?

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #3937

Special notes for your reviewer:
cri-o 新 1.31.0 版本已发布,另外在 1.31.0 版本中 MultiCIDRServiceAllocator 从 Alpha 变更到 Beta 版本,在 CI 中 runtimeConfig 配置限制了 api/alpha: "true" 版本,导致 beta 功能不可用,同步增加 api/beta: "true"
https://kubernetes.io/docs/reference/command-line-tools-reference/feature-gates/

@ty-dc ty-dc added the release/none no release note label Sep 22, 2024
@ty-dc ty-dc requested a review from weizhoublue as a code owner September 22, 2024 05:14
@ty-dc ty-dc force-pushed the e2e/add-1-31 branch 5 times, most recently from 6618d1b to 1608c41 Compare September 22, 2024 15:27
@ty-dc ty-dc closed this Sep 22, 2024
@ty-dc ty-dc reopened this Sep 24, 2024
@ty-dc ty-dc added the pr/not-ready not ready for merging label Sep 24, 2024
@ty-dc
Copy link
Collaborator Author

ty-dc commented Sep 24, 2024

在 k8s 1.31.0 版本中,启用了 MultiCIDRServiceAllocator 功能。然后创建 IPv6 的 service 时,无法成功

Error from server (InternalError): error when creating "test-service.yaml": Internal error occurred: failed to allocate a serviceIP: range is full

是 k8s 的一个bug,在 k8s repo 提交了 issue,当前已修复

@ty-dc ty-dc force-pushed the e2e/add-1-31 branch 2 times, most recently from 84e538f to dc93767 Compare September 25, 2024 03:20
@ty-dc
Copy link
Collaborator Author

ty-dc commented Sep 25, 2024

There is still a DRA problem. Waiting for rebase ##4110

Signed-off-by: ty-dc <tao.yang@daocloud.io>
Copy link
Contributor

This pull request has been automatically marked as stale because it
has not had recent activity. It will be closed if no further activity
occurs. Thank you for your contributions.

@github-actions github-actions bot added the pr/stale This pull is inactive label Nov 24, 2024
@aojea
Copy link

aojea commented Nov 27, 2024

@ty-dc @cyclinder are you using MultiCIDRServiceAllocator in prod?

I want to move the feature to ga and I need feedback from real world usage, so if you are doing it or you know someone tha is doing it please let me now

@github-actions github-actions bot removed the pr/stale This pull is inactive label Nov 27, 2024
@cyclinder
Copy link
Collaborator

@aojea MultuiServiceCIDR requires a higher version of k8s, we haven't received any examples of end-users using it yet, and we've been testing it more in our development environment.

@aojea
Copy link

aojea commented Nov 28, 2024

we've been testing it more in our development environment.

Thanks, I want to move it to ga in 1.33 and I'm collecting feedback, so I assume that at least you were testing it and the only issue found so far was the mentioned one

@ty-dc
Copy link
Collaborator Author

ty-dc commented Nov 28, 2024

Thanks, I want to move it to ga in 1.33 and I'm collecting feedback, so I assume that at least you were testing it and the only issue found so far was the mentioned one

Yes, only the issues mentioned above have been found so far. Looking forward to its official GA, we are happy to do something for MultiCIDRServiceAllocator feature if you need it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/not-ready not ready for merging release/none no release note
Projects
None yet
Development

Successfully merging this pull request may close these issues.

kind supports v1.31
3 participants