Skip to content

Commit 7fa240d

Browse files
committed
update nested_node_nics to force 10.0.0.0 for nic0 on each vm
1 parent d365c10 commit 7fa240d

5 files changed

Lines changed: 12 additions & 9 deletions

File tree

lisa/sut_orchestrator/azure/arm_template.bicep

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -269,11 +269,11 @@ resource virtual_network_name_resource 'Microsoft.Network/virtualNetworks@2024-0
269269
)
270270
}
271271
subnets: [for j in range(0, subnet_count): {
272-
name: '10.${resource_group_index}.${j}.0/24'
272+
name: j == 0 ? '10.0.0.0' : '10.${resource_group_index}.${j}.0'
273273
properties: {
274274
addressPrefixes: concat(
275-
['10.${resource_group_index}.${j}.0/24'],
276-
use_ipv6 ? ['2001:db8:${resource_group_index}:${j}::/64'] : []
275+
['10.${j ==0 ? '0' : resource_group_index}.${j}.0/24'],
276+
use_ipv6 ? ['2001:db8:${j == 0 ? j : resource_group_index}:${j}::/64'] : []
277277
)
278278
defaultOutboundAccess: enable_vm_nat
279279
networkSecurityGroup: {

lisa/sut_orchestrator/azure/autogen_arm_template.json

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
"_generator": {
77
"name": "bicep",
88
"version": "0.40.2.10011",
9-
"templateHash": "16491672364169834990"
9+
"templateHash": "6973919806372738793"
1010
}
1111
},
1212
"functions": [
@@ -596,9 +596,9 @@
596596
"name": "subnets",
597597
"count": "[length(range(0, parameters('subnet_count')))]",
598598
"input": {
599-
"name": "[format('10.{0}.{1}.0/24', parameters('resource_group_index'), range(0, parameters('subnet_count'))[copyIndex('subnets')])]",
599+
"name": "[if(equals(range(0, parameters('subnet_count'))[copyIndex('subnets')], 0), '10.0.0.0', format('10.{0}.{1}.0', parameters('resource_group_index'), range(0, parameters('subnet_count'))[copyIndex('subnets')]))]",
600600
"properties": {
601-
"addressPrefixes": "[concat(createArray(format('10.{0}.{1}.0/24', parameters('resource_group_index'), range(0, parameters('subnet_count'))[copyIndex('subnets')])), if(parameters('use_ipv6'), createArray(format('2001:db8:{0}:{1}::/64', parameters('resource_group_index'), range(0, parameters('subnet_count'))[copyIndex('subnets')])), createArray()))]",
601+
"addressPrefixes": "[concat(createArray(format('10.{0}.{1}.0/24', if(equals(range(0, parameters('subnet_count'))[copyIndex('subnets')], 0), '0', parameters('resource_group_index')), range(0, parameters('subnet_count'))[copyIndex('subnets')])), if(parameters('use_ipv6'), createArray(format('2001:db8:{0}:{1}::/64', if(equals(range(0, parameters('subnet_count'))[copyIndex('subnets')], 0), range(0, parameters('subnet_count'))[copyIndex('subnets')], parameters('resource_group_index')), range(0, parameters('subnet_count'))[copyIndex('subnets')])), createArray()))]",
602602
"defaultOutboundAccess": "[parameters('enable_vm_nat')]",
603603
"networkSecurityGroup": {
604604
"id": "[resourceId('Microsoft.Network/networkSecurityGroups', format('{0}-nsg', toLower(parameters('virtual_network_name'))))]"
@@ -933,7 +933,7 @@
933933
"_generator": {
934934
"name": "bicep",
935935
"version": "0.40.2.10011",
936-
"templateHash": "1909517540330820286"
936+
"templateHash": "18100026266609217689"
937937
}
938938
},
939939
"functions": [
@@ -971,7 +971,7 @@
971971
],
972972
"output": {
973973
"type": "string",
974-
"value": "[format('10.{0}.{1}.0', parameters('resource_group_index'), parameters('nic_index'))]"
974+
"value": "[if(equals(parameters('nic_index'), 0), '10.0.0.0', format('10.{0}.{1}.0', parameters('resource_group_index'), parameters('nic_index')))]"
975975
}
976976
}
977977
}

lisa/sut_orchestrator/azure/common.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1262,6 +1262,7 @@ class AzureArmParameter:
12621262
enable_vm_nat: bool = False
12631263
create_public_address: bool = True
12641264
source_address_prefixes: List[str] = field(default_factory=list)
1265+
resource_group_index: int = 0
12651266

12661267
def __post_init__(self, *args: Any, **kwargs: Any) -> None:
12671268
add_secret(self.admin_username, PATTERN_HEADTAIL)

lisa/sut_orchestrator/azure/nested_nodes_nics.bicep

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ func getPublicIpAddress(vmName string, publicIpName string) object => {
1515
var publicIpAddress = getPublicIpAddress(vmName, '${vmName}-public-ip')
1616
var publicIpAddressV6 = getPublicIpAddress(vmName, '${vmName}-public-ipv6')
1717

18-
func getSubnetName(resource_group_index int, nic_index int) string => '10.${resource_group_index}.${nic_index}.0'
18+
func getSubnetName(resource_group_index int, nic_index int) string => nic_index == 0 ? '10.0.0.0' : '10.${resource_group_index}.${nic_index}.0'
1919

2020
resource vm_nics 'Microsoft.Network/networkInterfaces@2023-06-01' = [for i in range(0, nic_count): {
2121
name: '${vmName}-nic-${i}'

lisa/sut_orchestrator/azure/platform_.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -605,6 +605,7 @@ def _deploy_environment(self, environment: Environment, log: Logger) -> None:
605605
assert self._azure_runbook
606606

607607
environment_context = get_environment_context(environment=environment)
608+
608609
if self._azure_runbook.resource_group_name:
609610
resource_group_name = self._azure_runbook.resource_group_name
610611
else:
@@ -1217,6 +1218,7 @@ def _create_deployment_parameters(
12171218
self._azure_runbook.virtual_network_name or AZURE_VIRTUAL_NETWORK_NAME
12181219
)
12191220
arm_parameters.use_ipv6 = self._azure_runbook.use_ipv6
1221+
arm_parameters.resource_group_index = int(environment.id)
12201222

12211223
is_windows: bool = False
12221224
arm_parameters.admin_username = self.runbook.admin_username

0 commit comments

Comments
 (0)