- 21 Dec, 2017 12 commits
-
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 57292, 56274, 57435, 57438, 57429). 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 a typo Just a typo, `timemoutseconds` should be `timeoutSeconds`.
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 57292, 56274, 57435, 57438, 57429). 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>. enable podpreset by default in local up cluster **What this PR does / why we need it**: This PR enables PodPreset in Admission control and also for that to work on the apiserver level enalbes the API group settings.k8s.io/v1alpha1. **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>. Add pkg/util/ipvs OWNERS file **What this PR does / why we need it**: This PR adds `pkg/util/ipvs/OWNERS` file, including two people: @thockin, @m1093782566(me). I created `pkg/util/ipvs` package for wrapping netlink IPVS call, which is used by IPVS proxier. Because reviewing this package needs some IPVS background knowledge, I create this OWNERS file for efficient code review. And, I am willing to share code review burden of this util package :) **Which issue(s) this PR fixes**: Fixes #57361 **Special notes for your reviewer**: **Release note**: ```release-note NONE ``` /sig network
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 57215, 57489). 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>. Print/return the text from a number of errors that were silent before. @rootfs @andyzhangx @khendiak @feiskyer
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 57215, 57489). 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 scheduler benchmark tests for PodAntiAffinity rules **What this PR does / why we need it**: This adds a benchmark test that measures the scheduling latency of pods with PodAntiAffinity rules where topologyKey=kubernetes.io/hostname. Additionally, this PR imposes a minimum `b.N` value to make the test results more meaningful. The benchmark is needed to validate any performance improvements made for #54189. Command used to invoke: ```sh make test-integration WHAT=./test/integration/scheduler_perf KUBE_TEST_ARGS="-run=xxxx -bench=." ``` Sample output: ``` pkg: k8s.io/kubernetes/test/integration/scheduler_perf BenchmarkScheduling/100Nodes/0Pods-40 100 36316838 ns/op BenchmarkScheduling/100Nodes/1000Pods-40 100 69460001 ns/op BenchmarkScheduling/1000Nodes/0Pods-40 100 193755560 ns/op BenchmarkScheduling/1000Nodes/1000Pods-40 100 343451472 ns/op BenchmarkSchedulingAntiAffinity/500Nodes/250Pods-40 250 127236420 ns/op BenchmarkSchedulingAntiAffinity/500Nodes/5000Pods-40 250 478661925 ns/op PASS ok k8s.io/kubernetes/test/integration/scheduler_perf 354.008s ``` **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 57434, 57221, 57417, 57474, 57481). 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 nfsprovisioner image to v1.0.9 to fix annotation race with pv controller fix https://github.com/kubernetes/kubernetes/issues/55780 . Summary: the provisioner must wait for the pv controller's annotation to be set as described by the 'protocol' doc, fail to wait and the pv controller may not be ready. The relevant change to the image was https://github.com/kubernetes-incubator/external-storage/pull/521 ```release-note NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 57434, 57221, 57417, 57474, 57481). 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>. Cleanup api service before namespace deletion. Fixes https://github.com/kubernetes/kubernetes/issues/57486. https://github.com/kubernetes/kubernetes/pull/57254 helps but is not enough for the fix. After https://github.com/kubernetes/kubernetes/pull/57254, I saw another failure https://k8s-gubernator.appspot.com/build/kubernetes-jenkins/logs/ci-cri-containerd-e2e-ubuntu-gce/1057. Before, all test after Aggregator test would fail. Now all tests before/after Aggregator tests are passing. However, during the Aggregator test waiting for namespace cleanup, all the other tests are failing. Aggregator test takes 10min to wait for namespace cleanup. The reason is that, `BeforeEach`s run in LIFO order, `AfterEach`s run in FIFO order. `NewDefaultFramework` inserts an `AfterEach` function to do namespace cleanup. In the current code, the namespace cleanup `AfterEach` is inserted by `NewDefaultFramework` before `cleanupTest`, it means that the test will try to delete namespace first, and then cleanup the api service. However, as known to us, if api service is not cleaned up, namespace deletion will wait forever. In this PR, we reverse the `AfterEach` order. A better solution is to put `BeforeEach` and `AfterEach` in an internal `Describe`. However, since we don't have an internal `Describe` in this test, I just reverse the `AfterEach` order directly. /cc @roycaihw @cheftako @kubernetes/sig-api-machinery-bugs **Release note**: ```release-note none ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 57434, 57221, 57417, 57474, 57481). 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>. fabiano no longer a thing **What this PR does / why we need it**: Removes me from lists of reviewers and approvers. **Release note**: ```release-note NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 57434, 57221, 57417, 57474, 57481). 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>. Send an event just before the Kubelet restarts to use a new config **What this PR does / why we need it**: This PR makes the Kubelet send events for configuration changes. This makes it much easier to see a recent history of configuration changes for the 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 #56895 **Special notes for your reviewer**: **Release note**: ```release-note NONE ``` /cc @dchen1107 @liggitt @dashpole
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 57434, 57221, 57417, 57474, 57481). 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>. Node e2e test: do not return error if Docker's check-config.sh fails **What this PR does / why we need it**: This PR fixes the issue in https://k8s-gubernator.appspot.com/build/kubernetes-jenkins/logs/ci-kubernetes-e2enode-ubuntudev2-k8sbeta-gkespec/1#k8sio-gke-system-requirements-conformance-featuregkeenv-the-docker-configuration-validation-should-pass https://github.com/moby/moby/pull/28007 changed the Docker's check-config.sh to return a non-zero exit code if the check fails. But we expect certain configs to be missing, so we should ignore the exit code. This needs to be cherry-picked into 1.9. **Release note**: ```release-note None ``` /area node-e2e /kind bug /assign @dchen1107
-
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>. outdent err block **What this PR does / why we need it**: **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**: ```NONE ```
-
Brendan Burns authored
-
- 20 Dec, 2017 28 commits
-
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 57349, 57368). 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 '/version/' to the system:discovery role, since that's what the spec says. See: https://github.com/kubernetes-client/java/issues/153 ```release-note Add the path '/version/' to the `system:discovery` cluster role. ```
-
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 watch to requirements for quota-able resources Since we use informers, if you can't be watched, you can't be quota'ed.
-
Matthew Wong authored
-
Jonathan Basseri authored
-
Lantao Liu 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>. Update code-of-conduct.md Refer to kubernetes/community as authoritative source for code of conduct ref: kubernetes/community#1527
-
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 problem accessing private docker registries **What this PR does / why we need it**: In 027c8b9e, we added code to move from .dockercfg to config.json file. But we forgot to use the right secret type and the key to store the base64'ed creds **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 #57427 #57273 **Special notes for your reviewer**: **Release note**: ```release-note Fixes issue creating docker secrets with kubectl 1.9 for accessing docker private registries. ```
-
Michael Taufen authored
-
Jonathan Basseri authored
This adds new benchmark tests that measure scheduler latency of pods that use affinity rules. Specifically, this tests affinity rules with topologyKey="kubernetes.io/hostname".
-
Jonathan Basseri authored
Under the default behavior of Go benchmark tests, all our scheduler_perf benchmark tests run with b.N=1, which is lower than we would like. This adds a lower bound to b.N so that the results are more meaningful. The alternative to this change is to always run these tests with the -benchtime flag set to a duration which will force b.N to increase. That would cause any test setup to be executed repeatedly as b.N ramps up, and -timout would probably also need to be set higher.
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 54278, 56259, 56762). 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>. New release for pause container **What this PR does / why we need it**: This PR bumps the TAG for the pause container so that we can build a new release with affecting older Kubernetes releases. Once the new release has been pushed to gcr.io/google_containers, I will follow up with a PR to update the kubelet/runtimes to use the new container. In addition to bumping the tag, this PR also: * Updates the pause Makefile to reflect a name change in the arm compiler used in kube-cross * Adds a very simple version string to pause.c to help troubleshoot issues like #50865 in the future. **Which issue(s) this PR fixes**: First of two PRs for #56253, #50865 **Special notes for your reviewer**: **Release note**: ```release-note NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 54278, 56259, 56762). 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 NODE_LOCAL_SSDS_EXT to config-test **What this PR does / why we need it**: Add NODE_LOCAL_SSDS_EXT to config-test so we can specify it for CI. **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 #57468 **Special notes for your reviewer**: **Release note**: ```release-note NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 54278, 56259, 56762). 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>. make sure of deleting archive **What this PR does / why we need it**: Exit() causes the current program to exit with the given status code, but deferred function does not run.
-
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>. Typo **What this PR does / why we need it**: Fix typo in "Kuberntes" ```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>. Evicted pods should not be marked as failing since they respawn **What this PR does / why we need it**: Juju deployments should not report evicted pods as failing. **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**: ```NONE ```
-
Brendan Burns authored
-
Brendan Burns authored
-
Aaron Crickenberger authored
Refer to kubernetes/community as authoritative source for code of conduct
-
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>. Renamed func name according to TODO. ```release-note None ```
-
Davanum Srinivas authored
In 027c8b9e, we added code to move from .dockercfg to config.json file. But we forgot to use the right secret type and the key to store the base64'ed creds
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 57252, 57168). 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>. Implementing predicates ordering **What this PR does / why we need it**: implements predicates ordering for the scheduler **Which issue(s) this PR fixes** : Fixes #53812 **Special notes for your reviewer**: @bsalamat @gmarek @resouer as discussed on slack, to implement ordering we have to choices: - use a layered approach with a list that indexes the order of the predicates map - change the underlying data structure used to represent a collection of predicates (a map in our case) into a list of predicates objects. Going with this solution might be "cleaner" but it will require a lot of changes and will increase the cost for accessing predicates from O(1) to O(n) (n being the number of predicates used by the scheduler). we might go with this solution for now. If the number of predicates start growing, we might switch to the second option. **Release note**: ```release-note adding predicates ordering for the kubernetes scheduler. ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 57252, 57168). 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>. Reduce CPU and memory requests for Metrics Server Nanny **What this PR does / why we need it**: Metrics Server Nanny is a sidecar container that performs small computations every 5 minutes to scale Metrics Server resource requirements when cluster size changes. This change reduces the CPU and memory requests to free up unused resource. **Release note**: ```release-note Free up CPU and memory requested but unused by Metrics Server Pod Nanny. ```
-
Konstantinos Tsakalozos authored
-
Konstantinos Tsakalozos authored
-
Konstantinos Tsakalozos authored
-
Konstantinos Tsakalozos authored
-
Yassine TIJANI authored
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 56681, 57384). 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>. Deprecate the alpha Accelerators feature gate. Encourage people to use DevicePlugins instead. /kind cleanup Related to https://github.com/kubernetes/features/issues/192 and https://github.com/kubernetes/features/issues/368 **Release note**: ```release-note The alpha Accelerators feature gate is deprecated and will be removed in v1.11. Please use device plugins instead. They can be enabled using the DevicePlugins feature gate. ``` /sig node /sig scheduling /area hw-accelerators
-