1. 11 Jan, 2017 27 commits
    • Pengfei Ni's avatar
      dockertools: fix build on OSX · 1f17a47f
      Pengfei Ni authored
      1f17a47f
    • Kubernetes Submit Queue's avatar
      Merge pull request #39497 from ymqytw/fix_evictions_test · c0241245
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 39694, 39383, 39651, 39691, 39497)
      
      Fix evictions test
      
      **What this PR does / why we need it**:
      Fixes bugs in evictions test. Make vet happy.
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #39412
      
      Ref: #39452
      
      cc: @calebamiles
      c0241245
    • Kubernetes Submit Queue's avatar
      Merge pull request #39691 from yujuhong/bump_timeout · ebc8e406
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 39694, 39383, 39651, 39691, 39497)
      
      Bump container-linux and gci timeout for docker health check
      
      The command `docker ps` can take longer time to respond under heavy load or
      when encountering some known issues. In these cases, the containers are running
      fine, so aggressive health check could cause serious disruption. Bump the
      timeout to 60s to be consistent with the debian-based containerVM.
      
      This addresses #38588
      ebc8e406
    • Kubernetes Submit Queue's avatar
      Merge pull request #39651 from liggitt/passwordfile-groups · 95968754
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 39694, 39383, 39651, 39691, 39497)
      
      Add support for groups to passwordfile
      
      As we move deployment methods to using RBAC, it is useful to be able to place the admin user in the bootstrap kubeconfig files in a superuser group. The tokencsv file supports specifying group membership, but the basicauth file does not. This adds it for parity.
      
      I plan to update the generated password file to put the admin user in a group (similar to the way https://github.com/kubernetes/kubernetes/pull/39537 puts that user in a group in the token file)
      
      ```release-note
      --basic-auth-file supports optionally specifying groups in the fourth column of the file
      ```
      95968754
    • Kubernetes Submit Queue's avatar
      Merge pull request #39383 from liggitt/bind-check · 3f2a02cf
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 39694, 39383, 39651, 39691, 39497)
      
      Allow rolebinding/clusterrolebinding with explicit bind permission check
      
      Fixes https://github.com/kubernetes/kubernetes/issues/39176
      Fixes https://github.com/kubernetes/kubernetes/issues/39258
      
      Allows creating/updating a rolebinding/clusterrolebinding if the user has explicitly been granted permission to perform the "bind" verb against the referenced role/clusterrole (previously, they could only bind if they already had all the permissions in the referenced role via an RBAC role themselves)
      
      ```release-note
      To create or update an RBAC RoleBinding or ClusterRoleBinding object, a user must:
      1. Be authorized to make the create or update API request
      2. Be allowed to bind the referenced role, either by already having all of the permissions contained in the referenced role, or by having the "bind" permission on the referenced role.
      ```
      3f2a02cf
    • Kubernetes Submit Queue's avatar
      Merge pull request #39694 from DirectXMan12/bug/hpa-panic · bb7d07a3
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 39694, 39383, 39651, 39691, 39497)
      
      HPA Controller: Check for 0-sum request value
      
      In certain conditions in which the set of metrics returned by Heapster
      is completely disjoint from the set of pods returned by the API server,
      we can have a request sum of zero, which can cause a panic (due to
      division by zero).  This checks for that condition.
      
      Fixes #39680
      
      **Release note**:
      
      ```release-note
      Fixes an HPA-related panic due to division-by-zero.
      ```
      bb7d07a3
    • Kubernetes Submit Queue's avatar
      Merge pull request #39686 from deads2k/rbac-38-snip-bad-dep · ea18d5c3
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 39673, 39536, 39617, 39540, 39686)
      
      remove API to server library dependency
      
      A client library (which must include api types), should not depend on our server library.  This duplicates one constant for conversion to avoid the link.
      
      @smarterclayton @liggitt
      
      The import-boss rule to keep us from forming these dependencies is non-trivial since some of the bits under /apis are server only (validation for instance).
      ea18d5c3
    • Kubernetes Submit Queue's avatar
      Merge pull request #39540 from pwittrock/kubectldocs · 748e8f6b
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 39673, 39536, 39617, 39540, 39686)
      
      Improve kubectl help with examples.
      
      ```release-note
      NONE
      ```
      748e8f6b
    • Kubernetes Submit Queue's avatar
      Merge pull request #39617 from apprenda/kubeadm_dns_upgrade · c425f772
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 39673, 39536, 39617, 39540, 39686)
      
      kubeadm: updated DNS deployment.
      
      **What this PR does / why we need it**: Updates KubeDNS deployment to match upstream.
      
      **Special notes for your reviewer**: It was tested manually by bootstrapping a new cluster, running a busybox container and making sure one could `nslookup` from within the container to find `kubernetes` and other services.
      c425f772
    • Kubernetes Submit Queue's avatar
      Merge pull request #39536 from screeley44/add-storage-class · 4d9bd12c
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 39673, 39536, 39617, 39540, 39686)
      
      Adding storageclass to resource printers for get pv and pvc
      
      Give ability to see what Storage Class a PV is using, as well as what StorageClass a PVC has requested
      
      ```
      [root@screeley-sc1 gce]# kubectl get pv
      NAME                                       CAPACITY   ACCESSMODES   RECLAIMPOLICY   STATUS      CLAIM                            STORAGECLASS   REASON    AGE
      pv-gce                                     2Gi        RWO           Retain          Available                                    anything                 41m
      pvc-774217fd-d440-11e6-9d11-42010af00010   2Gi        RWO           Delete          Bound       default/gce-claim-storageclass   slow                     0s
      
      [root@screeley-sc1 gce]# kubectl get pvc
      NAME                              STATUS    VOLUME                                     CAPACITY   ACCESSMODES   REQUESTEDSTORAGECLASS   AGE
      gce-claim-class1                  Pending                                                                                      		6m
      gce-claim-classless               Pending                                                                                      		6m
      gce-claim-storageclass            Bound     pvc-774217fd-d440-11e6-9d11-42010af00010   2Gi        RWO           slow           		6m
      gce-claim-storageclass-nonexist   Pending                                                                       superfly       		4s
      ```
      @kubernetes/sig-storage
      4d9bd12c
    • Kubernetes Submit Queue's avatar
      Merge pull request #39673 from liggitt/deads2k-fix-bindata · 3a7af8fb
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 39673, 39536, 39617, 39540, 39686)
      
      Generate stable bindata output
      
      fixes #30621
      
      bindata generation produces spurious diffs when run with different go versions because gzip output is not stable between versions.
      
      It also produces spurious diffs when invoked directly vs via make/go:generate.
      
      This PR:
      * adds data to bindata uncompressed (makes translation files human-readable, and the zip files were already compressed... for reference the file was 27.7 KB before, and 20.1 KB with this change)
      * runs the generation from KUBE_ROOT, so it doesn't matter whether it is invoked directly or via go:generate
      
      supercedes https://github.com/kubernetes/kubernetes/pull/39631
      3a7af8fb
    • Kubernetes Submit Queue's avatar
      Merge pull request #39005 from brendandburns/windows · d3c0914a
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 38212, 38792, 39641, 36390, 39005)
      
      Set MemorySwap to zero on Windows
      
      Fixes https://github.com/kubernetes/kubernetes/issues/39003
      
      @dchen1107 @michmike @kubernetes/sig-node-misc
      d3c0914a
    • Kubernetes Submit Queue's avatar
      Merge pull request #36390 from nikhiljindal/cascDelSvc · 94cca273
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 38212, 38792, 39641, 36390, 39005)
      
      Updating federated service controller to support cascading deletion
      
      Ref https://github.com/kubernetes/kubernetes/issues/33612
      
      Service controller is special than other federation controllers because it does not use federatedinformer and updater to sync services (it was written before we had those frameworks).
      Updating service controller code to instantiate these frameworks and then use deletion helper to perform cascading deletion.
      Note that, I havent changed the queuing logic in this PR so we still dont use federated informer to manage the queue. Will do that in the next PR.
      
      cc @kubernetes/sig-federation-misc  @mwielgus @quinton-hoole
      
      
      ```release-note
      federation: Adding support for DeleteOptions.OrphanDependents for federated services. Setting it to false while deleting a federated service also deletes the corresponding services from all registered clusters.
      ```
      94cca273
    • Kubernetes Submit Queue's avatar
      Merge pull request #39641 from liggitt/node-controller-status · 49a0cf7f
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 38212, 38792, 39641, 36390, 39005)
      
      Allow node-controller to update node status
      
      ref: #39639 
      
      * adds required permissions to node-controller
       * fixes typo in role name for pod-garbage-collector role
      * adds event watching permissions to persistent volume controller
      * adds event permissions to node proxier
      49a0cf7f
    • Kubernetes Submit Queue's avatar
      Merge pull request #38792 from mikedanese/bzl-test · ae62c5f0
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 38212, 38792, 39641, 36390, 39005)
      
      build test binary with bazel
      
      I think... this should work
      ae62c5f0
    • Kubernetes Submit Queue's avatar
      Merge pull request #38212 from mikedanese/kubeletauth · addc6cae
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 38212, 38792, 39641, 36390, 39005)
      
      Generate a kubelet CA and kube-apiserver cert-pair for kubelet auth.
      
      cc @cjcullen
      addc6cae
    • Kubernetes Submit Queue's avatar
      Merge pull request #39615 from gmarek/density_load_configs · 234c4358
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 39695, 37054, 39627, 39546, 39615)
      
      Add configs that run more advanced density and load tests
      
      Wojtek is on vacation this week - @timothysc can you please take a look? It's rather terrible, but I don't have a better idea on how to make parametric tests.
      
      cc @wojtek-t
      234c4358
    • Kubernetes Submit Queue's avatar
      Merge pull request #39546 from dashpole/dynamic_config_eviction_hard · a2da4f0c
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 39695, 37054, 39627, 39546, 39615)
      
      Use Dynamic Config in e2e_node inode eviction test
      
      Alternative solution to #39249.  Similar to solution proposed by @vishh in #36828.
      
      @Random-Liu @mtaufen
      a2da4f0c
    • Kubernetes Submit Queue's avatar
      Merge pull request #39627 from alejandroEsc/ae/kubelet/log · 87b9f6fa
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 39695, 37054, 39627, 39546, 39615)
      
      fixed error log that would cause two log lines to run into each other…
      
      **What this PR does / why we need it**:
      Fixed error log that would cause two log lines to run into each other. Logs running into each other can be reproduced by running local-up-cluster.sh on osx.
      
      
      **Release note**:
      ```NONE
      ```
      87b9f6fa
    • Kubernetes Submit Queue's avatar
      Merge pull request #37054 from tanshanshan/remove-repeat1 · 5aa177a8
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 39695, 37054, 39627, 39546, 39615)
      
      remove repeating const declaration
      
      **What this PR does / why we need it**:
      
      remove repeating const declaration  , and avoid const declaration in  loop
      
      Thanks.
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      ```
      5aa177a8
    • Kubernetes Submit Queue's avatar
      Merge pull request #39695 from ixdy/docker-build-pull · 844fa1b9
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 39695, 37054, 39627, 39546, 39615)
      
      Always --pull in docker build to ensure recent base images
      
      **What this PR does / why we need it**: By default, Docker will use a locally cached image rather than looking for a newer image. This can be problematic when using a common base image like debian:jessie, since you may end up unintentionally using a very old base image.
      
      I think everything here is only used on release paths, so it shouldn't affect any regular development workflows.
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      844fa1b9
    • Jeff Grafton's avatar
      Update to debian-iptables-*:v5 · 9172bf63
      Jeff Grafton authored
      9172bf63
    • Jeff Grafton's avatar
    • Kubernetes Submit Queue's avatar
      Merge pull request #39491 from jayunit100/sched_Histogram_error · add3a08a
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 34488, 39511, 39619, 38342, 39491)
      
      Update FitError as a message component into the PodConditionUpdater.
      
      Fixes #20064 , after a roundabout volley of ideas, we ended up digging into existing Conditions for this, rather then a first class API object.  This is just a quick sketch of the skeleton minimal implementation, it should pretty much "just work".  I'll test it more later today.
      
      
      Release Note:
      ```
      Histogram data of predicate failures is contained in pod conditions and thus available to users by kubectl commands.
      ```
      add3a08a
    • Kubernetes Submit Queue's avatar
      Merge pull request #38342 from ymqytw/make_SPatch_delete_all_duplicates · ebf1a533
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 34488, 39511, 39619, 38342, 39491)
      
      Make StrategicPatch delete all matching maps in a merging list
      
      fixes #38332
      
      ```release-note
      NONE
      ```
      
      cc: @lavalamp @pwittrock
      ebf1a533
    • Kubernetes Submit Queue's avatar
      Merge pull request #39619 from deads2k/fed-20-rename · 609e3e38
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 34488, 39511, 39619, 38342, 39491)
      
      rename kubernetes-discovery to kube-aggregator
      
      Rename `kubernetes-discovery` to `kube-aggregator`.  Move and bulk rename.
      
      @kubernetes/sig-api-machinery-misc
      609e3e38
    • Kubernetes Submit Queue's avatar
      Merge pull request #39511 from zhouhaibing089/lru-time-ut · 7d2f7980
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 34488, 39511, 39619, 38342, 39491)
      
      use fake clock in lruexpiration cache test
      
      when the system clock is extremely slow(usually see in VMs), this [check](https://github.com/kubernetes/kubernetes/blob/master/pkg/util/cache/lruexpirecache.go#L74) might still return the value.  
      
      ```go
      if c.clock.Now().After(e.(*cacheEntry).expireTime) {
      	go c.remove(key)
      	return nil, false
      }
      ```
      
      that means even we set the ttl to be 0 second, the after check might still be false(because the clock is too slow, and thus equals).
      
      the change here helps to reduce flakes.
      7d2f7980
  2. 10 Jan, 2017 13 commits