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

[Hub Generated] Review request for Microsoft.HybridCompute to add version preview/2023-10-03-preview #26059

Merged
merged 62 commits into from
Nov 14, 2023

remove secrets

539cbd0
Select commit
Loading
Failed to load commit list.
Merged

[Hub Generated] Review request for Microsoft.HybridCompute to add version preview/2023-10-03-preview #26059

remove secrets
539cbd0
Select commit
Loading
Failed to load commit list.
openapi-pipeline-app / Swagger LintDiff succeeded Nov 14, 2023 in 3m 0s

Swagger LintDiff succeeded

LintDiff
View more details.

Details

compared tags (via openapi-validator v2.1.6) new version base version
package-preview-2023-10 package-preview-2023-10(539cbd0) default(main)

[must fix]The following errors/warnings are introduced by current PR:
Only 50 items are listed, please refer to log for more details.

Rule Message Related RPC [For API reviewers]
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L1936
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L1954
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L1957
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L1991
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L2013
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L2031
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L2034
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L2065
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L2084
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L2093
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L2096
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L2124
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L2146
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L2155
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L2158
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L2171
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L2191
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L2204
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L2207
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L2220
⚠️ 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.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L3108
⚠️ 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.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L3150
⚠️ 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.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L3384
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L4373
⚠️ 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.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L4405
⚠️ ParameterNotUsingCommonTypes Not using the common-types defined parameter 'location'.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L713
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L715
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L731
⚠️ ParameterNotUsingCommonTypes Not using the common-types defined parameter 'location'.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L754
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L756
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L787
⚠️ ParameterNotUsingCommonTypes Not using the common-types defined parameter 'location'.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L806
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L808
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L833
⚠️ RequiredReadOnlySystemData The response of operation:'NetworkSecurityPerimeterConfigurations_GetByPrivateLinkScope' is defined without 'systemData'. Consider adding the systemData to the response.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L840
⚠️ ParameterNotUsingCommonTypes Not using the common-types defined parameter 'location'.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L846
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L848
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L851
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L854
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L873
⚠️ ParameterNotUsingCommonTypes Not using the common-types defined parameter 'location'.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L891
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L893
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L896
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L899
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L915
⚠️ ParameterNotUsingCommonTypes Not using the common-types defined parameter 'location'.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L936
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L938
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L941
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L944
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L975


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

Rule Message
❌ PostResponseCodes Long-running POST operations must have responses with 202 and default return codes. They must also have a 200 return code if only if the final response is intended to have a schema, if not the 200 return code must not be specified. They also must not have other response codes.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L38
❌ OperationIdNounVerb Per the Noun_Verb convention for Operation Ids, the noun 'Licenses' should not appear after the underscore. Note: If you have already shipped an SDK on top of this spec, fixing this warning may introduce a breaking change.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L42
❌ ProvisioningStateSpecifiedForLROPut 201 response schema in long running PUT operation is missing ProvisioningState property. A LRO PUT operations response schema must have ProvisioningState specified for the 200 and 201 status codes.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L84
❌ 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.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L84
❌ PatchResponseCodes Long-running PATCH operations must have responses with 200, 202 and default return codes. They also must not have other response codes.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L134
❌ LroPatch202 The async patch operation should return 202.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L163
❌ 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.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L224
❌ 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.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L348
❌ ParametersInPointGet Query parameter $expand should be removed. Point Get's MUST not have query parameters other than api version.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L520
❌ ResourceNameRestriction The resource name parameter 'extensionName' should be defined with a 'pattern' restriction.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L1103
❌ ProvisioningStateSpecifiedForLROPut 201 response schema in long running PUT operation is missing ProvisioningState property. A LRO PUT operations response schema must have ProvisioningState specified for the 200 and 201 status codes.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L1104
❌ 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.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L1104
❌ LroLocationHeader A 202 response should include an Location response header.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L1154
❌ 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.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L1253
❌ PostResponseCodes 200 return code does not have a schema specified. LRO POST must have a 200 return code if only if the final response is intended to have a schema, if not the 200 return code must not be specified.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L1441
❌ GetCollectionOnlyHasValueAndNextLink Get endpoints for collections of resources must only have the value and nextLink properties in their model.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L1617
❌ OperationsApiSchemaUsesCommonTypes Operations API path must follow the schema provided in the common types.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L1654
❌ GetCollectionOnlyHasValueAndNextLink Get endpoints for collections of resources must only have the value and nextLink properties in their model.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L1705
❌ BodyTopLevelProperties Top level properties should be one of name, type, id, location, properties, tags, plan, sku, etag, managedBy, identity, zones. Model definition 'Machine' has extra properties ['resources'].
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L2789
❌ TrackedResourcePatchOperation Tracked resource 'License' must have patch operation that at least supports the update of tags.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L2897
❌ ArmResourcePropertiesBag Top level property names should not be repeated inside the properties bag for ARM resource 'MachineExtension'. Properties [properties.type] conflict with ARM top level properties. Please rename these.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L3558
❌ 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.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L116
❌ ResourceNameRestriction The resource name parameter 'groupName' should be defined with a 'pattern' restriction.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L356
❌ ResourceNameRestriction The resource name parameter 'privateEndpointConnectionName' should be defined with a 'pattern' restriction.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L401
❌ ProvisioningStateSpecifiedForLROPut 201 response schema in long running PUT operation is missing ProvisioningState property. A LRO PUT operations response schema must have ProvisioningState specified for the 200 and 201 status codes.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L449
❌ 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.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L449
❌ LroLocationHeader A 202 response should include an Location response header.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L491
❌ 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.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L508
❌ PathForNestedResource The path for nested resource doest not meet the valid resource pattern.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L659
❌ ResourceNameRestriction The resource name parameter 'machineName' should be defined with a 'pattern' restriction.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/privateLinkScopes.json#L659
⚠️ LongRunningOperationsOptionsValidator A LRO Post operation with return schema must have 'x-ms-long-running-operation-options' extension enabled.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L37
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L46
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L49
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L71
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L92
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L95
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L98
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L123
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L142
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L145
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L148
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L173
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L192
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L195
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L198
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L214
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L232
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L235
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L238
⚠️ LatestVersionOfCommonTypesMustBeUsed Use the latest version v5 of types.json.
Location: Microsoft.HybridCompute/preview/2023-10-03-preview/HybridCompute.json#L254
View more details on openapi-pipeline-app