Skip to content

Commit

Permalink
Rename Firewall Policy child resource RuleGroup to RuleCollectionGroup (
Browse files Browse the repository at this point in the history
#9045)

* Rename firewall policy sub resources

s/RuleGroup/RuleCollectionGroup
s/Rule/RuleCollection
s/RuleCondition/Rule

This was based on customer feedback - the feature is currently in preview.

* rename files

* Update examples with new names for the subresource and children

* Fix model validation error

* Update examples
  • Loading branch information
gimotwanMSFT authored Apr 22, 2020
1 parent 35b06e5 commit b49caab
Show file tree
Hide file tree
Showing 12 changed files with 229 additions and 229 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,9 @@
"properties": {
"provisioningState": "Succeeded",
"threatIntelMode": "Alert",
"ruleGroups": [
"ruleCollectionGroups": [
{
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleGroups/ruleGroup1"
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleCollectionGroups/ruleCollectionGroup1"
}
],
"firewalls": []
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,9 @@
"properties": {
"provisioningState": "Succeeded",
"threatIntelMode": "Alert",
"ruleGroups": [
"ruleCollectionGroups": [
{
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleGroups/ruleGroup1"
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleCollectionGroups/ruleCollectionGroup1"
}
],
"firewalls": []
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,9 @@
"properties": {
"provisioningState": "Succeeded",
"threatIntelMode": "Alert",
"ruleGroups": [
"ruleCollectionGroups": [
{
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleGroups/ruleGroup1"
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleCollectionGroups/ruleCollectionGroup1"
}
],
"firewalls": []
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,12 +28,12 @@
"properties": {
"provisioningState": "Succeeded",
"threatIntelMode": "Alert",
"ruleGroups": [
"ruleCollectionGroups": [
{
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleGroups/rulegroup1"
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleCollectionGroups/ruleCollectionGroup1"
},
{
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleGroups/rulegroup2"
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleCollectionGroups/ruleCollectionGroup2"
}
],
"firewalls": []
Expand All @@ -53,12 +53,12 @@
"properties": {
"provisioningState": "Succeeded",
"threatIntelMode": "Alert",
"ruleGroups": [
"ruleCollectionGroups": [
{
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleGroups/ruleGroup1"
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleCollectionGroups/ruleCollectionGroup1"
},
{
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleGroups/rulegroup2"
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleCollectionGroups/ruleCollectionGroup2"
}
],
"firewalls": []
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"parameters": {
"ruleGroupName": "ruleGroup1",
"ruleCollectionGroupName": "ruleCollectionGroup1",
"firewallPolicyName": "firewallPolicy",
"resourceGroupName": "rg1",
"api-version": "2020-04-01",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"parameters": {
"ruleGroupName": "ruleGroup1",
"ruleCollectionGroupName": "ruleCollectionGroup1",
"firewallPolicyName": "firewallPolicy",
"resourceGroupName": "rg1",
"api-version": "2020-04-01",
Expand All @@ -9,24 +9,24 @@
"responses": {
"200": {
"body": {
"name": "ruleGroup1",
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleGroups/ruleGroup1",
"name": "ruleCollectionGroup1",
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleCollectionGroups/ruleCollectionGroup1",
"etag": "W/\"72090554-7e3b-43f2-80ad-99a9020dcb11\"",
"properties": {
"provisioningState": "Succeeded",
"priority": 110,
"rules": [
"ruleCollections": [
{
"ruleType": "FirewallPolicyFilterRule",
"name": "Example-Filter-Rule",
"ruleCollectionType": "FirewallPolicyFilterRuleCollection",
"name": "Example-Filter-Rule-Collection",
"priority": 200,
"action": {
"type": "Deny"
},
"ruleConditions": [
"rules": [
{
"ruleConditionType": "NetworkRuleCondition",
"name": "network-condition1",
"ruleType": "NetworkRule",
"name": "network-rule1",
"sourceAddresses": [
"10.1.25.0/24"
],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,25 +10,25 @@
"body": {
"value": [
{
"name": "ruleGroup1",
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleGroups/ruleGroup1",
"name": "ruleCollectionGroup1",
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleCollectionGroups/ruleCollectionGroup1",
"etag": "W/\"72090554-7e3b-43f2-80ad-99a9020dcb11\"",
"properties": {
"provisioningState": "Succeeded",
"priority": 110,
"rules": [
"ruleCollections": [
{
"name": "Example-Filter-Rule",
"ruleType": "FirewallPolicyFilterRule",
"name": "Example-Filter-Rule-Collection",
"ruleCollectionType": "FirewallPolicyFilterRuleCollection",
"priority": 120,
"action": {
"type": "Deny"
},
"ruleConditions": [
"rules": [
{
"name": "network-condition-1",
"ruleConditionType": "NetworkRuleCondition",
"description": "Network rule condition",
"name": "network-rule-1",
"ruleType": "NetworkRule",
"description": "Network rule",
"destinationAddresses": [
"*"
],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,21 +4,21 @@
"subscriptionId": "subid",
"resourceGroupName": "rg1",
"firewallPolicyName": "firewallPolicy",
"ruleGroupName": "ruleGroup1",
"ruleCollectionGroupName": "ruleCollectionGroup1",
"parameters": {
"properties": {
"priority": 110,
"rules": [
"ruleCollections": [
{
"ruleType": "FirewallPolicyFilterRule",
"name": "Example-Filter-Rule",
"ruleCollectionType": "FirewallPolicyFilterRuleCollection",
"name": "Example-Filter-Rule-Collection",
"action": {
"type": "Deny"
},
"ruleConditions": [
"rules": [
{
"ruleConditionType": "NetworkRuleCondition",
"name": "network-condition1",
"ruleType": "NetworkRule",
"name": "network-rule1",
"sourceAddresses": [
"10.1.25.0/24"
],
Expand All @@ -41,23 +41,23 @@
"responses": {
"200": {
"body": {
"name": "ruleGroup1",
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleGroups/ruleGroup1",
"name": "ruleCollectionGroup1",
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleCollectionGroups/ruleCollectionGroup1",
"etag": "w/\\00000000-0000-0000-0000-000000000000\\",
"properties": {
"provisioningState": "Succeeded",
"priority": 110,
"rules": [
"ruleCollections": [
{
"ruleType": "FirewallPolicyFilterRule",
"name": "Example-Filter-Rule",
"ruleCollectionType": "FirewallPolicyFilterRuleCollection",
"name": "Example-Filter-Rule-Collection",
"action": {
"type": "Deny"
},
"ruleConditions": [
"rules": [
{
"ruleConditionType": "NetworkRuleCondition",
"name": "network-condition1",
"ruleType": "NetworkRule",
"name": "network-rule1",
"sourceAddresses": [
"10.1.25.0/24"
],
Expand Down Expand Up @@ -85,17 +85,17 @@
"properties": {
"provisioningState": "Succeeded",
"priority": 110,
"rules": [
"ruleCollections": [
{
"ruleType": "FirewallPolicyFilterRule",
"name": "Example-Filter-Rule",
"ruleCollectionType": "FirewallPolicyFilterRuleCollection",
"name": "Example-Filter-Rule-Collection",
"action": {
"type": "Deny"
},
"ruleConditions": [
"rules": [
{
"ruleConditionType": "NetworkRuleCondition",
"name": "network-condition1",
"ruleType": "NetworkRule",
"name": "network-rule1",
"sourceAddresses": [
"10.1.25.0/24"
],
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"parameters": {
"ruleGroupName": "ruleGroup1",
"ruleCollectionGroupName": "ruleGroup1",
"firewallPolicyName": "firewallPolicy",
"resourceGroupName": "rg1",
"api-version": "2020-04-01",
Expand All @@ -15,18 +15,18 @@
"properties": {
"provisioningState": "Succeeded",
"priority": 110,
"rules": [
"ruleCollections": [
{
"ruleType": "FirewallPolicyFilterRule",
"name": "Example-Filter-Rule",
"ruleCollectionType": "FirewallPolicyFilterRuleCollection",
"name": "Example-Filter-Rule-Collection",
"priority": 200,
"action": {
"type": "Deny"
},
"ruleConditions": [
"rules": [
{
"ruleConditionType": "NetworkRuleCondition",
"name": "network-condition1",
"ruleType": "NetworkRule",
"name": "network-rule1",
"ipProtocols": [
"TCP"
],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,25 +10,25 @@
"body": {
"value": [
{
"name": "ruleGroup1",
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleGroups/ruleGroup1",
"name": "ruleCollectionGroup1",
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleCollectionGroups/ruleCollectionGroup1",
"etag": "W/\"72090554-7e3b-43f2-80ad-99a9020dcb11\"",
"properties": {
"provisioningState": "Succeeded",
"priority": 110,
"rules": [
"ruleCollections": [
{
"name": "Example-Filter-Rule",
"ruleType": "FirewallPolicyFilterRule",
"name": "Example-Filter-Rule-Collection",
"ruleCollectionType": "FirewallPolicyFilterRuleCollection",
"priority": 120,
"action": {
"type": "Deny"
},
"ruleConditions": [
"rules": [
{
"name": "network-condition-1",
"ruleConditionType": "NetworkRuleCondition",
"description": "Network rule condition",
"name": "network-rule-1",
"ruleType": "NetworkRule",
"description": "Network rule",
"sourceIpGroups": [
"/subscriptions/subid/providers/Microsoft.Network/resourceGroup/rg1/ipGroups/ipGroups1"
],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,21 +4,21 @@
"subscriptionId": "subid",
"resourceGroupName": "rg1",
"firewallPolicyName": "firewallPolicy",
"ruleGroupName": "ruleGroup1",
"ruleCollectionGroupName": "ruleCollectionGroup1",
"parameters": {
"properties": {
"priority": 110,
"rules": [
"ruleCollections": [
{
"ruleType": "FirewallPolicyFilterRule",
"name": "Example-Filter-Rule",
"ruleCollectionType": "FirewallPolicyFilterRuleCollection",
"name": "Example-Filter-Rule-Collection",
"action": {
"type": "Deny"
},
"ruleConditions": [
"rules": [
{
"ruleConditionType": "NetworkRuleCondition",
"name": "network-condition1",
"ruleType": "NetworkRule",
"name": "network-1",
"ipProtocols": [
"TCP"
],
Expand All @@ -41,23 +41,23 @@
"responses": {
"200": {
"body": {
"name": "ruleGroup1",
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleGroups/ruleGroup1",
"name": "ruleCollectionGroup1",
"id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.Network/firewallPolicies/firewallPolicy/ruleCollectionGroups/ruleCollectionGroup1",
"etag": "w/\\00000000-0000-0000-0000-000000000000\\",
"properties": {
"provisioningState": "Succeeded",
"priority": 110,
"rules": [
"ruleCollections": [
{
"ruleType": "FirewallPolicyFilterRule",
"name": "Example-Filter-Rule",
"ruleCollectionType": "FirewallPolicyFilterRuleCollection",
"name": "Example-Filter-Rule-Collection",
"action": {
"type": "Deny"
},
"ruleConditions": [
"rules": [
{
"ruleConditionType": "NetworkRuleCondition",
"name": "network-condition1",
"ruleType": "NetworkRule",
"name": "network-1",
"ipProtocols": [
"TCP"
],
Expand Down Expand Up @@ -85,17 +85,17 @@
"properties": {
"provisioningState": "Succeeded",
"priority": 110,
"rules": [
"ruleCollections": [
{
"ruleType": "FirewallPolicyFilterRule",
"name": "Example-Filter-Rule",
"ruleCollectionType": "FirewallPolicyFilterRuleCollection",
"name": "Example-Filter-Rule-Collection",
"action": {
"type": "Deny"
},
"ruleConditions": [
"rules": [
{
"ruleConditionType": "NetworkRuleCondition",
"name": "network-condition1",
"ruleType": "NetworkRule",
"name": "network-1",
"ipProtocols": [
"TCP"
],
Expand Down
Loading

0 comments on commit b49caab

Please sign in to comment.