From f7e347d66c158c90b8405f31d63162ec589bf453 Mon Sep 17 00:00:00 2001
From: Antoine Gosselin <angos@microsoft.com>
Date: Thu, 20 Feb 2020 11:16:55 -0800
Subject: [PATCH] Use tracked and proxy resource types

---
 .../2015-08-31-preview/ManagedIdentity.json   | 36 +++++++-------
 .../stable/2018-11-30/ManagedIdentity.json    | 47 ++++++++++++-------
 2 files changed, 48 insertions(+), 35 deletions(-)

diff --git a/specification/msi/resource-manager/Microsoft.ManagedIdentity/preview/2015-08-31-preview/ManagedIdentity.json b/specification/msi/resource-manager/Microsoft.ManagedIdentity/preview/2015-08-31-preview/ManagedIdentity.json
index b51e07e16e42..0c5cbed37da1 100644
--- a/specification/msi/resource-manager/Microsoft.ManagedIdentity/preview/2015-08-31-preview/ManagedIdentity.json
+++ b/specification/msi/resource-manager/Microsoft.ManagedIdentity/preview/2015-08-31-preview/ManagedIdentity.json
@@ -353,17 +353,6 @@
   "definitions": {
     "Identity": {
       "properties": {
-        "location": {
-          "type": "string",
-          "description": "The Azure region where the resource lives."
-        },
-        "tags": {
-          "type": "object",
-          "additionalProperties": {
-            "type": "string"
-          },
-          "description": "Resource tags"
-        },
         "properties": {
           "x-ms-client-flatten": true,
           "readOnly": true,
@@ -371,12 +360,9 @@
           "$ref": "#/definitions/IdentityProperties"
         }
       },
-      "required": [
-        "location"
-      ],
       "allOf": [
         {
-          "$ref": "../../../../../common-types/resource-management/v1/types.json#/definitions/Resource"
+          "$ref": "../../../../../common-types/resource-management/v1/types.json#/definitions/TrackedResource"
         }
       ],
       "description": "Describes an identity resource."
@@ -385,7 +371,23 @@
       "properties": {
         "location": {
           "type": "string",
-          "description": "The Azure region where the resource lives."
+          "x-ms-mutability": [
+            "read",
+            "create"
+          ],
+          "description": "The geo-location where the resource lives"
+        },
+        "tags": {
+          "type": "object",
+          "additionalProperties": {
+            "type": "string"
+          },
+          "x-ms-mutability": [
+            "read",
+            "create",
+            "update"
+          ],
+          "description": "Resource tags"
         },
         "properties": {
           "x-ms-client-flatten": true,
@@ -399,7 +401,7 @@
       ],
       "allOf": [
         {
-          "$ref": "../../../../../common-types/resource-management/v1/types.json#/definitions/Resource"
+          "$ref": "../../../../../common-types/resource-management/v1/types.json#/definitions/ProxyResource"
         }
       ],
       "description": "Describes a system assigned identity resource."
diff --git a/specification/msi/resource-manager/Microsoft.ManagedIdentity/stable/2018-11-30/ManagedIdentity.json b/specification/msi/resource-manager/Microsoft.ManagedIdentity/stable/2018-11-30/ManagedIdentity.json
index d63b972a49fd..16695c387b30 100644
--- a/specification/msi/resource-manager/Microsoft.ManagedIdentity/stable/2018-11-30/ManagedIdentity.json
+++ b/specification/msi/resource-manager/Microsoft.ManagedIdentity/stable/2018-11-30/ManagedIdentity.json
@@ -353,17 +353,6 @@
   "definitions": {
     "Identity": {
       "properties": {
-        "location": {
-          "type": "string",
-          "description": "The Azure region where the resource lives."
-        },
-        "tags": {
-          "type": "object",
-          "additionalProperties": {
-            "type": "string"
-          },
-          "description": "Resource tags"
-        },
         "properties": {
           "x-ms-client-flatten": true,
           "readOnly": true,
@@ -371,12 +360,9 @@
           "$ref": "#/definitions/UserAssignedIdentityProperties"
         }
       },
-      "required": [
-        "location"
-      ],
       "allOf": [
         {
-          "$ref": "../../../../../common-types/resource-management/v1/types.json#/definitions/Resource"
+          "$ref": "../../../../../common-types/resource-management/v1/types.json#/definitions/TrackedResource"
         }
       ],
       "description": "Describes an identity resource."
@@ -385,13 +371,22 @@
       "properties": {
         "location": {
           "type": "string",
-          "description": "The Azure region where the resource lives."
+          "x-ms-mutability": [
+            "read",
+            "create"
+          ],
+          "description": "The geo-location where the resource lives"
         },
         "tags": {
           "type": "object",
           "additionalProperties": {
             "type": "string"
           },
+          "x-ms-mutability": [
+            "read",
+            "create",
+            "update"
+          ],
           "description": "Resource tags"
         },
         "properties": {
@@ -412,7 +407,23 @@
       "properties": {
         "location": {
           "type": "string",
-          "description": "The Azure region where the resource lives."
+          "x-ms-mutability": [
+            "read",
+            "create"
+          ],
+          "description": "The geo-location where the resource lives"
+        },
+        "tags": {
+          "type": "object",
+          "additionalProperties": {
+            "type": "string"
+          },
+          "x-ms-mutability": [
+            "read",
+            "create",
+            "update"
+          ],
+          "description": "Resource tags"
         },
         "properties": {
           "x-ms-client-flatten": true,
@@ -426,7 +437,7 @@
       ],
       "allOf": [
         {
-          "$ref": "../../../../../common-types/resource-management/v1/types.json#/definitions/Resource"
+          "$ref": "../../../../../common-types/resource-management/v1/types.json#/definitions/ProxyResource"
         }
       ],
       "description": "Describes a system assigned identity resource."