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

fix: E2E scale scenario broken if update is false #3852

Merged
merged 2 commits into from
Sep 22, 2020
Merged

fix: E2E scale scenario broken if update is false #3852

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
86c02d9
fix: E2E scale scenario broken if update is false
jackfrancis Sep 21, 2020
e926a95
wrong comparison
jackfrancis Sep 21, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion test/e2e/cluster.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -302,8 +302,8 @@ fi
if [ "${SCALE_CLUSTER}" = "true" ]; then
nodepools=$(jq -r '.properties.agentPoolProfiles[].name' < _output/$RESOURCE_GROUP/apimodel.json)
for ((i = 0; i <= ${#nodepools[@]}; ++i)); do
nodepool=$(jq -r --arg i $i '. | .properties.agentPoolProfiles[$i | tonumber].name' < _output/$RESOURCE_GROUP/apimodel.json)
if [ -n "$UPDATE_NODE_POOLS" ]; then
nodepool=$(jq -r --arg i $i '. | .properties.agentPoolProfiles[$i | tonumber].name' < _output/$RESOURCE_GROUP/apimodel.json)
# modify the master VM SKU to simulate vertical vm scaling via upgrade
docker run --rm \
-v $(pwd):${WORK_DIR} \
Expand Down