Skip to content

Commit 689e34c

Browse files
authored
Merge pull request #184438 from osedlacek/patch-3
Added missing parameters and tip
2 parents 37ff18b + b851988 commit 689e34c

File tree

1 file changed

+10
-1
lines changed

1 file changed

+10
-1
lines changed

articles/virtual-machines/workloads/sap/automation-deploy-workload-zone.md

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,6 +112,8 @@ export spn_id="<appID>"
112112
export spn_secret="<password>"
113113
export tenant_id="<tenant>"
114114
export region_code="WEEU"
115+
export storageaccount="<storageaccount>"
116+
export keyvault="<keyvault>"
115117

116118
export DEPLOYMENT_REPO_PATH="${HOME}/Azure_SAP_Automated_Deployment/sap-automation"
117119
export ARM_SUBSCRIPTION_ID="${subscriptionID}"
@@ -125,7 +127,9 @@ ${DEPLOYMENT_REPO_PATH}/deploy/scripts/install_workloadzone.sh
125127
--subscription "${subscriptionID}" \
126128
--spn_id "${spn_id}" \
127129
--spn_secret "${spn_secret}" \
128-
--tenant_id "${tenant_id}"
130+
--tenant_id "${tenant_id}" \
131+
--keyvault "${keyvault}" \
132+
--storageaccountname "${storageaccount}"
129133
```
130134
# [Windows](#tab/windows)
131135

@@ -166,6 +170,11 @@ New-SAPWorkloadZone -Parameterfile DEV-$region_code-SAP01-INFRASTRUCTURE.tfvars
166170
> Replace `<storageaccount>` with the name of the storage account containing the Terraform state files
167171
> Replace `<statefile_subscription>` with the subscription ID for the storage account containing the Terraform state files
168172
173+
---
174+
175+
> [!TIP]
176+
> If the scripts fail to run, it can sometimes help to clear the local cache files by removing `~/.sap_deployment_automation/` and `~/.terraform.d/` directories before running the scripts again.
177+
169178
## Next step
170179

171180
> [!div class="nextstepaction"]

0 commit comments

Comments
 (0)