feat: add support for Kubernetes 1.17.5 #3088
Conversation
/hold This has a temporary commit for testing that should be discarded before merge. |
Codecov Report
@@ Coverage Diff @@
## master #3088 +/- ##
=======================================
Coverage 70.63% 70.63%
=======================================
Files 147 147
Lines 25410 25410
=======================================
Hits 17949 17949
Misses 6340 6340
Partials 1121 1121
Continue to review full report at Codecov.
|
This passed CI, so I will discard the temporary testing commit and it should be ready for review. |
e49060e
to
a3c6595
Compare
@@ -81,9 +81,9 @@ function Get-FilesToCacheOnVHD | |||
"https://kubernetesartifacts.azureedge.net/kubernetes/v1.16.6/windowszip/v1.16.6-1int.zip", | |||
"https://kubernetesartifacts.azureedge.net/kubernetes/v1.16.7/windowszip/v1.16.7-1int.zip", | |||
"https://kubernetesartifacts.azureedge.net/kubernetes/v1.16.8/windowszip/v1.16.8-1int.zip", | |||
"https://kubernetesartifacts.azureedge.net/kubernetes/v1.17.2/windowszip/v1.17.2-1int.zip", | |||
"https://kubernetesartifacts.azureedge.net/kubernetes/v1.17.3/windowszip/v1.17.3-1int.zip", |
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.
I think we need to take one more version away here
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.
AFAICT we have been keeping three patch versions around consistently in the Windows VHD. Intentional, or sloppy? @marosset?
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.
Intentional. So I think this PR is ready as-is.
7381778
to
9ad948c
Compare
9ad948c
to
8fdba1f
Compare
Reason for Change:
See https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG/CHANGELOG-1.17.md#changelog-since-v1174
Issue Fixed:
Requirements:
Notes: