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
Graduate DevicePluginCDIDevices to Beta #121254
Graduate DevicePluginCDIDevices to Beta #121254
Conversation
better to wait for #119280. @elezar are you still working adding e2e test for CDI devices? This sounds like a bar for alpha.(Probably, this can be the beta promotion criteria IMO). loop in KEP reviewers and approver: |
/sig node |
/triage accepted |
I tend to agree, addition of the e2e tests should be Beta promotion criteria. We should make sure that #119280 (once ready and reviewed) is merged before this PR. |
@pacoxu @swatisehgal Thanks for pointing out to #119280. I'll work on adding e2e tests. btw, should it be a pre-submit or periodic test or both? |
/retest |
I think it would be good for us to have both as periodic jobs help us track the health of the jobs on a regular basis and pre-submit jobs helps us prevent regressions when new PRs are submitted. This could be a good topic for discussion in the SIG Node CI subgroup and give a heads-up that we'd be introducing new jobs for testing of this feature. |
LGTM label has been added. Git tree hash: ce0d30cd64d48ef011079110495671fc7587f2d9
|
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bart0sh, dims, klueska The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/unhold |
What type of PR is this?
/kind feature
What this PR does / why we need it:
This PR graduates DevicePluginCDIDevices to Beta in 1.29
Special notes for your reviewer:
KEP update PR has been already merged: kubernetes/enhancements#4238
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: