-
Notifications
You must be signed in to change notification settings - Fork 558
Conversation
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
@@ -22,7 +22,7 @@ RUN mkdir /tmp/azurecli \ | |||
|
|||
RUN curl -fsSL https://get.docker.com/ | sh | |||
|
|||
ENV KUBECTL_VERSION 1.6.6 | |||
ENV KUBECTL_VERSION 1.7.5 |
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.
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.
Dockerfile used only in dev workflow AFAIK, and yes I think it's better for dev workflow to orient toward newer versions. Not attached to this, can remove in this PR.
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 actually like it here.
@@ -22,7 +22,7 @@ RUN mkdir /tmp/azurecli \ | |||
|
|||
RUN curl -fsSL https://get.docker.com/ | sh | |||
|
|||
ENV KUBECTL_VERSION 1.6.6 | |||
ENV KUBECTL_VERSION 1.7.5 |
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 actually like it here.
Bumps v1.7 release to v1.7.5:
I've built the Windows zip which is available at https://acsmirror.blob.core.windows.net/wink8s/v1.7.5-1intwinnat.zip (non-CDN URL).
@lachie83 @jackfrancis @JiangtianLi