1. 03 Nov, 2017 22 commits
    • Nikhita Raghunath's avatar
      Remove TPR remnants · 3b0b95ec
      Nikhita Raghunath authored
      There are still TPR types and generated client
      in the extensions group. It is better to clean
      that up, now that it has been removed from master.
      3b0b95ec
    • Kubernetes Submit Queue's avatar
      Merge pull request #55041 from wackxu/remdup · 12752f31
      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 duplicate healthz check register
      
      **What this PR does / why we need it**:
      
      We registe healthz check twice, another is 
      
      https://github.com/kubernetes/kubernetes/blob/master/cmd/kube-controller-manager/app/controllermanager.go#L203
      
      we should remove the duplicate one
      
      **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
      ```
      12752f31
    • Kubernetes Submit Queue's avatar
      Merge pull request #55054 from krzysztof-jastrzebski/e2e · db177095
      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 scale down to 1 e2e test.
      
      This PR adds test checking scale to 1 functionality.
      db177095
    • Kubernetes Submit Queue's avatar
      Merge pull request #55000 from deads2k/admission-10-split · 6fe3a4d8
      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>.
      
      make easy validation admission plugins into validators
      
      This switches "easy" admission plugins that need to be validators to be validators.  I also did one simple plugin to do both mutation and validation for practice.
      
      @kubernetes/sig-api-machinery-pr-reviews @caesarxuchao
      6fe3a4d8
    • David Eads's avatar
      75c448db
    • Krzysztof Jastrzebski's avatar
      Add scale down to 1 e2e test. · 7a5e9582
      Krzysztof Jastrzebski authored
      7a5e9582
    • Kubernetes Submit Queue's avatar
      Merge pull request #54991 from krzysztof-jastrzebski/master · aa66d8cb
      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>.
      
      Node autoprovisioning e2e test.
      
      This PR adds test scenario for cluster-autoscaler in GKE  for node autoprovisioning.
      aa66d8cb
    • Kubernetes Submit Queue's avatar
      Merge pull request #54956 from vladimirvivien/scaleio-remove-drv_cfg-dep · 6af21e8e
      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 dependency on drv_cfg binary for querying ScaleIO devices
      
      **What this PR does / why we need it**:
      This PR fixes the issue where a ScleIO kubernetes plugin required additional binary `drv_cfg` to be present on the local node to work properly, making it harder for the kubelet binary to be containerized without providing access to local paths inside the container.
      
      **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 #54954 
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      ScaleIO driver completely removes dependency on drv_cfg binary so a Kubernetes cluster can easily run a containerized kubelet.
      ```
      6af21e8e
    • Kubernetes Submit Queue's avatar
      Merge pull request #54848 from xiangpengzhao/add-ipvsscheduler-testcase · 676e28ff
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 54906, 54120, 54934, 54915, 54848). 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>.
      
      Improve unit test coverage for kube-proxy configuration.
      
      **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**:
      /cc @ncdc 
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      676e28ff
    • Kubernetes Submit Queue's avatar
      Merge pull request #54915 from xiangpengzhao/np-ut-cov · eacca001
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 54906, 54120, 54934, 54915, 54848). 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>.
      
      Improve unit test coverage for network policy validation.
      
      **What this PR does / why we need it**:
      `ok  	k8s.io/kubernetes/pkg/apis/networking/validation	0.074s	coverage: 87.8% of statements`
      
      to 
      
      `ok  	k8s.io/kubernetes/pkg/apis/networking/validation	0.082s	coverage: 100.0% of statements`
      
      **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
      ```
      eacca001
    • Kubernetes Submit Queue's avatar
      Merge pull request #54934 from akosiaris/master · 9ca2bda5
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 54906, 54120, 54934, 54915, 54848). 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>.
      
      Only parse ClusterCIDR, ServiceCIDR if AllocateNodeCIDRs
      
      **What this PR does / why we need it**:
      
      Avoid unnecessary spam in kube-controller-manager log if --cluster-cidr is not specified and --allocate-node-cidrs is false. Add clarification in kube-controller-manager help about that.
      
      **Release note**
      ```release-note
      Avoid unnecessary spam in kube-controller-manager log if --cluster-cidr is not specified and --allocate-node-cidrs is false.
      ```
      9ca2bda5
    • Kubernetes Submit Queue's avatar
      Merge pull request #54120 from m1093782566/fix-proxy-mode · b006bc5d
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 54906, 54120, 54934, 54915, 54848). 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 kube-proxy fall back to iptables firstly when unable to load kernel modules
      
      **What this PR does / why we need it**:
      
      Fix kube-proxy fall back to iptables firstly when unable to load kernel modules
      
      **Which issue this PR fixes**: 
      
      fixes #54121
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      
      /kind bug
      /sig network
      /area kube-proxy
      b006bc5d
    • Kubernetes Submit Queue's avatar
      Merge pull request #54906 from xiangpengzhao/sc-beta-v1 · 63cc6006
      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 storageclass version to v1 in examples
      
      **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**:
      /cc @jsafrane 
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      63cc6006
    • Kubernetes Submit Queue's avatar
      Merge pull request #54972 from m1093782566/ipvs-ipv6 · e19dbba8
      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>.
      
      clean up legacy ipv4/32 in ipvs proxy
      
      **What this PR does / why we need it**:
      
      clean up legacy ipv4/32 in ipvs proxy
      
      **Which issue(s) this PR fixes**: closes #51866
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      /sig network
      
      /area ipv6
      
      /assign @thockin 
      
      cc @danehans
      e19dbba8
    • Kubernetes Submit Queue's avatar
      Merge pull request #54976 from m1093782566/ipvs-kubeadm · 1fc64162
      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 kubeadm way in ipvs proxy README
      
      **What this PR does / why we need it**:
      
      As #53962 which support specify proxy mode for kubeadm is already in, we should add ipvs proxy kubeadm way in README.md.
      
      **Which issue(s) this PR fixes**:
      Fixes #54978
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      1fc64162
    • wackxu's avatar
      remove duplicate healthz check register · 22a706ed
      wackxu authored
      22a706ed
    • Kubernetes Submit Queue's avatar
      Merge pull request #55020 from derekwaynecarr/fix-spam · 0575f728
      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>.
      
      StopPodSandbox should not log when container is already removed
      
      **What this PR does / why we need it**:
      StopPodSandbox should not log when a container is already gone.  It should only log if it could not stop and the container was still present.
      
      Fixes https://github.com/kubernetes/kubernetes/issues/55021
      
      **Special notes for your reviewer**:
      This was seen in our production logs, need to eliminate spam.
      
      **Release note**:
      ```release-note
      NONE
      ```
      0575f728
    • m1093782566's avatar
      fix kube-proxy mode · ea930dbe
      m1093782566 authored
      ea930dbe
    • Kubernetes Submit Queue's avatar
      Merge pull request #54996 from mwielgus/metadata-proxy · 63c40972
      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>.
      
      Reduce metadata-proxy cpu requests to 30m
      
      After the recent change enabling metadata-proxy in tests (https://github.com/kubernetes/kubernetes/pull/54150) we started seeing problems with scheduling cluster autoscaler on master. Metadata-proxy eats all of the available space leaving nothing for CA to run on. 
      
      This PR reduces the cpu requests for metadata-proxy allowing other components to fit in.
      
      cc: @kubernetes/sig-autoscaling-bugs
      63c40972
    • Kubernetes Submit Queue's avatar
      Merge pull request #54964 from Random-Liu/add-containerd-e2e · 7a28aaff
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 54488, 54838, 54964). 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 support to for alternative container runtime in `kube-up.sh`
      
      For https://github.com/kubernetes/features/issues/286.
      
      This PR added 4 new environment variables in `kube-up.sh` to support alternative container runtime:
      1) `KUBE_MASTER_EXTRA_METADATA` and `KUBE_NODE_EXTRA_METADATA`. Add extra metadata on master and node instance. With this we could specify different cloud-init for a different container runtime, and also add extra metadata for the new cloud-init, e.g. [master.yaml](https://github.com/Random-Liu/cri-containerd/blob/7d739662141cc137f8b1e82a9824b18be2e5df21/test/e2e/master.yaml)
      2) `KUBE_CONTAINER_RUNTIME_ENDPOINT`. Specify different sock for different container runtime. It's only used when it's not empty.
      3) `KUBE_LOAD_IMAGE_COMMAND`. Specify different load image command for different container runtime.
      
      An example for cri-containerd:
      ```
      export KUBE_MASTER_EXTRA_METADATA="user-data=${GOPATH}/src/github.com/kubernetes-incubator/cri-containerd/test/e2e/master.yaml,cri-containerd-configure-sh=${GOPATH}/src/github.com/kubernetes-incubator/cri-containerd/test/configure.sh"
      export KUBE_NODE_EXTRA_METADATA="user-data=${GOPATH}/src/github.com/kubernetes-incubator/cri-containerd/test/e2e/node.yaml,cri-containerd-configure-sh=${GOPATH}/src/github.com/kubernetes-incubator/cri-containerd/test/configure.sh"
      export KUBE_CONTAINER_RUNTIME="remote"
      export KUBE_CONTAINER_RUNTIME_ENDPOINT="/var/run/cri-containerd.sock"
      export KUBE_LOAD_IMAGE_COMMAND="/home/cri-containerd/usr/local/bin/cri-containerd load"
      export NETWORK_POLICY_PROVIDER="calico"
      ```
      Signed-off-by: 's avatarLantao Liu <lantaol@google.com>
      
      ```release-note
      none
      ```
      /cc @yujuhong @dchen1107 @feiskyer @mikebrow @abhi @mrunalp @runcom 
      /cc @kubernetes/sig-node-pr-reviews
      7a28aaff
    • Kubernetes Submit Queue's avatar
      Merge pull request #54838 from Random-Liu/fix-calico-network-policy · 52ec6f42
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 54488, 54838, 54964). 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 calico network policy for opensource.
      
      For https://github.com/kubernetes/features/issues/286
      
      This PR:
      1) Add `NON_MASTER_NODE_LABELS` env, and only apply calico node label on non-master nodes.
      2) Set ip masq rules in cloud init, so that we don't need the ip masq agent. @dchen1107 @dnardo As we discussed.
      3) Let master use `${NETWORK_PROVIDER}` instead of fixed cni, because we won't run calico node agent on master. The master network should be configured separately (kubenet by default).
      
      With this PR, I could bring up a cluster with `NETWORK_POLICY_PROVIDER=calico` on GCE now.
      ```console
      $ cluster/kubectl.sh get pods --all-namespaces
      NAMESPACE     NAME                                                  READY     STATUS    RESTARTS   AGE
      kube-system   calico-node-9bxbv                                     2/2       Running   0          13m
      kube-system   calico-node-kjxtw                                     2/2       Running   0          13m
      kube-system   calico-node-vertical-autoscaler-67fb4f45bd-hcjmw      1/1       Running   0          16m
      kube-system   calico-node-xs2s2                                     2/2       Running   0          13m
      kube-system   calico-typha-7c4d876ddf-d4dtx                         1/1       Running   0          15m
      kube-system   calico-typha-horizontal-autoscaler-5f477cdc66-qwwph   1/1       Running   0          16m
      kube-system   calico-typha-vertical-autoscaler-58f7d686f7-pn72s     1/1       Running   0          16m
      kube-system   etcd-empty-dir-cleanup-e2e-test-lantaol-master        1/1       Running   0          16m
      kube-system   etcd-server-e2e-test-lantaol-master                   1/1       Running   0          16m
      kube-system   etcd-server-events-e2e-test-lantaol-master            1/1       Running   0          16m
      kube-system   event-exporter-v0.1.7-9d4dbb69c-m76v5                 2/2       Running   0          16m
      kube-system   fluentd-gcp-v2.0.10-25dmf                             2/2       Running   0          16m
      kube-system   fluentd-gcp-v2.0.10-kgxsk                             2/2       Running   0          16m
      kube-system   fluentd-gcp-v2.0.10-p75xg                             2/2       Running   0          16m
      kube-system   fluentd-gcp-v2.0.10-xzh77                             2/2       Running   0          16m
      kube-system   heapster-v1.5.0-beta.0-5cf4d9dff7-dmvm7               4/4       Running   0          13m
      kube-system   kube-addon-manager-e2e-test-lantaol-master            1/1       Running   0          15m
      kube-system   kube-apiserver-e2e-test-lantaol-master                1/1       Running   0          16m
      kube-system   kube-controller-manager-e2e-test-lantaol-master       1/1       Running   0          16m
      kube-system   kube-dns-79bdcb6c9f-2bpc8                             3/3       Running   0          15m
      kube-system   kube-dns-79bdcb6c9f-gr686                             3/3       Running   0          16m
      kube-system   kube-dns-autoscaler-996dcfc9d-pfs4s                   1/1       Running   0          16m
      kube-system   kube-proxy-e2e-test-lantaol-minion-group-3khw         1/1       Running   0          16m
      kube-system   kube-proxy-e2e-test-lantaol-minion-group-6878         1/1       Running   0          16m
      kube-system   kube-proxy-e2e-test-lantaol-minion-group-j9rq         1/1       Running   0          16m
      kube-system   kube-scheduler-e2e-test-lantaol-master                1/1       Running   0          16m
      kube-system   kubernetes-dashboard-765c6f47bd-lsw5r                 1/1       Running   0          16m
      kube-system   l7-default-backend-6d477bf555-x54zf                   1/1       Running   0          16m
      kube-system   l7-lb-controller-v0.9.7-e2e-test-lantaol-master       1/1       Running   0          16m
      kube-system   metrics-server-v0.2.0-9c4f8c48d-gkl79                 2/2       Running   0          13m
      kube-system   monitoring-influxdb-grafana-v4-54df94856c-krkvb       2/2       Running   0          16m
      kube-system   rescheduler-v0.3.1-e2e-test-lantaol-master            1/1       Running   0          16m
      ```
      
      **Note that with this PR, master node will be using kubenet by default. And network policy will not apply on master node.**
      
      **We need this to unblock `cri-containerd` integration with `kube-up.sh`.**
      /cc @dchen1107 @dnardo Please take a look.
      @kubernetes/sig-network-misc @kubernetes/sig-cluster-lifecycle-misc 
      Signed-off-by: 's avatarLantao Liu <lantaol@google.com>
      
      ```release-note
      None
      ```
      52ec6f42
    • Kubernetes Submit Queue's avatar
      Merge pull request #54488 from lichuqiang/plugin_base · 2084f7f4
      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 admission handler for device resources allocation
      
      **What this PR does / why we need it**:
      Add admission handler for device resources allocation to fail fast during pod creation
      
      **Which issue this PR fixes** 
      fixes #51592
      
      **Special notes for your reviewer**:
      @jiayingz Sorry, there is something wrong with my branch in #51895. And I think the existing comments in the PR might be too long for others to view. So I closed it and opened the new one, as we have basically reach an agreement on the implement :)
      I have covered the functionality and unit test part here, and would set about the e2e part ASAP
      
      /cc @jiayingz @vishh @RenaudWasTaken 
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      2084f7f4
  2. 02 Nov, 2017 18 commits