1. 16 Mar, 2018 9 commits
    • Maciej Szulik's avatar
      37073388
    • Kubernetes Submit Queue's avatar
      Merge pull request #61269 from x13n/fluentd-gcp-scaler · ca06cc43
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 60722, 61269). 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>.
      
      Bump fluentd-gcp-scaler version
      
      **What this PR does / why we need it**:
      This version fixes a bug in which scaler was setting resources for all containers in the pod, not only fluentd-gcp one.
      
      **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 #60763
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      ca06cc43
    • Kubernetes Submit Queue's avatar
      Merge pull request #60722 from filbranden/fluentd1 · 7e063329
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue. 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 mapping to /host/lib from fluentd-gcp container.
      
      **What this PR does / why we need it**:
      
      This mapping is no longer needed since fluentd-gcp v2.0.16, in which it started using a container image based on Debian Stretch, in which the systemd libraries already include support for all the supported
      compression algorithms.
      
      The `/run.sh` in the image no longer accesses `/host/lib` anyways, so let's stop mapping it here.
      
      Related changes:
      - fluentd-gcp on GoogleCloudPlatform/k8s-stackdriver#101
      - fluentd-es on GoogleCloudPlatform/google-fluentd#80
      
      /assign @timstclair 
      /cc @crassirostris @bmoyles0117 
      
      **Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:
      N/A
      
      **Special notes for your reviewer**:
      N/A
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      7e063329
    • Daniel Kłobuszewski's avatar
      Bump fluentd-gcp-scaler version · e430520d
      Daniel Kłobuszewski authored
      Fixes #60763
      
      This version fixes a bug in which scaler was setting resources for all containers in the pod, not only fluentd-gcp one.
      e430520d
    • Kubernetes Submit Queue's avatar
      Merge pull request #61243 from verult/pd-multizone-cluster · feac98a1
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue. 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>.
      
      Fixes 'Zone is empty' errors in PD upgrade tests; skips pd tests with inline volume in multizone clusters
      
      **What this PR does / why we need it**: Fixes regional cluster upgrade test failures.
      
      PV upgrade tests were failing because a "" zone is passed to the GCE PD create disk call. In a multizone setting the test must select from a managed zone.
      
      PD tests were failing because it uses inline GCE PD volumes, which should not be used in multizone clusters.
      
      **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 #61242 
      
      /release-note-none
      /assign @saad-ali 
      /cc @wojtek-t 
      /sig storage
      /sig gcp
      feac98a1
    • Kubernetes Submit Queue's avatar
      Merge pull request #60985 from soltysh/issue59918 · 5d672225
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 60978, 60985). 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>.
      
      Backoff only when failed pod shows up
      
      **What this PR does / why we need it**:
      Upon introducing the backoff policy we started to delay sync runs for the job when it failed several times before. This leads to failed jobs not reporting status right away in cases that are not related to failed pods, eg. a successful run. This PR ensures the backoff is applied only when `updatePod` receives a failed pod.
      
      **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 #59918 #59527
      
      /assign @janetkuo @kow3ns 
      
      **Release note**:
      ```release-note
      None
      ```
      5d672225
    • Kubernetes Submit Queue's avatar
      Merge pull request #60978 from danwinship/fix-iptables-restore-wait · dce8d416
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 60978, 60985). 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>.
      
      Fix use of "-w" flag to iptables-restore
      
      iptables accepts "-w5" but iptables-restore requires "-w 5", so kube-proxy is currently broken for people with an iptables-restore new enough that kube-proxy tries to use the new flags.
      
      Fixes #58956
      
      **Release note**:
      ```release-note
      Fixed kube-proxy to work correctly with iptables 1.6.2 and later.
      ```
      dce8d416
    • Kubernetes Submit Queue's avatar
      Merge pull request #61071 from jingxu97/Mar/podvolumerace · 9fe565ab
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 61203, 61071). 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>.
      
      Fix issue with race condition during pod deletion
      
      This PR fixes two issues
      1. When desired_state_populator removes podvolume state, it should check
      whether the actual state already has the volume before deleting it to
      make sure actual state has a chance to add the volume into the state
      2. When checking podVolume still exists, it not only checks the actual
      state, but also the volume disk directory because actual state might not
      reflect the real world when kubelet starts.
      
      fixes issue #60645
      9fe565ab
    • Kubernetes Submit Queue's avatar
      Merge pull request #61203 from rmmh/fix-mode-flag · 5f780938
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 61203, 61071). 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>.
      
      Fix deprecated gcloud compute networks --mode switches.
      
      "create --mode" becomes "create --subnet-mode", and switch-mode has been
      folded into "update".
      
      Create --mode was deprecated in October and will be removed in the next
      gcloud release. It is already failing in staging tests.
      **What this PR does / why we need it**:
      
      **Which issue(s) this PR fixes** Fixes #54238
      
      **Special notes for your reviewer**:
      
      **Release note**:
      ```release-note
      NONE
      ```
      5f780938
  2. 15 Mar, 2018 5 commits
    • Jing Xu's avatar
      Fix issue with race condition during pod deletion · 9bd006de
      Jing Xu authored
      This PR fixes two issues
      1. When desired_state_populator removes podvolume state, it should check
      whether the actual state already has the volume before deleting it to
      make sure actual state has a chance to add the volume into the state
      2. When checking podVolume still exists, it not only checks the actual
      state, but also the volume disk directory because actual state might not
      reflect the real world when kubelet starts.
      9bd006de
    • Kubernetes Submit Queue's avatar
      Merge pull request #61225 from x13n/fluentd-gcp-scaler · 448f9a37
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 60888, 61225). 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>.
      
      Bump fluentd-gcp-scaler version
      
      **What this PR does / why we need it**:
      This version verifies on its own whether resources should be updated or not, instead of relying on `kubectl set resources`.
      
      **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 #61190
      
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      
      cc @shyamjvs
      448f9a37
    • Kubernetes Submit Queue's avatar
      Merge pull request #60888 from jsafrane/fix-asw-reconstruct-inuse · 78e58025
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 60888, 61225). 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>.
      
      Mark reconstructed volumes as reported InUse
      
      When a newly started kubelet finds a directory where a volume should be,
      it can be fairly confident that the volume was mounted by previous kubelet
      and therefore the volume must have been in node.status.volumesInUse.
      
      Therefore we can mark reconstructed volumes as already reported so
      subsequent reconcile() can fix the directory and put the mounted volume
      into actual state of world.
      
      Fixes: #60645
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      
      /sig storage
      /sig node
      cc: @gnufied @jingxu97
      78e58025
    • Cheng Xing's avatar
      Fixes 'Zone is empty' errors in PD upgrade tests; skips pd tests with inline… · fe76c9f7
      Cheng Xing authored
      Fixes 'Zone is empty' errors in PD upgrade tests; skips pd tests with inline volume in multizone clusters
      fe76c9f7
    • Daniel Kłobuszewski's avatar
      Bump fluentd-gcp-scaler version · 3d9e9e96
      Daniel Kłobuszewski authored
      Fixes #61190.
      
      This version verifies on its own whether resources should be updated or not, instead of relying on `kubectl set resources`.
      3d9e9e96
  3. 14 Mar, 2018 7 commits
  4. 13 Mar, 2018 17 commits
  5. 12 Mar, 2018 2 commits
    • Kubernetes Submit Queue's avatar
      Merge pull request #60962 from technicianted/flex_race_fix · 6571be13
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue. 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>.
      
      Added missing error check that could cause kubelet to crash
      
      **What this PR does / why we need it**:
      Adds missing error check. An error can happen due to a race condition when watched files change, or become inaccessible. This can happen if a file was added to the driver directory then quickly removed, in which case the callback will be called with non-nil `err` and nil `info`, which is not checked, causing kubelet to crash.
      
      **Which issue(s) this PR fixes**:
      Fixes #60861
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      6571be13
    • Kubernetes Submit Queue's avatar
      Merge pull request #60994 from joejulian/add_remote_flag_option · a1c69341
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue. 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>.
      
      Add missing container-runtime "remote" option
      
      **What this PR does / why we need it**:
      Added the "remote" option to the auto-generated documentation for the
      `--container-runtime` flag.
      
      The kubelet flag `--container-runtime` lists the possible values as part of the auto-generated documentation but is missing the "remote" possibility.
      
      **Which issue(s) this PR fixes** :
      Fixes #60992
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      a1c69341