-
Notifications
You must be signed in to change notification settings - Fork 519
chore: updating windows VHD to include 3B patches + march k8s packages #2902
Conversation
"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", | ||
"https://kubernetesartifacts.azureedge.net/kubernetes/v1.18.0-beta.1/windowszip/v1.18.0-beta.1-1int.zip" | ||
"https://kubernetesartifacts.azureedge.net/kubernetes/v1.17.4/windowszip/v1.17.4-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.
Are we intentionally leaving 3 versions in 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.
3 patch releases per minor version, I should say
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 not keeping older versions is negatively impacting AKS cluster creation times.
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 don't have any data to back that up tho.
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.
Being conservative is fine w/ me
Codecov Report
@@ Coverage Diff @@
## master #2902 +/- ##
=========================================
+ Coverage 72.48% 72.5% +0.02%
=========================================
Files 140 141 +1
Lines 25694 25744 +50
=========================================
+ Hits 18624 18666 +42
- Misses 5994 5998 +4
- Partials 1076 1080 +4 |
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
Reason for Change:
Issue Fixed:
Requirements:
Notes: