[BUG] `APIM API Diagnostic` - `loggerId` of `DiagnosticContractProperties` should not be a required property

[sinbai]

API Spec link

azure-rest-api-specs/specification/apimanagement/resource-manager/Microsoft.ApiManagement/stable/2022-08-01/definitions.json

Line 2707 in 7b8c7b5

"required": [

API Spec version

Both 2022-08-01 and 2024-05-01

Describe the bug

For ApiDiagnostic_CreateOrUpdate api, the loggerId property of DiagnosticContractProperties should not be a required property when the DiagnosticId is local. (The corresponding feature are shown in the Azure Portal settings below, where the Diagnostics Logs is specified as Local.)

Expected behavior

The loggerId should not be listed as a required property because loggerId does not need to be specified when setting Diagnostics Logs to Local. (loggerId is listed as a required property, which makes TF unable to specify loggerId as null)

Actual behavior

The loggerId be listed as a required property.

Reproduction Steps

Create a APIM API Diagnostic by ApiDiagnostic_CreateOrUpdate api, specify DiagnosticId as local, do not specify loggerId, and the resource is created successfully.

Environment

No response