Kubernetes Submit Queue
5af04d1dd1
Merge pull request #32876 from errordeveloper/more-cert-utils
...
Automatic merge from submit-queue
Refactor cert utils into one pkg, add funcs from bootkube for kubeadm to use
**What this PR does / why we need it**:
We have ended-up with rather incomplete and fragmented collection of utils for handling certificates. It may be worse to consider using `cfssl` for doing all of these things, but for now there is some functionality that we need in `kubeadm` that we can borrow from bootkube. It makes sense to move the utils from bookube into core, as discussed in #31221 .
**Special notes for your reviewer**: I've taken the opportunity to review names of existing funcs and tried to make some improvements in that area (with help from @peterbourgon).
**Release note**:
```release-note
NONE
```
2016-09-22 01:29:46 -07:00
Ilya Dmitrichenko
386fae4592
Refactor utils that deal with certs
...
- merge `pkg/util/{crypto,certificates}`
- add funcs from `github.com/kubernetes-incubator/bootkube/pkg/tlsutil`
- ensure naming of funcs is fairly consistent
2016-09-19 09:03:42 +01:00
Michail Kargakis
2fd3c490df
controller: a couple of fixes for csr
...
Fixes:
* delete resource handler wasn't taking into account tombstones
* csr would requeue twice on update failure
2016-09-18 22:48:46 +02:00
Kubernetes Submit Queue
920581d964
Merge pull request #32664 from m1093782566/m109-certificates-hot-loop
...
Automatic merge from submit-queue
[Controller Manager] Fix certificates controller hotloop and use utilruntime.HandleError to replace glog.Errorf
<!-- Thanks for sending a pull request! Here are some tips for you:
1. If this is your first time, read our contributor guidelines https://github.com/kubernetes/kubernetes/blob/master/CONTRIBUTING.md and developer guide https://github.com/kubernetes/kubernetes/blob/master/docs/devel/development.md
2. If you want *faster* PR reviews, read how: https://github.com/kubernetes/kubernetes/blob/master/docs/devel/faster_reviews.md
3. Follow the instructions for writing a release note: https://github.com/kubernetes/kubernetes/blob/master/docs/devel/pull-requests.md#release-notes
-->
**What this PR does / why we need it**:
Fix certificates controller hotloop on unexpected API server rejections.
**Which issue this PR fixes**
Related issue is #30629
**Special notes for your reviewer**:
@deads2k @derekwaynecarr PTAL.
I find there is no unit test for certificates controller, and I will implement unit tests for it later.
2016-09-17 21:00:59 -07:00
Mike Danese
a765d59932
move informer and controller to pkg/client/cache
...
Signed-off-by: Mike Danese <mikedanese@google.com >
2016-09-15 12:50:08 -07:00
d00369826
fea0c79054
fix certificates controller hotloop on unexpected API server rejections
...
Change-Id: Ib7d2e18bcaa498bddfc785f3ff12958dfaaecbc3
2016-09-15 20:10:21 +08:00
deads2k
4317173d3f
add names for workqueues to gather controller latency/depth metrics
2016-08-30 09:51:50 -04:00
Mike Danese
9f379df76b
add an option to controller-manager to auto approve all CSRs
2016-08-22 11:46:01 -07:00
George Tankersley
803c7ac299
certificates: implement certificates controller
2016-07-19 11:25:38 -07:00
