-
Notifications
You must be signed in to change notification settings - Fork 38.8k
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
[Federation][(Un)join-01] Refactor common functions and structs into a util package. #35592
[Federation][(Un)join-01] Refactor common functions and structs into a util package. #35592
Conversation
LGTM, thanks @madhusudancs |
@madhusudancs PS you have some verification and unit test failures to resolve before merging. |
…ertutil-triple Automatic merge from submit-queue Implement package `triple` with utilities to generate certificate-key pairs for CA, server and clients. Please review only the last commit here. This is based on PRs #35592 which will be reviewed independently. Design Doc: PR #34484 cc @kubernetes/sig-cluster-federation @quinton-hoole @mwielgus
Also, add unit tests for `kubefed unjoin`.
82baefd
to
419fc9d
Compare
Jenkins Kubemark GCE e2e failed for commit 419fc9de7eb4f89c478d684a9786e18f31b9b769. Full PR test history. The magic incantation to run this job again is |
Jenkins GKE smoke e2e failed for commit 419fc9de7eb4f89c478d684a9786e18f31b9b769. Full PR test history. The magic incantation to run this job again is |
419fc9d
to
fe36da1
Compare
Jenkins GCI GKE smoke e2e failed for commit fe36da1e38917c4de7997f39db4bed074f4293d7. Full PR test history. The magic incantation to run this job again is |
Jenkins GCI GCE e2e failed for commit fe36da1e38917c4de7997f39db4bed074f4293d7. Full PR test history. The magic incantation to run this job again is |
Jenkins GCE e2e failed for commit fe36da1e38917c4de7997f39db4bed074f4293d7. Full PR test history. The magic incantation to run this job again is |
fe36da1
to
963a6ad
Compare
Jenkins unit/integration failed for commit 963a6ad82f94198d618ca6faccb04a71fa3d1497. Full PR test history. The magic incantation to run this job again is |
963a6ad
to
142e5a6
Compare
Also: 1. Add it to the client build targets list. 2. Register `kubefed join` and `kubefed unjoin` commands.
142e5a6
to
c044cae
Compare
Jenkins verification failed for commit c044cae. Full PR test history. The magic incantation to run this job again is |
c044cae
to
9a0fc0c
Compare
Adding |
Automatic merge from submit-queue |
Please review only the last commit here. This is based on PR #35495 which will be reviewed independently.
Design Doc: PR #34484
cc @kubernetes/sig-cluster-federation @quinton-hoole @nikhiljindal
This change is![Reviewable](https://camo.githubusercontent.com/2d899f4291d07d3cd2fa4aaae1e3b243f164c23fce87d30a589ace0d496a444c/68747470733a2f2f72657669657761626c652e6b756265726e657465732e696f2f7265766965775f627574746f6e2e737667)