From b0c935f1ac91edcaf4fbaa26675cc12dcc339ff1 Mon Sep 17 00:00:00 2001
From: Jeff Hackshaw <jhackshaw@microsoft.com>
Date: Thu, 22 Apr 2021 09:56:44 -0700
Subject: [PATCH 1/2] document tenantId query parameter for cross-tenant role
 assignment get and delete calls

---
 .../authorization-RoleAssignmentsCalls.json   | 32 +++++++++++++++++++
 .../authorization-RoleAssignmentsCalls.json   | 32 +++++++++++++++++++
 .../authorization-RoleAssignmentsCalls.json   | 32 +++++++++++++++++++
 3 files changed, 96 insertions(+)

diff --git a/specification/authorization/resource-manager/Microsoft.Authorization/preview/2020-04-01-preview/authorization-RoleAssignmentsCalls.json b/specification/authorization/resource-manager/Microsoft.Authorization/preview/2020-04-01-preview/authorization-RoleAssignmentsCalls.json
index f887263b7292..b03a9c48a781 100644
--- a/specification/authorization/resource-manager/Microsoft.Authorization/preview/2020-04-01-preview/authorization-RoleAssignmentsCalls.json
+++ b/specification/authorization/resource-manager/Microsoft.Authorization/preview/2020-04-01-preview/authorization-RoleAssignmentsCalls.json
@@ -48,6 +48,9 @@
           {
             "$ref": "#/parameters/ResourceProviderNamespaceParameter"
           },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
+          },
           {
             "name": "parentResourcePath",
             "in": "path",
@@ -118,6 +121,9 @@
         "operationId": "RoleAssignments_ListForResourceGroup",
         "description": "List role assignments for a resource group.",
         "parameters": [
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
+          },
           {
             "$ref": "../../../../../common-types/resource-management/v2/types.json#/parameters/ResourceGroupNameParameter"
           },
@@ -168,6 +174,9 @@
         "operationId": "RoleAssignments_Delete",
         "description": "Delete a role assignment.",
         "parameters": [
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
+          },
           {
             "name": "scope",
             "in": "path",
@@ -272,6 +281,9 @@
         "operationId": "RoleAssignments_Get",
         "description": "Get the specified role assignment.",
         "parameters": [
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
+          },
           {
             "name": "scope",
             "in": "path",
@@ -320,6 +332,9 @@
         "operationId": "RoleAssignments_DeleteById",
         "description": "Delete a role assignment.",
         "parameters": [
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
+          },
           {
             "name": "roleId",
             "in": "path",
@@ -410,6 +425,9 @@
         "operationId": "RoleAssignments_GetById",
         "description": "Gets a role assignment by ID.",
         "parameters": [
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
+          },
           {
             "name": "roleId",
             "in": "path",
@@ -451,6 +469,9 @@
         "operationId": "RoleAssignments_List",
         "description": "Gets all role assignments for the subscription.",
         "parameters": [
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
+          },
           {
             "name": "$filter",
             "in": "query",
@@ -498,6 +519,9 @@
         "operationId": "RoleAssignments_ListForScope",
         "description": "Gets role assignments for a scope.",
         "parameters": [
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
+          },
           {
             "name": "scope",
             "in": "path",
@@ -753,6 +777,14 @@
       "type": "string",
       "description": "The namespace of the resource provider.",
       "x-ms-parameter-location": "method"
+    },
+    "TenantIdQueryParameter": {
+      "name": "tenantId",
+      "in": "query",
+      "required": false,
+      "type": "string",
+      "description": "Tenant ID for cross-tenant request",
+      "x-ms-parameter-location": "method"
     }
   }
 }
diff --git a/specification/authorization/resource-manager/Microsoft.Authorization/preview/2020-08-01-preview/authorization-RoleAssignmentsCalls.json b/specification/authorization/resource-manager/Microsoft.Authorization/preview/2020-08-01-preview/authorization-RoleAssignmentsCalls.json
index b3782ecaf2e1..3dba0536d271 100644
--- a/specification/authorization/resource-manager/Microsoft.Authorization/preview/2020-08-01-preview/authorization-RoleAssignmentsCalls.json
+++ b/specification/authorization/resource-manager/Microsoft.Authorization/preview/2020-08-01-preview/authorization-RoleAssignmentsCalls.json
@@ -50,6 +50,9 @@
           },
           {
             "$ref": "#/parameters/RoleAssignmentFilterParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {
@@ -96,6 +99,9 @@
           },
           {
             "$ref": "#/parameters/RoleAssignmentFilterParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {
@@ -151,6 +157,9 @@
           },
           {
             "$ref": "#/parameters/RoleAssignmentFilterParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {
@@ -194,6 +203,9 @@
           },
           {
             "$ref": "../../../../../common-types/resource-management/v2/types.json#/parameters/ApiVersionParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {
@@ -283,6 +295,9 @@
           },
           {
             "$ref": "../../../../../common-types/resource-management/v2/types.json#/parameters/ApiVersionParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {
@@ -370,6 +385,9 @@
           },
           {
             "$ref": "../../../../../common-types/resource-management/v2/types.json#/parameters/ApiVersionParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {
@@ -410,6 +428,9 @@
           },
           {
             "$ref": "../../../../../common-types/resource-management/v2/types.json#/parameters/ApiVersionParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {
@@ -487,6 +508,9 @@
           },
           {
             "$ref": "../../../../../common-types/resource-management/v2/types.json#/parameters/ApiVersionParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {
@@ -770,6 +794,14 @@
       },
       "description": "Parameters for the role assignment.",
       "x-ms-parameter-location": "method"
+    },
+    "TenantIdQueryParameter": {
+      "name": "tenantId",
+      "in": "query",
+      "required": false,
+      "type": "string",
+      "description": "Tenant ID for cross-tenant request",
+      "x-ms-parameter-location": "method"
     }
   }
 }
diff --git a/specification/authorization/resource-manager/Microsoft.Authorization/preview/2020-10-01-preview/authorization-RoleAssignmentsCalls.json b/specification/authorization/resource-manager/Microsoft.Authorization/preview/2020-10-01-preview/authorization-RoleAssignmentsCalls.json
index c723a1e60b81..04835777fcf4 100644
--- a/specification/authorization/resource-manager/Microsoft.Authorization/preview/2020-10-01-preview/authorization-RoleAssignmentsCalls.json
+++ b/specification/authorization/resource-manager/Microsoft.Authorization/preview/2020-10-01-preview/authorization-RoleAssignmentsCalls.json
@@ -50,6 +50,9 @@
           },
           {
             "$ref": "#/parameters/RoleAssignmentFilterParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {
@@ -96,6 +99,9 @@
           },
           {
             "$ref": "#/parameters/RoleAssignmentFilterParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {
@@ -151,6 +157,9 @@
           },
           {
             "$ref": "#/parameters/RoleAssignmentFilterParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {
@@ -194,6 +203,9 @@
           },
           {
             "$ref": "../../../../../common-types/resource-management/v2/types.json#/parameters/ApiVersionParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {
@@ -283,6 +295,9 @@
           },
           {
             "$ref": "../../../../../common-types/resource-management/v2/types.json#/parameters/ApiVersionParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {
@@ -370,6 +385,9 @@
           },
           {
             "$ref": "../../../../../common-types/resource-management/v2/types.json#/parameters/ApiVersionParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {
@@ -410,6 +428,9 @@
           },
           {
             "$ref": "../../../../../common-types/resource-management/v2/types.json#/parameters/ApiVersionParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {
@@ -487,6 +508,9 @@
           },
           {
             "$ref": "../../../../../common-types/resource-management/v2/types.json#/parameters/ApiVersionParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {
@@ -771,6 +795,14 @@
       },
       "description": "Parameters for the role assignment.",
       "x-ms-parameter-location": "method"
+    },
+    "TenantIdQueryParameter": {
+      "name": "tenantId",
+      "in": "query",
+      "required": false,
+      "type": "string",
+      "description": "Tenant ID for cross-tenant request",
+      "x-ms-parameter-location": "method"
     }
   }
 }

From f8f290d65eab7f4e09ba53c1a1de20b03ca88ca2 Mon Sep 17 00:00:00 2001
From: Jeff Hackshaw <jhackshaw@microsoft.com>
Date: Thu, 22 Apr 2021 10:05:43 -0700
Subject: [PATCH 2/2] parameter order

---
 .../authorization-RoleAssignmentsCalls.json   | 48 +++++++++----------
 1 file changed, 24 insertions(+), 24 deletions(-)

diff --git a/specification/authorization/resource-manager/Microsoft.Authorization/preview/2020-04-01-preview/authorization-RoleAssignmentsCalls.json b/specification/authorization/resource-manager/Microsoft.Authorization/preview/2020-04-01-preview/authorization-RoleAssignmentsCalls.json
index b03a9c48a781..3e0e3a787eee 100644
--- a/specification/authorization/resource-manager/Microsoft.Authorization/preview/2020-04-01-preview/authorization-RoleAssignmentsCalls.json
+++ b/specification/authorization/resource-manager/Microsoft.Authorization/preview/2020-04-01-preview/authorization-RoleAssignmentsCalls.json
@@ -48,9 +48,6 @@
           {
             "$ref": "#/parameters/ResourceProviderNamespaceParameter"
           },
-          {
-            "$ref": "#/parameters/TenantIdQueryParameter"
-          },
           {
             "name": "parentResourcePath",
             "in": "path",
@@ -86,6 +83,9 @@
           },
           {
             "$ref": "../../../../../common-types/resource-management/v2/types.json#/parameters/SubscriptionIdParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {
@@ -121,9 +121,6 @@
         "operationId": "RoleAssignments_ListForResourceGroup",
         "description": "List role assignments for a resource group.",
         "parameters": [
-          {
-            "$ref": "#/parameters/TenantIdQueryParameter"
-          },
           {
             "$ref": "../../../../../common-types/resource-management/v2/types.json#/parameters/ResourceGroupNameParameter"
           },
@@ -139,6 +136,9 @@
           },
           {
             "$ref": "../../../../../common-types/resource-management/v2/types.json#/parameters/SubscriptionIdParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {
@@ -174,9 +174,6 @@
         "operationId": "RoleAssignments_Delete",
         "description": "Delete a role assignment.",
         "parameters": [
-          {
-            "$ref": "#/parameters/TenantIdQueryParameter"
-          },
           {
             "name": "scope",
             "in": "path",
@@ -194,6 +191,9 @@
           },
           {
             "$ref": "../../../../../common-types/resource-management/v2/types.json#/parameters/ApiVersionParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {
@@ -281,9 +281,6 @@
         "operationId": "RoleAssignments_Get",
         "description": "Get the specified role assignment.",
         "parameters": [
-          {
-            "$ref": "#/parameters/TenantIdQueryParameter"
-          },
           {
             "name": "scope",
             "in": "path",
@@ -301,6 +298,9 @@
           },
           {
             "$ref": "../../../../../common-types/resource-management/v2/types.json#/parameters/ApiVersionParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {
@@ -332,9 +332,6 @@
         "operationId": "RoleAssignments_DeleteById",
         "description": "Delete a role assignment.",
         "parameters": [
-          {
-            "$ref": "#/parameters/TenantIdQueryParameter"
-          },
           {
             "name": "roleId",
             "in": "path",
@@ -345,6 +342,9 @@
           },
           {
             "$ref": "../../../../../common-types/resource-management/v2/types.json#/parameters/ApiVersionParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {
@@ -425,9 +425,6 @@
         "operationId": "RoleAssignments_GetById",
         "description": "Gets a role assignment by ID.",
         "parameters": [
-          {
-            "$ref": "#/parameters/TenantIdQueryParameter"
-          },
           {
             "name": "roleId",
             "in": "path",
@@ -438,6 +435,9 @@
           },
           {
             "$ref": "../../../../../common-types/resource-management/v2/types.json#/parameters/ApiVersionParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {
@@ -469,9 +469,6 @@
         "operationId": "RoleAssignments_List",
         "description": "Gets all role assignments for the subscription.",
         "parameters": [
-          {
-            "$ref": "#/parameters/TenantIdQueryParameter"
-          },
           {
             "name": "$filter",
             "in": "query",
@@ -484,6 +481,9 @@
           },
           {
             "$ref": "../../../../../common-types/resource-management/v2/types.json#/parameters/SubscriptionIdParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {
@@ -519,9 +519,6 @@
         "operationId": "RoleAssignments_ListForScope",
         "description": "Gets role assignments for a scope.",
         "parameters": [
-          {
-            "$ref": "#/parameters/TenantIdQueryParameter"
-          },
           {
             "name": "scope",
             "in": "path",
@@ -539,6 +536,9 @@
           },
           {
             "$ref": "../../../../../common-types/resource-management/v2/types.json#/parameters/ApiVersionParameter"
+          },
+          {
+            "$ref": "#/parameters/TenantIdQueryParameter"
           }
         ],
         "responses": {