-
Notifications
You must be signed in to change notification settings - Fork 558
Conversation
pkg/acsengine/k8s_versions.go
Outdated
@@ -8,6 +8,33 @@ import ( | |||
|
|||
// KubeConfigs represents Docker images used for Kubernetes components based on Kubernetes versions (major.minor.patch) | |||
var KubeConfigs = map[string]map[string]string{ | |||
common.KubernetesVersion1Dot9Beta1: { | |||
"hyperkube": "hyperkube-amd64:v1.9.0-beta.1", | |||
"ccm": "cloud-controller-manager-amd64:v1.9.0", |
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.
Here should be 'cloud-controller-manager-amd64:v1.9.0-beta.1'.
Another thing to note is azure's 'ccm' image release is planned to be shipped via public dockerhub, which does not share 'gcrio.azureedge.net' prefix then. This could be addressed later when we move to that image.
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
pkg/acsengine/k8s_versions.go
Outdated
"pause": "pause-amd64:3.0", | ||
DefaultTillerAddonName: DefaultTillerImage, | ||
DefaultReschedulerAddonName: DefaultReschedulerImage, | ||
"windowszip": "v1.8.4-1int.zip", // TODO add Windows image |
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.
You can change to v1.9.0-1int.zip
pkg/api/common/const.go
Outdated
@@ -147,6 +150,7 @@ var AllKubernetesWindowsSupportedVersions = map[string]bool{ | |||
KubernetesVersion1Dot8Dot1: true, | |||
KubernetesVersion1Dot8Dot2: true, | |||
KubernetesVersion1Dot8Dot4: true, | |||
// TODO add KubernetesVersion1Dot9Dot0 when Windows image is built |
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.
uncomment?
What this PR does / why we need it: brings Kubernetes v1.9.0 support
Release note: