[BodyTopLevelProperties] Add "managedByExtended" to allowed list of top-level properties

docs/body-top-level-properties.md

@@ -15,7 +15,7 @@ ARM OpenAPI(swagger) specs
 ## Description
 
 Top level properties should be one of `name`, `type`, `id`, `location`, `properties`, `tags`, `plan`, `sku`, `etag`,
-`managedBy`, `identity`, `systemData`, `extendedlocation`. Per [ARM guidelines](https://github.com/cloud-and-ai-microsoft/resource-provider-contract/blob/master/v1.0/resource-api-reference.md),
+`managedBy`, `managedByExtended`, `identity`, `kind`, `zones`, `systemData`, `extendedLocation`. Per [ARM guidelines](https://github.com/cloud-and-ai-microsoft/resource-provider-contract/blob/master/v1.0/resource-api-reference.md),
 top level properties of a resource should be only ones from the allowed set.
 
 ## How to fix

docs/rules.md

@@ -154,7 +154,7 @@ Please refer to [body-properties-names-camel-case.md](./body-properties-names-ca
 ### BodyTopLevelProperties
 
 Top level properties should be one of `name`, `type`, `id`, `location`, `properties`, `tags`, `plan`, `sku`, `etag`,
-`managedBy`, `identity`, `systemData`, `extendedlocation`. Per [ARM guidelines](https://github.com/cloud-and-ai-microsoft/resource-provider-contract/blob/master/v1.0/resource-api-reference.md),
+`managedBy`, `managedByExtended`, `identity`, `kind`, `zones`, `systemData`, `extendedLocation`. Per [ARM guidelines](https://github.com/cloud-and-ai-microsoft/resource-provider-contract/blob/master/v1.0/resource-api-reference.md),
 top level properties of a resource should be only ones from the allowed set.
 
 Please refer to [body-top-level-properties.md](./body-top-level-properties.md) for details.

packages/rulesets/src/native/functions/arm-resource-validation.ts

@@ -77,6 +77,7 @@ export function* armResourcePropertiesBag(openapiSection: any, options: {}, ctx:
     "sku",
     "etag",
     "managedby",
+    "managedbyextended",
     "identity",
     "kind",
     "zones",
@@ -144,6 +145,7 @@ export function* bodyTopLevelProperties(openapiSection: any, options: {}, ctx: R
       "sku",
       "etag",
       "managedby",
+      "managedbyextended",
       "identity",
       "kind",
       "zones",
@@ -155,7 +157,7 @@ export function* bodyTopLevelProperties(openapiSection: any, options: {}, ctx: R
       if (!allowedBodyTopLevelProperties.includes(propName.toLowerCase())) {
         yield {
           location: ["definitions", re.modelName],
-          message: `Top level properties should be one of name, type, id, location, properties, tags, plan, sku, etag, managedBy, identity, zones. Model definition '${re.modelName}' has extra properties ['${propName}'].`,
+          message: `Top level properties should be one of name, type, id, location, properties, tags, plan, sku, etag, managedBy, managedByExtended, identity, kind, zones, systemData, extendedLocation. Model definition '${re.modelName}' has extra properties ['${propName}'].`,
         }
       }
     }

packages/rulesets/src/native/tests/resources/body-top-level-properties-real-swagger.json

@@ -140,6 +140,13 @@
           "type": "string",
           "description": "The escaped name of the namespace."
         },
+        "managedByExtended": {
+          "type": "array",
+          "items": {
+            "type": "string"
+          },
+          "description": "The fully qualified resource IDs of the resources that manage this resource"
+        },
         "classification": {
           "$ref": "#/definitions/NamespaceClassification",
           "description": "Kind of namespace"