Skip to content

Commit 5d6cb34

Browse files
NetworkSecurityGroup as an optional parameter in NetworkInterface.
1 parent 7c247af commit 5d6cb34

File tree

2 files changed

+8
-6
lines changed

2 files changed

+8
-6
lines changed

src/ResourceManager/Common/Commands.Common.Strategies/Network/NetworkInterfaceStrategy.cs

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ public static ResourceConfig<NetworkInterface> CreateNetworkInterfaceConfig(
3535
string name,
3636
NestedResourceConfig<Subnet, VirtualNetwork> subnet,
3737
ResourceConfig<PublicIPAddress> publicIPAddress,
38-
ResourceConfig<NetworkSecurityGroup> networkSecurityGroup)
38+
ResourceConfig<NetworkSecurityGroup> networkSecurityGroup = null)
3939
=> Strategy.CreateConfig(
4040
resourceGroup,
4141
name,
@@ -53,10 +53,12 @@ public static ResourceConfig<NetworkInterface> CreateNetworkInterfaceConfig(
5353
}
5454
}
5555
},
56-
NetworkSecurityGroup = new NetworkSecurityGroup
57-
{
58-
Id = networkSecurityGroup.GetId(subscription).IdToString()
59-
}
56+
NetworkSecurityGroup = networkSecurityGroup == null
57+
? null
58+
: new NetworkSecurityGroup
59+
{
60+
Id = networkSecurityGroup.GetId(subscription).IdToString()
61+
}
6062
},
6163
new IEntityConfig[] { subnet, publicIPAddress, networkSecurityGroup });
6264
}

src/ResourceManager/Common/Commands.Common.Strategies/ResourceConfigExtensions.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ public static ResourceConfig<TModel> CreateConfig<TModel>(
5050
resourceGroup.Name,
5151
name,
5252
createModel,
53-
dependencies.EmptyIfNull().Concat(new[] { resourceGroup }));
53+
dependencies.EmptyIfNull().Where(d => d != null).Concat(new[] { resourceGroup }));
5454

5555
public static async Task<TModel> GetAsync<TModel>(
5656
this ResourceConfig<TModel> config,

0 commit comments

Comments
 (0)