Skip to content
This repository has been archived by the owner on Jan 11, 2023. It is now read-only.

bumped addonmanager tag to be consistent with other releases #1064

Merged
merged 1 commit into from
Jul 20, 2017

Conversation

lachie83
Copy link
Member

@lachie83 lachie83 commented Jul 20, 2017

This change is Reviewable

@msftclas
Copy link

@lachie83,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@acs-bot
Copy link

acs-bot commented Jul 20, 2017

Can one of the admins verify this patch? Say "@acs-bot test this please" to start tests.

@seanknox
Copy link
Contributor

@acs-bot test this please

@@ -109,7 +109,7 @@ var KubeImages = map[api.OrchestratorVersion]map[string]string{
"addonresizer": "addon-resizer:2.0",
"heapster": "heapster:v1.4.0",
"dns": "k8s-dns-kube-dns-amd64:1.14.4",
"addonmanager": "kube-addon-manager-amd64:v6.1.2",
"addonmanager": "kube-addon-manager-amd64:v6.4-beta.2",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is there a reason we are using a beta version?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's the version that currently supports using kubectl 1.6 which is required for creation of RBAC resource types using addon-manager . See this for the changelog --
https://github.com/kubernetes/kubernetes/blob/master/cluster/addons/addon-manager/CHANGELOG.md#version-64-beta2--mon-june-12-2017-jeff-grafton-jgraftongooglecom. We used the 6.4 tag but which I thought would cover this beta since 6.4 hasn't actually been releases and the timestamp on 6.4 is much older than the 6.4-beta.2 -- https://gcr.io/v2/google_containers/kube-addon-manager/tags/list

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So we need 6.4 and it is still in beta. Is that correct? I guess this is okay, but just makes me kind of nervous to use beta versions.

@@ -109,7 +109,7 @@ var KubeImages = map[api.OrchestratorVersion]map[string]string{
"addonresizer": "addon-resizer:2.0",
"heapster": "heapster:v1.4.0",
"dns": "k8s-dns-kube-dns-amd64:1.14.4",
"addonmanager": "kube-addon-manager-amd64:v6.1.2",
"addonmanager": "kube-addon-manager-amd64:v6.4-beta.2",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So we need 6.4 and it is still in beta. Is that correct? I guess this is okay, but just makes me kind of nervous to use beta versions.

@lachie83
Copy link
Member Author

Yes that's correct and I don't think we have any other option at this point if we would like to run this tool.

@seanknox seanknox merged commit 42b4fa3 into Azure:master Jul 20, 2017
@seanknox
Copy link
Contributor

Kubernetes components at "beta" are already heavily tested—Deployments are one of many examples. Kubernetes beta is many more times rigorous than just about any other software project in existence.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants