-
Notifications
You must be signed in to change notification settings - Fork 558
Fix Windows sku + offer + publisher k8s and swarm variables #2955
Conversation
"agentWindowsSku": "2016-Datacenter-with-Containers", | ||
"agentWindowsPublisher": "[parameters('agentWindowsPublisher')]", | ||
"agentWindowsOffer": "[parameters('agentWindowsOffer')]", | ||
"agentWindowsSku": "[parameters('agentWindowsSku')]", |
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.
This is changing the SKU default from 2016-Datacenter-with-Containers
to Datacenter-Core-1709-with-Containers-smalldisk
and the offer default from WindowsServer
to WindowsServerSemiAnnual
, consistent with #2664 FYI cc: @JiangtianLi
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
45a52ae
to
5cfe59b
Compare
Codecov Report
@@ Coverage Diff @@
## master #2955 +/- ##
=========================================
Coverage ? 50.78%
=========================================
Files ? 97
Lines ? 14703
Branches ? 0
=========================================
Hits ? 7467
Misses ? 6539
Partials ? 697 Continue to review full report at Codecov.
|
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
What this PR does / why we need it: PR #2664 allowed specifying a custom sku + offer + publisher for DCOS, this PR is extending that to k8s and swarm as well.
Which issue this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close that issue when PR gets merged): fixes #2953Special notes for your reviewer:
If applicable:
Release note: