1. 06 Jun, 2018 16 commits
    • Rostislav M. Georgiev's avatar
      kubeadm: Don't match DNS versions to K8s versions · 19d2bbbc
      Rostislav M. Georgiev authored
      Some code in kubeadm was designed with the intent, that in the future CoreDNS
      and kube-dns versions will match to specific K8s versions. This code is not
      functional, since it does not perform any version matching. As of this moment,
      no version matching is planned and a lot of boilerplate code is left useless.
      The solution is simple - remove the unneeded parts to simplify the flow.
      Signed-off-by: 's avatarRostislav M. Georgiev <rostislavg@vmware.com>
      19d2bbbc
    • Kubernetes Submit Queue's avatar
      Merge pull request #64800 from dashpole/cadvisor_godep · f4668d28
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 63717, 64646, 64792, 64784, 64800). 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 cadvisor godeps to v0.30.0
      
      **What this PR does / why we need it**:
      cAdvisor godep update corresponding to 1.11
      
      **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 #63204
      
      **Release note**:
      ```release-note
      Use IONice to reduce IO priority of du and find
      cAdvisor ContainerReference no longer contains Labels. Use ContainerSpec instead.
      Fix a bug where cadvisor failed to discover a sub-cgroup that was created soon after the parent cgroup.
      ```
      
      /sig node
      /kind bug
      /priority critical-urgent
      
      /assign @dchen1107
      f4668d28
    • Kubernetes Submit Queue's avatar
      Merge pull request #64784 from jiayingz/status-ready · a32e5b6a
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 63717, 64646, 64792, 64784, 64800). 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>.
      
      Reconcile extended resource capacity after kubelet restart.
      
      **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 https://github.com/kubernetes/kubernetes/issues/64632
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      Kubelet will set extended resource capacity to zero after it restarts. If the extended resource is exported by a device plugin, its capacity will change to a valid value after the device plugin re-connects with the Kubelet. If the extended resource is exported by an external component through direct node status capacity patching, the component should repatch the field after kubelet becomes ready again. During the time gap, pods previously assigned with such resources may fail kubelet admission but their controller should create new pods in response to such failures.
      ```
      a32e5b6a
    • Kubernetes Submit Queue's avatar
      Merge pull request #64792 from luxas/patch_node_crisocket · 7d782400
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 63717, 64646, 64792, 64784, 64800). 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>.
      
      kubeadm: Upload CRISocket information in kubeadm init/join
      
      **What this PR does / why we need it**:
      
      As a side-effect, `kubeadm join` will become blocking on the kubelet doing the TLS bootstrap. This partially also fixes problems when users run `kubeadm join` and it returns successfully without anything happening as the kubelet is actually unhealthy. If that happens now `kubeadm join` will exit with a non-zero code.
      
      What this PR does is it uploads the CRISocket information to the Node API object as a workaround until we have something like https://github.com/kubernetes/kubernetes/issues/64460 in place that will solve this problem for real. This way we won't lose the CRISocket information which we would otherwise do.
      This can be used for `kubeadm upgrade` or `kubeadm reset` in future releases.
      
      **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**:
      Depends on https://github.com/kubernetes/kubernetes/pull/64624
      
      **Release note**:
      
      ```release-note
      [action required] TODO
      ```
      @kubernetes/sig-cluster-lifecycle-pr-reviews
      7d782400
    • Kubernetes Submit Queue's avatar
      Merge pull request #64646 from freehan/pod-ready-plus2-new · 0b8394a1
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 63717, 64646, 64792, 64784, 64800). 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 ContainersReady condition into Pod Status
      
      **Last 3 commits are new**
      
      Follow up PR of: https://github.com/kubernetes/kubernetes/pull/64057 and https://github.com/kubernetes/kubernetes/pull/64344
      
      Have a single PR for adding ContainersReady per https://github.com/kubernetes/kubernetes/pull/64344#issuecomment-394038384
      
      ```release-note
      Introduce ContainersReady condition in Pod Status
      ```
      
      
      /assign yujuhong for review
      /assign thockin for the tiny API change
      0b8394a1
    • Kubernetes Submit Queue's avatar
      Merge pull request #63717 from ingvagabund/promote-sysctl-annotations-to-fields · b6f75ac3
      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>.
      
      Promote sysctl annotations to fields
      
      #
      
      
      **What this PR does / why we need it**:
      
      Promoting experimental sysctl feature from annotations to API fields.
      
      **Special notes for your reviewer**:
      
      Following sysctl KEP: https://github.com/kubernetes/community/pull/2093
      
      **Release note**:
      
      ```release-note
      The Sysctls experimental feature has been promoted to beta (enabled by default via the `Sysctls` feature flag). PodSecurityPolicy and Pod objects now have fields for specifying and controlling sysctls. Alpha sysctl annotations will be ignored by 1.11+ kubelets. All alpha sysctl annotations in existing deployments must be converted to API fields to be effective.
      ```
      
      **TODO**:
      
      * [x] - Promote sysctl annotation in Pod spec
      * [x] - Promote sysctl annotation in PodSecuritySpec spec
      * [x] - Feature gate the sysctl
      * [x] - Promote from alpha to beta
      * [x] - docs PR - https://github.com/kubernetes/website/pull/8804
      b6f75ac3
    • Kubernetes Submit Queue's avatar
      Merge pull request #64670 from stealthybox/feature/kubeadm_882-etcd-zero-probe · d3a797a0
      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>.
      
      kubeadm: When etcd is listening on all interfaces, set the etcd probe to use loopback
      
      **What this PR does / why we need it**:
      When constructing the etcd liveness probe, if the user passes an IPv4 or IPv6 address,
      we set the `etcdctl` liveness probe to use the respective IPv4 or IPv6 loopback address for `--endpoints`.
      
      The etcd probe is now always formatted with the https:// protocol and square brackets around the IP (required for IPv6 / compatible with IPv4).
      
      `::1` is now also included in the etcd serving cert SAN by default.
      
      /kind bug
      /area kubeadm
      /area etcd
      /priority important-soon
      
      /sig cluster-lifecycle
      /assign @fabriziopandini
      
      **Which issue(s) this PR fixes**
      Fixes https://github.com/kubernetes/kubeadm/issues/882
      
      **Special notes for your reviewer**:
      ```bash
      root@vagrant:~# /vagrant/bin/882_kubeadm init --config /dev/stdin << EOF |& tail -n5
      etcd:
        extraArgs:
          listen-client-urls: https://[::]:2379
      EOF
      I0603 19:52:15.666594   24743 tlsbootstrap.go:50] [bootstraptoken] configured RBAC rules to allow Node Bootstrap tokens to post CSRs in order for nodes to get long term certificate credentials
      I0603 19:52:15.671424   24743 tlsbootstrap.go:72] [bootstraptoken] configured RBAC rules to allow the csrapprover controller automatically approve CSRs from a Node Bootstrap Token
      I0603 19:52:15.674607   24743 tlsbootstrap.go:95] [bootstraptoken] configured RBAC rules to allow certificate rotation for all node client certificates in the cluster
      I0603 19:52:15.677551   24743 clusterinfo.go:43] [bootstraptoken] creating the "cluster-info" ConfigMap in the "kube-public" namespace
      [addons] Applied essential addon: CoreDNS
      [addons] Applied essential addon: kube-proxy
      root@vagrant:~# cat /etc/kubernetes/manifests/etcd.yaml |grep -C4 listen
      spec:
        containers:
        - command:
          - etcd
          - --listen-client-urls=https://[::]:2379
          - --advertise-client-urls=https://127.0.0.1:2379
          - --cert-file=/etc/kubernetes/pki/etcd/server.crt
          - --client-cert-auth=true
          - --data-dir=/var/lib/etcd
      root@vagrant:~# cat /etc/kubernetes/manifests/etcd.yaml |grep -C4 etcdctl
            exec:
              command:
              - /bin/sh
              - -ec
              - ETCDCTL_API=3 etcdctl --endpoints=https://[::1]:2379 --cacert=/etc/kubernetes/pki/etcd/ca.crt
                --cert=/etc/kubernetes/pki/etcd/healthcheck-client.crt --key=/etc/kubernetes/pki/etcd/healthcheck-client.key
                get foo
            failureThreshold: 8
            initialDelaySeconds: 15
      ```
      
      **Release note**:
      ```release-note
      kubeadm now configures the etcd liveness probe correctly when etcd is listening on all interfaces
      ```
      d3a797a0
    • Kubernetes Submit Queue's avatar
      Merge pull request #63650 from soltysh/issue62382 · 34759c2d
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 64009, 64780, 64354, 64727, 63650). 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>.
      
      Never clean backoff in job controller
      
      **What this PR does / why we need it**:
      In https://github.com/kubernetes/kubernetes/pull/60985 I've added a mechanism which allows immediate job status update, unfortunately that broke the backoff logic seriously. I'm sorry for that. I've changed the `immediate` mechanism so that it NEVER cleans the backoff, but for the cases when we want fast status update it uses a zero backoff. 
      
      **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 #62382
      
      **Special notes for your reviewer**:
      /assign @janetkuo 
      
      **Release note**:
      ```release-note
      None
      ```
      34759c2d
    • Kubernetes Submit Queue's avatar
      Merge pull request #64727 from yguo0905/tpu-v1-api · 81b5db9d
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 64009, 64780, 64354, 64727, 63650). 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>.
      
      GCE: Update cloud provider to use TPU v1 API
      
      **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**:
      
      ```release-note
      GCE: Update cloud provider to use TPU v1 API
      ```
      
      /assign @vishh 
      /assign @cheftako
      81b5db9d
    • Kubernetes Submit Queue's avatar
      Merge pull request #64354 from mtaufen/dkcfg-safe-fields · 0e44d8c4
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 64009, 64780, 64354, 64727, 63650). 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>.
      
      per-field dynamic config advice
      
      Dynamic Kubelet config gives cluster admins and k8s-as-a-service providers a lot of flexibility around reconfiguring the Kubelet in live environments. With great power comes great responsibility. These comments intend to provide more nuanced guidance around using dynamic Kubelet config by adding items to consider when changing various fields and pointing out where cluster admins and k8s-as-service providers should maintain extra caution.
      
      @kubernetes/sig-node-pr-reviews PLEASE provide feedback and help fill in the blanks here, I don't have domain expertise in all of these features.
      
      https://github.com/kubernetes/features/issues/281
      
      ```release-note
      NONE
      ```
      0e44d8c4
    • Kubernetes Submit Queue's avatar
      Merge pull request #64780 from luxas/kubeadm_new_dropins · 216009fb
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 64009, 64780, 64354, 64727, 63650). 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>.
      
      kubeadm: Update the dropin for the kubelet in v1.11
      
      **What this PR does / why we need it**:
      One of the final pieces of https://github.com/kubernetes/kubeadm/issues/851, https://github.com/kubernetes/kubeadm/issues/847 and https://github.com/kubernetes/kubeadm/issues/822
      
      **Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:
      (partially)
      Fixes https://github.com/kubernetes/kubeadm/issues/822
      
      **Special notes for your reviewer**: Please check whether this release note makes sense to you.
      
      **Release note**:
      
      ```release-note
      [action required] The structure of the kubelet dropin in the kubeadm deb package has changed significantly.
      Instead of hard-coding the parameters for the kubelet in the dropin, a structured configuration file
      for the kubelet is used, and is expected to be present in `/var/lib/kubelet/config.yaml`.
      For runtime-detected, instance-specific configuration values, a environment file with
      dynamically-generated flags at `kubeadm init` or `kubeadm join` run time is used.
      Finally, if the user wants to override something specific for the kubelet that can't be done via
      the kubeadm Configuration file (which is preferred), they might add flags to the 
      `KUBELET_EXTRA_ARGS` environment variable in either `/etc/default/kubelet`
      or `/etc/sysconfig/kubelet`, depending on the system you're running on.
      ```
      @kubernetes/sig-cluster-lifecycle-pr-reviews
      216009fb
    • Kubernetes Submit Queue's avatar
      Merge pull request #64009 from feiskyer/windows-security-context · 999b2da4
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 64009, 64780, 64354, 64727, 63650). 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>.
      
      Kubelet: Add security context for Windows containers
      
      **What this PR does / why we need it**:
      
      This PR adds windows containers to Kubelet CRI and also implements security context setting for docker containers.
      
      **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**:
      
      RunAsUser from Kubernetes API only accept int64 today, which is not supported on Windows. It should be changed to intstr for working with both Windows and Linux containers in a separate PR.
      
      **Release note**:
      
      ```release-note
      Kubelet: Add security context for Windows containers
      ```
      
      /cc @PatrickLang @taylorb-microsoft @michmike @JiangtianLi @yujuhong @dchen1107
      999b2da4
    • leigh schrandt's avatar
    • Kubernetes Submit Queue's avatar
      Merge pull request #64732 from jingxu97/June/attachVolumeLimit · ad9722c4
      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>.
      
      Set GCE PD attachable volume limit based on machineType
      
      This PR implements the function to return attachable volume limit based
      on machineType for GCE PD. This is part of the design in kubernetes/community#2051/
      ad9722c4
    • David Ashpole's avatar
      disable process scheduler metrics · 4afcccd2
      David Ashpole authored
      4afcccd2
    • David Ashpole's avatar
      update cadvisor godeps to v0.30.0 · 4df5bd09
      David Ashpole authored
      4df5bd09
  2. 05 Jun, 2018 24 commits