Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable dynamicJsonColumns for log analytics configuration #26432

Merged
merged 3 commits into from
Nov 2, 2023

Fix code style

9c835cc
Select commit
Loading
Failed to load commit list.
Merged

Enable dynamicJsonColumns for log analytics configuration #26432

Fix code style
9c835cc
Select commit
Loading
Failed to load commit list.
openapi-pipeline-app / Swagger LintDiff succeeded Oct 27, 2023 in 3m 4s

Swagger LintDiff succeeded

LintDiff
View more details.

Details

compared tags (via openapi-validator v2.1.6) new version base version
package-preview-2023-08 package-preview-2023-08(9c835cc) package-preview-2023-08(release-app-Microsoft.App-2023-08-01-preview)

[must fix]The following errors/warnings are introduced by current PR:

Rule Message Related RPC [For API reviewers]
⚠️ EnumInsteadOfBoolean Booleans properties are not descriptive in all cases and can make them to use, evaluate whether is makes sense to keep the property as boolean or turn it into an enum.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L1411


The following errors/warnings exist before current PR submission:
Only 50 items are listed, please refer to log for more details.

Rule Message
❌ ResourceNameRestriction The resource name parameter 'environmentName' should be defined with a 'pattern' restriction.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L101
❌ LroErrorContent Error response content of long running operations must follow the error schema provided in the common types v2 and above.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L197
❌ DeleteResponseCodes Long-running delete operations must have responses with 202, 204 and default return codes. They also must have no other response codes.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L211
❌ LroLocationHeader A 202 response should include an Location response header.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L240
❌ LroErrorContent Error response content of long running operations must follow the error schema provided in the common types v2 and above.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L249
❌ PatchBodyParametersSchema Properties of a PATCH request body must not be required, property:location.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L286
❌ LroLocationHeader A 202 response should include an Location response header.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L301
❌ LroErrorContent Error response content of long running operations must follow the error schema provided in the common types v2 and above.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L307
❌ ResourceNameRestriction The resource name parameter 'environmentName' should be defined with a 'pattern' restriction.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L319
❌ ResourceNameRestriction The resource name parameter 'environmentName' should be defined with a 'pattern' restriction.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L369
❌ ResourceNameRestriction The resource name parameter 'certificateName' should be defined with a 'pattern' restriction.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L369
❌ PutResponseCodes Synchronous and long-running PUT operations must have responses with 200, 201 and default return codes. They also must not have other response codes.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L422
❌ ResourceNameRestriction The resource name parameter 'environmentName' should be defined with a 'pattern' restriction.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L597
❌ ResourceNameRestriction The resource name parameter 'managedCertificateName' should be defined with a 'pattern' restriction.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L597
❌ PutResponseCodes Synchronous and long-running PUT operations must have responses with 200, 201 and default return codes. They also must not have other response codes.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L650
❌ NoErrorCodeResponses Invalid status code specified. Please refer to the documentation for the allowed set.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L703
❌ LroErrorContent Error response content of long running operations must follow the error schema provided in the common types v2 and above.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L710
❌ ResourceNameRestriction The resource name parameter 'environmentName' should be defined with a 'pattern' restriction.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L839
❌ ResourceNameRestriction The resource name parameter 'environmentName' should be defined with a 'pattern' restriction.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L889
❌ ResourceNameRestriction The resource name parameter 'environmentName' should be defined with a 'pattern' restriction.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L948
❌ ResourceNameRestriction The resource name parameter 'environmentName' should be defined with a 'pattern' restriction.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L995
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L28
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L31
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L68
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L71
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L74
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L111
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L114
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L124
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L156
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L159
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L178
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L220
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L223
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L233
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L269
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L272
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L291
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L329
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L332
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L342
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L379
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L382
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L399
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L431
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L434
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L459
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L491
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L494
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.App/preview/2023-08-01-preview/ManagedEnvironments.json#L511
View more details on openapi-pipeline-app