Skip to content

Commit

Permalink
Merge pull request #2292 from monianshouhou/modify_location_1.1
Browse files Browse the repository at this point in the history
[release-1.1] fix: switch all ado tests location to eastus
  • Loading branch information
k8s-ci-robot committed Sep 15, 2022
2 parents 8a77945 + 89a5aa0 commit 939887e
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 11 deletions.
7 changes: 2 additions & 5 deletions .pipelines/scripts/run-e2e.sh
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ az extension add -n aks-preview
az login --service-principal -u "${AZURE_CLIENT_ID}" -p "${AZURE_CLIENT_SECRET}" --tenant "${AZURE_TENANT_ID}"

get_random_location() {
local LOCATIONS=("eastus" "eastus2" "southcentralus" "westus3")
local LOCATIONS=("eastus")
echo "${LOCATIONS[${RANDOM} % ${#LOCATIONS[@]}]}"
}

Expand All @@ -43,9 +43,6 @@ cleanup() {
trap cleanup EXIT

export AZURE_LOCATION="$(get_random_location)"
if [[ "${CLUSTER_TYPE}" =~ "autoscaling" ]]; then
export AZURE_LOCATION="australiaeast"
fi

IMAGE_TAG="$(git rev-parse --short=7 HEAD)"
CLUSTER_CONFIG_PATH="${REPO_ROOT}/.pipelines/templates/basic-lb.json"
Expand Down Expand Up @@ -90,7 +87,7 @@ echo "Running e2e"

# TODO: We should do it in autoscaling-multipool.json
if [[ "${CLUSTER_TYPE}" == "autoscaling-multipool" ]]; then
az aks update --resource-group "${RESOURCE_GROUP}" --name "${CLUSTER_NAME}" --cluster-autoscaler-profile balance-similar-node-groups=true
az aks update --subscription ${AZURE_SUBSCRIPTION_ID} --resource-group "${RESOURCE_GROUP}" --name "${CLUSTER_NAME}" --cluster-autoscaler-profile balance-similar-node-groups=true
fi

export E2E_ON_AKS_CLUSTER=true
Expand Down
6 changes: 0 additions & 6 deletions tests/e2e/autoscaling/autoscaler.go
Original file line number Diff line number Diff line change
Expand Up @@ -272,9 +272,6 @@ var _ = Describe("Cluster size autoscaler", Label(utils.TestSuiteLabelFeatureAut

targetNodeCount := initNodeCount
expectedNode := 50
if os.Getenv(utils.AKSTestCCM) != "" {
expectedNode = 40
}
for {
*scaleUpDeployment.Spec.Replicas += 5
targetNodeCount += 5
Expand Down Expand Up @@ -314,9 +311,6 @@ var _ = Describe("Cluster size autoscaler", Label(utils.TestSuiteLabelFeatureAut

targetNodeCount := initNodeCount
expectedNode := 50
if os.Getenv(utils.AKSTestCCM) != "" {
expectedNode = 38
}
for {
*scaleUpDeployment.Spec.Replicas += int32(expectedNode / 2)
targetNodeCount += expectedNode / 2
Expand Down

0 comments on commit 939887e

Please sign in to comment.