1. 13 Sep, 2017 21 commits
    • Aaron Crickenberger's avatar
      Workaround go-junit-report bug for TestApps · eb08dffc
      Aaron Crickenberger authored
      Blatant copy-pasta of 83ff8f2d
      eb08dffc
    • Kubernetes Submit Queue's avatar
      Merge pull request #52413 from aleksandra-malinowska/autoscaling-tests-extra-logs-2 · 5af069b7
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Add logging gcloud command error in e2e tests
      
      This adds extra log line to help with debugging GKE tests.
      5af069b7
    • Kubernetes Submit Queue's avatar
      Merge pull request #52078 from sttts/sttts-codegen-owners · 35b38a34
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      code-generator: add myself to OWNERS
      35b38a34
    • Kubernetes Submit Queue's avatar
      Merge pull request #52375 from jiayingz/deviceplugin-e2e · 991afb24
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 52316, 52289, 52375)
      
      Extends GPUDevicePlugin e2e test to exercise device plugin restarts.
      
      **What this PR does / why we need it**:
      This is part of issue #52189 but does not fix 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
      ```
      991afb24
    • Kubernetes Submit Queue's avatar
      Merge pull request #52289 from crassirostris/sd-logging-trim-long-lines · c9759ae3
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 52316, 52289, 52375)
      
      [fluentd-gcp addon] Trim too long log entries due to Stackdriver limitations
      
      Stackdriver doesn't support log entries bigger than 100KB, so by default fluentd plugin just drops such entries. To avoid that and increase the visibility of this problem it's suggested to trim long lines instead.
      
      /cc @igorpeshansky
      
      ```release-note
      [fluentd-gcp addon] Fluentd will trim lines exceeding 100KB instead of dropping them.
      ```
      c9759ae3
    • Kubernetes Submit Queue's avatar
      Merge pull request #52316 from jpbetz/salt-request-timeout-quickfix · a789fc77
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 52316, 52289, 52375)
      
      Small fix in salt manifest for kube-apiserver for request-timeout flag
      
      **What this PR does / why we need it**:
      
      Fixes a minor bug in salt manifest (typo from #51480)
      
      **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
      ```
      
      xref: #51355
      a789fc77
    • Aleksandra Malinowska's avatar
      log gcloud command error · c1732966
      Aleksandra Malinowska authored
      c1732966
    • Mik Vyatskov's avatar
    • Kubernetes Submit Queue's avatar
      Merge pull request #52201 from timothysc/ephemeral_gate · be78d113
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Version gates the ephemeral storage e2e test
      
      Version gates the ephemeral storage e2e test.
      
      **Release note**:
      ```
      NONE
      ```
      
      @kubernetes/sig-testing-pr-reviews
      be78d113
    • Kubernetes Submit Queue's avatar
      Merge pull request #52301 from tallclair/psp-seccomp · dc02dfe5
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 52339, 52343, 52125, 52360, 52301)
      
      '*' is valid for allowed seccomp profiles
      
      **What this PR does / why we need it**:
      This should be valid on a PodSecurityPolicy, but is currently rejected:
      ```
      seccomp.security.alpha.kubernetes.io/allowedProfileNames: '*'
      ```
      
      **Which issue this PR fixes**: fixes #52300
      
      ```release-note
      NONE
      ```
      dc02dfe5
    • Kubernetes Submit Queue's avatar
      Merge pull request #52360 from shyamjvs/add-debug-statements · 83c2f358
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 52339, 52343, 52125, 52360, 52301)
      
      Make log-dump use 'gcloud ssh' for GKE also
      
      Fixes https://github.com/kubernetes/test-infra/issues/4323
      
      I tested it locally (with some hacking for mimicking gke's DumpClusterLogs function in kubetest) and it worked.
      
      cc @ericchiang
      83c2f358
    • Kubernetes Submit Queue's avatar
      Merge pull request #52125 from yujuhong/fix-file-sync · c6a9b1e1
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 52339, 52343, 52125, 52360, 52301)
      
      dockershim: check if f.Sync() returns an error and surface it
      
      ```release-note
      dockershim: check the error when syncing the checkpoint.
      ```
      c6a9b1e1
    • Kubernetes Submit Queue's avatar
      Merge pull request #52343 from crassirostris/audit-policy-switch-to-beta · e81aeb59
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 52339, 52343, 52125, 52360, 52301)
      
      Switch default audit policy to beta and omit RequestReceived stage
      
      Related to https://github.com/kubernetes/kubernetes/issues/52265
      
      ```release-note
      By default, clusters on GCE no longer sends RequestReceived audit event, if advanced audit is configured.
      ```
      e81aeb59
    • Kubernetes Submit Queue's avatar
      Merge pull request #52339 from liggitt/alpha-test · 5bc9d7b4
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 52339, 52343, 52125, 52360, 52301)
      
      Prevent enabling alpha APIs by default
      
      related to #47691
      This is a follow up to #51839 to add a check that we do not enable alpha APIs by default
      5bc9d7b4
    • Kubernetes Submit Queue's avatar
      Merge pull request #52352 from enisoc/sts-deflake · 96365221
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 48226, 52046, 52231, 52344, 52352)
      
      StatefulSet: Deflake e2e RunHostCmd more.
      
      It turns out that at some points while the Node is recovering from a reboot, we get a different kind of error ("unable to upgrade connection"). Since we can't distinguish these transient errors from an error encountered after successfully executing the remote command, let's just retry all errors for 5min. If this doesn't work, I'm gonna blame it on sig-node.
      
      ref #48031
      96365221
    • Kubernetes Submit Queue's avatar
      Merge pull request #52344 from smarterclayton/no_log_pull · b04f81d3
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 48226, 52046, 52231, 52344, 52352)
      
      Log at higher verbosity levels some common SyncPod errors
      
      This log message was 90% of all glog.Errorf level statements reported on a production cluster, hiding other more impactful errors. We already log it in start container, but for extra caution we continue to log it at v(3) here (the downside of not logging a start container error is worse than some log spam at higher levels).
      
      HandleError() is intended only for unknown and unexpected errors.
      
      ```release-note
      NONE
      ```
      
      @derekwaynecarr @sjenning
      b04f81d3
    • Kubernetes Submit Queue's avatar
      Merge pull request #52231 from mkumatag/guestbook_multiarch · 434fffb6
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 48226, 52046, 52231, 52344, 52352)
      
      Port Guestbook tests to mutiarch
      
      **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 #52232 
      
      **Special notes for your reviewer**:
      
      **Release note**:
      
      ```NONE
      NONE
      ```
      434fffb6
    • Kubernetes Submit Queue's avatar
      Merge pull request #52046 from dashpole/soft_eviction · 32f1521c
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 48226, 52046, 52231, 52344, 52352)
      
      [BugFix] Soft Eviction timer works correctly
      
      fixes #51516
      
      thresholdsMet should not exclude previously met thresholds when we do not have new stats for a threshold.
      
      /assign @vishh @derekwaynecarr 
      cc @kubernetes/sig-node-bugs
      32f1521c
    • Kubernetes Submit Queue's avatar
      Merge pull request #48226 from wongma7/pd-predicate-log · 83b4c0ac
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue (batch tested with PRs 48226, 52046, 52231, 52344, 52352)
      
      Log get PVC/PV errors in MaxPD predicate only at high verbosity
      
      The error is effectively ignored since even if a PVC/PV doesn't exist it gets counted, and it's rarely actionable either so let's reduce the verbosity.
      
      Basically a user somewhere on the cluster will have to have done something "wrong" for this error to occur, e.g. if *,while the pod is running, pod's PVC is deleted or pods' PVC's PV is deleted. And from that point forward the logs will be spammed every time the predicate is evaluated on a node where that "wrong" pod exists
      
      **Release note**:
      
      ```release-note
      NONE
      ```
      83b4c0ac
    • Kubernetes Submit Queue's avatar
      Merge pull request #52248 from zjj2wry/set-env-list · 58b126d9
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      fix kubectl set env --list description
      
      **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
      none
      ```
      58b126d9
    • Kubernetes Submit Queue's avatar
      Merge pull request #51252 from andyzhangx/azuredisk-windows · 39659ac1
      Kubernetes Submit Queue authored
      Automatic merge from submit-queue
      
      Azuredisk mount on windows node
      
      **What this PR does / why we need it**:
      This PR will enable azure disk on windows node, customer could create a pod mounted with azure disk on windows node. 
      There are a few pending items still left:
      1) Current fstype would be forced as NTFS, will change if there is such requirement
      2) GetDeviceNameFromMount function is not implemented(empty) because in Linux, we could use "cat /proc/mounts" to read all mounting points in OS easily, but in Windows, there is no such place, I am still figuring out. The empty function would cause a few warning logging, but it will not affect the main logic now.
      
      **Special notes for your reviewer**:
      1. This PR depends on https://github.com/kubernetes/kubernetes/pull/51240, which allow windows mount path in config validation
      2. There is a bug in docker on windows(https://github.com/moby/moby/issues/34729), the ContainerPath could only be a drive letter now(e.g. D:), dir path would fail in the end.
      
      The example pod with mount path is like below:
      
      ```
      kind: Pod
      apiVersion: v1
      metadata:
        name: pod-uses-shared-hdd-5g
        labels:
          name: storage
      spec:
        containers:
        - image: microsoft/iis
          name: az-c-01
          volumeMounts:
          - name: blobdisk01
            mountPath: 'F:'
        nodeSelector:
          beta.kubernetes.io/os: windows
        volumes:
        - name: blobdisk01
          persistentVolumeClaim:
            claimName: pv-dd-shared-hdd-5
      ```
      
      **Release note**:
      
      ```release-note
      39659ac1
  2. 12 Sep, 2017 19 commits