1. 21 Aug, 2017 1 commit
  2. 19 Aug, 2017 4 commits
  3. 18 Aug, 2017 35 commits
    • Kubernetes Submit Queue's avatar
      Merge pull request #47991 from ncdc/fix-initial-exec-terminal-dimensions · 2ab7ad14
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Fix initial exec terminal dimensions
      
      **What this PR does / why we need it**:
      Delay attempting to send a terminal resize request to docker until after
      the exec has started; otherwise, the initial resize request will fail.
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #47990
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      2ab7ad14
    • Michael Taufen's avatar
    • Kubernetes Submit Queue's avatar
      Merge pull request #50101 from bowei/owners · 683f09c6
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Update OWNERS files for networking components
      
      This will reduce the approval load for the top level tree owners
      
      ```release-note
      NONE
      ```
      683f09c6
    • Kubernetes Submit Queue's avatar
      Merge pull request #50490 from liyinan926/master · 5cc3cda0
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Use CollisionCount for collision avoidance in StatefulSet controller
      
      **What this PR does / why we need it**:
      This PR uses the newly added `CollisionCount` in `StatefulSetStatus` for name collision avoidance when the `StatefulSet` controller creates `ControllerRevision`s. The `CreateControllerRevision` method of the `ControllerHistory` interface was augmented to use a user-specified `collisionCount` instead of the internal probe that always starts with 0. The `StatefulSet` controller uses the `CreateControllerRevision` to create `ControllerRevision`s and when it calls it, it passes in the `CollisionCount` of the `StatefulSet` it manipulates. 
        
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #49909.
      
      **Special notes for your reviewer**:
      /assign @kow3ns 
      
      **Release note**:
      ```release-note
      Use CollisionCount for collision avoidance when creating ControllerRevisions in StatefulSet controller
      ```
      5cc3cda0
    • Kubernetes Submit Queue's avatar
      Merge pull request #50860 from wlan0/master · 9553e89a
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Proposal to add @wlan0 to appropriate owner files
      
      @thockin @luxas
      9553e89a
    • Kubernetes Submit Queue's avatar
      Merge pull request #50839 from tengqm/fix-admission-controller-reg · 0ade03bc
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 50911, 50839)
      
      Fix admission plugin registration
      
      **What this PR does / why we need it**:
      The current registration of admission plug-ins happen too late so the
      command line help is not yielding correct/useful information to users.
      This is already causing confusion as mentioned in some issues.
      
      This patch fixes it by moving plugins.go down to options package because
      "plugins" in this context are themselves options for users. Registration
      of plugins is not an expensive operation and it is already done in most
      execution paths.
      
      **Which issue this PR fixes**
      fixes #50833 
      
      **Special notes for your reviewer**:
      There might be other thoughts on how to better align this with other
      efforts. This PR is leaving them to future work. For example:
      in future, we may want to revisit these plugins and migrate them to the
      shared apiserver repo when appropriate.
      
      **Release note**:
      I'm not sure if a release note is needed for this. Maybe there is a need.
      
      ```release-note
      ```
      0ade03bc
    • Kubernetes Submit Queue's avatar
      Merge pull request #50911 from deads2k/api-01-diff · 09a1eb3f
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      add diff details to pod validation error
      
      This adds a basic (large) diff to the `ValidatePodUpdate` method, but we've been waiting on the "perfect" diff for a long time and it hasn't appeared.
      09a1eb3f
    • Kubernetes Submit Queue's avatar
      Merge pull request #50691 from fabriziopandini/kubeadm-phases-certs3 · cc90d34f
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 50904, 50691)
      
      Make Kubeadm phase certs codebase consistent with other phases
      
      **What this PR does / why we need it:**
      This PR is a refactoring of [#48196](https://github.com/kubernetes/kubernetes/pull/48196), that makes this part of kubeadm more consistent with other parts of kubeadm recently changed, e.g. [controlplane & etcd phase #50302](https://github.com/kubernetes/kubernetes/pull/50302) and [kubeconfig phase #49419](https://github.com/kubernetes/kubernetes/pull/49419)
      
      **Which issue this PR fixes:**
      none
      
      **Special notes for your reviewer:**
      cc @luxas
      cc90d34f
    • Kubernetes Submit Queue's avatar
      Merge pull request #50904 from crassirostris/sd-logging-e2e-system-logs · 26eb7c94
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 50904, 50691)
      
      Stackdriver Logging e2e: Explicitly check for docker and kubelet logs presence
      
      Check for kubelet and docker logs explicitly in the Stackdriver Logging e2e tests
      26eb7c94
    • Yinan Li's avatar
    • David Eads's avatar
      add diff details to pod validation error · 53348c05
      David Eads authored
      53348c05
    • Mik Vyatskov's avatar
    • Kubernetes Submit Queue's avatar
      Merge pull request #50876 from enj/enj/f/owners_reviewer · afabd098
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Add enj as reviewer to OWNERS
      
      Adding myself as a reviewer for the following areas:
      
      - API
      - auth
      - registry
      - storage (etcd)
      Signed-off-by: 's avatarMonis Khan <mkhan@redhat.com>
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      
      @kubernetes/sig-api-machinery-pr-reviews
      @kubernetes/sig-auth-pr-reviews
      afabd098
    • Kubernetes Submit Queue's avatar
      Merge pull request #48263 from CaoShuFeng/useless_argument · c3a39f76
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      remove useless argument "name"
      
      **Release note**:
      
      ```
      NONE
      ```
      c3a39f76
    • Kubernetes Submit Queue's avatar
      Merge pull request #50885 from CaoShuFeng/webhook_dead_code · 2ba796fe
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 50255, 50885)
      
      remove dead code for cloner
      
      I found some dead code in audit webhook backend.
      This change do some clean work for: https://github.com/kubernetes/kubernetes/pull/48544/commits/2bbe72d4e09f7c95e1ad851187d4733a54644fbe
      
      **Release note**:
      ```
      NONE
      ```
      2ba796fe
    • Kubernetes Submit Queue's avatar
      Merge pull request #50255 from mattlandis/aws-duplicate-subnets · c573d819
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 50255, 50885)
      
      AWS: Arbitrarily choose first (lexicographically) subnet in AZ
      
      When there is more than one subnet for an AZ on AWS choose arbitrarily
      chose the first one lexicographically for consistency.
      
      
      
      **What this PR does / why we need it**:
      If two subnets were to be used appear in the same aws az which one is chosen is currently not consistent.  This could lead to difficulty in diagnosing issues.
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #45983
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      ```
      c573d819
    • Kubernetes Submit Queue's avatar
      Merge pull request #50856 from shyamjvs/save-route-controller-memory · 4e416812
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 50303, 50856)
      
      Make route-controller list only relevant routes instead of all of them
      
      Ref https://github.com/kubernetes/kubernetes/issues/50854 (somewhat related issue)
      
      IIUC from the code, route-controller memory is mainly being used in storing routes and nodes (also CIDRs, but that's not much).
      This should help reduce that memory usage (particularly when running in a project with large no. of routes), by moving filtering to server-side.
      For e.g in kubernetes-scale project we have ~5000 routes (each about 600B) => 3 MB of routes
      
      This doesn't help with reducing time to list the routes as filtering is also linear.
      
      cc @kubernetes/sig-scalability-misc @wojtek-t @gmarek
      4e416812
    • Kubernetes Submit Queue's avatar
      Merge pull request #50303 from zouyee/newb · 2c6cb24e
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      delete ineffectual assginment
      
      **What this PR does / why we need it**:
      delete ineffectual assignment
      **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**:
      2c6cb24e
    • Kubernetes Submit Queue's avatar
      Merge pull request #50873 from enj/patch-1 · e472a758
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Add enj to OWNERS for test/integration/etcd/etcd_storage_path_test.go
      
      @deads2k is the bot smart enough to not spam me with every test change?  Perhaps I should create an `OWNERS` file in `test/integration/etcd`?
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      
      @kubernetes/sig-api-machinery-pr-reviews
      e472a758
    • Kubernetes Submit Queue's avatar
      Merge pull request #50765 from sttts/sttts-kube-gen-rename · d543a7c7
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Rename k8s.io/{kube-gen -> code-generator}
      d543a7c7
    • fabriziopandini's avatar
      Autogenerated bazel files · c6bb8fbb
      fabriziopandini authored
      c6bb8fbb
    • fabriziopandini's avatar
      Main work -- cleanup certs CLI command · 00fa026b
      fabriziopandini authored
      00fa026b
    • fabriziopandini's avatar
      Main work -- refactor certs phase · 5fac458f
      fabriziopandini authored
      5fac458f
    • Kubernetes Submit Queue's avatar
      Merge pull request #49617 from duan-yue/code_cls · 38053c3e
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      fix typo
      
      **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
      NONE
      ```
      38053c3e
    • Kubernetes Submit Queue's avatar
      Merge pull request #50575 from dixudx/CollisionCount_int64_to_int32 · e553d6eb
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      CollisionCount should have type int32 across controllers that use it for collision avoidance
      
      **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 #50530
      
      **Special notes for your reviewer**:
      /cc @liyinan926
      /assign @kow3ns @thockin @janetkuo 
      
      **Release note**:
      
      ```release-note
      Change CollisionCount from int64 to int32 across controllers
      ```
      e553d6eb
    • Kubernetes Submit Queue's avatar
      Merge pull request #49993 from mathspanda/master · 4564f3d8
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      verify pkg/util contains no code
      
      **What this PR does / why we need it**:
      verify util packages contain no go codes
      
      related issue: #49923
      4564f3d8
    • Kubernetes Submit Queue's avatar
      Merge pull request #50867 from mrballcb/newrelic-infrastructure-example · 67fb74d2
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 50277, 50823, 50376, 50867)
      
      NR Infrastructure agent example daemonset
      
      Copy of previous newrelic example, then modified to use the new agent
      "newrelic-infra" instead of "nrsysmond".
      
      Also maps all of host node's root fs into /host in the container (ro,
      but still exposes underlying node info into a container).
      
      Updates to README
      
      
      
      **What this PR does / why we need it**:  It provides an example basic configuration to get New Relic Infrastructure monitoring agent installed as a daemonset in containers on a cluster.  It started as a clone of examples/newrelic/, then updated to use the "infrastructure" docker image provided by New Relic.
      
      **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**:
      
      ```NONE
      ```
      67fb74d2
    • Kubernetes Submit Queue's avatar
      Merge pull request #50376 from guangxuli/add_sig_prefix_to_e2e_test · bf69aafd
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 50277, 50823, 50376, 50867)
      
      Move e2e taints test file to sig-scheduling
      
      **What this PR does / why we need it**:
      Move taint test file to e2e scheduling and add sig-scheduling prefix.
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #
      Ref Umbrella issue #49161
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      none
      bf69aafd
    • Kubernetes Submit Queue's avatar
      Merge pull request #50823 from zjj2wry/set-image · 89abb48f
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 50277, 50823, 50376, 50867)
      
      fix issue(#50821)Add image check, if image not changed, transform false
      
      **What this PR does / why we need it**:
      closes #50821 
      **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
      ```
      89abb48f
    • Kubernetes Submit Queue's avatar
      Merge pull request #50277 from yguo0905/live-restore-test · a4f6ae44
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Add node e2e test for Docker's live-restore
      
      Ref: https://github.com/kubernetes/kubernetes/issues/42926
      
      This PR adds a test for docker live-restore. If this is fine, we can close the unfinished PR https://github.com/kubernetes/kubernetes/pull/40364.
      
      **Release note**:
      ```
      None
      ```
      a4f6ae44
    • Cao Shufeng's avatar
      remove dead code for cloner · 7b5c7bb7
      Cao Shufeng authored
      I found some dead code in audit webhook backend.
      This change do some clean work for: https://github.com/kubernetes/kubernetes/pull/48544/commits/2bbe72d4e09f7c95e1ad851187d4733a54644fbe
      7b5c7bb7
    • Tim(Xiaoyu) Zhang's avatar
      Reduce one time url direction · 623d0efb
      Tim(Xiaoyu) Zhang authored
      Reduce one time url direction
      623d0efb
    • Di Xu's avatar
      auto-gen · b36320be
      Di Xu authored
      b36320be
    • Kubernetes Submit Queue's avatar
      Merge pull request #50837 from tizhou86/master · a1234aa2
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Fix several typos in CHANGELOG.md file
      
      **What this PR does / why we need it**:
      
      Fixed several typos in CHANGELOG.md file.
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #
      
      NONE
      
      **Special notes for your reviewer**:
      
      NONE
      
      **Release note**:
      
      ```release-note
      
      NONE
      
      ```
      a1234aa2
    • Kubernetes Submit Queue's avatar
      Merge pull request #50218 from dixudx/fix_GPU_resource_zero_limit_validation · 075d209e
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      fix GPU resource validation that incorrectly allows zero limits
      
      **What this PR does / why we need it**:
      
      The validation logic for GPUs is not run if limits is not set for GPUs.
      We need to check limits equals requests even if just request is set for GPUs.
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #50182
      
      **Special notes for your reviewer**:
      /assign @vishh 
      /cc @jiayingz 
      
      **Release note**:
      
      ```release-note
      fix GPU resource validation that incorrectly allows zero limits
      ```
      075d209e