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
update golang to v1.17 #1912
update golang to v1.17 #1912
Conversation
I met this error, too. |
I reviewed the UT file allocate_test.go and it was not changed recently. So I guess there may be some wrong update with the allocate.go. @justadogistaken I noticed that only this PR #1906 was merged with this file recently. So can you help give a test about this bug? I've tested for serval times and found that the allocations are not stable. Namely, the output allocations are not unique. There are small probability to be same with expected. |
I reset my commit to the pr ahead of PR #1906, this error exists previously. But this may not be an error. Pods of c1/p1, c1/p2, c2/p1, c2/p2 have same requests. This error will happen if c1/p2 gets into tasks(priorityQueue) first or c2/p2 gets into first instead of p1. |
Let's check this out.
|
/lgtm |
/approve |
There was a pr #1898 removing the follwing block.
|
/cc @xiaoanyunfei Pls help take a look and make corresponding UT cases changed if the allocate logic changes. THX. |
Signed-off-by: Thor-wl <13164644535@163.com>
Signed-off-by: Thor-wl <13164644535@163.com>
2486f4a
to
40f5156
Compare
/lgtm |
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
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: huone1, shinytang6 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 |
As kubernetes-sigs/controller-tools#643 mentions, there may be some bugs in package
encoding
for golang v1.16.x. This PR should be merged before #1910Signed-off-by: Thor-wl 13164644535@163.com