Skip to content

Commit 8559028

Browse files
committed
updating TF variables
1 parent ac8b38f commit 8559028

File tree

4 files changed

+50
-19
lines changed

4 files changed

+50
-19
lines changed

quickstart/202-machine-learning-moderately-secure-existing-VNet/compute.tf

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ resource "azurerm_machine_learning_compute_instance" "compute_instance" {
1212
location = azurerm_resource_group.default.location
1313
machine_learning_workspace_id = azurerm_machine_learning_workspace.default.id
1414
virtual_machine_size = "STANDARD_DS2_V2"
15-
subnet_resource_id = var.training_subnet_resource_id
15+
subnet_resource_id = "${data.azurerm_subnet.training.id}"
1616

1717
depends_on = [
1818
azurerm_private_endpoint.mlw_ple
@@ -26,7 +26,7 @@ resource "azurerm_machine_learning_compute_cluster" "compute" {
2626
machine_learning_workspace_id = azurerm_machine_learning_workspace.default.id
2727
vm_priority = "Dedicated"
2828
vm_size = "STANDARD_DS2_V2"
29-
subnet_resource_id = var.training_subnet_resource_id
29+
subnet_resource_id = "${data.azurerm_subnet.training.id}"
3030

3131
identity {
3232
type = "SystemAssigned"

quickstart/202-machine-learning-moderately-secure-existing-VNet/network.tf

Lines changed: 24 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,23 @@
1+
# Data Sources
2+
3+
data "azurerm_subnet" "training" {
4+
name = var.training_subnet_name
5+
virtual_network_name = var.vnet_name
6+
resource_group_name = var.vnet_resource_group_name
7+
}
8+
9+
data "azurerm_subnet" "aks" {
10+
name = var.aks_subnet_name
11+
virtual_network_name = var.vnet_name
12+
resource_group_name = var.vnet_resource_group_name
13+
}
14+
15+
data "azurerm_subnet" "ml" {
16+
name = var.ml_subnet_name
17+
virtual_network_name = var.vnet_name
18+
resource_group_name = var.vnet_resource_group_name
19+
}
20+
121
# Network Security Groups
222
resource "azurerm_network_security_group" "nsg-training" {
323
name = "nsg-training"
@@ -30,7 +50,7 @@ resource "azurerm_network_security_group" "nsg-training" {
3050
}
3151

3252
resource "azurerm_subnet_network_security_group_association" "nsg-training-link" {
33-
subnet_id = var.training_subnet_resource_id
53+
subnet_id = "${data.azurerm_subnet.training.id}"
3454
network_security_group_id = azurerm_network_security_group.nsg-training.id
3555
}
3656

@@ -43,7 +63,7 @@ resource "azurerm_network_security_group" "nsg-aks" {
4363
}
4464

4565
resource "azurerm_subnet_network_security_group_association" "nsg-aks-link" {
46-
subnet_id = var.aks_subnet_resource_id
66+
subnet_id = "${data.azurerm_subnet.aks.id}"
4767
network_security_group_id = azurerm_network_security_group.nsg-aks.id
4868
}
4969

@@ -81,7 +101,7 @@ resource "azurerm_route" "training-BatchRoute" {
81101
}
82102

83103
resource "azurerm_subnet_route_table_association" "rt-training-link" {
84-
subnet_id = var.training_subnet_resource_id
104+
subnet_id = "${data.azurerm_subnet.training.id}"
85105
route_table_id = azurerm_route_table.rt-training.id
86106
}
87107

@@ -101,6 +121,6 @@ resource "azurerm_route" "aks-Internet-Route" {
101121
}
102122

103123
resource "azurerm_subnet_route_table_association" "rt-aks-link" {
104-
subnet_id = var.aks_subnet_resource_id
124+
subnet_id = "${data.azurerm_subnet.aks.id}"
105125
route_table_id = azurerm_route_table.rt-aks.id
106126
}

quickstart/202-machine-learning-moderately-secure-existing-VNet/variables.tf

Lines changed: 18 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -21,20 +21,31 @@ variable "image_build_compute_name" {
2121
default = "image-builder"
2222
}
2323

24-
# Existing subnets variables
25-
variable "training_subnet_resource_id" {
24+
# Existing vnet and subnets variables
25+
26+
variable "vnet_resource_group_name" {
27+
type = string
28+
description = "Name of the resource group for the existing VNet"
29+
}
30+
31+
variable "vnet_name" {
32+
type = string
33+
description = "Name of the existing VNet"
34+
}
35+
36+
variable "training_subnet_name" {
2637
type = string
27-
description = "Resource ID of the existing training subnet"
38+
description = "Name of the existing training subnet"
2839
}
2940

30-
variable "aks_subnet_resource_id" {
41+
variable "aks_subnet_name" {
3142
type = string
32-
description = "Resource ID of the existing aks subnet"
43+
description = "Name of the existing aks subnet"
3344
}
3445

35-
variable "ml_subnet_resource_id" {
46+
variable "ml_subnet_name" {
3647
type = string
37-
description = "Resource ID of the existing ML workspace subnet"
48+
description = "Name of the existing ML workspace subnet"
3849
}
3950

4051

quickstart/202-machine-learning-moderately-secure-existing-VNet/workspace.tf

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ resource "azurerm_private_endpoint" "kv_ple" {
7070
name = "ple-${var.name}-${var.environment}-kv"
7171
location = azurerm_resource_group.default.location
7272
resource_group_name = azurerm_resource_group.default.name
73-
subnet_id = var.ml_subnet_resource_id
73+
subnet_id = "${data.azurerm_subnet.ml.id}"
7474

7575
private_dns_zone_group {
7676
name = "private-dns-zone-group"
@@ -89,7 +89,7 @@ resource "azurerm_private_endpoint" "st_ple_blob" {
8989
name = "ple-${var.name}-${var.environment}-st-blob"
9090
location = azurerm_resource_group.default.location
9191
resource_group_name = azurerm_resource_group.default.name
92-
subnet_id = var.ml_subnet_resource_id
92+
subnet_id = "${data.azurerm_subnet.ml.id}"
9393

9494
private_dns_zone_group {
9595
name = "private-dns-zone-group"
@@ -108,7 +108,7 @@ resource "azurerm_private_endpoint" "storage_ple_file" {
108108
name = "ple-${var.name}-${var.environment}-st-file"
109109
location = azurerm_resource_group.default.location
110110
resource_group_name = azurerm_resource_group.default.name
111-
subnet_id = var.ml_subnet_resource_id
111+
subnet_id = "${data.azurerm_subnet.ml.id}"
112112

113113
private_dns_zone_group {
114114
name = "private-dns-zone-group"
@@ -127,7 +127,7 @@ resource "azurerm_private_endpoint" "cr_ple" {
127127
name = "ple-${var.name}-${var.environment}-cr"
128128
location = azurerm_resource_group.default.location
129129
resource_group_name = azurerm_resource_group.default.name
130-
subnet_id = var.ml_subnet_resource_id
130+
subnet_id = "${data.azurerm_subnet.ml.id}"
131131

132132
private_dns_zone_group {
133133
name = "private-dns-zone-group"
@@ -146,7 +146,7 @@ resource "azurerm_private_endpoint" "mlw_ple" {
146146
name = "ple-${var.name}-${var.environment}-mlw"
147147
location = azurerm_resource_group.default.location
148148
resource_group_name = azurerm_resource_group.default.name
149-
subnet_id = var.ml_subnet_resource_id
149+
subnet_id = "${data.azurerm_subnet.ml.id}"
150150

151151
private_dns_zone_group {
152152
name = "private-dns-zone-group"
@@ -172,7 +172,7 @@ resource "azurerm_machine_learning_compute_cluster" "image-builder" {
172172
vm_priority = "LowPriority"
173173
vm_size = "Standard_DS2_v2"
174174
machine_learning_workspace_id = azurerm_machine_learning_workspace.default.id
175-
subnet_resource_id = var.training_subnet_resource_id
175+
subnet_resource_id = "${data.azurerm_subnet.training.id}"
176176

177177
scale_settings {
178178
min_node_count = 0

0 commit comments

Comments
 (0)