Skip to content
This repository has been archived by the owner on Feb 9, 2022. It is now read-only.

Commit

Permalink
ci: fix aks k8s version selection
Browse files Browse the repository at this point in the history
the `kversion` variable was not being properly bound because of closures as a result of
which the wrong k8s version was being picked for the aks tests

(cherry picked from commit 46fa8f2)
  • Loading branch information
Sameer Naik committed Apr 30, 2019
1 parent 422d434 commit 770c525
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions Jenkinsfile
Original file line number Diff line number Diff line change
Expand Up @@ -293,7 +293,8 @@ spec:
// See:
// gcloud container get-server-config
def gkeKversions = ["1.11"]
for (kversion in gkeKversions) {
for (x in gkeKversions) {
def kversion = x // local bind required because closures
def project = 'bkprtesting'
def zone = 'us-east1-b'
def platform = "gke-" + kversion
Expand Down Expand Up @@ -383,7 +384,8 @@ spec:
// See:
// az aks get-versions -l centralus --query 'sort(orchestrators[?orchestratorType==`Kubernetes`].orchestratorVersion)'
def aksKversions = ["1.10", "1.11"]
for (kversion in aksKversions) {
for (x in aksKversions) {
def kversion = x // local bind required because closures
def resourceGroup = 'jenkins-bkpr-rg'
def location = "eastus"
def platform = "aks-" + kversion
Expand Down Expand Up @@ -500,7 +502,8 @@ spec:
}

def eksKversions = ["1.10", "1.11"]
for (kversion in eksKversions) {
for (x in eksKversions) {
def kversion = x // local bind required because closures
def awsRegion = "us-east-1"
def awsUserPoolId = "${awsRegion}_zkRzdsjxA"
def awsZones = ["us-east-1b", "us-east-1f"]
Expand Down

0 comments on commit 770c525

Please sign in to comment.