1. 03 Jul, 2017 18 commits
  2. 02 Jul, 2017 2 commits
    • Kubernetes Submit Queue's avatar
      Merge pull request #48281 from hogepodge/configure-swift-store · 9848cdb3
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Add configuration for swift container name
      
      **What this PR does / why we need it:**
      This review updates the OpenStack Heat provider to allow for configuring the name of the Swift object store.
      
      **Which issue this PR fixes:**
      fixes #47966
      
      **Special notes for your reviewer**:
      Note that the terminology for OpenStack Swift conflicts with K8S terminology. In this instance, container is referring to the organization structure of Swift storage objects.
      
      **Release note**:
      ```release-note
      Adds configuration option for Swift object store container name to OpenStack Heat provider.
      ```
      9848cdb3
    • Kubernetes Submit Queue's avatar
      Merge pull request #48304 from mengqiy/kubectl_slice · 950a09d9
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      split util/slice for kubectl
      
      Split util/slice to remove the dependency of kubectl on `k8s.io/kubernetes/pkg/util/slice`.
      
      ```release-note
      NONE
      ```
      /assign @apelisse @monopole 
      
      cc: @pwittrock 
      
      ref #48209
      950a09d9
  3. 01 Jul, 2017 18 commits
    • Nikhita Raghunath's avatar
      36271f98
    • ymqytw's avatar
      split util/slice · 8dac9639
      ymqytw authored
      8dac9639
    • Kubernetes Submit Queue's avatar
      Merge pull request #48056 from luxas/kubeadm_remove_old_label · 5eccc7ae
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Remove old node role label that is not used by kubeadm
      
      **What this PR does / why we need it**:
      This label hasn't been used by kubeadm since v1.5.
      v1.5 support was dropped when v1.6 arrived due to a potential security flaw and the earlier alpha status.
      So this label hasn't been used for around three months.
      It makes sense to remove it in time for v1.8, could already have been done for v1.7
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      @kubernetes/sig-cluster-lifecycle-pr-reviews
      5eccc7ae
    • Kubernetes Submit Queue's avatar
      Merge pull request #47489 from Mashimiao/cm-remove-unneeded · f663c4c8
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 48264, 48324, 48125, 47944, 47489)
      
      remove unneeded variable
      Signed-off-by: 's avatarMa Shimiao <mashimiao.fnst@cn.fujitsu.com>
      
      
      
      **What this PR does / why we need it**:
      small fix, remove unneeded variable
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      f663c4c8
    • Kubernetes Submit Queue's avatar
      Merge pull request #47944 from allencloud/fix-CopyStrings-and-ShuffleStrings-when-slice-is-nil · ed8993e3
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 48264, 48324, 48125, 47944, 47489)
      
      fix CopyStrings and ShuffleStrings for slice when slice is nil
      Signed-off-by: 's avatarallencloud <allen.sun@daocloud.io>
      
      
      
      **What this PR does / why we need it**:
      This PR fixes two functions in util/slice.go, in which I think `CopyStrings` and `ShuffleStrings` miss one case. The case is input data is nil, in this case I think the data returned should be nil as well rather than a non-nil slice with 0 element.
      In addition, I added some test code for this.
       
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #
      NONE, I did not raise a issue for this code. I ran into this when code learning.
      
      **Special notes for your reviewer**:
      NONE
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      ed8993e3
    • Kubernetes Submit Queue's avatar
      Merge pull request #48125 from xiangpengzhao/downwardapi-poduid · faf4e57f
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 48264, 48324, 48125, 47944, 47489)
      
      Add Pod UID (metadata.uid) to downward API env var
      
      **What this PR does / why we need it**:
      Exposing Pod UID by downward API.
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #28918 
      
      **Special notes for your reviewer**:
      Generated files aren't committed. I'd like CI to tell me what scripts should I run to generate these files.
      
      /cc @smarterclayton @vishh @dubstack
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      faf4e57f
    • Kubernetes Submit Queue's avatar
      Merge pull request #48324 from jamie-liu/master · 9cde4bd6
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 48264, 48324, 48125, 47944, 47489)
      
      Remove duplicated line from ceph-secret-admin.yaml
      
      **What this PR does / why we need it**:
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      ```
      9cde4bd6
    • Kubernetes Submit Queue's avatar
      Merge pull request #48264 from johscheuer/set-quota-for-volumes · 4fe05d81
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 48264, 48324, 48125, 47944, 47489)
      
      Set quota for volumes
      
      **What this PR does / why we need it**:
      This PR allows users of the Quobyte Storage class to specify if automatically a Quota for the volume should be created. With a Quota a Quobyte volume can only grow in the specified size.
      
      **Special notes for your reviewer**:
      Update the Quobyte API version for the needed functionality.
      4fe05d81
    • Kubernetes Submit Queue's avatar
      Merge pull request #48115 from shikhi7/master · 754ed208
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 48317, 48313, 48351, 48357, 48115)
      
      Formatted Dockerfile to be cleaner and precise
      
      **What this PR does / why we need it**:
      This PR includes good practices which should be followed while writing Dockerfiles as it involves cleaner and clearer syntax.
      
      **Special notes for your reviewer**:
      Includes good practices of writing Dockerfile. Maybe we may allow these types of changes to all old Dockerfiles.
      754ed208
    • Kubernetes Submit Queue's avatar
      Merge pull request #48357 from faraazkhan/master · 36342629
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 48317, 48313, 48351, 48357, 48115)
      
      allow heapster clusterrole to see deployments
      
      **What this PR does / why we need it**:
      
      Currently the default `system:heapster` ClusterRole does not allow access to the `deployment.extensions`. This limits metric collection and results in `DENY` messages in the kube-api logs like these:
      
      ```
      [kube-apiserver-ip-10-0-10-12.us-west-2.compute.internal] I0630 19:22:16.088301       1 rbac.go:87] RBAC DENY: user "system:serviceaccount:kube-system:heapster" groups [system:serviceaccounts system:serviceaccounts:kube-system system:authenticated] cannot "get" resource "deployments.extensions" named "heapster-v1.4.0" in namespace "kube-system"
      ```
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #  https://github.com/kubernetes/heapster/issues/1710
      
      **Special notes for your reviewer**: NONE
      
      **Release note**:
      
      ```release-note
      Allow the system:heapster ClusterRole read access to deployments
      ```
      36342629
    • Kubernetes Submit Queue's avatar
      Merge pull request #48351 from juju-solutions/bug/get-pass · dc597291
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 48317, 48313, 48351, 48357, 48115)
      
      Ensure get_password is accessing a file that exists.
      
      **What this PR does / why we need it**: get_password will throw an exception instead of returning None in case the basic_auth.csv file is missing but /root/cdk/ is there in a juju deployment.
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/324
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```
      Fix race condition where /root/cdk is not yet initialised in kubernetes-master setup by Juju  
      ```
      dc597291
    • Kubernetes Submit Queue's avatar
      Merge pull request #48313 from timstclair/lint · 21038e36
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 48317, 48313, 48351, 48357, 48115)
      
      Cleanup lint errors in the pkg/kubelet/server/... directory
      
      Cleanup some issues that were found while experimenting with https://github.com/alecthomas/gometalinter on the `pkg/kubelet/server/...` directories.
      21038e36
    • Kubernetes Submit Queue's avatar
      Merge pull request #48317 from CaoShuFeng/impersonation_group · fbba6709
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      remove useless check from impersonation filter
      
      When groupsSpecified is false, that means no other groups are added
      rather than the service account groups. So this check doesn't make
      any sense.
      
      **Release note**:
      
      ```
      NONE
      ```
      fbba6709
    • Kubernetes Submit Queue's avatar
      Merge pull request #48299 from mengqiy/kubectl_term · 73a94eac
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 47918, 47964, 48151, 47881, 48299)
      
      move term to kubectl/util
      
      move term from pkg/util/term to pkg/kubectl/util/term
      
      remove dependency of `k8s.io/kubernetes/pkg/util/term` for `pkg/kubelet/dockershim/exec.go` and `pkg/kubelet/dockershim/exec.go`
      
      Ref: https://github.com/kubernetes/kubernetes/issues/48209
      
      ```release-note
      NONE
      ```
      /assign @apelisse @monopole 
      
      cc: @pwittrock
      73a94eac
    • Kubernetes Submit Queue's avatar
      Merge pull request #47881 from cadmuxe/endpoint · c0337c92
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 47918, 47964, 48151, 47881, 48299)
      
      Add ApiEndpoint support to GCE config.
      
      **What this PR does / why we need it**:
      Add the ability to change ApiEndpoint  for GCE.
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #
      
      **Special notes for your reviewer**:
      
      **Release note**:
      ```release-note
      None
      ```
      c0337c92
    • Kubernetes Submit Queue's avatar
      Merge pull request #48151 from apelisse/accept-encoding-gzip · 19724f0b
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 47918, 47964, 48151, 47881, 48299)
      
      GZip openapi schema if accepted by client
      
      **What this PR does / why we need it**: Uses gzip "Accept-Encoding" flag rather than specific path to download gzipped openapi schema.
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #48375
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      ```
      19724f0b
    • Kubernetes Submit Queue's avatar
      Merge pull request #47964 from hasanatkazmi/master · 9a303eae
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 47918, 47964, 48151, 47881, 48299)
      
      removed 'Storage' option from 'kubectl top' like options
      
      **What this PR does / why we need it**:
      https://github.com/kubernetes/kubectl/issues/34
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      ```
      9a303eae
    • Kubernetes Submit Queue's avatar
      Merge pull request #47918 from jiayingz/unit-test · 57dddce8
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 47918, 47964, 48151, 47881, 48299)
      
      Add unit test coverage for nvidiaGPUManager initialization
      
      Part of #47750 
      
      ```release-note
      NONE
      ```
      57dddce8
  4. 30 Jun, 2017 2 commits
    • Tim Hockin's avatar
      Merge pull request #47934 from wlan0/master · a0db2872
      Tim Hockin authored
      enable docs and man page autogeneration for cloud-controller-manager
      a0db2872
    • Kubernetes Submit Queue's avatar
      Merge pull request #47058 from dcbw/remove-dead-kubelet-shaping · 4b354d48
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 43558, 48261, 42376, 46803, 47058)
      
      kubelet: remove unused bandwidth shaping teardown code
      
      Since v1.5 and the removal of --configure-cbr0:
      
      0800df74 "Remove the legacy networking mode --configure-cbr0"
      
      kubelet hasn't done any shaping operations internally.  They
      have all been delegated to network plugins like kubenet or
      external CNI plugins.  But some shaping code was still left
      in kubelet, so remove it now that it's unused.
      
      @freehan @thockin @bprashanth
      4b354d48