1. 14 Aug, 2017 17 commits
  2. 13 Aug, 2017 4 commits
  3. 12 Aug, 2017 19 commits
    • Kubernetes Submit Queue's avatar
      Merge pull request #50362 from k82cn/k8s_50360 · 2820b45c
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Moved node condition filter into a predicates.
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #50360 
      
      **Release note**:
      ```release-note
      A new predicates, named 'CheckNodeCondition', was added to replace node condition filter. 'NetworkUnavailable', 'OutOfDisk' and 'NotReady' maybe reported as a reason when failed to schedule pods.
      ```
      2820b45c
    • Kubernetes Submit Queue's avatar
      Merge pull request #50479 from yguo0905/node-perf-m60 · cf80b91a
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 49847, 49743, 49853, 50225, 50479)
      
      Add node benchmark tests for cos-m60 with docker 1.12.6
      
      Ref: https://github.com/kubernetes/kubernetes/issues/42926
      
      This PR adds a benchmark tests against cos-m60 with docker 1.12.6 on http://node-perf-dash.k8s.io. This test is useful for docker validation -- we can compare the performance of different dockers on the same OS.
      
      cos-m60 comes with docker 1.13.1 by default, so we need to use cloud-init to downgrade the version to 1.12.6.
      
      **Release note**:
      ```
      None
      ```
      
      /assign @dchen1107
      cf80b91a
    • Kubernetes Submit Queue's avatar
      Merge pull request #50225 from tcharding/kubectl-run-log · a1933f8e
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 49847, 49743, 49853, 50225, 50479)
      
      Remove duplicate logging code
      
      **What this PR does / why we need it**:
      
      Currently function `handleAttachPod` contains duplicate code which copies the AttachOptions output writer to the pod logging writer. This code can be refactored into a separate function. 
      
      **Special notes for your reviewer**:
      
      Refactor only, does not change program logic.
      
      **Release note**:
      ```release-note
      NONE
      ```
      
      /sig cli
      /kind cleanup
      a1933f8e
    • Kubernetes Submit Queue's avatar
      Merge pull request #49853 from duan-yue/capabilities · 6d91ad2d
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 49847, 49743, 49853, 50225, 50479)
      
      refactor capabilities to a singleton struct
      
      **What this PR does / why we need it**:
      refactor
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #
      refactor
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      6d91ad2d
    • Kubernetes Submit Queue's avatar
      Merge pull request #49743 from euank/hostname · 1f2185fb
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 49847, 49743, 49853, 50225, 50479)
      
      cloudprovider/photon: remove unneeded bash exec
      
      **Release note**:
      ```release-note
      NONE
      ```
      1f2185fb
    • Kubernetes Submit Queue's avatar
      Merge pull request #49847 from m1093782566/fix-win-proxy · 2d4c609a
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Fix winspace proxier wrong comment message
      
      **What this PR does / why we need it**:
      
      Since winspace proxier has nothing to do with iptables, this PR remove the wrong comment message on iptables.
      
      **Which issue this PR fixes**: 
      
      Fixes #50524
      2d4c609a
    • Kubernetes Submit Queue's avatar
      Merge pull request #50353 from… · 527c4488
      Kubernetes Submit Queue authored
      Merge pull request #50353 from zhangxiaoyu-zidif/Delete-redundant-expectError-for-controller-ref-test
      
      Automatic merge from submit-queue
      
      Delete redundant test para. for controller_ref_manager_test
      
      **What this PR does / why we need it**:
      The test does not use para. expectError.
      
      **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
      ```
      527c4488
    • Kubernetes Submit Queue's avatar
      Merge pull request #49719 from dixudx/fix_apps_deploymentspec_conversion · c1da492a
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      fix apps DeploymentSpec conversion issue
      
      **What this PR does / why we need it**:
      
      When working on #49645, I found current conversion for `v1.PodTemplateSpec` to `api.PodTemplateSpec` did not work properly. It should function as [L244-L246](https://github.com/kubernetes/kubernetes/blob/master/pkg/apis/apps/v1beta1/conversion.go#L244-L246).
      
      **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**:
      /cc @janetkuo 
      
      **Release note**:
      
      ```release-note
      fix apps DeploymentSpec conversion issue
      ```
      c1da492a
    • Kubernetes Submit Queue's avatar
      Merge pull request #50520 from shyamjvs/allow-kubemark-test-args · 00145732
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Add variables for passing test args to kubemark master components
      
      cc @msau42 - This change will enable us to turn on extender in the scheduler in kubemark-scale job
      00145732
    • Kubernetes Submit Queue's avatar
      Merge pull request #50506 from bskiba/external_config · 276bfb8c
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 50485, 49951, 50508, 50511, 50506)
      
      Pass config to external Kubemark cluster in e2e tests
      
      When cluster autoscaler is used in kubemark tests,
      pass default kubeconfig as external cluster config.
      
      @shyamjvs @gmarek 
      
      **Release note**:
      ```
      NONE
      ```
      276bfb8c
    • Kubernetes Submit Queue's avatar
      Merge pull request #50511 from crassirostris/fix-event-exporter · b0e066ee
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 50485, 49951, 50508, 50511, 50506)
      
      Update Stackdriver event exporter version
      
      Fixes https://github.com/kubernetes/kubernetes/issues/50510
      b0e066ee
    • Kubernetes Submit Queue's avatar
      Merge pull request #50508 from YuxiJin-tobeyjin/kubectltypo · 9f7ddb64
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 50485, 49951, 50508, 50511, 50506)
      
      fix a typo
      
      **What this PR does / why we need it**:
      fix a small typo
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #
      verions->versions
      **Special notes for your reviewer**:
      
      **Release note**:
      NONE
      ```release-note
      ```NONE
      9f7ddb64
    • Kubernetes Submit Queue's avatar
      Merge pull request #49951 from mkumatag/multiarch_nonewprivs · bbc74a34
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 50485, 49951, 50508, 50511, 50506)
      
      Multiarch nonewprivs test image
      
      **What this PR does / why we need it**:
      This PR is for converting nonewprivs image which pushed very recently part of https://github.com/kubernetes/kubernetes/pull/47019.
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #
      Fixes #50498 
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```NONE```
      bbc74a34
    • Kubernetes Submit Queue's avatar
      Merge pull request #50485 from jianglingxia/jlx081110 · 369d5357
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      get_test.go fix error format and info
      
      **What this PR does / why we need it**:
      there left the only one need modify,thanks
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #49481 
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      369d5357
    • Kubernetes Submit Queue's avatar
      Merge pull request #50205 from dixudx/fix_kubectl_edit_panic_nil_list · 577fdf91
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 50537, 49699, 50160, 49025, 50205)
      
      not allowing "kubectl edit <resource>" when you got an empty list
      
      **What this PR does / why we need it**:
      `kubectl edit` will panic when adding an empty list.
      
      > panic: runtime error: index out of range
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #50147
      
      **Special notes for your reviewer**:
      /assign @errordeveloper @mengqiy @janetkuo @fabianofranz
      /cc @rootfs @soltysh @sttts
      
      **Release note**:
      
      ```release-note
      not allowing "kubectl edit <resource>" when you got an empty list
      ```
      577fdf91
    • Kubernetes Submit Queue's avatar
      Merge pull request #49025 from danwinship/non-cloud-node-ip · b91f1918
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 50537, 49699, 50160, 49025, 50205)
      
      When not using a CloudProvider, set both InternalIP and ExternalIP on Nodes
      
      #36095 changed all of the cloudproviders to set both InternalIP and ExternalIP on Nodes, but the non-cloudprovider fallback code now only sets InternalIP.
      
      This causes the test "should be able to create a functioning NodePort service" in test/e2e/service.go to fail on cloud-provider-less clusters, because (with LegacyHostIP gone), it now will only try to work with ExternalIPs, and will fail if the node has only an InternalIP.
      
      There isn't much other code that assumes that ExternalIP will always be set (there's something in pkg/master/master.go, but I don't know what it's doing, so maybe it's only useful in the case where InternalIP != ExternalIP anyway). But given that several of the cloudproviders (mesos, ovirt, rackspace) now explicitly set both InternalIP and ExternalIP to the same value always, it seemed right to do that in the fallback case too.
      
      @deads2k FYI
      
      **Release note**:
      ```release-note
      NONE
      ```
      b91f1918
    • Kubernetes Submit Queue's avatar
      Merge pull request #50160 from apelisse/openapi-validation · 937fc0d1
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 50537, 49699, 50160, 49025, 50205)
      
      openapi: Add validation logic
      
      This allows validation of a yaml/json object against an openapi schema.
      A lot more testing would be needed to validate the logic, and also this
      is not plumbed in, so it can't be used by kubectl yet.
      
      **What this PR does / why we need it**: This is implementing validation against the openapi swagger spec rather than the old swagger spec.
      
      **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/kubectl/issues/49
      
      **Special notes for your reviewer**:
      
      **Release note**:
      ```release-note
      NONE
      ```
      937fc0d1
    • Kubernetes Submit Queue's avatar
      Merge pull request #49699 from xingzhou/kube-49384-bug · be1f1439
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 50537, 49699, 50160, 49025, 50205)
      
      AddOrUpdateTaint should ignore duplicate Taint.
      
      The parameter of AddOrUpdateTaint is Taint pointer, so should use
      Taint object itself to compare with the node's taint list to ignore
      duplicate taint.
      
      While doing #49384, found this issue and fixed.
      
      Fixed part of #49384, other test cases will be added in the following patch
      
      **Release note**:
      ```
      None
      ```
      be1f1439
    • Kubernetes Submit Queue's avatar
      Merge pull request #50537 from liggitt/kubefed-rbac · c207dd5a
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 50537, 49699, 50160, 49025, 50205)
      
      select an RBAC version for kubefed it knows how to speak
      
      kubefed tries to speak whatever version of RBAC the server has, regardless of whether it knows about that version or not. the version discovery it does has to select a version both it and the server speak.
      
      related to https://github.com/kubernetes/kubernetes/issues/50534
      
      ```release-note
      fixes kubefed's ability to create RBAC roles in version-skewed clusters
      ```
      c207dd5a