1. 03 Nov, 2017 22 commits
    • chenpengdev's avatar
      fix exported method run's comment · 3429d895
      chenpengdev authored
      3429d895
    • 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