- 06 Nov, 2017 10 commits
-
-
George Kraft authored
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 54535, 54950, 55081). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Bug/departing **What this PR does / why we need it**: With this PR we fix the bugs we had with multiple masters setup with juju. **Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*: Fixes https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/422 **Special notes for your reviewer**: Related PR: https://github.com/juju-solutions/interface-kube-control/pull/15 **Release note**: ```NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 54535, 54950, 55081). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. code-gen: add +groupGoName to allow unique Go identifiers in clientsets and informers We use the first segement of the GroupName as Go name. Hence, a GroupName "policy.k8s.io" and "policy.authorization.k8s.io" could not live in the same clientset or shared informer factory. This PR add another tag: ``` // +groupGoName=AuthorizationPolicy ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. apiextensions/examples: add typed client It was discussed in https://github.com/kubernetes/kubernetes/pull/52753#issuecomment-337840762 to add a typed client example to apiextensions-apiserver and remove the controller part. For an example controller, we have [`k8s.io/sample-controller`](https://github.com/kubernetes/kubernetes/tree/master/staging/src/k8s.io/sample-controller) now. **Release note**: ```release-note NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Remove kubectl dependency internal version **What this PR does / why we need it**: ref kubernetes/kubectl#83 **Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*: Fixes # **Special notes for your reviewer**: use `k8s.io/api/core/v1` replace `k8s.io/kubernetes/pkg/api` use `k8s.io/client-go/kubernetes/typed/core/v1` replace `k8s.io/kubernetes/pkg/client/clientset_generated/internalclientset/typed/core/internalversion` **Release note**: ```release-note NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Remove the wrongly checked in binary This is awkward :(
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Fix scheduler predicates test that may violate DNS label rules **What this PR does / why we need it**: This commit fixes an issue where in clusters which have FQDN as the node names, one of the scheduling predicates tests will fail because it will try and run a pod with a name that violates DNS-1123 rules. As an example, one such pod name could look like "filler-pod-kube-node-0.kubelet.mesos". **Which issue(s) this PR fixes**: Fixes #55117 **Special notes for your reviewer**: As soon as this is approved, I will create the cherry-pick PRs for 1.7.x. It doesn't apply on 1.8.x. cc @aveshagarwal @kubernetes/sig-scheduling-pr-reviews **Release note**: ``` NONE ```
-
Dr. Stefan Schimanski authored
-
Dr. Stefan Schimanski authored
-
Dr. Stefan Schimanski authored
-
- 05 Nov, 2017 11 commits
-
-
Chao Xu authored
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Stop using the PersistentVolumeLabel admission controller in v1.9 **What this PR does / why we need it**: Stop using the PersistentVolumeLabel admission controller in v1.9, as in v1.9, we're targeting out-of-tree cloud providers to be ready to use more generally as part of moving to beta. **Which issue this PR fixes**: fixes kubernetes/kubeadm#444
-
root authored
-
Paulo Pires authored
This commit fixes an issue where in clusters which have FQDN as the node names, one of the scheduling predicates tests will fail because it will try and run a pod with a name that violates DNS-1123 rules. As an example, one such pod name could look like "filler-pod-kube-node-0.kubelet.mesos". Signed-off-by:Paulo Pires <pjpires@gmail.com>
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. correct the error info when resourceName equals to hugepage **What this PR does / why we need it**: when the resourceName eqauls to hugepage, the error info outputs as NvidiaGPU, which should be corrected.
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. [trival]remove duplicated code from unit test **Release note**: ``` NONE ```
-
Nikhita Raghunath authored
-
Nikhita Raghunath authored
-
Nikhita Raghunath authored
-
Nikhita Raghunath authored
-
Cao Shufeng authored
-
- 04 Nov, 2017 4 commits
-
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Add comment for preempt sunction. **What this PR does / why we need it**: Add explain for preempt sunction. **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 ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. kubeadm: reset: use crictl to reset containers @luxas PTAL Signed-off-by:
Antonio Murdaca <runcom@redhat.com> **What this PR does / why we need it**: This patch makes kubeadm to try and reset containers using `crictl` first instead of docker. The reason is that kubeadm reset is ineffective with new container runtimes using the CRI (like CRI-O). This patch uses `crictl` first and falls back to `docker` in case `crictl` isn't available. **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes # Fix https://github.com/kubernetes/kubeadm/issues/508 **Special notes for your reviewer**: **Release note**: ```release-note kubeadm: reset: use crictl to reset containers ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Adds Support for Configurable Kubeadm Probes. **What this PR does / why we need it**: Allows kubeadm liveness probes to be configurable using extra args. Needed to provide deployment flexibility where services are not bound to `127.0.0.1`. **Which issue this PR fixes**: fixes https://github.com/kubernetes/kubeadm/issues/473 **Special notes for your reviewer**: Needed for IPv6 support. /sig network /area ipv6 **Release note**: ```release-note ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Check for available volume before attach/delete in EBS We should check for available volume before performing attach or delete of EBS volume. Mutable API calls such as Attach Volume or Delete Volume has much lower quota than immutable API calls such as DescribeVolume or DescribeInstance. Checking for available state before attach or delete ensures that we are not making useless API calls. Fixes https://github.com/kubernetes/kubernetes/issues/55014 ```release-note Check for available volume before attach/delete operation in EBS ```
-
- 03 Nov, 2017 15 commits
-
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 53679, 51063). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Fixes to enable Windows CNI **What this PR does / why we need it**: This PR has fixed which enables Kubelet to use Windows CNI plugin. **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes # #49646 **Special notes for your reviewer**: **Release note**: ```release-note ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Workloads V1 **What this PR does / why we need it**: This PR promotes the Deployment, ReplicaSet, and DaemonSet StatefulSet, ControllerRevision kinds to the apps/v1 group version. https://github.com/kubernetes/features/issues/353 **Special notes for your reviewer**: There will be at least two followups to this PR. The first to add a scale sub-resource when the correct location is resolved, and the second to deal with Conditions in the workloads API. While it would have been preferable to move the kinds individually providing a lesser burden on reviewers, this proved impracticable due to the intricacies of version resolution in kubectl for objects of the different kinds in the same group. ```release-note DaemonSet, Deployment, ReplicaSet, and StatefulSet have been promoted to GA and are available in the apps/v1 group version. ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Rename Detach() parameter. `Detach()` does not get device name, it gets volume name. Parameters named `deviceMountPath` or `deviceName` just confuses developers. Note that this PR just renames parameters here and there, there should be no behavior change. @kubernetes/sig-storage-pr-reviews /assign @gnufied @jingxu97 **Release note**: ```release-note NONE ```
-
Konstantinos Tsakalozos authored
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Add prometheus-to-sd-exporter to metadata-proxy addon; bump to v0.1.4 **What this PR does / why we need it**: Add metrics exporters to the metadata-proxy addon for GCE. Work toward #8867. **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 ```
-
Marcin Wielgus authored
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 55050, 53464, 54936, 55028, 54928). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Fix panic in kubelet because of uninitialized map **What this PR does / why we need it**: Initialized the uninitialized map in kubelet **Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*: Fixes [#54927](https://github.com/kubernetes/kubernetes/issues/54927) **Special notes for your reviewer**: The default value of --enable-controller-attach-detach is true, map will be initialized like: ``` if kl.enableControllerAttachDetach { if node.Annotations == nil { node.Annotations = make(map[string]string) } ... } ``` if set --enable-controller-attach-detach to false, map will have no Initialized. **Release note**: ```release-note NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 55050, 53464, 54936, 55028, 54928). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. kubelet: dockershim: remove orphaned checkpoint files Fixes https://github.com/kubernetes/kubernetes/issues/55070 Currently, `ListPodSandbox()` returns a combined list of sandboxes populated from both the runtime and the dockershim checkpoint files. However the sandboxes in the checkpoint files might not exist anymore. The kubelet sees the sandbox returned by `ListPodSandbox()` and determines it shouldn't be running and calls `StopPodSandbox()` on it. This generates an error when `StopContainer()` is called as the container does not exist. However the checkpoint file is not cleaned up. This leads to subsequent calls to `StopPodSandbox()` that fail in the same way each time. This PR removes the checkpoint file if StopContainer fails due to container not found. The only other place `RemoveCheckpoint()` is called, except if it is corrupt, is from `RemoveSandbox()`. If the container does not exist, what `RemoveSandbox()` would have done has been effectively been done already. So this is just clean up. @derekwaynecarr @eparis @freehan @dcbw
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 55050, 53464, 54936, 55028, 54928). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Update deletionTimestamp with information about finalizer effect **What this PR does / why we need it**: ref https://github.com/kubernetes/community/pull/1177 **Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:Fixes # https://github.com/kubernetes/website/issues/6128 **Special notes for your reviewer**: **Release note**: ```release-note NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 55050, 53464, 54936, 55028, 54928). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. output empty creationTimestamp as null **Release note** ```release-note NONE ``` Updates the value of the `creationTimestamp` field to be `null` when empty, to keep parity between it and `deletionTimestamp`. Adds a round-trip test to ensure that unstructured objects containing empty metadata fields are able to be re-converted back into internal or external objects. Prior to the proposed patch in this PR, an unstructured object whose `.metadata.creationTimestamp` value had been set through the metadata accessor to an empty value (`metav1.Time{}` in this case), was unable to be re-converted to an internal or external type using the runtime decoder. Conversion would fail with the error: ``` unstructured_test.go:177: FromUnstructured failed: parsing time "" as "2006-01-02T15:04:05Z07:00": cannot parse "" as "2006" ``` cc @liggitt @fabianofranz
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Remove version check in kubectl e2e test. **What this PR does / why we need it**: We don't need to check these versions for kubectl e2e tests in current cycle. **Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*: ref: #55053 **Special notes for your reviewer**: /cc @liggitt since you're also from sig-cli-maintainers :) **Release note**: ```release-note NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 51401, 54056, 54977, 55017, 55052). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. refactor admission handler and add UT **What this PR does / why we need it**: remove Redundant code, and add UT case. **Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*: Fixes # **Special notes for your reviewer**: **Release note**: ```release-note NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 51401, 54056, 54977, 55017, 55052). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. extensions: remove TPR remnants The extensions group still had the TPR types + generated client. Having this in the codebase doesn't create any problems but would be good to clean up, especially since TPR access has been removed in 1.8. **Release note**: ```release-note NONE ``` /assign @sttts @deads2k
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 51401, 54056, 54977, 55017, 55052). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. [test/e2e_node]Redirect dl.k8s.io to the kubernetes-release GCS bucket **What this PR does / why we need it**: fixes [#33726](https://github.com/kubernetes/kubernetes/pull/33726)Redirect dl.k8s.io to the kubernetes-release GCS bucket ref :[kubernetes/k8s.io#15](https://github.com/kubernetes/k8s.io/pull/15) **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**: ``` NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 51401, 54056, 54977, 55017, 55052). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Fix example and comments for kubectl convert Signed-off-by:
yuexiao-wang <wang.yuexiao@zte.com.cn> **What this PR does / why we need it**: There are some erros in example and comments in convert **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes # **Release note**: ```release-note NONE ```
-