1. 31 Mar, 2018 7 commits
    • Kubernetes Submit Queue's avatar
      Merge pull request #61369 from rramkumar1/expose-kubelet-health-checks · 20f7f37c
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 61894, 61369). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.
      
      Expose kubelet health checks using new prometheus endpoint
      
      **What this PR does / why we need it**:
      Expose the results of kubelet liveness and readiness probes through a new endpoint on the kubelet called /containerHealth. This endpoint will expose a Prometheus metric. Below is a snippet of output when that endpoint is queried.
      
      ```
      rramkumar@e2e-test-rramkumar-master ~ $ curl localhost:10255/metrics/probes
      # HELP prober_probe_result The result of a liveness or readiness probe for a container.
      # TYPE prober_probe_result gauge
      prober_probe_result{container_name="kube-apiserver",namespace="kube-system",pod_name="kube-apiserver-e2e-test-rramkumar-master",pod_uid="949e11ad296ad9e3c842fd900f8cc723",probe_type="Liveness"} 0
      prober_probe_result{container_name="kube-controller-manager",namespace="kube-system",pod_name="kube-controller-manager-e2e-test-rramkumar-master",pod_uid="0abfc37840bba279706ec39ae53a924c",probe_type="Liveness"} 0
      prober_probe_result{container_name="kube-scheduler",namespace="kube-system",pod_name="kube-scheduler-e2e-test-rramkumar-master",pod_uid="0cd4171f9c806808291e6e24f99f0454",probe_type="Liveness"} 0
      prober_probe_result{container_name="l7-lb-controller",namespace="kube-system",pod_name="l7-lb-controller-v0.9.8-alpha.2-e2e-test-rramkumar-master",pod_uid="968c792f4c1772566c71403dca2407f9",probe_type="Liveness"} 0
      ```
      
      **Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:
      Fixes #58235
      
      
      
      **Release note**:
      ```release-note
      Kubelet now exposes a new endpoint /metrics/probes which exposes a Prometheus metric containing the liveness and/or readiness probe results for a container.
      ```
      20f7f37c
    • Kubernetes Submit Queue's avatar
      Merge pull request #61894 from atlassian/misc-cleanups · 7ce753aa
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 61894, 61369). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.
      
      Use range in loops; misc fixes
      
      **What this PR does / why we need it**:
      It is cleaner to use `range` in for loops to iterate over channel until it is closed.
      
      **Release note**:
      ```release-note
      NONE
      ```
      /kind cleanup
      7ce753aa
    • Kubernetes Submit Queue's avatar
      Merge pull request #60525 from ravisantoshgudimetla/scheduler-pvc · 9847c8ee
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 54997, 61869, 61816, 61909, 60525). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.
      
      Balanced resource allocation priority to include volume count on nodes.
      
      Scheduler balanced resource allocation priority to include volume count on nodes.
      
      /cc @aveshagarwal @abhgupta
      
      
      
      **What this PR does / why we need it**:
      
      **Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:
      Fixes #58232
      
      
      **Release note**:
      
      ```release-note
      Balanced resource allocation priority in scheduler to include volume count on node 
      ```
      9847c8ee
    • Kubernetes Submit Queue's avatar
      Merge pull request #61909 from nikhiljindal/testMCI · 1c008c58
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 54997, 61869, 61816, 61909, 60525). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.
      
      Updating multicluster e2e test to ensure that controller only creates instance groups
      
      Ref https://github.com/kubernetes/ingress-gce/issues/182
      
      Updating the multicluster ingress e2e test to also verify that the controller does not create any other resource than instance group.
      
      I was planning to do that using annotations first, but turns out that we have the right check for annotations: https://github.com/kubernetes/ingress-gce/blob/61d574a2c74fbb6f47e1aec0fae385cf2ca97aaf/pkg/controller/controller.go#L317 :)
      
      Am using `length(ListResource) == 0` now but have still kept the annotation check.
      
      cc @nicksardo @MrHohn @G-Harmon
      
      **Release note**:
      ```release-note
      NONE
      ```
      1c008c58
    • Kubernetes Submit Queue's avatar
      Merge pull request #61816 from wackxu/rman · ac8a02f5
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 54997, 61869, 61816, 61909, 60525). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.
      
      Remove alpha annotation for volume node affinity
      
      **What this PR does / why we need it**:
      
      **Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:
      Fixes # https://github.com/kubernetes/kubernetes/issues/61785
      
      **Special notes for your reviewer**:
      /assign @msau42 
      
      **Release note**:
      
      ```release-note
      ACTION REQUIRED: Alpha annotation for PersistentVolume node affinity has been removed.  Update your PersistentVolumes to use the beta PersistentVolume.nodeAffinity field before upgrading to this release
      ```
      ac8a02f5
    • Kubernetes Submit Queue's avatar
      Merge pull request #61869 from mikedanese/serverauth1 · 9c67d76b
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 54997, 61869, 61816, 61909, 60525). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.
      
      certs: only append locally discovered addresses when we get none from the cloudprovider
      
      The cloudprovider is right, and only cloudprovider addresses can be verified centrally, so don't add any extra when we have them.
      9c67d76b
    • Kubernetes Submit Queue's avatar
      Merge pull request #54997 from wackxu/addvaltest · edce49fb
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 54997, 61869, 61816, 61909, 60525). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.
      
      Improve api validation unit test coverage
      
      **What this PR does / why we need it**:
      
      Add UT for ValidatePodLogOptions and AccumulateUniqueHostPorts
      
      Before
      
      ```
      coverage: 60.0% of statements
      ok  	k8s.io/kubernetes/pkg/api/v1/validation	0.058s
      ```
      
      After:
      ```
      coverage: 92.9% of statements
      ok  	k8s.io/kubernetes/pkg/api/v1/validation	0.057s
      ```
      
      **Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:Fixes #
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      edce49fb
  2. 30 Mar, 2018 33 commits