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

chore: remove support for Kubernetes 1.13 #3059

Merged
merged 1 commit into from
Apr 13, 2020
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
5 changes: 0 additions & 5 deletions cmd/generate_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -540,11 +540,6 @@ func TestExampleAPIModels(t *testing.T) {
apiModelPath: "../examples/kubernetes-msi-userassigned/kube-vmss.json",
setArgs: defaultSet,
},
{
name: "1.13 example",
apiModelPath: "../examples/kubernetes-releases/kubernetes1.13.json",
setArgs: defaultSet,
},
{
name: "1.14 example",
apiModelPath: "../examples/kubernetes-releases/kubernetes1.14.json",
Expand Down
6 changes: 3 additions & 3 deletions cmd/get_versions_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ func TestGetVersionsCmd_ShouldSupportJSON(t *testing.T) {

command := &getVersionsCmd{
orchestrator: "kubernetes",
version: "1.13.3",
version: "1.15.11",
output: "json",
}
err := command.run(nil, nil)
Expand All @@ -44,7 +44,7 @@ func TestGetVersionsCmd_ShouldBeHumanReadable(t *testing.T) {

command := &getVersionsCmd{
orchestrator: "kubernetes",
version: "1.13.3",
version: "1.15.11",
output: "human",
}
err := command.run(nil, nil)
Expand All @@ -57,7 +57,7 @@ func TestGetVersionsCmd_ShouldErrorIfInvalidOption(t *testing.T) {

command := &getVersionsCmd{
orchestrator: "kubernetes",
version: "1.13.3",
version: "1.15.11",
output: "yaml",
}
err := command.run(nil, nil)
Expand Down
2 changes: 1 addition & 1 deletion cmd/rotate_certs_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -257,7 +257,7 @@ func TestWriteArtifacts(t *testing.T) {
t.Parallel()

g := NewGomegaWithT(t)
cs := api.CreateMockContainerService("testcluster", "1.13.12", 3, 2, false)
cs := api.CreateMockContainerService("testcluster", "1.15.11", 3, 2, false)
_, err := cs.SetPropertiesDefaults(api.PropertiesDefaultsParams{
IsScale: false,
IsUpgrade: false,
Expand Down
2 changes: 1 addition & 1 deletion docs/community/example-launch.json
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@
"id": "kubernetesVersion",
"type": "promptString",
"description": "Kubernetes version:",
"default": "1.13.6"
"default": "1.18.1"
},
{
"id": "location",
Expand Down
2 changes: 1 addition & 1 deletion examples/addons/nvidia-device-plugin/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ This is the [NVIDIA Device Plugin](https://github.com/NVIDIA/k8s-device-plugin)
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13",
"orchestratorRelease": "1.15",
"kubernetesConfig": {
"addons": [
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13",
"orchestratorRelease": "1.15",
"kubernetesConfig": {
"addons": [
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorVersion": "1.13.11",
"orchestratorVersion": "1.15.11",
"kubernetesConfig": {
"enableRbac" : true,
"enableAggregatedAPIs": true,
Expand Down
2 changes: 1 addition & 1 deletion examples/e2e-tests/kubernetes/gpu-enabled/definition.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13"
"orchestratorRelease": "1.15"
},
"masterProfile": {
"count": 1,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13"
"orchestratorRelease": "1.15"
},
"masterProfile": {
"count": 5,
Expand Down
2 changes: 1 addition & 1 deletion examples/e2e-tests/kubernetes/zones/definition.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13"
"orchestratorRelease": "1.15"
},
"masterProfile": {
"count": 5,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease" : "1.13",
"orchestratorRelease": "1.15",
"kubernetesConfig": {
"useManagedIdentity": true,
"userAssignedID": "aksenginetestid"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease" : "1.13",
"orchestratorRelease": "1.15",
"kubernetesConfig": {
"useManagedIdentity": true,
"userAssignedID": "aksenginetestid"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease" : "1.13",
"orchestratorRelease": "1.15",
"kubernetesConfig": {
"useManagedIdentity": true,
"userAssignedID": "aksenginetestid"
Expand Down
2 changes: 1 addition & 1 deletion examples/feature-gates/kubernetes-featuresgates.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13",
"orchestratorRelease": "1.15",
"kubernetesConfig": {
"kubeletConfig" : {
"--feature-gates": "MountPropagation=true,DebugContainers=true"
Expand Down
2 changes: 1 addition & 1 deletion examples/ipvs/kubernetes-msi.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13",
"orchestratorRelease": "1.15",
"kubernetesConfig": {
"useManagedIdentity": true,
"kubeProxyMode" : "ipvs"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13",
"orchestratorRelease": "1.15",
"kubernetesConfig": {
"useCloudControllerManager": true
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13",
"orchestratorRelease": "1.15",
"kubernetesConfig": {
"enableEncryptionWithExternalKms": true,
"keyVaultSku": "Premium"
Expand Down
2 changes: 1 addition & 1 deletion examples/kubernetes-config/kubernetes-standardlb.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13",
"orchestratorRelease": "1.15",
"kubernetesConfig": {
"loadBalancerSku": "Standard",
"excludeMasterFromStandardLB": true
Expand Down
2 changes: 1 addition & 1 deletion examples/kubernetes-containerd.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13",
"orchestratorRelease": "1.15",
"kubernetesConfig": {
"networkPlugin": "flannel",
"containerRuntime": "containerd",
Expand Down
2 changes: 1 addition & 1 deletion examples/kubernetes-kubenet-containerd.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13",
"orchestratorRelease": "1.15",
"kubernetesConfig": {
"networkPlugin": "kubenet",
"containerRuntime": "containerd"
Expand Down
2 changes: 1 addition & 1 deletion examples/kubernetes-labels/kubernetes.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13"
"orchestratorRelease": "1.15"
},
"masterProfile": {
"count": 1,
Expand Down
35 changes: 0 additions & 35 deletions examples/kubernetes-releases/kubernetes1.13.json

This file was deleted.

2 changes: 1 addition & 1 deletion examples/kubernetes-vmss-master/kubernetes.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13"
"orchestratorRelease": "1.15"
},
"masterProfile": {
"count": 1,
Expand Down
2 changes: 1 addition & 1 deletion examples/kubernetes-vmss-master/windows.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13"
"orchestratorRelease": "1.15"
},
"masterProfile": {
"count": 1,
Expand Down
2 changes: 1 addition & 1 deletion examples/kubernetes-vmss-spot/kubernetes.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13",
"orchestratorRelease": "1.15",
"kubernetesConfig": {
"useManagedIdentity": true
}
Expand Down
2 changes: 1 addition & 1 deletion examples/kubernetes-vmss/kubernetes.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13",
"orchestratorRelease": "1.15",
"kubernetesConfig": {
"useManagedIdentity": true
}
Expand Down
2 changes: 1 addition & 1 deletion examples/largeclusters/kubernetes.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13",
"orchestratorRelease": "1.15",
"kubernetesConfig": {
"cloudProviderBackoff": true,
"cloudProviderBackoffRetries": 6,
Expand Down
2 changes: 1 addition & 1 deletion examples/managed-identity/kubernetes-msi.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13",
"orchestratorRelease": "1.15",
"kubernetesConfig": {
"useManagedIdentity": true
}
Expand Down
2 changes: 1 addition & 1 deletion examples/multiple-nodepools/multipool.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13"
"orchestratorRelease": "1.15"
},
"masterProfile": {
"count": 1,
Expand Down
2 changes: 1 addition & 1 deletion examples/networkpolicy/kubernetes-antrea.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13",
"orchestratorRelease": "1.15",
"kubernetesConfig": {
"networkPolicy": "antrea"
}
Expand Down
2 changes: 1 addition & 1 deletion examples/service-mesh/istio.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13",
"orchestratorRelease": "1.15",
"kubernetesConfig": {
"apiServerConfig": {
"--admission-control": "NamespaceLifecycle,LimitRanger,ServiceAccount,DefaultStorageClass,DefaultTolerationSeconds,MutatingAdmissionWebhook,ValidatingAdmissionWebhook,ResourceQuota,DenyEscalatingExec,AlwaysPullImages"
Expand Down
2 changes: 1 addition & 1 deletion examples/windows/kubernetes-hyperv.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13",
"orchestratorRelease": "1.15",
"kubernetesConfig": {
"apiServerConfig" : {
"--feature-gates": "HyperVContainer=true"
Expand Down
8 changes: 4 additions & 4 deletions pkg/api/apiloader_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -197,7 +197,7 @@ func TestLoadContainerServiceWithEmptyLocationCustomCloud(t *testing.T) {
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13",
"orchestratorRelease": "1.15",
"kubernetesConfig": {
"kubernetesImageBase": "msazurestackqa/",
"useInstanceMetadata": false,
Expand Down Expand Up @@ -281,7 +281,7 @@ func TestLoadContainerServiceWithEmptyLocationCustomCloud(t *testing.T) {
"properties": {
"orchestratorProfile": {
"orchestratorType": "Kubernetes",
"orchestratorRelease": "1.13",
"orchestratorRelease": "1.15",
"kubernetesConfig": {
"kubernetesImageBase": "msazurestackqa/",
"useInstanceMetadata": false,
Expand Down Expand Up @@ -430,7 +430,7 @@ func TestLoadContainerServiceForAgentPoolOnlyClusterWithRawJSON(t *testing.T) {
Translator: &i18n.Translator{},
}

cs, _, err := apiloader.LoadContainerServiceForAgentPoolOnlyCluster(rawJSON, v20170831.APIVersion, true, false, "1.13.11", nil)
cs, _, err := apiloader.LoadContainerServiceForAgentPoolOnlyCluster(rawJSON, v20170831.APIVersion, true, false, "1.15.11", nil)

if err != nil {
t.Errorf("unexpected error while executing LoadContainerServiceForAgentPoolOnlyCluster: %s", err.Error())
Expand All @@ -456,7 +456,7 @@ func TestLoadContainerServiceForAgentPoolOnlyClusterWithRawJSON(t *testing.T) {
rawJSON20180331 := []byte(`{"id":"sampleID","location":"westus2","plan":{"name":"sampleRPPlan","product":"fooProduct","promotionCode":"barPromoCode","publisher":"bazPublisher"},"tags":{"123":"456","abc":"def"},"type":"sampleType","properties":{"provisioningState":"Succeeded","kubernetesVersion":"","dnsPrefix":"blueorange","fqdn":"blueorange.azure.com","agentPoolProfiles":[{"name":"sampleagent","count":0,"vmSize":"Standard_DS1_v1","osDiskSizeGB":512,"storageProfile":"ManagedDisks","vnetSubnetID":"/subscriptions/SUB_ID/resourceGroups/RG_NAME/providers/Microsoft.Network/virtualNetworks/sampleVnet/subnets/sampleVnetSubnetID","osType":"Linux"}],"windowsProfile":{"adminUsername":"azureuser","adminPassword":"azurepassword"},"servicePrincipalProfile":{"clientId":"sampleClientID","secret":"sampleSecret"}}}`)

//Test with version v20180331
cs, _, err = apiloader.LoadContainerServiceForAgentPoolOnlyCluster(rawJSON20180331, v20180331.APIVersion, true, false, "1.13.11", nil)
cs, _, err = apiloader.LoadContainerServiceForAgentPoolOnlyCluster(rawJSON20180331, v20180331.APIVersion, true, false, "1.15.11", nil)

if err != nil {
t.Errorf("unexpected error while executing LoadContainerServiceForAgentPoolOnlyCluster: %s", err.Error())
Expand Down
2 changes: 1 addition & 1 deletion pkg/api/common/versions.go
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@ var AllKubernetesSupportedVersions = map[string]bool{
"1.13.8": false,
"1.13.9": false,
"1.13.10": false,
"1.13.11": true,
"1.13.11": false,
"1.13.12": false, // disabled because of https://github.com/Azure/aks-engine/issues/2312
"1.14.0-alpha.1": false,
"1.14.0-alpha.2": false,
Expand Down
2 changes: 1 addition & 1 deletion pkg/api/convertertoagentpoolonlyapi_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ import (
)

func TestConvertV20180331AgentPoolOnlyOrchestratorProfile(t *testing.T) {
kubernetesVersion := "1.13.11"
kubernetesVersion := "1.15.11"
networkPlugin := v20180331.Azure
networkPolicy := v20180331.NetworkPolicyCalico
networkPluginKubenet := v20180331.Kubenet
Expand Down
2 changes: 1 addition & 1 deletion pkg/api/defaults-kubelet_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -295,7 +295,7 @@ func TestKubeletConfigAzureStackDefaults(t *testing.T) {
}

func TestKubeletConfigDefaultsRemovals(t *testing.T) {
cs := CreateMockContainerService("testcluster", common.RationalizeReleaseAndVersion(Kubernetes, "1.13", "", false, false), 3, 2, false)
cs := CreateMockContainerService("testcluster", common.RationalizeReleaseAndVersion(Kubernetes, "1.15", "", false, false), 3, 2, false)
poolProfile := &AgentPoolProfile{}
poolProfile.Count = 1
poolProfile.Name = "agentpool2"
Expand Down