- 20 Apr, 2017 18 commits
-
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 42177, 42176, 44721) Removed fluentd-gcp manifest pod ```release-note Fluentd manifest pod is no longer created on non-registered master when creating clusters using kube-up.sh. ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 42177, 42176, 44721) Job: Respect ControllerRef **What this PR does / why we need it**: This is part of the completion of the [ControllerRef](https://github.com/kubernetes/community/blob/master/contributors/design-proposals/controller-ref.md) proposal. It brings Job into full compliance with ControllerRef. See the individual commit messages for details. **Which issue this PR fixes**: This ensures that Job does not fight with other controllers over control of Pods. Ref: #24433 **Special notes for your reviewer**: **Release note**: ```release-note Job controller now respects ControllerRef to avoid fighting over Pods. ``` cc @erictune @kubernetes/sig-apps-pr-reviews
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 42177, 42176, 44721) CronJob: Respect ControllerRef **What this PR does / why we need it**: This is part of the completion of the [ControllerRef](https://github.com/kubernetes/community/blob/master/contributors/design-proposals/controller-ref.md) proposal. It brings CronJob into compliance with ControllerRef. See the individual commit messages for details. **Which issue this PR fixes**: This ensures that other controllers do not fight over control of objects that a CronJob owns. **Special notes for your reviewer**: **Release note**: ```release-note CronJob controller now respects ControllerRef to avoid fighting with other controllers. ``` cc @erictune @kubernetes/sig-apps-pr-reviews
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 44555, 44238) openstack: remove field flavor_to_resource I believe there is no usage about `flavor_to_resource`, and I think there is no need to build that information, too. cc @anguslees **Release note:** ``` NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue Remove the old docker-multinode files that were built into the hyperkube image **What this PR does / why we need it**: ref: https://goo.gl/VxSaKx **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes # **Special notes for your reviewer**: **Release note**: ```release-note The hyperkube image has been slimmed down and no longer includes addon manifests and other various scripts. These were introduced for the now removed docker-multinode setup system. ``` cc @jbeda @brendandburns @bgrant0607 @justinsb @mikedanese
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue Switch to pointer to policy rule, visit and short circuit during authorization Ref #40015 * Switches policy rule helper methods to work with pointers * Switches authorization to use a short-circuiting visitor Best-case, authorization short-circuits early and avoids accumulating rules it never needs to check Worst-case (a forbidden request), it still checks all the applicable rules, but requires less allocation to do so $ go test ./plugin/pkg/auth/authorizer/rbac/... -bench=. -benchmem -run Bench on master: ``` BenchmarkAuthorize/allow_list_pods-8 300000 4373 ns/op 3840 B/op 26 allocs/op BenchmarkAuthorize/allow_update_pods/status-8 300000 5121 ns/op 3840 B/op 26 allocs/op BenchmarkAuthorize/forbid_educate_dolphins-8 300000 4706 ns/op 3840 B/op 26 allocs/op ``` with short-circuiting and policy rule pointer changes: ``` BenchmarkAuthorize/allow_list_pods-8 2000000 930 ns/op 64 B/op 2 allocs/op BenchmarkAuthorize/allow_update_pods/status-8 1000000 1656 ns/op 64 B/op 2 allocs/op BenchmarkAuthorize/forbid_educate_dolphins-8 500000 3395 ns/op 1488 B/op 25 allocs/op ```
-
Anthony Yeh authored
-
Jordan Liggitt authored
-
Piotr Szczesniak authored
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue Refactoring reorganize taints function in kubectl to expose operations **What this PR does / why we need it**: This adds some UX functionality when specifying taints using kubectl. For example: ``` ./kubectl.sh taint nodes XYZ dedicated1=abca2:NoSchedule node "XYZ" tainted ./kubectl.sh taint nodes XYZ dedicated1=abca1:NoSchedule --overwrite=True node "XYZ overwritten ./kubectl.sh taint nodes XYZ dedicated1- node "XYZ" untainted ./kubectl.sh taint nodes XYZ dedicated=abca1:NoSchedule dedicated1- node "XYZ" modified ``` **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #43167 **Release note**: ``` Fixed the output of kubectl taint node command with minor improvements. ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue adding test for volume fstype validation **What this PR does / why we need it**: This PR is adding a test for volume fstype validation. Test verifies fstype specified in storage-class is being honored after volume creation. Steps: 1. Create StorageClass with fstype set to valid type (default case included). 2. Create PVC which uses the StorageClass created in step 1. 3. Wait for PV to be provisioned. 4. Wait for PVC's status to become Bound. 5. Create pod using PVC on specific node. 6. Wait for Disk to be attached to the node. 7. Execute command in the pod to get fstype. 8. Delete pod and Wait for Volume Disk to be detached from the Node. 9. Delete PVC, PV and Storage Class. **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes # **Special notes for your reviewer**: **Release note**: ```release-note None ``` cc: @jeffvance @tusharnt
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 42272, 44696) e2e test fix: Wait longer when first creating ELB On any cloud (GCE or AWS), a lag between creating the LoadBalancer and having it actually start serving traffic is expected. On AWS the lag is larger, and we weren't correctly using the longer wait on our first request. Use a longer wait period on our first request. Fix #44695 ```release-note NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue apiserver: Update genericapiserver to panic on listener error Previously runServer would try to listen again if a listener error occurred. This commit changes the response to a panic to allow a process manager (systemd/kubelet/etc) to react to the failure. **Release note**: ```release-note The Kubernetes API server now exits if it encounters a networking failure (e.g. the networking interface hosting its address goes away) to allow a process manager (systemd/kubelet/etc) to react to the problem. Previously the server would log the failure and try again to bind to its configured address:port. ``` cc: @liggitt @sttts @deads2k @derekwaynecarr
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue e2e: Prefer kubeconfig host to default Previously it was necessary to pass ``-host`` to ``e2e.test`` even if ``-kubeconfig`` was specified since otherwise a localhost default would be used. This change ensures that the default is only used when kubeconfig is not set. cc: @jayunit100
-
Justin Santa Barbara authored
On any cloud (GCE or AWS), a lag between creating the LoadBalancer and having it actually start serving traffic is expected. On AWS the lag is larger, and we weren't correctly using the longer wait on our first request. Use a longer wait period on our first request. Fix #44695
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 44687, 44689, 44661) Fix panic when using `kubeadm init` with vsphere cloud-provider **What this PR does / why we need it**: Check if the reference is nil when finding machine reference by UUID. **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #44603 **Special notes for your reviewer**: This is just a quick fix for the panic. **Release note**: ```release-note NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 44687, 44689, 44661) Retry in get-kube.sh to avoid download flakes. GCS has up to 2% 5xx rates, so retrying is critical. This is currently failing about 8 times per day [according to the dashboard](https://storage.googleapis.com/k8s-gubernator/triage/index.html?test=Extract#be2f33fb1e6dd2389d12). It could be backported to reduce the flake rate. Relase note: ```release-note NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue Add hack/lib to kubernetes release tarball **What this PR does / why we need it**: Add hack/lib to kubernetes release tarball, to fix an issue with https://get.k8s.io/ script introduced in #42748. **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: Fixes https://github.com/kubernetes/kubernetes/pull/42748#issuecomment-295412268 **Special notes for your reviewer**: I'm new to bazel, so hopefully I'm not off-base here :) **Release note**: ```release-note NONE ``` cc: @ixdy @dcbw @smarterclayton
-
- 19 Apr, 2017 22 commits
-
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue Implement LRU for AWS device allocator On failure to attach do not use device from pool In AWS environment when attach fails on the node lets not use device from the pool. This makes sure that a bigger pool of devices is available. -
Kubernetes Submit Queue authored
Automatic merge from submit-queue select one api endpoint at random when deploying kubernetes-core charm **What this PR does / why we need it**: Fixes a bug in the kubernetes-worker Juju charm code that attempted to give kube-proxy more than one api endpoint. **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes # **Special notes for your reviewer**: https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/255 **Release note**: ```release-note Fixes a bug in the kubernetes-worker Juju charm code that attempted to give kube-proxy more than one api endpoint. ```
-
Anthony Yeh authored
-
Anthony Yeh authored
Currently, an e2e test is the only way to ensure we have the proper RBAC permissions to adopt Jobs.
-
Anthony Yeh authored
This prevents a race with the GC while it orphans dependents.
-
Anthony Yeh authored
-
Anthony Yeh authored
This is needed now that the default is OrphanDependents.
-
Christoph Blecker authored
-
Ryan Hitchman authored
-
Anthony Yeh authored
-
Maru Newby authored
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue Update gcr.io/google_containers/porter image to 4524579c **What this PR does / why we need it**: updates the porter image to one built at 4524579c using go1.8.1. This incorporates #44638, which has a new dummy certificate that is compliant with go1.8+. Image has already been pushed. **Release note**: ```release-note NONE ``` /assign @liggitt /cc @luxas @lavalamp
-
Anthony Yeh authored
An e2e test is currently the only way to ensure we have the correct RBAC permissions to PATCH Pods.
-
Anthony Yeh authored
This is needed to update ControllerRef during adopt/release.
-
Anthony Yeh authored
This is necessary to avoid racing with the GC when it orphans dependents.
-
Anthony Yeh authored
-
Anthony Yeh authored
-
Anthony Yeh authored
Now that the default delete option for Job is OrphanDependents, Job deletion is asynchronous.
-
Anthony Yeh authored
-
Anthony Yeh authored
The Job Listers still use selectors, because this is the behavior expected by callers. This clarifies the meaning of the returned list. Some callers may need to switch to using GetControllerOf() instead, but that is a separate, case-by-case issue.
-
Anthony Yeh authored
This is part of the completion of ControllerRef, as described here: https://github.com/kubernetes/community/blob/master/contributors/design-proposals/controller-ref.md#watches
-
Hemant Kumar authored
In AWS environment when attach fails on the node lets not use device from the pool. This makes sure we don't reuse recently freed devices
-