Skip to content

Commit 0719c2e

Browse files
Abdul-MicrosoftAbdul-Microsoft
committed
Merge remote-tracking branch 'origin/dev' into psl-logging-enhancement
2 parents 7fb7f6c + 57c1324 commit 0719c2e

11 files changed

Lines changed: 31 additions & 44 deletions

‎docs/CustomizingAzdParameters.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ By default this template will use the environment name as the prefix to prevent
1717
| `AZURE_ENV_MODEL_CAPACITY` | integer | `150` | Set the Model Capacity (choose a number based on available GPT model capacity in your subscription). |
1818
| `AZURE_ENV_LOG_ANALYTICS_WORKSPACE_ID` | string | Guide to get your [Existing Workspace ID](/docs/re-use-log-analytics.md) | Set this if you want to reuse an existing Log Analytics Workspace instead of creating a new one. |
1919
| `AZURE_ENV_IMAGETAG` | string | `latest` | Set the Image tag Like (allowed values: latest, dev, hotfix) |
20-
| `AZURE_ENV_JUMPBOX_SIZE` | string | `Standard_DS2_v2` | Specifies the size of the Jumpbox Virtual Machine. Set a custom value if `enablePrivateNetworking` is `true`. |
20+
| `AZURE_ENV_VM_SIZE` | string | `Standard_D2s_v5` | Specifies the size of the Jumpbox Virtual Machine (e.g., `Standard_D2s_v5`, `Standard_D2s_v4`). Set a custom value if `enablePrivateNetworking` is `true`. |
2121
| `AZURE_ENV_JUMPBOX_ADMIN_USERNAME` | string | `JumpboxAdminUser` | Specifies the administrator username for the Jumpbox Virtual Machine. |
2222
| `AZURE_ENV_JUMPBOX_ADMIN_PASSWORD` | string | `JumpboxAdminP@ssw0rd1234!` | Specifies the administrator password for the Jumpbox Virtual Machine. |
2323
| `AZURE_ENV_COSMOS_SECONDARY_LOCATION` | string | *(not set by default)* | Specifies the secondary region for Cosmos DB. Required if `enableRedundancy` is `true`. |

‎docs/DeploymentGuide.md‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -202,11 +202,11 @@ Copy the contents from the production configuration file to your main parameters
202202

203203
> **Note:** This section only applies if you selected **Production** deployment type in section 3.1. VMs are not deployed in the default Development/Testing configuration.
204204

205-
By default, random GUIDs are generated for VM credentials. To set custom credentials:
205+
By default, hard-coded fallback values are used for VM credentials (`JumpboxAdminUser` / `JumpboxAdminP@ssw0rd1234!`). To set custom credentials:
206206

207207
```shell
208-
azd env set AZURE_ENV_VM_ADMIN_USERNAME <your-username>
209-
azd env set AZURE_ENV_VM_ADMIN_PASSWORD <your-password>
208+
azd env set AZURE_ENV_JUMPBOX_ADMIN_USERNAME <your-username>
209+
azd env set AZURE_ENV_JUMPBOX_ADMIN_PASSWORD <your-password>
210210
```
211211

212212
### 3.3 Advanced Configuration (Optional)

‎docs/TroubleShootingSteps.md‎

Lines changed: 3 additions & 3 deletions
Large diffs are not rendered by default.

‎infra/main.bicep‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -636,9 +636,9 @@ module virtualMachine 'br/public:avm/res/compute/virtual-machine:0.20.0' = if (e
636636
enableTelemetry: enableTelemetry
637637
computerName: take(virtualMachineResourceName, 15)
638638
osType: 'Windows'
639-
vmSize: vmSize ?? 'Standard_D2s_v3'
640-
adminUsername: vmAdminUsername ?? 'JumpboxAdminUser'
641-
adminPassword: vmAdminPassword ?? 'JumpboxAdminP@ssw0rd1234!'
639+
vmSize: !empty(vmSize) ? vmSize : 'Standard_D2s_v5'
640+
adminUsername: !empty(vmAdminUsername) ? vmAdminUsername : 'JumpboxAdminUser'
641+
adminPassword: !empty(vmAdminPassword) ? vmAdminPassword : 'JumpboxAdminP@ssw0rd1234!'
642642
managedIdentities: {
643643
systemAssigned: true
644644
}

‎infra/main.json‎

Lines changed: 7 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
"_generator": {
77
"name": "bicep",
88
"version": "0.41.2.15936",
9-
"templateHash": "5257255376706775962"
9+
"templateHash": "3093757051086668797"
1010
},
1111
"name": "Modernize Your Code Solution Accelerator",
1212
"description": "CSA CTO Gold Standard Solution Accelerator for Modernize Your Code. \r\n"
@@ -12895,11 +12895,11 @@
1289512895
},
1289612896
"dependsOn": [
1289712897
"applicationInsights",
12898-
"[format('avmPrivateDnsZones[{0}]', variables('dnsZoneIndex').monitor)]",
12899-
"[format('avmPrivateDnsZones[{0}]', variables('dnsZoneIndex').oms)]",
1290012898
"[format('avmPrivateDnsZones[{0}]', variables('dnsZoneIndex').storageBlob)]",
1290112899
"[format('avmPrivateDnsZones[{0}]', variables('dnsZoneIndex').ods)]",
1290212900
"[format('avmPrivateDnsZones[{0}]', variables('dnsZoneIndex').agentSvc)]",
12901+
"[format('avmPrivateDnsZones[{0}]', variables('dnsZoneIndex').monitor)]",
12902+
"[format('avmPrivateDnsZones[{0}]', variables('dnsZoneIndex').oms)]",
1290312903
"dataCollectionEndpoint",
1290412904
"logAnalyticsWorkspace",
1290512905
"virtualNetwork"
@@ -16617,15 +16617,9 @@
1661716617
"osType": {
1661816618
"value": "Windows"
1661916619
},
16620-
"vmSize": {
16621-
"value": "[coalesce(parameters('vmSize'), 'Standard_D2s_v3')]"
16622-
},
16623-
"adminUsername": {
16624-
"value": "[coalesce(parameters('vmAdminUsername'), 'JumpboxAdminUser')]"
16625-
},
16626-
"adminPassword": {
16627-
"value": "[coalesce(parameters('vmAdminPassword'), 'JumpboxAdminP@ssw0rd1234!')]"
16628-
},
16620+
"vmSize": "[if(not(empty(parameters('vmSize'))), createObject('value', parameters('vmSize')), createObject('value', 'Standard_D2s_v5'))]",
16621+
"adminUsername": "[if(not(empty(parameters('vmAdminUsername'))), createObject('value', parameters('vmAdminUsername')), createObject('value', 'JumpboxAdminUser'))]",
16622+
"adminPassword": "[if(not(empty(parameters('vmAdminPassword'))), createObject('value', parameters('vmAdminPassword')), createObject('value', 'JumpboxAdminP@ssw0rd1234!'))]",
1662916623
"managedIdentities": {
1663016624
"value": {
1663116625
"systemAssigned": true
@@ -31922,9 +31916,9 @@
3192231916
},
3192331917
"dependsOn": [
3192431918
"aiServices",
31919+
"[format('avmPrivateDnsZones[{0}]', variables('dnsZoneIndex').openAI)]",
3192531920
"[format('avmPrivateDnsZones[{0}]', variables('dnsZoneIndex').cognitiveServices)]",
3192631921
"[format('avmPrivateDnsZones[{0}]', variables('dnsZoneIndex').aiServices)]",
31927-
"[format('avmPrivateDnsZones[{0}]', variables('dnsZoneIndex').openAI)]",
3192831922
"virtualNetwork"
3192931923
]
3193031924
},

‎infra/main.parameters.json‎

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -38,15 +38,6 @@
3838
"azureAiServiceLocation": {
3939
"value": "${AZURE_ENV_AI_SERVICE_LOCATION}"
4040
},
41-
"vmSize": {
42-
"value": "${AZURE_ENV_JUMPBOX_SIZE}"
43-
},
44-
"vmAdminUsername": {
45-
"value": "${AZURE_ENV_JUMPBOX_ADMIN_USERNAME}"
46-
},
47-
"vmAdminPassword": {
48-
"value": "${AZURE_ENV_JUMPBOX_ADMIN_PASSWORD}"
49-
},
5041
"backendExists": {
5142
"value": "${SERVICE_BACKEND_RESOURCE_EXISTS=false}"
5243
},

‎infra/main.waf.parameters.json‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,15 +38,15 @@
3838
"azureAiServiceLocation": {
3939
"value": "${AZURE_ENV_AI_SERVICE_LOCATION}"
4040
},
41-
"vmSize": {
42-
"value": "${AZURE_ENV_JUMPBOX_SIZE}"
43-
},
4441
"vmAdminUsername": {
4542
"value": "${AZURE_ENV_JUMPBOX_ADMIN_USERNAME}"
4643
},
4744
"vmAdminPassword": {
4845
"value": "${AZURE_ENV_JUMPBOX_ADMIN_PASSWORD}"
4946
},
47+
"vmSize": {
48+
"value": "${AZURE_ENV_VM_SIZE}"
49+
},
5050
"backendExists": {
5151
"value": "${SERVICE_BACKEND_RESOURCE_EXISTS=false}"
5252
},

‎infra/main_custom.bicep‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -565,9 +565,9 @@ module virtualMachine 'br/public:avm/res/compute/virtual-machine:0.20.0' = if (e
565565
enableTelemetry: enableTelemetry
566566
computerName: take(virtualMachineResourceName, 15)
567567
osType: 'Windows'
568-
vmSize: vmSize ?? 'Standard_D2s_v3'
569-
adminUsername: vmAdminUsername ?? 'JumpboxAdminUser'
570-
adminPassword: vmAdminPassword ?? 'JumpboxAdminP@ssw0rd1234!'
568+
vmSize: !empty(vmSize) ? vmSize : 'Standard_D2s_v5'
569+
adminUsername: !empty(vmAdminUsername) ? vmAdminUsername : 'JumpboxAdminUser'
570+
adminPassword: !empty(vmAdminPassword) ? vmAdminPassword : 'JumpboxAdminP@ssw0rd1234!'
571571
managedIdentities: {
572572
systemAssigned: true
573573
}

‎infra/modules/virtualNetwork.bicep‎

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -170,10 +170,12 @@ param resourceSuffix string
170170

171171
// VM Size Notes:
172172
// 1 B-series VMs (like Standard_B2ms) do not support accelerated networking.
173-
// 2 Pick a VM size that does support accelerated networking (the usual jump-box candidates):
174-
// Standard_DS2_v2 (2 vCPU, 7 GiB RAM, Premium SSD) // The most broadly available (it’s a legacy SKU supported in virtually every region).
175-
// Standard_D2s_v3 (2 vCPU, 8 GiB RAM, Premium SSD) // next most common
176-
// Standard_D2s_v4 (2 vCPU, 8 GiB RAM, Premium SSD) // Newest, so fewer regions available
173+
// 2 Pick a VM size that supports accelerated networking + Premium SSD (the usual jump-box candidates):
174+
// Standard_D2s_v5 (2 vCPU, 8 GiB RAM, Premium SSD/v2/Ultra) // DEFAULT - current-gen Intel, broad regional availability.
175+
// Standard_D2as_v5 (2 vCPU, 8 GiB RAM, Premium SSD/Ultra) // AMD alternative, typically ~15% cheaper.
176+
// Standard_D2s_v4 (2 vCPU, 8 GiB RAM, Premium SSD) // Previous gen, also broadly available.
177+
// Standard_DS2_v2 (2 vCPU, 7 GiB RAM, Premium SSD) // Legacy SKU, being retired from some regions - avoid for new deployments.
178+
// 3 A-series (Av2) is NOT suitable: no Premium SSD support, no accelerated networking.
177179

178180
// Subnet Classless Inter-Domain Routing (CIDR) Sizing Reference Table (Best Practices)
179181
// | CIDR | # of Addresses | # of /24s | Notes |

‎infra/samples/network-subnet-design.bicep‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ import { jumpBoxConfigurationType } from 'network/jumpbox.bicep'
110110
@description('Optional. Configuration for the Jumpbox VM. Leave null to omit Jumpbox creation.')
111111
param jumpboxConfiguration jumpBoxConfigurationType = {
112112
name: 'vm-jumpbox-${resourcesName}'
113-
size: 'Standard_D2s_v3' // Default size, can be overridden
113+
size: 'Standard_D2s_v5' // Default size, can be overridden
114114
username: vmAdminUsername
115115
password: vmAdminPassword
116116
subnet: {

0 commit comments

Comments
 (0)