[Bug fix] Remove read-only constraint for management policy, and change properties types from integer to number (Azure#6551)

zfchen95 · Juliehzl · commit df89eafa51c7 · 2019-07-17T16:12:47.000+08:00
* [Bug fix] Remove read-only constraint for management policy

* [Bug fix] Change daysAfterCreationGreaterThan and daysAfterModificationGreaterThan to float type

* change type to number

* Modify description, remove integer

* Fix existing build errors in Storage swagger
diff --git a/specification/storage/resource-manager/Microsoft.Storage/stable/2019-04-01/examples/BlobContainersDelete.json b/specification/storage/resource-manager/Microsoft.Storage/stable/2019-04-01/examples/BlobContainersDelete.json
@@ -8,11 +8,7 @@
     "monitor": "true"
   },
   "responses": {
-    "200": {
-      "body": ""
-    },
-    "204": {
-      "body": ""
-    }
+    "200": {},
+    "204": {}
   }
 }
diff --git a/specification/storage/resource-manager/Microsoft.Storage/stable/2019-04-01/examples/StorageAccountCreate.json b/specification/storage/resource-manager/Microsoft.Storage/stable/2019-04-01/examples/StorageAccountCreate.json
@@ -55,8 +55,6 @@
         "type": "Microsoft.Storage/storageAccounts"
       }
     },
-    "202": {
-      "body": ""
-    }
+    "202": {}
   }
 }
diff --git a/specification/storage/resource-manager/Microsoft.Storage/stable/2019-04-01/examples/StorageAccountDelete.json b/specification/storage/resource-manager/Microsoft.Storage/stable/2019-04-01/examples/StorageAccountDelete.json
@@ -7,11 +7,7 @@
     "monitor": "true"
   },
   "responses": {
-    "200": {
-      "body": ""
-    },
-    "204": {
-      "body": ""
-    }
+    "200": {},
+    "204": {}
   }
 }
diff --git a/specification/storage/resource-manager/Microsoft.Storage/stable/2019-04-01/examples/StorageAccountDeleteManagementPolicy.json b/specification/storage/resource-manager/Microsoft.Storage/stable/2019-04-01/examples/StorageAccountDeleteManagementPolicy.json
@@ -8,11 +8,7 @@
     "monitor": "true"
   },
   "responses": {
-    "200": {
-      "body": ""
-    },
-    "204": {
-      "body": ""
-    }
+    "200": {},
+    "204": {}
   }
 }
diff --git a/specification/storage/resource-manager/Microsoft.Storage/stable/2019-04-01/examples/StorageAccountFailover.json b/specification/storage/resource-manager/Microsoft.Storage/stable/2019-04-01/examples/StorageAccountFailover.json
@@ -7,11 +7,7 @@
     "monitor": "true"
   },
   "responses": {
-    "200": {
-      "body": ""
-    },
-    "202": {
-      "body": ""
-    }
+    "200": {},
+    "202": {}
   }
 }
diff --git a/specification/storage/resource-manager/Microsoft.Storage/stable/2019-04-01/storage.json b/specification/storage/resource-manager/Microsoft.Storage/stable/2019-04-01/storage.json
@@ -2160,7 +2160,6 @@
         "properties": {
           "$ref": "#/definitions/ManagementPolicyProperties",
           "x-ms-client-flatten": true,
-          "readOnly": true,
           "description": "Returns the Storage Account Data Policies Rules."
         }
       },
@@ -2317,9 +2316,10 @@
     "DateAfterModification": {
       "properties": {
         "daysAfterModificationGreaterThan": {
-          "type": "integer",
+          "type": "number",
+          "multipleOf": 1.0,
           "minimum": 0,
-          "description": "Integer value indicating the age in days after last modification"
+          "description": "Value indicating the age in days after last modification"
         }
       },
       "required": [
@@ -2330,9 +2330,10 @@
     "DateAfterCreation": {
       "properties": {
         "daysAfterCreationGreaterThan": {
-          "type": "integer",
+          "type": "number",
+          "multipleOf": 1.0,
           "minimum": 0,
-          "description": "Integer value indicating the age in days after creation"
+          "description": "Value indicating the age in days after creation"
         }
       },
       "required": [

Original file line number	Diff line number	Diff line change
`@@ -8,11 +8,7 @@`
`8`	`8`	`"monitor": "true"`
`9`	`9`	`},`
`10`	`10`	`"responses": {`
`11`		`- "200": {`
`12`		`- "body": ""`
`13`		`- },`
`14`		`- "204": {`
`15`		`- "body": ""`
`16`		`- }`
	`11`	`+ "200": {},`
	`12`	`+ "204": {}`
`17`	`13`	`}`
`18`	`14`	`}`
Original file line number	Diff line number	Diff line change
`@@ -55,8 +55,6 @@`
`55`	`55`	`"type": "Microsoft.Storage/storageAccounts"`
`56`	`56`	`}`
`57`	`57`	`},`
`58`		`- "202": {`
`59`		`- "body": ""`
`60`		`- }`
	`58`	`+ "202": {}`
`61`	`59`	`}`
`62`	`60`	`}`
Original file line number	Diff line number	Diff line change
`@@ -7,11 +7,7 @@`
`7`	`7`	`"monitor": "true"`
`8`	`8`	`},`
`9`	`9`	`"responses": {`
`10`		`- "200": {`
`11`		`- "body": ""`
`12`		`- },`
`13`		`- "204": {`
`14`		`- "body": ""`
`15`		`- }`
	`10`	`+ "200": {},`
	`11`	`+ "204": {}`
`16`	`12`	`}`
`17`	`13`	`}`