1. 18 May, 2017 17 commits
    • Kubernetes Submit Queue's avatar
      Merge pull request #45582 from CaoShuFeng/kubectl_set · 212150b6
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      avoid concrete examples for missingResourceError
      
      missingResourceError uses pod and services as an example in error message. However some sub-commands doesn't support pod/service, this change use `<resource> <name>` instead of concrete examples.
      Before this change:
      ```console
      $ kubectl set subject
      error: You must provide one or more resources by argument or filename.
      Example resource specifications include:
         '-f rsrc.yaml'
         '--filename=rsrc.json'
         'pods my-pod'
         'services'
      ```
      
      After this change:
      ```console
      $ kubectl set subject
      error: You must provide one or more resources by argument or filename.
      Example resource specifications include:
         '-f rsrc.yaml'
         '--filename=rsrc.json'
         '<resource> <name>'
         '<resource>'
      ```
      
      **Release note**:
      
      ```release-note
      avoid concrete examples for missingResourceError
      ```
      212150b6
    • Kubernetes Submit Queue's avatar
      Merge pull request #46013 from NickrenREN/openstack-reviewer · 7abb2641
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Add myself to openstack review pool
      
      **Release note**:
      ```release-note
      NONE
      ```
      7abb2641
    • Kubernetes Submit Queue's avatar
      Merge pull request #45642 from JiangtianLi/jiangtli-fixdns2 · f706b810
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Fix the issue in Windows kube-proxy when processing unqualified name. This is for DNS client such as ping or iwr that validate name in response and original question.
      
      **What this PR does / why we need it**:
      This PR is an additional fix to #41618 and [the corresponding commit](https://github.com/kubernetes/kubernetes/commit/b9dfb69dd7679ae31bad1fb4980561fea22f8ed4). The DNS client such as nslookup does not validate name matching in response and original question. That works fine when we append DNS suffix to unqualified name in DNS query in Windows kube-proxy. However, for DNS client such as ping or Invoke-WebRequest that validates name in response and original question, the issue arises and the DNS query fails although the received DNS response has no error. 
      
      This PR fixes the additional issue by restoring the original question name in DNS response. Further, this PR refactors DNS message routines by using miekg's DNS library.
      
      This PR affects the Windows kube-proxy only.
      
      **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #42605 
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      Fix DNS suffix search list support in Windows kube-proxy.
      ```
      f706b810
    • Kubernetes Submit Queue's avatar
      Merge pull request #43578 from vsiddharth/pause-container · e7b18148
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Improve pause-container warning message
      Signed-off-by: 's avatarVinothkumar Siddharth <sidvin@amazon.com>
      
      
      
      **What this PR does / why we need it**:
      This just improves the warning message currently emitted by pause
      
      **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
      ```
      e7b18148
    • Kubernetes Submit Queue's avatar
      Merge pull request #45869 from MrHohn/esipp-default-move · 7f27cebe
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Move defaulting logic for ExternalTrafficPolicy into defaults.go
      
      Link #45812.
      
      Per [#45812/comment](https://github.com/kubernetes/kubernetes/issues/45812#issuecomment-301632807), move defaulting logic to defaults.go so that federation will get the same code.
      
      This PR does not have any functional change :)
      
      /assign @freehan @thockin 
      /cc @madhusudancs @nikhiljindal @shashidharatd
      Also /cc @caesarxuchao per the client-go changes.
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      7f27cebe
    • NickrenREN's avatar
      Add myself to openstack review pool · 9370808a
      NickrenREN authored
      9370808a
    • Dawn Chen's avatar
      Merge pull request #46004 from MrHohn/revert-remove-sync-reason · dddc6b86
      Dawn Chen authored
      Revert "Remove reasons from iptables syncProxyRules"
      dddc6b86
    • Kubernetes Submit Queue's avatar
      Merge pull request #42975 from smarterclayton/time_namespace · 7df01780
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 40234, 45885, 42975)
      
      Log how much time it takes e2e tests to clean up the namespace
      7df01780
    • Kubernetes Submit Queue's avatar
      Merge pull request #45885 from zhangxiaoyu-zidif/kubelet_test-delete-redundant-para · faf556ec
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 40234, 45885, 42975)
      
      delete unused para.
      
      **What this PR does / why we need it**:
      delete unused para.
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      faf556ec
    • Kubernetes Submit Queue's avatar
      Merge pull request #40234 from emaildanwilson/fedTargetClusterByLabel · ca755211
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 40234, 45885, 42975)
      
      Fed target cluster by label for sync controller
      
      [use clusterselector w/ federated configmap deploys](https://github.com/kubernetes/kubernetes/pull/40234/commits/667dc7744475878749acdf972d54bc11f28b993d)
      
      **What this PR does / why we need it**: adds the ability to indicate objects are sent to subsets of federated clusters ref #29887 
      
      **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
      ```
      ca755211
    • Kubernetes Submit Queue's avatar
      Merge pull request #46006 from caesarxuchao/minor-BUILD · 6c1d7dab
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      restrict visibility of client-go/pkg/api
      
      The dependency was removed in a previous PR.
      6c1d7dab
    • Kubernetes Submit Queue's avatar
      Merge pull request #45678 from a-robinson/1.0 · 747b706c
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 45990, 45544, 45745, 45742, 45678)
      
      Add explicit image tag to cockroachdb example and test
      
      @gyliu513 
      
      ```release-note
      NONE
      ```
      747b706c
    • Kubernetes Submit Queue's avatar
      Merge pull request #45742 from janetkuo/deployment-integration-test · 4f56e8b7
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 45990, 45544, 45745, 45742, 45678)
      
      Add integration test for deployment
      
      We have no integration test for Deployment currently. In this PR, add an integration test which covers an e2e test (create a new RollingUpdate deployment), add more replicas to the Deployment, and set minReadySeconds so that we can test maxUnavailable. 
      
      Plan to add more integration tests that cover e2e tests after this initial PR is merged.
      
      @kubernetes/sig-apps-pr-reviews
      4f56e8b7
    • Kubernetes Submit Queue's avatar
      Merge pull request #45745 from marun/fed-test-cluster-addition · 8710f6e6
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 45990, 45544, 45745, 45742, 45678)
      
      [Federation]  Add integration testing for cluster addition
      
      This PR adds integration testing of the sync controller for cluster addition.  This ensures coverage equivalency between the integration tests and the old controller unit tests, so those tests are removed by this PR.
      
      Resolves #45257
      
      cc: @kubernetes/sig-federation-pr-reviews
      8710f6e6
    • Kubernetes Submit Queue's avatar
      Merge pull request #45544 from ianchakeres/reconciler-err-cleanup · 6dbe853e
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 45990, 45544, 45745, 45742, 45678)
      
      Refactor reconciler volume log and error messages
      
      **What this PR does / why we need it**:
      Utilizes volume-specific error and log messages introduced in #44969, inside files that also log volume information. 
      
      Specifically: 
      
      - pkg/kubelet/volumemanager/reconciler/reconciler.go, 
      - pkg/controller/volume/attachdetach/reconciler/reconciler.go, and
      - pkg/kubelet/volumemanager/populator/desired_state_of_world_populator.go
      
      
      **Which issue this PR fixes** : fixes #40905
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      ```
      NONE
      6dbe853e
    • Kubernetes Submit Queue's avatar
      Merge pull request #45990 from dashpole/fix_gc · 7f183142
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      [Bug Fix] Return quantity freed even when an error exists
      
      bug was added by #44986.  If we get an error from deleteImages, we try and use the bytes freed in a calculation.
      That PR changed the behavior from evicting a pod upon error, to panicking because of the nil pointer.
      
      This should fix inode eviction tests
      
      /assign @dchen1107
      7f183142
    • Chao Xu's avatar
      restrict visibility of client-go/pkg/api · 88aed19e
      Chao Xu authored
      88aed19e
  2. 17 May, 2017 23 commits