1. 10 Aug, 2018 13 commits
    • Aaron Crickenberger's avatar
      macOS and docker for mac don't play nicely with mktemp · 2de8daf0
      Aaron Crickenberger authored
      On macOS mktemp -d drops something in /var/folders, which isn't
      shared by default with Docker for Mac. Thus I can't run docker
      with that volume mounted to build binaries for test images. So
      instead, tell mktemp to use kubernetes/_tmp as its base, which
      is what I see some of the hack/verify-* scripts use.
      2de8daf0
    • Kubernetes Submit Queue's avatar
      Merge pull request #67176 from charrywanganthony/auditv1beta1Tov1_utils · 5de3896d
      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>.
      
      use v1 version of advanced audit policy in kubeadm
      
      audit api version has been updated to v1 #65891 
      
      **Release note**:
      ```release-note
      kubeadm uses audit policy v1 instead of v1beta1
      ```
      5de3896d
    • Kubernetes Submit Queue's avatar
      Merge pull request #67131 from rosti/kube-proxy-manifest-fix2 · cd06fd4a
      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: Deduplicate kube-proxy image logic
      
      **What this PR does / why we need it**:
      
      Until now, kube-proxy image was handled in two separate places:
          
      - In images.go along with the pre-pull code and without having the image override capabilities (via UnifiedControlPlaneImage)
      
      - In the kube-proxy manifest, where image override was possible.
      
      This duplicates the kube-proxy image logic and makes it prone to errors.
      Therefore, this change aims to deduplicate it and make it more straightforward.
      This is achieved in the following ways:
      
      - GetKubeControlPlaneImage is used for kube-proxy image fetching, thus allowing for the image to be overriden by UnifiedControlPlaneImage.
      
      - Remove duplicated logic from the manifest and use GetKubeControlPlaneImage to generate the image for the manifest.
      
      This PR also removes GetKubeControlPlaneImageNoOverride as it's no longer needed.
      
      **Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:
      None
      
      **Special notes for your reviewer**:
      
      /cc @kubernetes/sig-cluster-lifecycle-pr-reviews
      /area kubeadm
      /assign @luxas
      /assign @timothysc
      
      **Release note**:
      
      ```release-note
      kubeadm: make sure pre-pulled kube-proxy image and the one specified in its daemon set manifest are the same
      ```
      cd06fd4a
    • Kubernetes Submit Queue's avatar
      Merge pull request #66953 from Huang-Wei/fix-#66348 · ecc64f2e
      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>.
      
      ensure ScheduleDSPods respects tainted nodes
      
      - add PodToleratesNodeTaints to ~~nodeSelectionPredicates()~~ checkNodeFitness()
      - add integration testcase
      
      Fixes #66348, and would keep the behavior consistent with it was when ScheduleDSPods is disabled.
      
      ```release-note
      NONE
      ```
      ecc64f2e
    • Kubernetes Submit Queue's avatar
      Merge pull request #67209 from liggitt/clarify-forbidden-message · d0f278e1
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 67058, 67083, 67220, 67222, 67209). 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 the request attributes clearer in forbidden messages
      
      This makes the resource and API group components of a forbidden message explicit (including the "" core API group), to help people trying to match up forbidden errors to required policy changes.
      
      ```release-note
      NONE
      ```
      d0f278e1
    • Kubernetes Submit Queue's avatar
      Merge pull request #67222 from ixdy/qemu-ARCH-static-permissions-fixes · 35e3c782
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 67058, 67083, 67220, 67222, 67209). 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 permissions of qemu-ARCH-static in debian-base and other images
      
      **What this PR does / why we need it**: proper fix for the issue I found in #67215. Some machines (like apparently workstations at Google) have a restrictive umask, so the `qemu-ARCH-static` binaries were getting installed in images without world read/execute permissions, causing utilities like `apt-get` to fail.
      
      There was also a duplicate download/install of these binaries for `debian-iptables`, which further confused the issue. I've since removed that duplicate installation.
      
      Many thanks to @BenTheElder for asking the right question to get me to look at the permissions again.
      
      I haven't pushed any images yet. After merge, I'll build/promote `debian-base:0.3.2`, then update everything to use it, then push some more images, write some more PRs, ...
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      /assign @tallclair
      35e3c782
    • Kubernetes Submit Queue's avatar
      Merge pull request #67220 from verult/vsphere-panic · 6c77dce9
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 67058, 67083, 67220, 67222, 67209). 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>.
      
      Fixed unsafe type cast in vSphere volume plugin
      
      **What this PR does / why we need it**: Fixes the controller manager panic caused by vSphere volumes being used on the wrong cloud provider.
      
      **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 #67218
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      
      /assign @saad-ali
      6c77dce9
    • Kubernetes Submit Queue's avatar
      Merge pull request #67083 from mkumatag/docker-check · a8eb0066
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 67058, 67083, 67220, 67222, 67209). 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 a check for docker version to push fat manifest images
      
      **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 #
      This is for adding a check to avoid any corrupted fat manifest creation.
      
      **Special notes for your reviewer**:
      @dims @luxas 
      **Release note**:
      
      ```release-note
      NONE
      ```
      a8eb0066
    • Kubernetes Submit Queue's avatar
      Merge pull request #67058 from hanxiaoshuai/fix0806 · 422f9cb9
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 67058, 67083, 67220, 67222, 67209). 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>.
      
      use const v1.ProtocolTCP replace of string TCP
      
      **What this PR does / why we need it**:
      use const v1.ProtocolTCP replace of string TCP
      **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
      ```
      422f9cb9
    • Kubernetes Submit Queue's avatar
      Merge pull request #67077 from vikaschoudhary16/scope-selectors-beta · a1b3133b
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 67161, 67093, 67077). 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>.
      
      Graduate ResourceQuotaScopeSelectors to beta
      
      All the pre-requisite items listed in the tracker issue, https://github.com/kubernetes/kubernetes/issues/66673, have got merged. Therefore, opening this PR to graduate scope selectors field in the resource quota spec.
      
      **Release note**:
      
      ```release-note
      Graduate Resource Quota ScopeSelectors to beta, and enable it by default.
      ```
      /cc @derekwaynecarr @deads2k @smarterclayton
      a1b3133b
    • Kubernetes Submit Queue's avatar
      Merge pull request #67093 from juanvallejo/jvallejo/improve-config-context-update · 3587aa42
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 67161, 67093, 67077). 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 kubeconfig file modification time
      
      In cases where there are few destination filenames for a given
      amount of contexts, but a large amount of contexts, this patch
      prevents reading and writing to the same file (or small number
      of files) over and over again needlessly.
      
      **Release note**:
      ```release-note
      Decrease the amount of time it takes to modify kubeconfig files with large amounts of contexts
      ```
      
      cc @deads2k
      3587aa42
    • Kubernetes Submit Queue's avatar
      Merge pull request #67161 from dims/remove-local-manifest-list-after-push · 5158c181
      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 the local manifest list after push
      
      **What this PR does / why we need it**:
      
      Manifests seem sticky in docker, so let's try to purge so if
      we have re-push a fresh set of containers (with same version number as
      before) during testing, the manifests are created fresh.
      
      Change-Id: I41c010c08bd50b68ff6973a4ae1e004824fab178
      
      **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
      ```
      5158c181
    • Wei Huang's avatar
      ensure ScheduleDSPods respects tainted nodes · deb6d854
      Wei Huang authored
      - add PodToleratesNodeTaints to nodeSelectionPredicates()
      - add integration testcases
      deb6d854
  2. 09 Aug, 2018 27 commits