1. 19 May, 2017 37 commits
    • Kubernetes Submit Queue's avatar
      Merge pull request #46068 from zjj2wry/ccc · effca960
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      fix changelog link not work
      
      **What this PR does / why we need it**:
      i read other pr has fix it, but now reappear. how can i got generate tool ?
      
      **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
      ```
      effca960
    • Kubernetes Submit Queue's avatar
      Merge pull request #46038 from dnardo/ip-masq-agent · d3aa925c
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 44606, 46038)
      
      Add ip-masq-agent addon to the addons folder. 
      
      This also ensures that under gce we add this DaemonSet if the non-masq-cidr
      is set to 0/0.
      
      
      
      **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
      Add ip-masq-agent addon to the addons folder which is used in GCE if  --non-masquerade-cidr is set to 0/0
      ```
      d3aa925c
    • Kubernetes Submit Queue's avatar
      Merge pull request #44606 from ivan4th/fix-serialization-of-enforce-node-allocatable · 69ce7d84
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 44606, 46038)
      
      Fix serialization of EnforceNodeAllocatable
      
      EnforceNodeAllocatable being `nil` and `[]` are treated in different
      ways by kubelet. Namely, `nil` is replaced with `[]string{"pods"}` by
      the defaulting mechanism.
      
      E.g. if you run kubelet in Docker-in-Docker environment
      you may need to run it with the following options:
      `--cgroups-per-qos=false --enforce-node-allocatable=`
      (this corresponds to EnforceNodeAllocatable being empty array and not
      null) If you then grab kubelet configuration via /configz and try to
      reuse it for dynamic kubelet config, kubelet will think that
      EnforceNodeAllocatable is null, failing to run in the
      Docker-in-Docker environment.
      
      Encountered this while updating Virtlet for Kubernetes 1.6
      (the dev environment is based on kubeadm-dind-cluster)
      69ce7d84
    • Kubernetes Submit Queue's avatar
      Merge pull request #46104 from liggitt/node-admission · 65f5bff1
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 46028, 46104)
      
      Use name from node object on create
      
      GetName() isn't populated in admission attributes on create unless the rest storage is a NamedCreator (which only specific subresources are today)
      
      Fixes #46085
      65f5bff1
    • Kubernetes Submit Queue's avatar
      Merge pull request #46028 from humblec/iscsi-describe · ea828d05
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 46028, 46104)
      
      Add missing parameters of iscsi volume source to describe printer.
      ea828d05
    • Kubernetes Submit Queue's avatar
      Merge pull request #44898 from xingzhou/kube-44697 · 4d89212d
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 45908, 44898)
      
      While calculating pod's cpu limits, need to count in init-container.
      
      Need to count in init-container when calculating a pod's cpu limits.
      Otherwise, may cause pod start failure due to "invalid argument"
      error while trying to write "cpu.cfs_quota_us" file.
      
      Fixed #44697 
      
      Release note:
      ```
      NONE
      ```
      4d89212d
    • Kubernetes Submit Queue's avatar
      Merge pull request #45908 from ncdc/kube-proxy-write-config · 9a5694b4
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      kube-proxy: add --write-config-to flag
      
      Add --write-config-to flag to kube-proxy to write the default configuration
      values to the specified file location.
      
      @deads2k suggested I create my own scheme for this, so I followed the example he shared with me. The only bit currently still referring to `api.Scheme` is where we create the event broadcaster recorder. In order to use the custom private scheme, I either have to pass it in to `NewProxyServer()`, or I have to make `NewProxyServer()` a member of the `Options` struct. If the former, then I probably need to export `Options.scheme`. Thoughts?
      
      cc @mikedanese @sttts @liggitt @deads2k @smarterclayton @timothysc @kubernetes/sig-network-pr-reviews @kubernetes/sig-api-machinery-pr-reviews 
      
      ```release-note
      Add --write-config-to flag to kube-proxy to allow users to write the default configuration settings to a file.
      ```
      9a5694b4
    • Daniel Nardo's avatar
      Add ip-masq-agent addon to the addons folder. This also · 96ae3468
      Daniel Nardo authored
      ensures that under gce we add this daemonset if the non-masq-cidr
      is set to 0/0.
      96ae3468
    • Kubernetes Submit Queue's avatar
      Merge pull request #46097 from mwielgus/comparisons · 3c1c9268
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 46075, 46059, 46095, 46097)
      
      Use integer comparisons instead of string comparisons in autoscaler config validation
      
      **What this PR does / why we need it**:
      
      kube-up.sh doesn't work if MAX_INSTANCES_PER_MIG is specified and KUBE_AUTOSCALER_MAX_NODES is lexicographically smaller ("10" < "2").
      3c1c9268
    • Kubernetes Submit Queue's avatar
      Merge pull request #46095 from wojtek-t/avoid_sleep_in_endpoint_controller · ff7b3653
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 46075, 46059, 46095, 46097)
      
      Avoid sleep in endpoint controller
      ff7b3653
    • Kubernetes Submit Queue's avatar
      Merge pull request #46059 from nikhita/test-int-preserve · ff460734
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 46075, 46059, 46095, 46097)
      
      Integration test for kube-apiextensions-server: integers
      
      **What this PR does / why we need it**: Check if integers are present after decoding.
      Originally an issue for TPRs: #30213
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: for #45511 
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```
      NONE
      ```
      @sttts
      ff460734
    • Kubernetes Submit Queue's avatar
      Merge pull request #46075 from liggitt/fix-unbound-variable · 735742b5
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 46075, 46059, 46095, 46097)
      
      Fix unbound variable
      
      Fixes an unbound variable introduced in https://github.com/kubernetes/kubernetes/pull/45929/commits/6fd36792f185a2f846d08ff5547460b25435b413#diff-ed3df710e9af7cd30a185896a60897d9
      735742b5
    • Jordan Liggitt's avatar
      Use name from node object on create · a2689736
      Jordan Liggitt authored
      a2689736
    • Kubernetes Submit Queue's avatar
      Merge pull request #46051 from p0lyn0mial/admission_plugins_register_method · 03ccb4f0
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 45313, 46051)
      
      Change all the existing admission init blocks to call a Register function
      
      **What this PR does / why we need it**:
      this is a two stage refactor when done there will be no init block in admission plugins.
      Instead all plugins expose Register function which accept admission.Plugins instance.
      The registration to global plugin registry happens inside Register function
      
      **Release note**:
      
      ```
      NONE
      ```
      03ccb4f0
    • Kubernetes Submit Queue's avatar
      Merge pull request #45313 from realfake/cloudprovider-gce-metadata · 07dac20c
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      GCE for cloud-controller-manager 
      
      **What this PR does / why we need it**:
      This implements the `NodeAddressesByProviderID`and `InstanceTypeByProviderID` methods used by the cloud-controller-manager to the GCE provider.
      
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      07dac20c
    • Marcin Wielgus's avatar
    • Wojciech Tyczynski's avatar
      Avoid sleep in endpoint controller · d2529bb6
      Wojciech Tyczynski authored
      d2529bb6
    • Kubernetes Submit Queue's avatar
      Merge pull request #46092 from kargakis/redirect-kubeadm-issues · d651ef78
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Redirect kubeadm issues to kubeadm repo
      
      @kubernetes/sig-cluster-lifecycle-misc ptal
      d651ef78
    • Kubernetes Submit Queue's avatar
      Merge pull request #46088 from Mashimiao/remove-unreachable-code · 231c4a00
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      remove unreachable code
      Signed-off-by: 's avatarMa Shimiao <mashimiao.fnst@cn.fujitsu.com>
      
      
      **What this PR does / why we need it**:
      remove unreachable code
      231c4a00
    • Michail Kargakis's avatar
      Redirect kubeadm issues to kubeadm repo · bad2b97d
      Michail Kargakis authored
      Signed-off-by: 's avatarMichail Kargakis <mkargaki@redhat.com>
      bad2b97d
    • Ma Shimiao's avatar
      remove unreachable code · c0feba4f
      Ma Shimiao authored
      Signed-off-by: 's avatarMa Shimiao <mashimiao.fnst@cn.fujitsu.com>
      c0feba4f
    • Kubernetes Submit Queue's avatar
      Merge pull request #45004 from feiskyer/hostnetwork · 51f3ac1b
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Add node e2e tests for hostNetwork
      
      **What this PR does / why we need it**:
      
      Add node e2e tests for hostNetwork.
      
      **Which issue this PR fixes**
      
      Part of #44118.
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      
      /assign @Random-Liu @yujuhong
      51f3ac1b
    • Kubernetes Submit Queue's avatar
      Merge pull request #46061 from shyamjvs/master-metrics-fix · d9005df6
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Copy static variable 'verb' before instrumenting APIserver call to prevent overwriting
      
      Fixes https://github.com/kubernetes/kubernetes/issues/45566
      
      cc @wojtek-t @gmarek
      d9005df6
    • realfake's avatar
      Implement providerID node functions for gce · 250b2299
      realfake authored
      *Add splitProviderID helper function
      *Add getInstanceFromProjectInZoneByName function
      *Implement gce InstanceTypeByProviderID
      *Implement gce NodeAddressesByProviderID
      250b2299
    • Kubernetes Submit Queue's avatar
      Merge pull request #45960 from pmichali/issue45906 · d81da031
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      IPv6 support for hexCIDR()
      
      Includes these changes:
      - Modified so that IPv6 CIDRs can be converted correctly.
      - Added test cases for IPv6 addresses.
      - Split UTs for hexCIDR() and asciiCIDR() so that masking can be tested.
      - Add UTs for failure cases.
      
      Note: Some code that calls hexCIDR() builds a CIDR from the pod IP string
      and the concatenation of "/32". These should, in the future, use "128",
      if/when the pod IP is IPv6. Not addressed as part of this commit.
      
      
      
      **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 #45906
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-noteNONE
      ```
      d81da031
    • zhengjiajin's avatar
      fix changelog not work · 7dabb8b2
      zhengjiajin authored
      7dabb8b2
    • Kubernetes Submit Queue's avatar
      Merge pull request #46063 from madhusudancs/fed-kubefed-logv4 · 57adadd4
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Log kubefed operations at log level 4 in our test environments.
      
      This is useful for debugging test failures that involve federation control plane turn up/down.
      
      
      **Release note**:
      ```release-note
      NONE
      ```
      
      /assign @perotinus @csbell 
      /sig federation
      
      cc @kubernetes/sig-federation-pr-reviews
      57adadd4
    • Xing Zhou's avatar
      While calculating pod's cpu limits, need to count in init-container. · 22ab45b5
      Xing Zhou authored
      Need to count in init-container when calculating a pod's cpu limits.
      Otherwise, may cause pod start failure due to "invalid argument"
      error while trying to write "cpu.cfs_quota_us" file.
      22ab45b5
    • Jordan Liggitt's avatar
      Fix unbound variable · 20ccdfbd
      Jordan Liggitt authored
      20ccdfbd
    • Kubernetes Submit Queue's avatar
      Merge pull request #45987 from nicksardo/cloud-init-kubeclient · ead8c98c
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Initialize cloud providers with a K8s clientBuilder
      
      **What this PR does / why we need it**:
      This PR provides each cloud provider the ability to generate kubernetes clients. Either the full access or service account client builder is passed from the controller manager. Cloud providers could need to retrieve information from the cluster that isn't provided through defined interfaces, and this seems more preferable to adding parameters.
      
      Please leave your thoughts/comments.
      
      **Release note**:
      ```release-note
      NONE
      ```
      ead8c98c
    • Kubernetes Submit Queue's avatar
      Merge pull request #46056 from marun/fed-federated-updater-deletion-error · c0619947
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 41535, 45985, 45929, 45948, 46056)
      
      [Federation] Fix bad logic of deletion error handling for federated updater
      
      Reverts a change accidentally added in #45364.
      
      cc: @kubernetes/sig-federation-pr-reviews
      c0619947
    • Kubernetes Submit Queue's avatar
      Merge pull request #45948 from CaoShuFeng/underscore · 5290eece
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 41535, 45985, 45929, 45948, 46056)
      
      remove useless flags from hack/verify-flags/known-flags.txt
      
      Flags in known-flags.txt is used to check misspelling from "-" to "_" in
      workspace, so a flag with out "-" should not show up in this file.
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      5290eece
    • Kubernetes Submit Queue's avatar
      Merge pull request #45929 from liggitt/node-admission · a9fbeef6
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 41535, 45985, 45929, 45948, 46056)
      
      NodeRestriction admission plugin
      
      Adds an optional `NodeRestriction` admission plugin that limits identifiable kubelets to mutating their own Node object, and Pod objects bound to their node.
      
      This is the admission portion of https://github.com/kubernetes/community/blob/master/contributors/design-proposals/kubelet-authorizer.md and kubernetes/features#279
      
      ```release-note
      The `NodeRestriction` admission plugin limits the `Node` and `Pod` objects a kubelet can modify. In order to be limited by this admission plugin, kubelets must use credentials in the `system:nodes` group, with a username in the form `system:node:<nodeName>`. Such kubelets will only be allowed to modify their own `Node` API object, and only modify `Pod` API objects that are bound to their node.
      ```
      a9fbeef6
    • Kubernetes Submit Queue's avatar
      Merge pull request #45985 from deads2k/tpr-16-finalizer · 61eace2b
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 41535, 45985, 45929, 45948, 46056)
      
      add CRD finalizer to remove CRs
      
      Fixes https://github.com/kubernetes/kubernetes/issues/45878
      
      This adds a finalizer for customresourcedefinitions to prevent CRD deletion until all CR instances are gone.
      
      @sdminonne I lost track of your issue, but here's the fix
      
      I'm at a loss for how to test this.  It's tested from the outside by ensuring that a CRD delete removes its instances (integration test) and we could add more integration tests, but for unit tests I can't seem to find a mock `dynamic.ClientPool` and its not easily writeable at the moment.  I'm thinking about saying we just add more black box tests given the options.
      61eace2b
    • Kubernetes Submit Queue's avatar
      Merge pull request #41535 from aanm/adding-ipv6-brackets · f57f0388
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 41535, 45985, 45929, 45948, 46056)
      
      kubelet/envvars: Adding brackets to IPv6 addresses
      Signed-off-by: 's avatarAndré Martins <aanm90@gmail.com>
      
      **What this PR does / why we need it**: This adds IPv6 brackets on environments variables pods
      
      **Special notes for your reviewer**:
      
      Since the IP is a string I think the fastest way to detect if it's an IPv6 was to check for the presence of `:` in it. Let me know what you think.
      f57f0388
    • Kubernetes Submit Queue's avatar
      Merge pull request #45921 from pwittrock/kubectl-issues · b6211c6e
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Redirect users filing kubectl issues to the kubernetes/kubectl repo
      b6211c6e
    • Kubernetes Submit Queue's avatar
      Merge pull request #45839 from caesarxuchao/client-gen-vendor · 8214f7bb
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      let client-gen understand vendor
      
      This is extracted from #44784, where we move external api types to k8s.io/api. After the move, the types will locate at vendor/k8s.io/api/xxx. However, gengo index the parsed package using the import path, which is stripped of  the vendor/ prefix, so we'll need to strip the vendor/ prefix as necessary in client-gen.
      
      This PR doesn't produce any change to the generated clientset yet since all types are still in the kubernetes repo.
      8214f7bb
  2. 18 May, 2017 3 commits