Skip to content

Commit 59a7aa7

Browse files
removed unnecessary comments
1 parent 7f50235 commit 59a7aa7

2 files changed

Lines changed: 4 additions & 22 deletions

File tree

infra/main.bicep

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -220,9 +220,6 @@ resource resourceGroupTags 'Microsoft.Resources/tags@2025-04-01' = {
220220
}
221221
}
222222

223-
// AVM Telemetry: Standard Azure Verified Modules (AVM) no-op marker deployment.
224-
// Deploys zero actual resources — used only to track AVM module usage telemetry.
225-
// Gated by the enableTelemetry parameter (default: true). See https://aka.ms/avm/TelemetryInfo
226223
#disable-next-line no-deployments-resources
227224
resource avmTelemetry 'Microsoft.Resources/deployments@2024-03-01' = if (enableTelemetry) {
228225
name: '46d3xbcp.ptn.sa-convknowledgemining.${replace('-..--..-', '.', '-')}.${substring(uniqueString(deployment().name, location), 0, 4)}'
@@ -325,10 +322,6 @@ module applicationInsights 'br/public:avm/res/insights/component:0.7.1' = if (en
325322
}
326323
}
327324
// ========== Virtual Network and Networking Components ========== //
328-
// NOTE: All resources in this section (VNet, Bastion, Jumpbox VM, Private DNS Zones, Private Endpoints)
329-
// are gated behind enablePrivateNetworking (default: false). These are infrastructure-only resources
330-
// required for WAF-aligned deployments with private networking. They provide secure network connectivity
331-
// and operational access (jumpbox/bastion for RDP/SSH) — no application code references are expected.
332325

333326
// Virtual Network with NSGs and Subnets
334327
module virtualNetwork 'modules/virtualNetwork.bicep' = if (enablePrivateNetworking) {
@@ -343,7 +336,7 @@ module virtualNetwork 'modules/virtualNetwork.bicep' = if (enablePrivateNetworki
343336
enableTelemetry: enableTelemetry
344337
}
345338
}
346-
// Azure Bastion Host — provides secure RDP/SSH access to the Jumpbox VM without exposing public IPs
339+
// Azure Bastion Host
347340
var bastionHostName = 'bas-${solutionSuffix}'
348341
module bastionHost 'br/public:avm/res/network/bastion-host:0.8.2' = if (enablePrivateNetworking) {
349342
name: take('avm.res.network.bastion-host.${bastionHostName}', 64)
@@ -372,7 +365,7 @@ module bastionHost 'br/public:avm/res/network/bastion-host:0.8.2' = if (enablePr
372365
}
373366
}
374367

375-
// Jumpbox Virtual Machine — provides operational access to private network resources via Bastion
368+
// Jumpbox Virtual Machine
376369
var jumpboxVmName = take('vm-jumpbox-${solutionSuffix}', 15)
377370
module jumpboxVM 'br/public:avm/res/compute/virtual-machine:0.21.0' = if (enablePrivateNetworking) {
378371
name: take('avm.res.compute.virtual-machine.${jumpboxVmName}', 64)
@@ -432,8 +425,6 @@ module jumpboxVM 'br/public:avm/res/compute/virtual-machine:0.21.0' = if (enable
432425
}
433426

434427
// ========== Private DNS Zones ========== //
435-
// These DNS zones enable private endpoint name resolution for all Azure services in WAF deployments.
436-
// Each zone is linked to the VNet and allows private endpoints to resolve to internal IPs.
437428
var privateDnsZones = [
438429
'privatelink.cognitiveservices.azure.com'
439430
'privatelink.openai.azure.com'

infra/main_custom.bicep

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -202,9 +202,6 @@ resource resourceGroupTags 'Microsoft.Resources/tags@2025-04-01' = {
202202
}
203203
}
204204

205-
// AVM Telemetry: Standard Azure Verified Modules (AVM) no-op marker deployment.
206-
// Deploys zero actual resources — used only to track AVM module usage telemetry.
207-
// Gated by the enableTelemetry parameter (default: true). See https://aka.ms/avm/TelemetryInfo
208205
#disable-next-line no-deployments-resources
209206
resource avmTelemetry 'Microsoft.Resources/deployments@2024-03-01' = if (enableTelemetry) {
210207
name: '46d3xbcp.ptn.sa-convknowledgemining.${replace('-..--..-', '.', '-')}.${substring(uniqueString(deployment().name, location), 0, 4)}'
@@ -308,10 +305,6 @@ module applicationInsights 'br/public:avm/res/insights/component:0.7.1' = if (en
308305
}
309306
}
310307
// ========== Virtual Network and Networking Components ========== //
311-
// NOTE: All resources in this section (VNet, Bastion, Jumpbox VM, Private DNS Zones, Private Endpoints)
312-
// are gated behind enablePrivateNetworking (default: false). These are infrastructure-only resources
313-
// required for WAF-aligned deployments with private networking. They provide secure network connectivity
314-
// and operational access (jumpbox/bastion for RDP/SSH) — no application code references are expected.
315308

316309
// Virtual Network with NSGs and Subnets
317310
module virtualNetwork 'modules/virtualNetwork.bicep' = if (enablePrivateNetworking) {
@@ -326,7 +319,7 @@ module virtualNetwork 'modules/virtualNetwork.bicep' = if (enablePrivateNetworki
326319
enableTelemetry: enableTelemetry
327320
}
328321
}
329-
// Azure Bastion Host — provides secure RDP/SSH access to the Jumpbox VM without exposing public IPs
322+
// Azure Bastion Host
330323
var bastionHostName = 'bas-${solutionSuffix}'
331324
module bastionHost 'br/public:avm/res/network/bastion-host:0.8.2' = if (enablePrivateNetworking) {
332325
name: take('avm.res.network.bastion-host.${bastionHostName}', 64)
@@ -355,7 +348,7 @@ module bastionHost 'br/public:avm/res/network/bastion-host:0.8.2' = if (enablePr
355348
}
356349
}
357350

358-
// Jumpbox Virtual Machine — provides operational access to private network resources via Bastion
351+
// Jumpbox Virtual Machine
359352
var jumpboxVmName = take('vm-jumpbox-${solutionSuffix}', 15)
360353
module jumpboxVM 'br/public:avm/res/compute/virtual-machine:0.21.0' = if (enablePrivateNetworking) {
361354
name: take('avm.res.compute.virtual-machine.${jumpboxVmName}', 64)
@@ -415,8 +408,6 @@ module jumpboxVM 'br/public:avm/res/compute/virtual-machine:0.21.0' = if (enable
415408
}
416409

417410
// ========== Private DNS Zones ========== //
418-
// These DNS zones enable private endpoint name resolution for all Azure services in WAF deployments.
419-
// Each zone is linked to the VNet and allows private endpoints to resolve to internal IPs.
420411
var privateDnsZones = [
421412
'privatelink.cognitiveservices.azure.com'
422413
'privatelink.openai.azure.com'

0 commit comments

Comments
 (0)