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
remove system validators package from kubeadm and use k8s.io/system-validators #84718
Conversation
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.
Confirming there aren't any behavioral changes here?
Do you mind taking a look at the verify
build failures? thanks :)
6f595f0
to
55827de
Compare
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.
just some notes for @neolit123 and @Random-Liu
/lgtm |
…alidators instead Signed-off-by: Yassine TIJANI <ytijani@vmware.com>
727b010
to
b9fe59c
Compare
/remove-area apiserver |
/retest |
/assign @liggitt |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: BenTheElder, liggitt, neolit123, yastij 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 |
@neolit123 @liggitt - feel free to unhold, this should be good to go |
/hold cancel |
/retest |
2 similar comments
/retest |
/retest |
Signed-off-by: Yassine TIJANI ytijani@vmware.com
What type of PR is this?
/kind cleanup
/priority important-soon
What this PR does / why we need it: This PR removes the system validators package from kubeadm and uses instead
k8s.io/system-validators
Which issue(s) this PR fixes: part of kubernetes/kubeadm#1638
Special notes for your reviewer:
/assign @neolit123 @Random-Liu
Does this PR introduce a user-facing change?:
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: