-
Notifications
You must be signed in to change notification settings - Fork 498
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
Replace controller.Int32Ptr with k8s.io/utils/pointer.Int32Ptr (#2841) #2845
Replace controller.Int32Ptr with k8s.io/utils/pointer.Int32Ptr (#2841) #2845
Conversation
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@lichunzhu can you cherry-pick it manually? the refactor PR #2841 changed the code which only goes into v1.2, so the automatic cherry-picked PR has conflicts in release-1.1. |
okay |
I deleted the file |
it seems all goes well |
/merge |
/run-all-tests |
@ti-srebot merge failed. |
/merge |
/run-all-tests |
cherry-pick #2841 to release-1.1
What problem does this PR solve?
Resolve #2836
What is changed and how does it work?
Replace controller.Int32Ptr with k8s.io/utils/pointer.Int32Ptr
Check List
Tests
Related changes
Does this PR introduce a user-facing change?: