Initial network error fixes and suppressions

Azure · May 10, 2018 · 202886c · 202886c
1 parent b3f5a7a
commit 202886c
Show file tree

Hide file tree

Showing 16 changed files with 79 additions and 60 deletions.
diff --git a/...on/network/resource-manager/Microsoft.Network/stable/2018-05-01/checkDnsAvailability.json b/...on/network/resource-manager/Microsoft.Network/stable/2018-05-01/checkDnsAvailability.json
@@ -10,12 +10,10 @@
     "https"
   ],
   "consumes": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "produces": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "security": [
     {

diff --git a/...ion/network/resource-manager/Microsoft.Network/stable/2018-05-01/expressRouteCircuit.json b/...ion/network/resource-manager/Microsoft.Network/stable/2018-05-01/expressRouteCircuit.json
@@ -10,12 +10,10 @@
     "https"
   ],
   "consumes": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "produces": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "security": [
     {

diff --git a/...ork/resource-manager/Microsoft.Network/stable/2018-05-01/expressRouteCrossConnection.json b/...ork/resource-manager/Microsoft.Network/stable/2018-05-01/expressRouteCrossConnection.json
@@ -10,12 +10,10 @@
     "https"
   ],
   "consumes": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "produces": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "security": [
     {

diff --git a/specification/network/resource-manager/Microsoft.Network/stable/2018-05-01/loadBalancer.json b/specification/network/resource-manager/Microsoft.Network/stable/2018-05-01/loadBalancer.json
@@ -10,12 +10,10 @@
     "https"
   ],
   "consumes": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "produces": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "security": [
     {

diff --git a/specification/network/resource-manager/Microsoft.Network/stable/2018-05-01/network.json b/specification/network/resource-manager/Microsoft.Network/stable/2018-05-01/network.json
@@ -10,12 +10,10 @@
     "https"
   ],
   "consumes": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "produces": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "security": [
     {

diff --git a/...cation/network/resource-manager/Microsoft.Network/stable/2018-05-01/networkInterface.json b/...cation/network/resource-manager/Microsoft.Network/stable/2018-05-01/networkInterface.json
@@ -10,12 +10,10 @@
     "https"
   ],
   "consumes": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "produces": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "security": [
     {

diff --git a/...on/network/resource-manager/Microsoft.Network/stable/2018-05-01/networkSecurityGroup.json b/...on/network/resource-manager/Microsoft.Network/stable/2018-05-01/networkSecurityGroup.json
@@ -10,12 +10,10 @@
     "https"
   ],
   "consumes": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "produces": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "security": [
     {

diff --git a/...fication/network/resource-manager/Microsoft.Network/stable/2018-05-01/networkWatcher.json b/...fication/network/resource-manager/Microsoft.Network/stable/2018-05-01/networkWatcher.json
@@ -10,12 +10,10 @@
     "https"
   ],
   "consumes": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "produces": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "security": [
     {

diff --git a/...ication/network/resource-manager/Microsoft.Network/stable/2018-05-01/publicIpAddress.json b/...ication/network/resource-manager/Microsoft.Network/stable/2018-05-01/publicIpAddress.json
@@ -10,12 +10,10 @@
     "https"
   ],
   "consumes": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "produces": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "security": [
     {

diff --git a/specification/network/resource-manager/Microsoft.Network/stable/2018-05-01/routeTable.json b/specification/network/resource-manager/Microsoft.Network/stable/2018-05-01/routeTable.json
@@ -10,12 +10,10 @@
     "https"
   ],
   "consumes": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "produces": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "security": [
     {

diff --git a/specification/network/resource-manager/Microsoft.Network/stable/2018-05-01/usage.json b/specification/network/resource-manager/Microsoft.Network/stable/2018-05-01/usage.json
@@ -10,12 +10,10 @@
     "https"
   ],
   "consumes": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "produces": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "security": [
     {

diff --git a/...fication/network/resource-manager/Microsoft.Network/stable/2018-05-01/virtualNetwork.json b/...fication/network/resource-manager/Microsoft.Network/stable/2018-05-01/virtualNetwork.json
@@ -10,12 +10,10 @@
     "https"
   ],
   "consumes": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "produces": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "security": [
     {

diff --git a/...n/network/resource-manager/Microsoft.Network/stable/2018-05-01/virtualNetworkGateway.json b/...n/network/resource-manager/Microsoft.Network/stable/2018-05-01/virtualNetworkGateway.json
@@ -10,12 +10,10 @@
     "https"
   ],
   "consumes": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "produces": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "security": [
     {

diff --git a/...on/network/resource-manager/Microsoft.Network/stable/2018-05-01/vmssNetworkInterface.json b/...on/network/resource-manager/Microsoft.Network/stable/2018-05-01/vmssNetworkInterface.json
@@ -10,12 +10,10 @@
     "https"
   ],
   "consumes": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "produces": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "security": [
     {
@@ -395,6 +393,13 @@
       "required": true,
       "type": "string",
       "description": "The subscription credentials which uniquely identify the Microsoft Azure subscription. The subscription ID forms part of the URI for every service call."
+    },
+    "ApiVersionParameter": {
+      "name": "api-version",
+      "in": "query",
+      "required": true,
+      "type": "string",
+      "description": "Client API version."
     }
   }
 }
diff --git a/...ion/network/resource-manager/Microsoft.Network/stable/2018-05-01/vmssPublicIpAddress.json b/...ion/network/resource-manager/Microsoft.Network/stable/2018-05-01/vmssPublicIpAddress.json
@@ -10,12 +10,10 @@
     "https"
   ],
   "consumes": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "produces": [
-    "application/json",
-    "text/json"
+    "application/json"
   ],
   "security": [
     {
@@ -256,6 +254,13 @@
       "required": true,
       "type": "string",
       "description": "The subscription credentials which uniquely identify the Microsoft Azure subscription. The subscription ID forms part of the URI for every service call."
+    },
+    "ApiVersionParameter": {
+      "name": "api-version",
+      "in": "query",
+      "required": true,
+      "type": "string",
+      "description": "Client API version."
     }
   }
 }
diff --git a/specification/network/resource-manager/readme.md b/specification/network/resource-manager/readme.md
@@ -410,6 +410,41 @@ input-file:
 - Microsoft.Network/preview/2015-05-01-preview/network.json
 ```
 
+## Suppression  
+``` yaml
+directive:
+  - suppress: RequiredPropertiesMissingInResourceModel
+    reason: name, id and type properties are inherited from the upper level
+  - suppress: TrackedResourceListByImmediateParent
+    reason: Another list APIs naming approach is used over the specs
+  - suppress: EnumInsteadOfBoolean
+    reason: Booleans are used by networking APIs
+  - suppress: GetInOperationName
+    where: $.paths["/subscriptions/{subscriptionId}/providers/Microsoft.Network/locations/{location}/CheckDnsNameAvailability"].get.operationId
+    reason: Customized verb is used for API
+  - suppress: GetInOperationName
+    where: $.paths["/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Network/virtualNetworks/{virtualNetworkName}/CheckIPAddressAvailability"].get.operationId
+    reason: Customized verb is used for API
+  - suppress: PutInOperationName
+    where: $.paths["/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Network/connections/{virtualNetworkGatewayConnectionName}/sharedkey"].put.operationId
+    reason: Customized verb is used for API
+  - suppress: PostOperationIdContainsUrlVerb
+    from: networkWatcher.json
+    reason: Customized verbs are used for API
+  - suppress: PostOperationIdContainsUrlVerb
+    from: expressRouteCircuit.json
+    reason: Customized verbs are used for API
+  - suppress: PostOperationIdContainsUrlVerb
+    from: expressRouteCrossConnection.json
+    reason: Customized verbs are used for API
+  - suppress: OperationIdNounVerb
+    from: vmssPublicIpAddress.json
+    reason: VMSS specs have custom naming
+  - suppress: OperationIdNounVerb
+    from: vmssNetworkInterface.json
+    reason: VMSS specs have custom naming
+```
+
 ---
 # Code Generation