-
Notifications
You must be signed in to change notification settings - Fork 519
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
@mboersma actually I take that back, could you also remove 1.13 from the |
/hold |
Codecov Report
@@ Coverage Diff @@
## master #2749 +/- ##
=======================================
Coverage 72.34% 72.34%
=======================================
Files 137 137
Lines 25344 25344
=======================================
Hits 18334 18334
Misses 5953 5953
Partials 1057 1057 |
/hold release |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[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:
Approvers can indicate their approval by writing |
Reason for Change:
Kubernetes 1.13 has been off the support matrix for a while, so in advance of disabling it, let's stop using CI resources to test it.
AKS is dropping support for 1.13 on Feb. 28th.
Issue Fixed:
Requirements:
Notes: