-
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
Upgrade oldest k8s version supported to 1.13 #8154
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: priyawadhwa 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 |
kvm2 Driver Times for Minikube (PR 8154): [64.66103687 66.04992261499999 65.41625457599999] Averages Time Per Log
docker Driver Times for Minikube (PR 8154): [29.078301047 28.031170774 28.320757192000002] Averages Time Per Log
|
Codecov Report
@@ Coverage Diff @@
## master #8154 +/- ##
==========================================
- Coverage 35.27% 35.27% -0.01%
==========================================
Files 146 146
Lines 9326 9327 +1
==========================================
Hits 3290 3290
- Misses 5637 5638 +1
Partials 399 399
|
kvm2 Driver Times for Minikube (PR 8154): [28.521131311000005 27.843907519000005 28.066573526999996] Averages Time Per Log
|
Travis tests have failedHey @priyawadhwa, 1st Buildmake test
TravisBuddy Request Identifier: 79398140-a10d-11ea-8dbc-21d9b9d6c42b |
kvm2 Driver Times for Minikube (PR 8154): [64.178267218 59.217210967 65.91576447] Averages Time Per Log
docker Driver Times for Minikube (PR 8154): [26.960540546 26.782817828999995 27.188744450999998] Averages Time Per Log
|
Codecov Report
@@ Coverage Diff @@
## master #8154 +/- ##
==========================================
+ Coverage 34.35% 34.37% +0.01%
==========================================
Files 147 147
Lines 9456 9469 +13
==========================================
+ Hits 3249 3255 +6
- Misses 5808 5814 +6
- Partials 399 400 +1
|
kvm2 Driver Times for Minikube (PR 8154): [60.932249888 64.377975617 62.037485968999995] Averages Time Per Log
docker Driver Times for Minikube (PR 8154): [25.153019097000005 25.983420994999992 26.141533469] Averages Time Per Log
|
kvm2 Driver Times for Minikube (PR 8154): [63.69428047699999 63.961550158 64.097132399] Averages Time Per Log
docker Driver Times for Minikube (PR 8154): [26.955066086 27.232749281999997 26.638473627999996] Averages Time Per Log
|
Our old-k8s-version test has consistently been failing in CI (#8048) when testing k8s version 1.12
Unfortunately, I was unable to repro this bug locally, and the test consistently passed on my machine. It seems that for whatever reason, 1.13 passes on CI, so for now the current solution is to upgrade oldest k8s version to 1.13 so that we have reliable integration tests.
Users can still specify using k8s version 1.12 by running
closes #8048