1. 27 Apr, 2017 10 commits
    • NickrenREN's avatar
      Add cinder volume examples · f49fb20e
      NickrenREN authored
      I can not find cinder examples,add in packages examples/volume and examples/persistent-volume-provisioning
      f49fb20e
    • Kubernetes Submit Queue's avatar
      Merge pull request #43618 from xilabao/fix-kubectl-run-output · 65838085
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 44970, 43618)
      
      fix kubectl run output
      
      fixes https://github.com/kubernetes/kubernetes/issues/40440
      65838085
    • Kubernetes Submit Queue's avatar
      Merge pull request #44970 from Random-Liu/fix-stop-container-timeout · c3df35df
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 44970, 43618)
      
      CRI: Fix StopContainer timeout
      
      Fixes https://github.com/kubernetes/kubernetes/issues/44956.
      
      I verified this PR with the example provided in https://github.com/kubernetes/kubernetes/issues/44956, and now pod deletion will respect grace period timeout:
      ```
      NAME                         READY     STATUS        RESTARTS   AGE
      gracefully-terminating-pod   1/1       Terminating   0          6m
      ```
      
      @dchen1107 @yujuhong @feiskyer /cc @kubernetes/sig-node-bugs
      c3df35df
    • Kubernetes Submit Queue's avatar
      Merge pull request #44986 from dashpole/fix_image_gc · e885c77f
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Allow Partial Success for ImageGC
      
      Fixes #44951.  When the eviction manager is under disk pressure, it first attempts to reclaim disk space by deleting images.  However, if there are any errors during the image deletion process, the eviction manager treats that as a failed attempt delete images--even if some were successfully deleted.
      
      This change essentially makes the eviction manager ignore errors during image garbage collection, and instead rely solely on the quantity of resources reclaimed.  If image deletion completely fails, for example, then this should still work as it would return 0 bytes freed.  This allows for partial success, because any resources freed are counted, regardless of if some images fail to be deleted, for example.
      
      This does not require any changes to the image manager, as the current behavior is already to return the disk space freed along with any errors.
      
      ```release-note
      Fixes a bug where pods were evicted even after images are successfully deleted.
      ```
      
      cc @dchen1107 @vishh @kubernetes/kubernetes-release-managers
      
      note to reviewers: this is mostly whitespace changes, so it will make more sense in reviewable
      e885c77f
    • Kubernetes Submit Queue's avatar
      Merge pull request #44935 from yifan-gu/fix_poll · 2e7cc022
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 44940, 44974, 44935)
      
      apimachinery/pkg/util/wait: Fix potential goroutine leak in pollInternal().
      
      **What this PR does / why we need it**:
      
      Without the change, the wait function wouldn't exit until the timeout
      happens, so if the timeout is set to a big value and the Poll() is run
      inside a loop, then the total goroutines will increase indefinitely.
      
      This PR fixes the issue by closing the stop channel to tell the wait function
      to exit immediately if condition is true or any error happens.
      2e7cc022
    • Kubernetes Submit Queue's avatar
      Merge pull request #44974 from caesarxuchao/remove-client-go-api-listers · c446132a
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 44940, 44974, 44935)
      
      Remove import of internal api package in generated external-versioned listers
      
      Follow up of https://github.com/kubernetes/kubernetes/pull/44523
      
      One line change in cmd/libs/go2idl/lister-gen/generators/lister.go, and simple changes in pkg/apis/autoscaling/v2alpha1/register.go, other changes are generated.
      
      The internal api package will be eliminated from client-go, so these imports should be removed. Also, it's more correct to report the versioned resource in the error.
      c446132a
    • Kubernetes Submit Queue's avatar
      Merge pull request #44940 from sjenning/bump-runc · a92007b4
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Bump runc to d223e2a
      
      Fixes https://github.com/kubernetes/kubernetes/issues/43856
      
      @derekwaynecarr
      a92007b4
    • Kubernetes Submit Queue's avatar
      Merge pull request #43469 from enisoc/has-conflicts · 904b0207
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Fix mergepatch.HasConflicts().
      
      **What this PR does / why we need it**:
      
      This fixes some false negatives:
      
      * If a map had multiple entries, only the first was checked.
      * If a list had multiple entries, only the first was checked.
      
      **Which issue this PR fixes**:
      
      **Special notes for your reviewer**:
      
      **Release note**:
      ```release-note
      NONE
      ```
      904b0207
    • Kubernetes Submit Queue's avatar
      Merge pull request #43188 from vmware/VSANPolicyTest · 2c8a1565
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      e2e tests for VSAN policy support in Kubernetes for vSphere
      
      Following e2e test cases have been implemented for VSAN policy support in Kubernetes for vSphere. These e2e tests are for PR #42974 which out for review.
      
      A total of 8 test cases for below mentioned 5 different scenarios are implemented.
      
      Test cases:
      
      1. Validation of VSAN capabilities.
      - hostFailuresToTolerate : Minimum 1 and Max 3. Should be integer.
      - stripeWidth: Minimum is 1 and Maximum is 12. Should be integer.
      - proportionalCapacity: Expressed in percentage. Should be between 0 and 100. Should be integer.
      - iopsLimit: Should be greater than 0. Should be integer.
      
      2. Use a VSAN testbed setup. Try valid VSAN capabilities which are supported by VSAN testbed. Make sure the disk is created with policy configured with it.
      - Ex: Using hostFailuresToTolerate=0 and cacheReservation=12
      Ex: diskStripes=1 and objectSpaceReservation=30
      
      3. Use a VSAN testbed setup. Try valid VSAN capabilities which are not supported by VSAN testbed. Make sure that the disk creation fails and PVC status is pending.
      
      4. Try using VSAN capabilities on a non-VSAN datastore. PVC status will be pending and it errors to the user saying to VSAN capabilities are not supported on a non-VSAN testbed.
      
      5. Try all 1 to 5 with custom datastore specified by the user.
      
      @jeffvance  @divyenpatel
      
      **Release note**:
      
      ```release-note
      None
      ```
      2c8a1565
    • David Ashpole's avatar
  2. 26 Apr, 2017 30 commits
    • Kubernetes Submit Queue's avatar
      Merge pull request #44531 from pwittrock/kubectl-openapi · 433aec11
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      OpenAPI support for kubectl
      
      Support for openapi spec in kubectl.
      
      Includes:
      - downloading and caching openapi spec to a local file
      - parsing openapi spec into binary serializable datastructures (10x faster load times 600ms -> 40ms)
      - caching parsed openapi spec in memory for each command
      
      ```release-note
      NONE
      ```
      433aec11
    • Random-Liu's avatar
      Fix StopContainer timeout · cfd0efff
      Random-Liu authored
      cfd0efff
    • Kubernetes Submit Queue's avatar
      Merge pull request #44967 from chuckbutler/43461 · 15533fac
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Fixes #43461
      
      **What this PR does / why we need it**:
      The master-components started state triggers a daemon recycle. The guard
      was to prevent the daemons from being cycled too often and interrupting
      normal workflow. This additional state check is probing the etcd
      connection and if changing triggers a re-configure and recycle of the api-control 
      plane when etcd units are scaling up and down.
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #43461
      
      **Special notes for your reviewer**:
      Check the contents of /var/snap/kube-apiserver/current/args after scaling etcd both up and down and the values will have changed, and kube-apiserver will have recycled to read the new connection string.
      
      **Release note**:
      
      ```release-note
      kubernetes-master juju charm properly detects etcd-scale events and reconfigures appropriately.
      ```
      15533fac
    • Kubernetes Submit Queue's avatar
      Merge pull request #44451 from ncdc/spdy-follow-redirects · 274df99e
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Add redirect support to SpdyRoundTripper
      
      Add support for following redirects to the SpdyRoundTripper. This is
      necessary for clients using it directly (e.g. the apiserver talking
      directly to the kubelet) because the CRI streaming server issues a
      redirect for streaming requests.
      
      We need this in OpenShift because we have code that executes inside our apiserver that talks directly to the node to perform an attach request, and we need to be able to follow that redirect.
      
      This code was adapted from the upgrade-aware proxy handler.
      
      cc @smarterclayton @sttts @liggitt @timstclair @kubernetes/sig-api-machinery-pr-reviews
      274df99e
    • Chao Xu's avatar
      bazel · ee61ffb4
      Chao Xu authored
      ee61ffb4
    • Chao Xu's avatar
    • Kubernetes Submit Queue's avatar
      Merge pull request #44960 from shashidharatd/federation-service-controller-2 · ed0adbf3
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 43395, 44960)
      
      [Federation] Cleanup unused code in service controller
      
      Post merging the PR #41258, lot of unused code is left behind in federation service controller. These changes were segregated to this PR, so that the original PR was small and manageable.
      
      **Release note**:
      ```
      NONE
      ```
      
      cc @kubernetes/sig-federation-pr-reviews
      ed0adbf3
    • Kubernetes Submit Queue's avatar
      Merge pull request #43395 from sjenning/selinux-e2e-node-lifecycle · 7670341f
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 43395, 44960)
      
      e2e-node: refactor lifecycle test to avoid selinux issues
      
      Fixes #42905
      
      Previously, the exec hook tests mounted a HostPath volume from /tmp and touched a file as a indicator that the hook had run.  This is prohibited by selinux policy on Fedora/RHEL/Centos.
      
      This PR refactors the test to avoid filesystem indication and use the same indication that the HTTP hooks use; a GET to a http endpoint.  The exec hooks run `curl` to hit this endpoint and trigger the indication.  This simplifies this test quite a bit as well, removing over 85 lines of code.
      
      REVIEWER NOTE: The diff is a mess on this one.  Probably better to just review the new version of the file.
      
      @derekwaynecarr
      7670341f
    • Kubernetes Submit Queue's avatar
      Merge pull request #44964 from rmmh/docker-push · e8abe449
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 44962, 44964)
      
      Fix the last deprecated "gcloud docker push args" usage.
      
      This deprecated usage will break in March 2017.
      
      **Release note**:
      ```release-note
      NONE
      ```
      e8abe449
    • Balu Dontu's avatar
    • Kubernetes Submit Queue's avatar
      Merge pull request #44962 from redbaron/fix-misleading-error · 17688c7c
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Remove misleading error from CronJob controller when it can't find parent UID
      
      CronJob controller lists all `Jobs` and then tries to match then against `CronJobs` using parent UID. It reports error if parent UID can't be found which means it reports error  on every job which is not part of CronJob
      
      
      ```release-note
      NONE
      ```
      17688c7c
    • Charles Butler's avatar
      Fixes #43461 · 690be0fb
      Charles Butler authored
      The master-components started state triggers a daemon recycle. The guard
      was to prevent the daemons from being cycled too often and interrupting
      normal workflow. This additional state check is guarded against the etcd
      connection string from changing, allowing the current behavior but
      triggers a re-configure and recycle of the api-control plane when etcd
      units are scaling up and down.
      690be0fb
    • Seth Jennings's avatar
      bump runc to d223e2a · fda2c9fa
      Seth Jennings authored
      fda2c9fa
    • Ryan Hitchman's avatar
      0c6fd625
    • Kubernetes Submit Queue's avatar
      Merge pull request #44744 from yguo0905/ubuntu-gce-cluster-tests · 6d17ab32
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Support running Ubuntu image on GCE
      
      **What this PR does / why we need it**:
      
      This PR (on top of #44629) contains the script changes for running Ubuntu image on GCE.
      
      **Special notes for your reviewer**:
      
      We made change in `gci/node.yaml` and `gci/master.yaml` to ensure that Kubernetes jobs can start automatically after reboot. This is not needed for GCI but required by Ubuntu. See https://github.com/kubernetes/kubernetes/pull/44744#discussion_r113105970 for details. With this change, Ubuntu could use the same provisioning scripts as GCI's.
      
      Ran e2e tests using the following command and all tests passed.
      
      ```
      KUBE_GCE_NODE_IMAGE=ubuntu-gke-1604-xenial-v20170420-1 KUBE_GCE_NODE_PROJECT=ubuntu-os-gke-cloud KUBE_NODE_OS_DISTRIBUTION=ubuntu GINKGO_PARALLEL=y GINKGO_PARALLEL_NODES=30 go run hack/e2e.go -- -v --build --up --test --test_args="--ginkgo.skip=\[Slow\]|\[Serial\]|\[Disruptive\]|\[Flaky\]|\[Feature:.+\]" --down
      ```
      
      Also tested manually for both GCI and Ubuntu images.
      
      **Release note**:
      `Support Ubuntu 16.04 image on GCE`
      6d17ab32
    • Maxim Ivanov's avatar
    • Kubernetes Submit Queue's avatar
      Merge pull request #44945 from ktsakalozos/bug/dns-fix · b5caa6be
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Send dns details only after cdk-addons are configured
      
      **What this PR does / why we need it**: This is a bugfix on the deployment of Kubernetes via Juju. See issue below.
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #40386 and
      https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/262
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```
      Fix KubeDNS issue in Juju deployments. 
      ```
      b5caa6be
    • shashidharatd's avatar
      Auto generated bazel build file · b0f8de73
      shashidharatd authored
      b0f8de73
    • shashidharatd's avatar
      d2462c79
    • Kubernetes Submit Queue's avatar
      Merge pull request #41258 from shashidharatd/federation-service-controller-1 · 2d79d53f
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 44942, 41258)
      
      [Federation] Use federated informer for service controller and annotations to store lb ingress
      
      **What this PR does / why we need it**:
      This is breaking up of the PR #40296 into smaller one. please refer to #41253
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #
      Handles 2 tasks in #41253
      Fixes issues in #27623, #35827
      
      **Special notes for your reviewer**:
      
      **Release note**:
      ```
      NONE
      ```
      
      cc @quinton-hoole @nikhiljindal @kubernetes/sig-federation-pr-reviews
      2d79d53f
    • Kubernetes Submit Queue's avatar
      Merge pull request #44942 from YuPengZTE/devMaximum · 7f811a3a
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      fix typo
      Signed-off-by: 's avataryupengzte <yu.peng36@zte.com.cn>
      
      
      
      **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
      ```
      7f811a3a
    • Andy Goldstein's avatar
      Update bazel · fc2128c8
      Andy Goldstein authored
      fc2128c8
    • Andy Goldstein's avatar
      Add redirect support to SpdyRoundTripper · 715d5d9c
      Andy Goldstein authored
      Add support for following redirects to the SpdyRoundTripper. This is
      necessary for clients using it directly (e.g. the apiserver talking
      directly to the kubelet) because the CRI streaming server issues a
      redirect for streaming requests.
      
      Also extract common logic for following redirects.
      715d5d9c
    • Kubernetes Submit Queue's avatar
      Merge pull request #44866 from shyamjvs/prepare-for-logexporter · e917f4ed
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Allow disabling log dump for nodes (in preparation for using logexporter)
      
      This is, in part, a change required for allowing usage of [logexporter](https://github.com/kubernetes/test-infra/tree/master/logexporter) for dumping node logs to GCS directly, instead of doing it through log-dump.sh.
      
      cc @kubernetes/test-infra-maintainers @wojtek-t @gmarek @fejta
      e917f4ed
    • Kubernetes Submit Queue's avatar
      Merge pull request #44808 from shiywang/SemanticDeepEqual · 3e16a27d
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 44931, 44808)
      
      Update to use Semantic.DeepEqual in regsitry
      
      **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 https://github.com/kubernetes/kubernetes/issues/43402
      
      **Release note**:
      ```
      NONE
      ```
      
      /assign @janetkuo
      3e16a27d
    • Kubernetes Submit Queue's avatar
      Merge pull request #44931 from chuckbutler/worker-pause-action · 4fdee60b
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 44931, 44808)
      
      Closes #44392
      
      **What this PR does / why we need it**:
      
      Fix the pause action with regard to the new behavior where
      --delete-local-data=false by default. Historically --force was all that
      was required, this flag has changed to be more descriptive of the
      actions it's taking.
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #44392
      
      
      **Release note**:
      
      ```release-note
      Added support to the pause action in the kubernetes-worker charm for new flag --delete-local-data
      ```
      4fdee60b
    • Kubernetes Submit Queue's avatar
      Merge pull request #44731 from dmmcquay/kubeadm_manifest_cfg_usage · 508041e9
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      kubeadm: remove func arg that isn't being used
      
      **What this PR does / why we need it**: Functions had args passed to them that weren't being used. See https://github.com/kubernetes/kubernetes/pull/44601 for more details. 
      
      **Special notes for your reviewer**: @luxas
      
      **Release note**:
      ```release-note
      NONE
      ```
      508041e9
    • Kubernetes Submit Queue's avatar
      Merge pull request #43850 from xilabao/add-output-to-kubeadm-version · eb0bc857
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      add --output flag to `kubeadm version`
      
      ref to kubectl https://github.com/kubernetes/kubernetes/pull/39858
      eb0bc857
    • xilabao's avatar
      add output flag to kubeadm version · 3719840e
      xilabao authored
      3719840e
    • Konstantinos Tsakalozos's avatar