- 12 Jul, 2017 40 commits
-
-
Jun Xiang Tee authored
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 46738, 48827, 48831) Add test for kubectl resource filter. This should prevent regression of the bug fixed in #48786.
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 46738, 48827, 48831) Moving disruption controller e2es to workload/ Based on #45301 Moving to track sig-apps in a single directory cc @kubernetes/sig-contributor-experience-misc @kubernetes/sig-apps-misc @erictune @kow3ns @crimsonfaith91
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue StatefulSet upgrade test - replicated database (mysql) **What this PR does / why we need it**: Adds a new upgrade test. The test creates a statefulset with a replicated mysql database. It populates the database and then continually polls the data while performing an upgrade. Ultimately, this PR increases confidence of reliability during upgrades. It helps show that StatefulSets and Pod Disruption Budgets are doing what they're supposed to. Code to pay attention to this was added for #38336. Also vendors in a golang mysql client driver, for use in the test. **Release note**: ```release-note NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 48781, 48817, 48830, 48829, 48053) kubeadm: Start to remove old envparams **What this PR does / why we need it**: Historically, the envparams feature was used as a way to easily debug `kubeadm` while developing it in the v1.3-v1.4 timeframe. Since then some parameters have been hanging around and not being moved into the API as they should have. Note: This is a temporary step; moving things into the API. Still, the API is gonna change, this is not the end state. But this is better than keeping the envparams. I'm gonna deal with `KubernetesDir` in the next PR. **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes # Fixes: kubernetes/kubeadm#326 **Special notes for your reviewer**: **Release note**: ```release-note NONE ``` @timothysc @pipejakob
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 48781, 48817, 48830, 48829, 48053) vSphere for cloud-controller-manager **What this PR does / why we need it**: This is to implement the `NodeAddressesByProviderID` and `InstanceTypeByProviderID` methods for cloud-controller-manager for vSphere cloud provider. Currently vSphere cloud provider only supports VMs in the same folder. Thus `NodeAddressesByProviderID` is similar to `NodeAddresses` with a simple ProviderID to NodeName translation. `InstanceTypeByProviderID` returns nil as same as `InstanceType`. **Which issue this PR fixes** Part of Issue https://github.com/kubernetes/kubernetes/issues/47257 **Release note**: ```NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 48781, 48817, 48830, 48829, 48053) Fix yaml-quote typo Caught this looking through CI logs. /assign wojtek-t
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 48781, 48817, 48830, 48829, 48053) Move kubectl e2e tests to their own directory and prefix the test nam… ```release-note NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue Make storage e2e tests start with [sig-storage] instead of [k8s.io]. This makes understanding sig ownership from a test name very easy for tools and humans. - Use a SIGDescribe helper function that adds a [sig-storage] prefix instead of [k8s.io] for tests in storage/ - Move a test that should be in storage into storage. - Make tests owned by multiple SIGs (configmap test) have [sig-storage] instead of [Volume] labels. This means that all tests that sig-storage directly owns can be found with a simple regex. /cc @kubernetes/sig-storage-pr-reviews **What this PR does / why we need it**: This will be used to make a testgrid dashboard for sig-storage. **Release note**: ```release-note NONE ``` Issue #48779
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 48594, 47042, 48801, 48641, 48243) Add initial support for the Azure instance metadata service. Part of fixing #46632 @colemickens @rootfs @jdumars @kris-nova
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 48594, 47042, 48801, 48641, 48243) Prepare to introduce websockets for exec and portforward Refactor the code in remotecommand to better represent the structure of what is common between portforward and exec. Ref #48633
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 48594, 47042, 48801, 48641, 48243) Fix panic of DeleteRoute() Fix #48800 It should be 'addr_pairs', not 'routes'. **Release note**: ```release-note NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 48594, 47042, 48801, 48641, 48243) Validated expected event numbers for damoncontroller test. **What this PR does / why we need it**: **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #46629 **Special notes for your reviewer**: **Release note**: ```release-note none ```
-
Anirudh authored
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue Add node-name flag to `init` phase **What this PR does / why we need it**: Allow to specify a node-name instead of relaying in `os.Hostname()` This is useful where kubelet use the name given by the cloud-provider to register the node. **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: Partially fix: kubernetes/kubeadm#64 **Release note**: ```release-note Added new flag to `kubeadm init`: --node-name, that lets you specify the name of the Node object that will be created ```
-
Lucas Käldström authored
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue Remove dead code for OpenStack provider **Release note**: ```release-note NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 48279, 48566, 48319, 48794, 47952) Added pod evictors for new zone. **What this PR does / why we need it**: Added pod evictors for new zone which was added manually. **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #47791 **Release note**: ```release-note-none ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 48279, 48566, 48319, 48794, 47952) Removed mesos flags from known-flags.txt. **What this PR does / why we need it**: **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 (batch tested with PRs 48279, 48566, 48319, 48794, 47952) Helper methods dealing with ControllerRef **What this PR does / why we need it**: Adds helper methods for working with **controllerRef** (controller's `OwnerReference`). It is based on the existing code from Kubernetes plus extracting some common logic: - `NewControllerRef` is based on examples from [daemon controller](https://github.com/kubernetes/kubernetes/blob/master/pkg/controller/daemon/daemoncontroller.go#L1223), [deployment controller](https://github.com/kubernetes/kubernetes/blob/master/pkg/controller/deployment/sync.go#L649), [job controller](https://github.com/kubernetes/kubernetes/blob/master/pkg/controller/job/utils.go#L34) - `GetControllerOf` is copied from [controller_ref_manager.go](https://github.com/kubernetes/kubernetes/blob/master/pkg/controller/controller_ref_manager.go#L36) - `IsControlledBy` is a common logic extracted from resource controllers: [deployment_util.go#L568](https://github.com/kubernetes/kubernetes/blob/master/pkg/controller/deployment/util/deployment_util.go#L568), [history.go#L276](https://github.com/kubernetes/kubernetes/blob/master/pkg/kubectl/history.go#L276) and many others. It will also be useful for writing custom resource controllers, for example [service-catalog#979](https://github.com/kubernetes-incubator/service-catalog/pull/979)
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 48279, 48566, 48319, 48794, 47952) Add prometheus plugin on fluentd image. **What this PR does / why we need it**: This PR adds the prometheus plugin on Fluentd. **Special notes for your reviewer**: The plugin used was: https://github.com/kazegusuri/fluent-plugin-prometheus, on the latest stable version. All configs used are default. **Release note**: ```release-note Fluentd-es addon now exposes a /metrics endpoint for monitoring on port 24231. ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue Use Container-optimzed OS images for nodes by default Part of the deprecation of the debian-based ContainerVM images. ```release-note kube-up and kubemark will default to using cos (GCI) images for nodes. The previous default was container-vm (CVM, "debian"), which is deprecated. If you need to explicitly use container-vm for some reason, you should set KUBE_NODE_OS_DISTRIBUTION=debian ```
-
Anthony Yeh authored
-
Christian Bell authored
-
Miao Luo authored
Implement NodeAddressesByProviderID and InstanceTypeByProviderID for vsphere cloud provider.
-
Ryan Hitchman authored
-
Ryan Hitchman authored
-
Ryan Hitchman authored
-
Ryan Hitchman authored
This makes understanding sig ownership from a test name very easy for tools and humans.
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 48425, 41680, 48457, 48619, 48635) proxy/userspace: suppress "LoadBalancerRR: Removing endpoints" message Don't print it when there aren't any endpoints getting removed. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1468420 Fixes: https://github.com/kubernetes/kubernetes/issues/48816 ```release-note NONE ``` @eparis @thockin @kubernetes/rh-networking
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 48425, 41680, 48457, 48619, 48635) [Federation] Remove redundant e2e Now that federation of replicasets and deployments is implemented with the sync controller, the previous crud e2e duplicates coverage provided by the crudtester integration and e2e testing. cc: @kubernetes/sig-federation-pr-reviews
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 48425, 41680, 48457, 48619, 48635) "rbd: image xxx is locked by other nodes" is misleading **What this PR does / why we need it**: For RWO PV, RBD plugin tries to fence it first, but there are many situations which may cause lock to fail, e.g. - userSecret is incorrect - monitor addresses are incorrect or node cannot access ceph cluster temporarily - image is locked by other nodes - maybe more... So, original "image xxx is locked by other nodes" is incorrect in some cases and misleading in diagnosis. This PR change the error to be correct and not misleading first. We may add detail error descriptions later. **Special notes for your reviewer**: New FailedMount event example if RBD plugin cannot lock image: ``` ... FailedMount MountVolume.SetUp failed for volume "pvc-ee37a9c8-608e-11e7-b3a7-000c291fbe71" : rbd: failed to lock image kubernetes-dynamic-pvc-ee3b9911-608e-11e7-97b6-000c291fbe71 (maybe locked by other nodes), error exit status 22 ``` **Release note**: ```release-note NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 48425, 41680, 48457, 48619, 48635) Improved code coverage for pkg/kubelet/types/pod_update The test coverage for pod_update.go was imprved from 36% to 100%. **What this PR does / why we need it**: This fixed part of #40780 **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 Pass cluster name to Heapster with Stackdriver sink. **What this PR does / why we need it**: Passes cluster name as argument to Heapster when it's used with Stackdriver sink to allow setting resource label 'cluster_name' in exported metrics. **Release note**: ```release-note NONE ```
-
Phillip Wittrock authored
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 48672, 47140, 48709, 48786, 48757) Fix flaky test Test_Run_OneVolumeAttachAndDetachMultipleNodesWithReadWriteMany Only relying on the NewAttacher/Detacher call counts is not enough as they happen in parallel to the testing/verification code and thus the actual attaching/detaching may not be done yet, resulting in flaky test results. Fixes #46244
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 48672, 47140, 48709, 48786, 48757) Correctly filter terminated pods in kubectl We shouldn't use `Status.Reason` to determine whether the pod has terminated or not.
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 48672, 47140, 48709, 48786, 48757) glusterfs: retry without auto_unmount only when it's not supported GlusterFS volume plugin should not blindly retry all failed mounts without auto_unmount, it should retry them only when we are sure that auto_unmount was the reason the first attempt failed. @kubernetes/sig-storage-pr-reviews /assign @humblec ```release-note NONE ```
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 48672, 47140, 48709, 48786, 48757) Update Azure SDK. Updates to the Azure SDK to pick up new APIs.
-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 48672, 47140, 48709, 48786, 48757) use built-in path separator instead of hard coded **What this PR does / why we need it**: We should use built-in path separator to avoid hard coded strings. **Which issue this PR fixes** : **Special notes for your reviewer**: **Release note**: ```release-note None ```
-