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

add Kubernetes v1.7.16 support #2592

Merged
merged 3 commits into from
Apr 5, 2018
Merged

Conversation

jackfrancis
Copy link
Member

@jackfrancis jackfrancis commented Apr 4, 2018

What this PR does / why we need it: Kubernetes v1.7.16:

https://github.com/kubernetes/kubernetes/releases/tag/v1.7.16

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #

Special notes for your reviewer:

If applicable:

  • tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:

add Kubernetes v1.7.16 support

@ghost ghost assigned jackfrancis Apr 4, 2018
@ghost ghost added the in progress label Apr 4, 2018
@jackfrancis
Copy link
Member Author

This looks good, waiting on a Windows build @JiangtianLi

@@ -585,7 +585,7 @@ func TestWindowsVersions(t *testing.T) {
}

p = getK8sDefaultProperties(true)
p.OrchestratorProfile.OrchestratorVersion = "1.7.16"
Copy link
Member Author

Choose a reason for hiding this comment

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

We don't need to keep changing this every time we bump a 1.7 release :)

@@ -683,7 +683,7 @@ func TestLinuxVersions(t *testing.T) {
}

p = getK8sDefaultProperties(false)
p.OrchestratorProfile.OrchestratorVersion = "1.7.16"
Copy link
Member Author

Choose a reason for hiding this comment

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

ditto

Copy link
Contributor

@CecileRobertMichon CecileRobertMichon left a comment

Choose a reason for hiding this comment

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

lgtm

@jackfrancis jackfrancis merged commit 63ff4e7 into Azure:master Apr 5, 2018
@ghost ghost removed the in progress label Apr 5, 2018
@jackfrancis jackfrancis deleted the k8s-v1.7.16 branch April 5, 2018 18:25
jackfrancis added a commit that referenced this pull request Apr 6, 2018
* add Kubernetes v1.7.16 support

* updated tests so that they aren’t pinned to latest version

* enable 1.7.16 windows artifact
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.

2 participants