- 15 May, 2017 28 commits
-
-
Anthony Yeh authored
CHANGELOG: Add Known Issues for v1.6.3
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue Fix the outdated link to the Helm Charts repo **What this PR does / why we need it**: It is only a link fix, the repo URL has changed **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 ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue docs:examples:guestbook: don't advice untagged image Using untagged images means no version at all, this can cause weird problems. Right now, `redis:latest` is no longer compatible with the redis version used in the slave service of the example.
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 45807, 45814) Fix invalid bash script in unit test. This change fix such error when we run "make test": "find: invalid expression; you have used a binary operator '-o' with nothing before it." @deads2k **Release note**: ```release-note ``` -
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 45807, 45814) Fix typo.
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 45070, 45821, 45732, 45494, 45789) Rename vars scheduledJob to cronJob **What this PR does / why we need it**: Rename vars scheduledJob to cronJob **Special notes for your reviewer**: refer to #45480 **Release note**: ```release-note NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 45070, 45821, 45732, 45494, 45789) Fix lint errors in juju kubernetes master and e2e charms **What this PR does / why we need it**: Fixes style error in the Juju charms **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**: ``` Code style fixes in Juju charms ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 45070, 45821, 45732, 45494, 45789) apiextesions-server integration tests: test multiple registration **What this PR does / why we need it**: Add integration test for Multiple registrations XREF: #45511 **Special notes for your reviewer**: @deads2k my first plan was to submit one single PR for both tests but I'm facing one issue with resources deregistration. Get in touch via slack
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 45070, 45821, 45732, 45494, 45789) Minor fix in run-gcloud-compute-with-retries output piping To work with older versions of bash. cc @wojtek-t @gmarek
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue Uses container/heap for DelayingQueue The current implementation of DelayingQueue doesn't perform very well when a large number of items (at random delays) are inserted. The original authors seemed to be aware of this and noted it in a `TODO` comment. This is my attempt at switching the implementation to use a priority queue based on `container/heap`. Benchmarks from before the change: ``` ╰─ go test -bench=. -benchmem | tee /tmp/before.txt BenchmarkDelayingQueue_AddAfter-8 300000 256824 ns/op 520 B/op 3 allocs/op PASS ok k8s.io/kubernetes/staging/src/k8s.io/client-go/util/workqueue 77.237s ``` After: ``` ╰─ go test -bench=. -benchmem | tee /tmp/after.txt BenchmarkDelayingQueue_AddAfter-8 500000 3519 ns/op 406 B/op 4 allocs/op PASS ok k8s.io/kubernetes/staging/src/k8s.io/client-go/util/workqueue 2.969s ``` Comparison: ``` ╰─ benchcmp /tmp/before.txt /tmp/after.txt benchmark old ns/op new ns/op delta BenchmarkDelayingQueue_AddAfter-8 256824 3519 -98.63% benchmark old allocs new allocs delta BenchmarkDelayingQueue_AddAfter-8 3 4 +33.33% benchmark old bytes new bytes delta BenchmarkDelayingQueue_AddAfter-8 520 406 -21.92% ``` I also find the `container/heap`-based code a bit more easy to understand. The implementation of the PriorityQueue is based on the documentation for `container/heap`. Feedback definitely welcomed. This is one of my first contributions. ```release-note NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue kube-apiextensions-server: Fix potential SEGV with null delegate handler **What this PR does / why we need it**: In the kube-apiextensions-server there is a fallback value for `null` delegate to `http.NotFoundHandler()` in handling group and versions discovery, but no fallback for custom resources endpoint. It leads to SEGV when running with `genericapiserver.EmptyDelegate`.
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue Fix discovery version for autoscaling to be v1 The order of the storage setup blocks in the setup for the autoscaling API group was accidentally inverted, meaning that if the v2alpha1 API group was turned on, it would be set to the preferred API group-version for discovery. This was unintentional; the latest stable version should (v1) should be preferred instead. **Release note**: ```release-note Ensure that autoscaling/v1 is the preferred version for API discovery when autoscaling/v2alpha1 is enabled. ```
-
Salvatore Dario Minonne authored
-
Shyam Jeedigunta authored
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue [Federation] Fix federated service reconcilation issue due to addition of External… …TrafficPolicy field to v1.Service **What this PR does / why we need it**: New fields (ExternalTrafficPolicy) are introduced to v1.Service by this PR #41162. If this field is not specified in service spec, the service controller will assign default and updates the service spec. In federation, the service spec is not updated and we continuously try to reconcile as the federated service and the service in federated cluster do not match. **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #45795 **Special notes for your reviewer**: **Release note**: ``` NONE ``` cc @kubernetes/sig-federation-bugs @madhusudancs
-
Cao Shufeng authored
This change fix such error when we run "make test": "find: invalid expression; you have used a binary operator '-o' with nothing before it." -
Kubernetes Submit Queue authored
Automatic merge from submit-queue util/iptables: fix cross-build failures due to syscall.Flock() Fixes: https://github.com/kubernetes/kubernetes/issues/45554 Related: https://github.com/kubernetes/kubernetes/pull/44895 @luxas @k82cn @danwinship
-
Xing Zhou authored
Fixed typo.
-
shashidharatd authored
Fix federated service reconcilation issue due to addition of ExternalTrafficPolicy field to v1.Service
-
Dan Williams authored
Fixes: https://github.com/kubernetes/kubernetes/issues/45554
-
Nail Islamov authored
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue use StringArrayVar for ResourceName ref to https://github.com/kubernetes/kubernetes/pull/43903 ```release-note `kubectl create role` and `kubectl create clusterrole` no longer allow specifying multiple resource names as comma-separated arguments. Use repeated `--resource-name` arguments to specify multiple resource names. ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue Adding missing verify-*.sh scripts for autogenerated federation docs Fixes https://github.com/kubernetes/kubernetes/issues/41866
-
nikhiljindal authored
-
nikhiljindal authored
-
nikhiljindal authored
-
nikhiljindal authored
-
nikhiljindal authored
-
- 14 May, 2017 4 commits
-
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue Add plural name for CustomResources example **What this PR does / why we need it**: Adds plural name for CustomResources example. There is no field called `name` for CustomResources. Instead it is `plural`. This will also fail validation (https://github.com/kubernetes/kubernetes/blob/master/staging/src/k8s.io/kube-apiextensions-server/pkg/apis/apiextensions/validation/validation.go#L33) while executing the instructions mentioned here: https://github.com/kubernetes/kubernetes/issues/45510. **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: **Special notes for your reviewer**: **Release note**: ``` NONE ``` @sttts
-
Nikhita Raghunath authored
-
zhangxiaoyu-zidif authored
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue Add support for continuous testing of GPU node e2es in GCP. This PR adds support for testing Nvidia GPUs on Ubuntu 14.04 base image.
-
- 13 May, 2017 8 commits
-
-
Vishnu kannan authored
Signed-off-by:Vishnu kannan <vishnuk@google.com>
-
Andy Lindeman authored
-
Rober Morales-Chaparro authored
-
Andy Lindeman authored
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue Add LogPath for container status in CRI In order to get log path of container from CRI runtime, we need to add `LogPath` to `ContainerStatus` in CRI. @Random-Liu @feiskyer
-
heartlock authored
Signed-off-by:heartlock <21521209@zju.edu.cn>
-
heartlock authored
Signed-off-by:heartlock <21521209@zju.edu.cn>
-
Anthony Yeh authored
-