-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Run 'minikube logs' when sub-tests fail, remove some extraneous logging #8051
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: tstromberg 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:
Approvers can indicate their approval by writing |
/ok-to-test |
kvm2 Driver Times for Minikube (PR 8051): [65.620839079 65.342391438 67.151121313] Averages Time Per Log
docker Driver Times for Minikube (PR 8051): [27.19383085799999 29.298809534 26.837276436] Averages Time Per Log
|
Travis tests have failedHey @tstromberg, TravisBuddy Request Identifier: 3beab2a0-918b-11ea-9e78-458f7b3d5a30 |
kvm2 Driver Times for Minikube (PR 8051): [62.79598678700001 65.541904657 61.791106934] Averages Time Per Log
docker Driver Times for Minikube (PR 8051): [27.188582207999996 27.285879382000005 26.952988401000002] Averages Time Per Log
|
Codecov Report
@@ Coverage Diff @@
## master #8051 +/- ##
=======================================
Coverage 35.45% 35.45%
=======================================
Files 146 146
Lines 9284 9284
=======================================
Hits 3292 3292
Misses 5592 5592
Partials 400 400 |
kvm2 Driver Times for Minikube (PR 8051): [61.489142438 65.277431346 65.178181208] Averages Time Per Log
docker Driver Times for Minikube (PR 8051): [26.806373284000003 27.469437271 27.491883757999997] Averages Time Per Log
|
This is my attempt to make integration tests more relevant:
Sub-tests which mutate state should now show post-mortem logs at the completion of the sub-test. Which means we will now show logs closer to the point of failure.
Post-mortem logs now include "minikube logs" and "docker inspect". I removed "docker logs" and "df", since they aren't as useful. I also removed other logs which are duplicated by "minikube logs".
Post-mortem logs now describe only the pods which are not running, which reduces the amount of unimportant information.
Reduce verbosity for tests where it hasn't been useful. The added verbosity here was just including libmachine logs.