Skip to content
This repository has been archived by the owner on Oct 24, 2023. It is now read-only.

test: retry "get nodes that match regex" test #2120

Merged
merged 2 commits into from Oct 8, 2019

Conversation

jackfrancis
Copy link
Member

Reason for Change:

Observed this E2E flake:

$ k get nodes -o json
 �Failure [30.437 seconds]�
 2019/10/07 21:04:48 Error trying to run 'kubectl get nodes':
  - exit status 1
 2019/10/07 21:04:48 
  - {
     "apiVersion": "v1",
     "items": [],
     "kind": "List",
     "metadata": {
         "resourceVersion": "",
         "selfLink": ""
     }
 }
 Unable to connect to the server: dial tcp 13.81.26.234:443: i/o timeout
 �[BeforeSuite] BeforeSuite �[0m
 �/go/src/github.com/Azure/aks-engine/test/e2e/kubernetes/kubernetes_test.go:71�
 
   �Expected error:
       <*exec.ExitError | 0xc0003cbc40>: {
           ProcessState: {
               pid: 3198,
               status: 256,
               rusage: {
                   Utime: {Sec: 0, Usec: 150474},
                   Stime: {Sec: 0, Usec: 22361},
                   Maxrss: 37936,
                   Ixrss: 0,
                   Idrss: 0,
                   Isrss: 0,
                   Minflt: 3460,
                   Majflt: 0,
                   Nswap: 0,
                   Inblock: 0,
                   Oublock: 8,
                   Msgsnd: 0,
                   Msgrcv: 0,
                   Nsignals: 0,
                   Nvcsw: 349,
                   Nivcsw: 229,
               },
           },
           Stderr: nil,
       }
       exit status 1
   not to have occurred�
 
   /go/src/github.com/Azure/aks-engine/test/e2e/kubernetes/kubernetes_test.go:93

Issue Fixed:

Requirements:

Notes:

@codecov
Copy link

codecov bot commented Oct 7, 2019

Codecov Report

Merging #2120 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2120   +/-   ##
=======================================
  Coverage   76.58%   76.58%           
=======================================
  Files         135      135           
  Lines       20613    20613           
=======================================
  Hits        15787    15787           
  Misses       3903     3903           
  Partials      923      923

mboersma
mboersma previously approved these changes Oct 7, 2019
Copy link
Member

@mboersma mboersma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@acs-bot
Copy link

acs-bot commented Oct 7, 2019

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jackfrancis, mboersma

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:
  • OWNERS [jackfrancis,mboersma]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@acs-bot
Copy link

acs-bot commented Oct 7, 2019

New changes are detected. LGTM label has been removed.

@acs-bot acs-bot removed the lgtm label Oct 7, 2019
@jackfrancis jackfrancis merged commit 1ea906f into Azure:master Oct 8, 2019
@jackfrancis jackfrancis deleted the e2e-node-get-regex-retry branch October 8, 2019 00:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants