1. 17 Jul, 2017 26 commits
    • Zach Loafman's avatar
      cluster/gke: If NODE_INSTANCE_GROUP is set, don't execute any bash · 3a2e9d51
      Zach Loafman authored
      Transitional part of kubernetes/test-infra#3307, should be eliminated
      by kubernetes/test-infra#3330: Allow NODE_INSTANCE_GROUP to be set
      before we get here, which eliminates any cluster/gke use if
      KUBERNETES_CONFORMANCE_PROVIDER is set to "gke".
      3a2e9d51
    • Kubernetes Submit Queue's avatar
      Merge pull request #49021 from gmarek/master-size · 12ba9bdc
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Create 64-core masters for huge clusters
      
      cc @shyamjvs
      12ba9bdc
    • Kubernetes Submit Queue's avatar
      Merge pull request #46332 from feiskyer/Seccomp · 10dc1aac
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Kubelet CRI: move seccomp from annotations to security context
      
      **What this PR does / why we need it**:
      
      This is the final step for #39130, which moves seccomp from annotations to linux container security context. And it also fixes #36997 by set the full seccomp profile path for node-installed profiles.
      
      Note it doesn't include spec the seccomp profile format, which should be addressed at #39128. And a following PR is required for implementing in kuberuntime and dockershim.
      
      **Which issue this PR fixes** 
      
      Fixes #39130
      Fixes #36997
      
      **Special notes for your reviewer**:
      
      **Release note**:
      ```release-note
      Kubelet CRI: move seccomp from annotations to security context.
      ```
      10dc1aac
    • Kubernetes Submit Queue's avatar
      Merge pull request #48909 from caesarxuchao/owners · ce91f2ab
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Add approvers to pkg/controller/garbagecollector
      
      cc @deads2k
      ce91f2ab
    • Kubernetes Submit Queue's avatar
      Merge pull request #48733 from liggitt/namespace-deletion · e2eefe92
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 48494, 48733)
      
      Never prevent deletion of resources as part of namespace lifecycle
      
      Namespace lifecycle should not prevent deletion of resources... its purpose is to prevent creation of resources in a terminating namespace, or create/update of resources in a non-existent namespace.
      
      Fixes #49027
      e2eefe92
    • Kubernetes Submit Queue's avatar
      Merge pull request #48494 from mkumatag/webserver · 8b39fa9c
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 48494, 48733)
      
      Move test-webserver from contrib/for-demos to kubernetes/test/images
      
      **What this PR does / why we need it**:
      This PR is for
      - Moving the https://github.com/kubernetes/contrib/tree/master/for-demos/test-webserver to kubernetes/test/images - Refer https://github.com/kubernetes/contrib/pull/2544 for more information
      - Multi architecture support for test-webserver image
      
      **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
      ```
      8b39fa9c
    • Kubernetes Submit Queue's avatar
      Merge pull request #48908 from shyamjvs/reduce-services-loadtest · 5c32b7d1
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 48991, 48908)
      
      Group every two services into one in load test
      
      Ref https://github.com/kubernetes/kubernetes/issues/48938
      
      Following from discussion with @bowei and @freehan .
      This reduces #services to 8200 while keeping no. of backends same.
      
      /cc @wojtek-t @gmarek
      5c32b7d1
    • Kubernetes Submit Queue's avatar
      Merge pull request #48991 from smarterclayton/cleanup_restclient · d54ab221
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Remove old, core/v1 specific constructs from RESTClient
      
      Now that metav1 is abstracted from the APIs, RESTClient should also be agnostic to the core API.
      
      * Remove `LabelSelectorParam` and `FieldSelectorParam` - use `VersionedParams` with `ListOptions`
      * Remove `UintParam`
      * Remove all legacy field selector logic from `VersionedParams` - ParameterCodec now handles that
      * Remove special parameters (like `timeout`) which is no longer set by most clients
      d54ab221
    • gmarek's avatar
      Create 64-core masters for huge clusters · af648021
      gmarek authored
      af648021
    • Shyam Jeedigunta's avatar
      26006af4
    • Kubernetes Submit Queue's avatar
      Merge pull request #48805 from sakeven/use_const · a9afb931
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 48262, 48805)
      
      [Scheduler] Use const value maxPriority instead of immediate value 10
      Signed-off-by: 's avatarsakeven <jc5930@sina.cn>
      
      
      
      **What this PR does / why we need it**:
      
      Use const value maxPriority instead of immediate value 10.
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```
      NONE
      ```
      a9afb931
    • Kubernetes Submit Queue's avatar
      Merge pull request #48262 from MaciekPytel/fix_autoscaler_e2e_on_gke_2 · 3d24cf05
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 48262, 48805)
      
      Fix condition in autoscaler e2e
      
      Fix off-by-one in cluster-autoscaler that made the test trivially
      pass (without actually requiring scale-up to happen).
      3d24cf05
    • Kubernetes Submit Queue's avatar
      Merge pull request #48451 from sakeven/fix/ForgetPod_first_after_bind_failed · 4f6af5fa
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      forget pod first after binding failed
      Signed-off-by: 's avatarsakeven <jc5930@sina.cn>
      
      
      
      **What this PR does / why we need it**:
      
      In the implementation of scheduler cache, `FinishBinding` marks Pod expired, and then pod would be cleaned in ttl seconds. While `ForgetPod` checks Pod whether assumed,  if not, it reports an error.
      
      So if binding failed and ttl(now 30s) is too short,  the error will occur when `ForgetPod`, thus we won't record `BindingRejected` event.
      
      Although it's rare, we shouldn't depend on the value of ttl.
      
      **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
      ```
      4f6af5fa
    • Kubernetes Submit Queue's avatar
      Merge pull request #47087 from ravisantoshgudimetla/node_controller_flake · 1170b7c2
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Changes to fix the node eviction flake
      
      **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 #39975
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      1170b7c2
    • Maciej Pytel's avatar
      Fix condition in autoscaler e2e · f53be7bd
      Maciej Pytel authored
      Fix off-by-one in cluster-autoscaler that made the test trivially
      pass (without actually requiring scale-up to happen).
      f53be7bd
    • Kubernetes Submit Queue's avatar
      Merge pull request #48944 from liaoyw/renew-lease-errormsg · ba2bdb1a
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Log error when failed to renew lease.
      
      **What this PR does / why we need it**:
      
      Log detailed error when leaderelection can not renew release.
      It would add a little bit help to find direct reason of failing renew lease
      ```
      E0626 15:23:06.269198   46443 leaderelection.go:263] Failed to update lock: etcdserver: request timed out
      E0626 15:23:07.528206   46443 leaderelection.go:263] Failed to update lock: Operation cannot be fulfilled on endpoints "kube-scheduler": the object has been modified; please apply yo
      ur changes to the latest version and try again
      E0626 15:23:07.528260   46443 event.go:259] Could not construct reference to: '&v1.Endpoints{TypeMeta:v1.TypeMeta{Kind:"", APIVersion:""}, ObjectMeta:v1.ObjectMeta{Name:"", GenerateN
      ame:"", Namespace:"", SelfLink:"", UID:"", ResourceVersion:"", Generation:0, CreationTimestamp:v1.Time{Time:time.Time{sec:0, nsec:0, loc:(*time.Location)(nil)}}, DeletionTimestamp:(*
      v1.Time)(nil), DeletionGracePeriodSeconds:(*int64)(nil), Labels:map[string]string(nil), Annotations:map[string]string(nil), OwnerReferences:[]v1.OwnerReference(nil), Finalizers:[]str
      ing(nil), ClusterName:""}, Subsets:[]v1.EndpointSubset(nil)}' due to: 'selfLink was empty, can't make reference'. Will not report event: 'Normal' 'LeaderElection' 'gd6-k8s-noah-prod0
      01-master-s0004 stopped leading'
      I0626 15:23:07.528391   46443 leaderelection.go:208] failed to renew lease kube-system/kube-scheduler
      F0626 15:23:07.528422   46443 server.go:134] lost master
      ```
      ba2bdb1a
    • sakeven's avatar
      forget pod first after bind failed · e9aee2b2
      sakeven authored
      Signed-off-by: 's avatarsakeven <jc5930@sina.cn>
      e9aee2b2
    • Kubernetes Submit Queue's avatar
      Merge pull request #47187 from leblancd/v6_only_proxy_server · e8eb858b
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 47309, 47187)
      
      Add IPv6 test cases to kube-proxy server test.
      
      **What this PR does / why we need it**: This change adds some IPv6 test cases for the kube-proxy server. Also adds some test cases for negative conditions for better test coverage.
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #47313
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      ```
      e8eb858b
    • Kubernetes Submit Queue's avatar
      Merge pull request #47309 from xiang90/util · 94bca5ff
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 47309, 47187)
      
      scheduler/util: remove bad print format
      
      Fix https://github.com/kubernetes/kubernetes/issues/18834
      94bca5ff
    • sakeven's avatar
      Use const value maxPriority instead of immediate value 10 · 6aeb77aa
      sakeven authored
      Signed-off-by: 's avatarsakeven <jc5930@sina.cn>
      6aeb77aa
    • Kubernetes Submit Queue's avatar
      Merge pull request #48274 from superbrothers/fix-to-override-kubectl-flags · 4b4e9197
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 48381, 48274)
      
      Fix completions for --namespace to override kubectl flags
      
      **What this PR does / why we need it**:
      This PR fixes completions for --namespace to override kubectl flags. Due to not using __kubectl_parse_get, __kubectl_get_namespaces doesn't support to override kubectl flags.
      
      **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
      ```
      4b4e9197
    • Kubernetes Submit Queue's avatar
      Merge pull request #48381 from superbrothers/completion-kubectl-config-delete-cluster · 0049dd07
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Support completion for kubectl config delete-cluster
      
      **What this PR does / why we need it**:
      This PR supports completion for kubectl config delete-cluster.
      ```
      $ kubectl config delete-cluster <tab>
      cluster01  minikube
      ```
      
      **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
      Support completion for kubectl config delete-cluster
      ```
      0049dd07
    • Pengfei Ni's avatar
      Run hack/update-generated-runtime.sh · 79a964ee
      Pengfei Ni authored
      79a964ee
    • Pengfei Ni's avatar
      31ac1d8e
    • Kubernetes Submit Queue's avatar
      Merge pull request #48280 from MaciekPytel/ca_scale_to_0_e2e_2 · 396d8b11
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 46091, 48280)
      
      Added scale-down-to-0 e2e for cluster autoscaler
      396d8b11
    • Kubernetes Submit Queue's avatar
      Merge pull request #46091 from xilabao/new-output-in-edit · 8ce63785
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 46091, 48280)
      
      allow output patch string in edit command
      
      **What this PR does / why we need it**:
      allow user to get the patch from edit command if user is not familiar with the patch format.
      
      ```
      # ./cluster/kubectl.sh create role a --verb=get,list --resource=no
      role "a" created
      
      # ./cluster/kubectl.sh edit role a --output-patch=true
      Patch: {"rules":[{"apiGroups":[""],"resources":["nodes"],"verbs":["get","list","delete"]}]}
      role "a" edited
      
      # ./cluster/kubectl.sh create role b --verb=get,list --resource=no
      role "b" created
      
      # ./cluster/kubectl.sh patch role b -p '{"rules":[{"apiGroups":[""],"resources":["nodes"],"verbs":["get","list","delete"]}]}'
      role "b" patched
      ```
      **Which issue this PR fixes**: fixes #47173
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      Could get the patch from kubectl edit command
      ```
      8ce63785
  2. 16 Jul, 2017 14 commits
    • Kubernetes Submit Queue's avatar
      Merge pull request #46930 from k82cn/sched_integ_test · 0c74c36b
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 47417, 47638, 46930)
      
      Added scheduler integration test owners.
      
      **What this PR does / why we need it**:
      Add OWNER file into scheduler integration test.
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes # N/A
      
      **Release note**:
      ```release-note-none
      ```
      0c74c36b
    • Kubernetes Submit Queue's avatar
      Merge pull request #47638 from xilabao/update-hostports-log · 35794a8f
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 47417, 47638, 46930)
      
      update the err of hostPorts in psp
      
      **What this PR does / why we need it**:
      change `Allowed ports: [{8000 8080}]` to `Allowed ports: [8000-8080]`
      
      **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
      ```
      35794a8f
    • Kubernetes Submit Queue's avatar
      Merge pull request #47417 from xilabao/print-non-existent-key · 1cbb7b14
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 47417, 47638, 46930)
      
      print non-existent key in configmap
      
      **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 #41573
      
      **Special notes for your reviewer**:
      carry from #41574
      **Release note**:
      
      ```release-note
      NONE
      ```
      1cbb7b14
    • Kubernetes Submit Queue's avatar
      Merge pull request #48930 from crassirostris/sig-instrumentation-e2e-cleanup · 8d770eb8
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 47360, 48930)
      
      Sig-instrumentation e2e tests refactoring
      
      Consolidate all sig-intrumentation-owned tests to one directory with clean owners.
      
      /cc @piosz
      8d770eb8
    • Kubernetes Submit Queue's avatar
      Merge pull request #47360 from icereval/feature/fluentd-node-problem-detector · 1b0d1997
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      GCP Fluentd Logging of node-problem-detector
      
      **What this PR does / why we need it**:
      
      This PR adds `node-problem-detector` logging to the gcp fluentd configmap.
      
      Adding this logging GCE can now capture events and alert on process `OOMKilling` conditions when applying memory limits to containers.
      
      **Which issue this PR fixes**:
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      1b0d1997
    • icereval's avatar
      log node-problem-detector · 60872db8
      icereval authored
      60872db8
    • Clayton Coleman's avatar
      Remove Kube specific api constructs from restclient · 112e0fa9
      Clayton Coleman authored
      All callers must use VersionedParameters, which no longer has special
      behavior for Kube resources.
      112e0fa9
    • Clayton Coleman's avatar
      Remove "special" restclient parameters · f0e11c5b
      Clayton Coleman authored
      f0e11c5b
    • Clayton Coleman's avatar
      Remove use of (Label|Field)SelectorParam · b6d9815b
      Clayton Coleman authored
      b6d9815b
    • Mik Vyatskov's avatar
      Sig-instrumentation e2e tests refactoring · 9d3356c6
      Mik Vyatskov authored
      9d3356c6
    • Kubernetes Submit Queue's avatar
      Merge pull request #48993 from k82cn/rename_nodeutil_2_v1node · b516a521
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Renamed nodeutil to v1node.
      
      **What this PR does / why we need it**:
      Renamed `nodeutil` to `v1node` as there's already another similar package named `utilnode`.
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #N/A
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      b516a521
    • Klaus Ma's avatar
      Renamed nodeutil to v1node. · 01097dba
      Klaus Ma authored
      01097dba
    • Kubernetes Submit Queue's avatar
      Merge pull request #48556 from tamird/patch-1 · 269d7c3b
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 46973, 48556)
      
      Update CockroachDB tag to v1.0.3
      
      @a-robinson
      269d7c3b
    • Kubernetes Submit Queue's avatar
      Merge pull request #46973 from php-coder/improve_flex_logging · 86a42b87
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 46973, 48556)
      
      Improve error reporting when flex driver has failed to initialize
      
      **What this PR does / why we need it**:
      
      This PR improves error reporting for the case when flex driver is failing to initialize. There are 2 improvements:
      
      1) show only the plugin name instead of a full struct. This makes a message shorter and removes useless and internal information.
      
      Before:
      >E0605 16:44:59.330215   26786 plugins.go:359] Failed to load volume plugin &{k8s/nfs /usr/libexec/kubernetes/kubelet-plugins/volume/exec/k8s~nfs %!s(*kubelet.kubeletVolumeHost=&{0xc431ea5800 {{1 0} map[kubernetes.io/downward-api:0xc431ee3f20 kubernetes.io/aws-ebs:0xc431ee3eb0 kubernetes.io/git-repo:0xc431ee3ef0 kubernetes.io/host-path:0xc430e985f0 kubernetes.io/rbd:0xc42bfab840 kubernetes.io/quobyte:0xc431ee3f00 kubernetes.io/fc:0xc42bfab980 kubernetes.io/empty-dir:0xc431ee3ed0 kubernetes.io/nfs:0xc430e98640 kubernetes.io/iscsi:0xc42bfab720 kubernetes.io/glusterfs:0xc430faaba0 kubernetes.io/cinder:0xc42bfab8c0 kubernetes.io/gce-pd:0xc431ee3ee0 kubernetes.io/secret:0xc42bfab6a0 kubernetes.io/flocker:0xc431ee3f30 kubernetes.io/cephfs:0xc431ee3f10]} 0xc42698cf40}) %!s(*exec.executor=&{}) {%!s(int32=0) %!s(uint32=0)} []}, error: unexpected end of JSON input
      
      After:
      >E0605 16:59:45.520185   29041 plugins.go:359] Failed to load volume plugin k8s/nfs, error: unexpected end of JSON input
      
      2) quote script output. In case the output was empty, messages look a bit better:
      
      Before:
      > E0605 16:44:59.330077   26786 driver-call.go:212] Failed to unmarshal output for command: init, **output: **, error: unexpected end of JSON input
      > W0605 16:44:59.330170   26786 driver-call.go:140] FlexVolume: driver call failed: executable: /usr/libexec/kubernetes/kubelet-plugins/volume/exec/k8s\~nfs/nfs, args: [init], error: fork/exec /usr/libexec/kubernetes/kubelet-plugins/volume/exec/k8s\~nfs/nfs: permission denied, **output: **
      
      After:
      >E0605 16:59:45.519906   29041 driver-call.go:212] Failed to unmarshal output for command: init, **output: ""**, error: unexpected end of JSON input
      >W0605 16:59:45.520109   29041 driver-call.go:140] FlexVolume: driver call failed: executable: /usr/libexec/kubernetes/kubelet-plugins/volume/exec/k8s\~nfs/nfs, args: [init], error: fork/exec /usr/libexec/kubernetes/kubelet-plugins/volume/exec/k8s\~nfs/nfs: permission denied, **output: ""**
      86a42b87