1. 23 Nov, 2017 31 commits
  2. 22 Nov, 2017 9 commits
    • Yang Guo's avatar
    • Kubernetes Submit Queue's avatar
      Merge pull request #55643 from tallclair/psp-scale · 21584421
      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>.
      
      [PodSecurityPolicy] Optimize authorization check
      
      **What this PR does / why we need it**:
      
      Authorizing PodSecurityPolicy use may involve a remote call, and can be slow. Rather than authorizing the user / SA for every policy in the cluster, only test authz for the policies under which the pod is valid.
      
      This is a big improvement in the case where there are a lot of policies for which the pod is not valid (benchmark below), but should also help when the pod is valid under other policies, as it allows the authorization to short-circuit on the first accepted policy.
      
      **Benchmark:**
      Highlight from scale testing (see https://docs.google.com/document/d/1IIcHHE_No1KMAybW5krIphdN325eGa2sxF2eqg2YAPI/edit for the full results). These were run with 1000 policies under which the pods were not valid, and had no role bindings.
      
      | | method | resource | 50th percentile | 90th percentile | 99th percentile
      | -- | -- | -- | -- | -- | --
      | 1.8 HEAD | POST | pods | 8.696784s | 20.497659s | 22.472421s
      | 1.8 With fix | POST | pods | 25.454ms | 29.068ms | 85.817ms
      
      (I didn't benchmark master, but expect the difference to be more drastic, since the authorization is run twice - for both Admit and Validate)
      
      **Which issue(s) this PR fixes**:
      Fixes #55521
      
      **Special notes for your reviewer**:
      The validation errors are no longer totally accurate, as they may include errors from PSPs that the user/pod isn't authorized to use. However, I think this is a worthwhile tradeoff. If this is a big concern, we could authorize all policies in the case where none admitted /validated the pod.
      
      **Release note**:
      ```release-note
      Improved PodSecurityPolicy admission latency, but validation errors are no longer limited to only errors from authorized policies.
      ```
      21584421
    • Kubernetes Submit Queue's avatar
      Merge pull request #56109 from dashpole/update_cadvisor_godep · 399811c5
      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 cadvisor godeps to v0.28.2
      
      **What this PR does / why we need it**:
      cAdvisor cuts a release for each kubernetes release.  This cAdvisor release corresponds to cadvisor 1.9.
      
      **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 #55909 
      Fixes #55628
      
      **Special notes for your reviewer**:
      This is also a prerequisite for fixing https://github.com/kubernetes/kubernetes/issues/51745.
      This fixes the flaky node_e2e, which is submit queue blocking
      
      **Release note**:
      ```release-note
      Add containerd monitoring support
      ```
      399811c5
    • Tim Hockin's avatar
      Fix NewProxyServer · 79778288
      Tim Hockin authored
      Different OSes need different args.  This is not a great fix, but better
      than adding an arg to Windows which doesn't need it.
      79778288
    • Serguei Bezverkhi's avatar
      a63fadfb
    • Kubernetes Submit Queue's avatar
      Merge pull request #56201 from luxas/new_kubeadm_owners · 93dc3763
      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>.
      
      Restructure cmd/kubeadm/OWNERS
      
      **What this PR does / why we need it**:
      
      After a year of virtually the same people in `cmd/kubeadm/OWNERS`, the SIG has decided to refresh the list of reviewers/approvers a little. If you feel more/other changes are needed yet, please speak up here in this thread, in our weekly SIG meeting or ping me on Slack.
      
      Thanks all everyone that has helped out here and thanks to you who are stepping up more now!
      
      **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
      ```
      @kubernetes/sig-cluster-lifecycle-pr-reviews 
      
      Please ACK that you are ok with this officially here as well:
      @kad @timothysc @jamiehannaford @xiangpengzhao @lukemarsden @errordeveloper @mattmoyer @justinsb @fabriziopandini
      93dc3763
    • Kubernetes Submit Queue's avatar
      Merge pull request #54140 from wackxu/updds · 94d4cb27
      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 code comment that are out of date for apps/v1beta2
      
      **What this PR does / why we need it**:
      
      update comment that are out of date
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #54139
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      94d4cb27
    • Anthony Yeh's avatar
      Update CHANGELOG-1.6.md for v1.6.13. · 1ede59ac
      Anthony Yeh authored
      1ede59ac
    • Hemant Kumar's avatar
      Enable PersistentVolumeClaimResize admission plugin in default cluster settings · 5ee4d2bb
      Hemant Kumar authored
      The plugin itself will not do anything if resizing is not enabled.
      
      Move Resize plugin to start
      5ee4d2bb