From beed341bf11793e28d27d7d9c00821dcf1a2e354 Mon Sep 17 00:00:00 2001
From: Leonard Francis <leonard@greenqloud.com>
Date: Fri, 22 Mar 2019 17:24:01 +0000
Subject: [PATCH] Small correction to list handling (#5446)

* NFSAAS-1505 minor update and validation fixes

* NFSAAS-1505 bring in line with RP

Put (and patch) for snapshots.
Service level definition changes.

* NFSAAS-1505 correct resource reference and limits

* NFSAAS-1505 resource and property changes

Incorrect pool resource reference
Name elements of property not required
Correction to return status code

* NFSAAS-1505 status codes

* NFSAAS-1505 tidy up examples

* NFSAAS-1505 tidy up examples

* NFSAAS-1644 remove mt get and add put status code

* NFSAAS-1644 remove mt get and add put status code

* NFSAAS-1875 update to R3.5 standard

* NFSAAS-1875 Update to R3.5 Standard (#11)

* Nfsaas 1875 swagger update (#12)

* NFSAAS-1875 Update to R3.5 Standard

* NFSAAS-1875 update to R3.5 standard

* Nfsaas 1875 swagger update (#13)

* NFSAAS-1875 Update to R3.5 Standard

* NFSAAS-1875 update to R3.5 standard

* NFSAAS-1875 updated from review comments

* NFSAAS-1875 small correction
---
 .../examples/Accounts_CreateOrUpdate.json     | 10 +++++-
 .../2017-08-15/examples/Accounts_Update.json  |  2 +-
 .../preview/2017-08-15/netapp.json            | 34 +++++++++++++++----
 3 files changed, 37 insertions(+), 9 deletions(-)

diff --git a/specification/netapp/resource-manager/Microsoft.NetApp/preview/2017-08-15/examples/Accounts_CreateOrUpdate.json b/specification/netapp/resource-manager/Microsoft.NetApp/preview/2017-08-15/examples/Accounts_CreateOrUpdate.json
index fdb4fb8ecb7a..3859118ea072 100644
--- a/specification/netapp/resource-manager/Microsoft.NetApp/preview/2017-08-15/examples/Accounts_CreateOrUpdate.json
+++ b/specification/netapp/resource-manager/Microsoft.NetApp/preview/2017-08-15/examples/Accounts_CreateOrUpdate.json
@@ -7,11 +7,19 @@
     "body": {}
   },
   "responses": {
+    "200": {
+      "body": {
+        "location": "eastus",
+        "properties": {
+          "provisioningState": "Succeeded"
+        }
+      }
+    },
     "201": {
       "body": {
         "location": "eastus",
         "properties": {
-          "provisioningState": "Created"
+          "provisioningState": "Succeeded"
         }
       }
     },
diff --git a/specification/netapp/resource-manager/Microsoft.NetApp/preview/2017-08-15/examples/Accounts_Update.json b/specification/netapp/resource-manager/Microsoft.NetApp/preview/2017-08-15/examples/Accounts_Update.json
index fef82b77979e..ed2a522b4df6 100644
--- a/specification/netapp/resource-manager/Microsoft.NetApp/preview/2017-08-15/examples/Accounts_Update.json
+++ b/specification/netapp/resource-manager/Microsoft.NetApp/preview/2017-08-15/examples/Accounts_Update.json
@@ -11,7 +11,7 @@
       "body": {
         "location": "eastus",
         "properties": {
-          "provisioningState": "Created"
+          "provisioningState": "Succeeded"
         }
       }
     }
diff --git a/specification/netapp/resource-manager/Microsoft.NetApp/preview/2017-08-15/netapp.json b/specification/netapp/resource-manager/Microsoft.NetApp/preview/2017-08-15/netapp.json
index cc92d9f50787..56849cb2517e 100644
--- a/specification/netapp/resource-manager/Microsoft.NetApp/preview/2017-08-15/netapp.json
+++ b/specification/netapp/resource-manager/Microsoft.NetApp/preview/2017-08-15/netapp.json
@@ -169,6 +169,12 @@
           }
         ],
         "responses": {
+          "200": {
+            "description": "OK - account updated",
+            "schema": {
+              "$ref": "#/definitions/netAppAccount"
+            }
+          },
           "201": {
             "description": "Account created",
             "schema": {
@@ -1161,12 +1167,14 @@
         "activeDirectories": {
           "description": "Active Directories",
           "type": "array",
-          "$ref": "#/definitions/activeDirectories"
+          "items": {
+            "$ref": "#/definitions/activeDirectory"
+          }
         }
       }
     },
-    "activeDirectories": {
-      "description": "Active Directories",
+    "activeDirectory": {
+      "description": "Active Directory",
       "type": "object",
       "properties": {
         "activeDirectoryId": {
@@ -1527,8 +1535,14 @@
         },
         "exportPolicy": {
           "description": "Export policy rule",
-          "type": "array",
-          "$ref": "#/definitions/exportPolicyRule"
+          "properties": {
+            "rules": {
+              "type": "array",
+              "items": {
+              "$ref": "#/definitions/exportPolicyRule"
+              }
+            }
+          }
         },
         "provisioningState": {
           "type": "string",
@@ -1656,8 +1670,14 @@
         },
         "exportPolicy": {
           "description": "Export policy rule",
-          "type": "array",
-          "$ref": "#/definitions/exportPolicyRule"
+          "properties": {
+            "rules": {
+              "type": "array",
+              "items": {
+              "$ref": "#/definitions/exportPolicyRule"
+              }
+            }
+          }
         }
       }
     },