-
Notifications
You must be signed in to change notification settings - Fork 532
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 deps to 1.27 #608
Conversation
Skipping CI for Draft Pull Request. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Huang-Wei 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 |
@zwpaper you can take over if you have bandwidth. Atop this PR, UT in |
added a PR here Huang-Wei#1 to fix the failing test. will do a full test later @Huang-Wei |
Thanks a lot! I've merged that one in the fork repo, and push here. I'm going to tweak the CI to use go 1.20, and then have the CI start here: kubernetes/test-infra#30235 |
/retest |
Signed-off-by: Zhang Wei <kweizh@gmail.com>
CI failure:
|
@zwpaper ^^ FYI |
really strange, we did not change the logic related to the integration test, why is this keep failing and the origin tests work fine #610 it keeps report
|
/test pull-scheduler-plugins-integration-test-master |
@zwpaper I also spot this while debugging. I'm pretty sure it's a regression introduced in upstream v1.27. Let me file a fix in k/k. |
@zwpaper to unblock bumping master to 1.27, I commented out those preemption tests. We can revert them once the upstream fix is available in the next v1.27 patch release. PTAL. |
wow, I have tried and did not found that changes, let me have a look at the k/k issue and find out is there anything I can help. /lgtm |
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #606
Special notes for your reviewer:
Does this PR introduce a user-facing change?