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.

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.StorageSync to add version stable/2022-09-01 #26082

Conversation

ankushbindlish2
Copy link
Member

@ankushbindlish2 ankushbindlish2 commented Oct 4, 2023

ARM (Control Plane) API Specification Update Pull Request

PR review workflow diagram

Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

diagram

[1] ARM review queue (for merge queues, see [4])
The PRs are processed by time opened, ascending. Your PR may show up on 2nd or later page.
If you addressed Step 1 from the diagram and your PR is not showing up in the queue, ensure the label ARMChangesRequested
is removed from your PR. This should cause the label WaitForARMFeedback to be added.
[2] https://aka.ms/azsdk/support/specreview-channel
[3] List of SDK breaking changes approvers in pinned Teams announcement
[4] public repo merge queue, private repo merge queue (for ARM review queue, [1])

If you need further help with anything, see Getting help section below.

Purpose of this PR

What's the purpose of this PR? Check all that apply. This is mandatory!

  • New API version. (Such PR should have been generated with OpenAPI Hub, per this wiki doc.)
  • Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
  • Update existing version to fix swagger quality issues in S360.
  • Other, please clarify:
    • edit this with your clarification

Due diligence checklist

To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

Breaking changes review (Step 1)

  • If the automation determines you have breaking changes, i.e. Step 1 from the diagram applies to you,
    you must follow the breaking changes process.
    IMPORTANT This applies even if:
    • The tool fails while it shouldn't, e.g. due to runtime exception, or incorrect detection of breaking changes.
    • You believe there is no need for you to request breaking change approval, for any reason.
      Such claims must be reviewed, and the process is the same.

ARM API changes review (Step 2)

  • If this PR is in purview of ARM review then automation will add the ARMReview label.
  • If you want to force ARM review, add the label yourself.
  • Proceed according to the diagram at the top of this comment.

Viewing API changes

For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures

If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
Swagger-Suppression-Process
to get approval.

Getting help

@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Oct 4, 2023

Next Steps to Merge

✔️ All automated merging requirements have been met! Refer to step 4 in the PR workflow diagram (even if your PR is for data plane, not ARM).

@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Oct 4, 2023

Swagger Validation Report

️️✔️BreakingChange succeeded [Detail] [Expand]
There are no breaking changes.
compared swaggers (via Oad v0.10.4)] new version base version
types.json 4.0(e935be8) 4.0(main)
️❌Breaking Change(Cross-Version): 84 Errors, 240 Warnings failed [Detail]
compared swaggers (via Oad v0.10.4)] new version base version
storagesync.json 2022-09-01(e935be8) 2022-06-01(main)
storagesync.json 2022-09-01(e935be8) 2017-06-05-preview(main)

The following breaking changes are detected by comparison with the latest stable version:

Only 28 items are listed, please refer to log for more details.

Rule Message
1011 - AddingResponseCode The new version adds a response code 'default'.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L116:11
1011 - AddingResponseCode The new version adds a response code 'default'.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L562:11
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L85:9
Old: Microsoft.StorageSync/stable/2022-06-01/storagesync.json#L85:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L127:9
Old: Microsoft.StorageSync/stable/2022-06-01/storagesync.json#L121:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L4410:9
Old: Microsoft.StorageSync/stable/2022-06-01/storagesync.json#L4222:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: common-types/resource-management/v5/types.json#L14:9
Old: common-types/resource-management/v3/types.json#L14:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: common-types/resource-management/v5/types.json#L14:9
Old: common-types/resource-management/v3/types.json#L14:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L206:9
Old: Microsoft.StorageSync/stable/2022-06-01/storagesync.json#L200:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L260:9
Old: Microsoft.StorageSync/stable/2022-06-01/storagesync.json#L254:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L348:9
Old: Microsoft.StorageSync/stable/2022-06-01/storagesync.json#L342:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L430:9
Old: Microsoft.StorageSync/stable/2022-06-01/storagesync.json#L424:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L482:9
Old: Microsoft.StorageSync/stable/2022-06-01/storagesync.json#L476:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L541:9
Old: Microsoft.StorageSync/stable/2022-06-01/storagesync.json#L535:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: common-types/resource-management/v5/types.json#L14:9
Old: common-types/resource-management/v3/types.json#L14:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L583:9
Old: Microsoft.StorageSync/stable/2022-06-01/storagesync.json#L571:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L626:9
Old: Microsoft.StorageSync/stable/2022-06-01/storagesync.json#L614:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L704:9
Old: Microsoft.StorageSync/stable/2022-06-01/storagesync.json#L692:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L765:9
Old: Microsoft.StorageSync/stable/2022-06-01/storagesync.json#L753:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L824:9
Old: Microsoft.StorageSync/stable/2022-06-01/storagesync.json#L812:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: common-types/resource-management/v5/types.json#L14:9
Old: common-types/resource-management/v3/types.json#L14:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L883:9
Old: Microsoft.StorageSync/stable/2022-06-01/storagesync.json#L871:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: common-types/resource-management/v5/types.json#L14:9
Old: common-types/resource-management/v3/types.json#L14:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L953:9
Old: Microsoft.StorageSync/stable/2022-06-01/storagesync.json#L941:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L1014:9
Old: Microsoft.StorageSync/stable/2022-06-01/storagesync.json#L1002:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L1077:9
Old: Microsoft.StorageSync/stable/2022-06-01/storagesync.json#L1065:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: common-types/resource-management/v5/types.json#L14:9
Old: common-types/resource-management/v3/types.json#L14:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: common-types/resource-management/v5/types.json#L14:9
Old: common-types/resource-management/v3/types.json#L14:9
1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L1180:9
Old: Microsoft.StorageSync/stable/2022-06-01/storagesync.json#L1168:9


The following breaking changes are detected by comparison with the latest preview version:

Only 28 items are listed, please refer to log for more details.

Rule Message
⚠️ 1008 - ModifiedOperationId The operation id has been changed from 'CloudEndpoints_RestoreHeatbeat' to 'CloudEndpoints_restoreheartbeat'. This will impact generated code.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L1698:7
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L1272:7
⚠️ 1008 - ModifiedOperationId The operation id has been changed from 'ServerEndpoints_Recall' to 'ServerEndpoints_recallAction'. This will impact generated code.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L2431:7
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L1847:7
⚠️ 1010 - AddingRequiredParameter The required parameter 'parameters' was added in the new version.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L2463:11
⚠️ 1011 - AddingResponseCode The new version adds a response code '202'.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L171:11
⚠️ 1011 - AddingResponseCode The new version adds a response code '202'.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L313:11
⚠️ 1011 - AddingResponseCode The new version adds a response code '202'.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L390:11
⚠️ 1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L149:13
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L94:13
⚠️ 1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L3547:9
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L2467:13
⚠️ 1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L281:13
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L200:13
⚠️ 1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L4380:9
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L2467:13
⚠️ 1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L912:13
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L486:13
⚠️ 1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L3571:9
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L2486:13
⚠️ 1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L1113:13
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L687:13
⚠️ 1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L3592:9
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L2505:13
⚠️ 1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L2048:13
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L1464:13
⚠️ 1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L3644:9
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L2524:13
⚠️ 1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L2146:13
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L1562:13
⚠️ 1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L3815:9
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L2524:13
⚠️ 1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L2676:13
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L2083:13
⚠️ 1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L3728:9
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L2553:9
⚠️ 1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L127:9
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L72:9
⚠️ 1023 - TypeFormatChanged The new version has a different format than the previous one.
New: common-types/resource-management/v5/types.json#L14:9
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L2390:9
⚠️ 1023 - TypeFormatChanged The new version has a different format than the previous one.
New: common-types/resource-management/v5/types.json#L14:9
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L2390:9
⚠️ 1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L4410:9
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L2897:9
⚠️ 1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L206:9
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L125:9
⚠️ 1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L260:9
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L179:9
⚠️ 1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L348:9
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L241:9
⚠️ 1023 - TypeFormatChanged The new version has a different format than the previous one.
New: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L430:9
Old: Microsoft.StorageSync/preview/2017-06-05-preview/storagesync.json#L297:9
️️✔️CredScan succeeded [Detail] [Expand]
There is no credential detected.
️⚠️LintDiff: 6 Warnings warning [Detail]
compared tags (via openapi-validator v2.1.6) new version base version
package-2022-09 package-2022-09(e935be8) default(main)

[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.StorageSync/stable/2022-09-01/storagesync.json#L3608
⚠️ 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.StorageSync/stable/2022-09-01/storagesync.json#L3796
⚠️ 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.StorageSync/stable/2022-09-01/storagesync.json#L3806
⚠️ 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.StorageSync/stable/2022-09-01/storagesync.json#L4396
⚠️ 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.StorageSync/stable/2022-09-01/storagesync.json#L4426
⚠️ 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.StorageSync/stable/2022-09-01/storagesync.json#L4618


The following errors/warnings exist before current PR submission:

Only 28 items are listed, please refer to log for more details.

Rule Message
OperationsApiResponseSchema The response schema of operations API '/providers/Microsoft.StorageSync/operations' does not match the ARM specification. Please standardize the schema.
Location: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L43
OperationsApiSchemaUsesCommonTypes Operations API path must follow the schema provided in the common types.
Location: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L44
ResourceNameRestriction The resource name parameter 'locationName' should be defined with a 'pattern' restriction.
Location: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L70
ResourceNameRestriction The resource name parameter 'storageSyncServiceName' should be defined with a 'pattern' restriction.
Location: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L125
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.StorageSync/stable/2022-09-01/storagesync.json#L126
PutRequestResponseSchemeArm A PUT operation request body schema should be the same as its 200 response schema, to allow reusing the same entity between GET and PUT. If the schema of the PUT request body is a superset of the GET response body, make sure you have a PATCH operation to make the resource updatable. Operation: 'StorageSyncServices_Create' Request Model: 'parameters[4].schema' Response Model: 'responses[200].schema'
Location: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L126
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.StorageSync/stable/2022-09-01/storagesync.json#L126
LroErrorContent Error response content of long running operations must follow the error schema provided in the common types v2 and above.
Location: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L199
LroErrorContent Error response content of long running operations must follow the error schema provided in the common types v2 and above.
Location: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L341
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.StorageSync/stable/2022-09-01/storagesync.json#L347
LroErrorContent Error response content of long running operations must follow the error schema provided in the common types v2 and above.
Location: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L421
GetCollectionOnlyHasValueAndNextLink Get endpoints for collections of resources must only have the value and nextLink properties in their model.
Location: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L454
GetCollectionOnlyHasValueAndNextLink Get endpoints for collections of resources must only have the value and nextLink properties in their model.
Location: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L503
ResourceNameRestriction The resource name parameter 'storageSyncServiceName' should be defined with a 'pattern' restriction.
Location: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L529
GetCollectionOnlyHasValueAndNextLink Get endpoints for collections of resources must only have the value and nextLink properties in their model.
Location: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L558
ResourceNameRestriction The resource name parameter 'storageSyncServiceName' should be defined with a 'pattern' restriction.
Location: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L571
ResourceNameRestriction The resource name parameter 'privateEndpointConnectionName' should be defined with a 'pattern' restriction.
Location: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L571
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.StorageSync/stable/2022-09-01/storagesync.json#L615
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.StorageSync/stable/2022-09-01/storagesync.json#L615
LroErrorContent Error response content of long running operations must follow the error schema provided in the common types v2 and above.
Location: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L687
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.StorageSync/stable/2022-09-01/storagesync.json#L693
LroErrorContent Error response content of long running operations must follow the error schema provided in the common types v2 and above.
Location: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L756
ResourceNameRestriction The resource name parameter 'storageSyncServiceName' should be defined with a 'pattern' restriction.
Location: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L763
GetCollectionOnlyHasValueAndNextLink Get endpoints for collections of resources must only have the value and nextLink properties in their model.
Location: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L796
ResourceNameRestriction The resource name parameter 'storageSyncServiceName' should be defined with a 'pattern' restriction.
Location: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L822
GetCollectionOnlyHasValueAndNextLink Get endpoints for collections of resources must only have the value and nextLink properties in their model.
Location: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L855
ResourceNameRestriction The resource name parameter 'storageSyncServiceName' should be defined with a 'pattern' restriction.
Location: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L881
ResourceNameRestriction The resource name parameter 'syncGroupName' should be defined with a 'pattern' restriction.
Location: Microsoft.StorageSync/stable/2022-09-01/storagesync.json#L881
️️✔️Avocado succeeded [Detail] [Expand]
Validation passes for Avocado.
️️✔️SwaggerAPIView succeeded [Detail] [Expand]
️️✔️TypeSpecAPIView succeeded [Detail] [Expand]
️️✔️ModelValidation succeeded [Detail] [Expand]
Validation passes for ModelValidation.
️️✔️SemanticValidation succeeded [Detail] [Expand]
Validation passes for SemanticValidation.
️️✔️PoliCheck succeeded [Detail] [Expand]
Validation passed for PoliCheck.
️️✔️PrettierCheck succeeded [Detail] [Expand]
Validation passes for PrettierCheck.
️️✔️SpellCheck succeeded [Detail] [Expand]
Validation passes for SpellCheck.
️️✔️Lint(RPaaS) succeeded [Detail] [Expand]
Validation passes for Lint(RPaaS).
️️✔️PR Summary succeeded [Detail] [Expand]
Validation passes for Summary.
️️✔️Automated merging requirements met succeeded [Detail] [Expand]
Posted by Swagger Pipeline | How to fix these errors?

@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Oct 4, 2023

Swagger Generation Artifacts

️️✔️ApiDocPreview succeeded [Detail] [Expand]
 Please click here to preview with your @microsoft account. 
️❌SDK Breaking Change Tracking failed [Detail]

Breaking Changes Tracking


azure-sdk-for-python-track2 - track2_azure-mgmt-storagesync - Approved - 1.0.0
+	Model StorageSyncApiError no longer has parameter inner_error
azure-sdk-for-js - @azure/arm-storagesync - Approved - 10.0.0
+	Interface StorageSyncApiError no longer has parameter innerError
️️✔️ azure-sdk-for-net-track2 succeeded [Detail] [Expand]
️⚠️ azure-sdk-for-python-track2 warning [Detail]
  • ⚠️Warning [Logs]Release - Generate from 077f1f8. SDK Automation 14.0.0
    command	sh scripts/automation_init.sh ../azure-sdk-for-python_tmp/initInput.json ../azure-sdk-for-python_tmp/initOutput.json
    cmderr	[automation_init.sh] WARNING: Skipping azure-nspkg as it is not installed.
    command	sh scripts/automation_generate.sh ../azure-sdk-for-python_tmp/generateInput.json ../azure-sdk-for-python_tmp/generateOutput.json
    cmderr	[automation_generate.sh] npm notice
    cmderr	[automation_generate.sh] npm notice New major version of npm available! 9.8.1 -> 10.2.0
    cmderr	[automation_generate.sh] npm notice Changelog: <https://github.com/npm/cli/releases/tag/v10.2.0>
    cmderr	[automation_generate.sh] npm notice Run `npm install -g [email protected]` to update!
    cmderr	[automation_generate.sh] npm notice
  • ️✔️track2_azure-mgmt-storagesync [View full logs]  [Release SDK Changes] Breaking Change Detected
    info	[Changelog] ### Features Added
    info	[Changelog]
    info	[Changelog]   - Added operation CloudEndpointsOperations.afs_share_metadata_certificate_public_keys
    info	[Changelog]   - Added operation RegisteredServersOperations.begin_update
    info	[Changelog]   - Added operation group MicrosoftStorageSyncOperationsMixin
    info	[Changelog]   - Model CloudEndpoint has a new parameter change_enumeration_status
    info	[Changelog]   - Model CloudEndpoint has a new parameter system_data
    info	[Changelog]   - Model CloudEndpointCreateParameters has a new parameter system_data
    info	[Changelog]   - Model OperationResourceMetricSpecification has a new parameter lock_aggregation_type
    info	[Changelog]   - Model OperationResourceMetricSpecification has a new parameter supported_aggregation_types
    info	[Changelog]   - Model PrivateEndpointConnection has a new parameter group_ids
    info	[Changelog]   - Model PrivateEndpointConnection has a new parameter system_data
    info	[Changelog]   - Model PrivateLinkResource has a new parameter system_data
    info	[Changelog]   - Model ProxyResource has a new parameter system_data
    info	[Changelog]   - Model RegisteredServer has a new parameter active_auth_type
    info	[Changelog]   - Model RegisteredServer has a new parameter application_id
    info	[Changelog]   - Model RegisteredServer has a new parameter identity
    info	[Changelog]   - Model RegisteredServer has a new parameter latest_application_id
    info	[Changelog]   - Model RegisteredServer has a new parameter system_data
    info	[Changelog]   - Model RegisteredServerCreateParameters has a new parameter application_id
    info	[Changelog]   - Model RegisteredServerCreateParameters has a new parameter identity
    info	[Changelog]   - Model RegisteredServerCreateParameters has a new parameter system_data
    info	[Changelog]   - Model Resource has a new parameter system_data
    info	[Changelog]   - Model ServerEndpoint has a new parameter initial_upload_policy
    info	[Changelog]   - Model ServerEndpoint has a new parameter server_endpoint_provisioning_status
    info	[Changelog]   - Model ServerEndpoint has a new parameter system_data
    info	[Changelog]   - Model ServerEndpointBackgroundDataDownloadActivity has a new parameter started_timestamp
    info	[Changelog]   - Model ServerEndpointCloudTieringStatus has a new parameter low_disk_mode
    info	[Changelog]   - Model ServerEndpointCreateParameters has a new parameter initial_upload_policy
    info	[Changelog]   - Model ServerEndpointCreateParameters has a new parameter system_data
    info	[Changelog]   - Model ServerEndpointSyncActivityStatus has a new parameter session_minutes_remaining
    info	[Changelog]   - Model StorageSyncApiError has a new parameter innererror
    info	[Changelog]   - Model StorageSyncService has a new parameter identity
    info	[Changelog]   - Model StorageSyncService has a new parameter system_data
    info	[Changelog]   - Model StorageSyncService has a new parameter use_identity
    info	[Changelog]   - Model StorageSyncServiceCreateParameters has a new parameter id
    info	[Changelog]   - Model StorageSyncServiceCreateParameters has a new parameter identity
    info	[Changelog]   - Model StorageSyncServiceCreateParameters has a new parameter name
    info	[Changelog]   - Model StorageSyncServiceCreateParameters has a new parameter system_data
    info	[Changelog]   - Model StorageSyncServiceCreateParameters has a new parameter type
    info	[Changelog]   - Model StorageSyncServiceCreateParameters has a new parameter use_identity
    info	[Changelog]   - Model StorageSyncServiceUpdateParameters has a new parameter identity
    info	[Changelog]   - Model StorageSyncServiceUpdateParameters has a new parameter use_identity
    info	[Changelog]   - Model SyncGroup has a new parameter system_data
    info	[Changelog]   - Model SyncGroupCreateParameters has a new parameter system_data
    info	[Changelog]   - Model TrackedResource has a new parameter system_data
    info	[Changelog]   - Model Workflow has a new parameter system_data
    info	[Changelog]
    info	[Changelog] ### Breaking Changes
    info	[Changelog]
    info	[Changelog]   - Model StorageSyncApiError no longer has parameter inner_error
️⚠️ azure-sdk-for-java warning [Detail]
  • ⚠️Warning [Logs]Release - Generate from 077f1f8. SDK Automation 14.0.0
    command	./eng/mgmt/automation/init.sh ../azure-sdk-for-java_tmp/initInput.json ../azure-sdk-for-java_tmp/initOutput.json
    cmderr	[init.sh] [notice] A new release of pip is available: 23.0.1 -> 23.2.1
    cmderr	[init.sh] [notice] To update, run: pip install --upgrade pip
    cmderr	[init.sh] [notice] A new release of pip is available: 23.0.1 -> 23.2.1
    cmderr	[init.sh] [notice] To update, run: pip install --upgrade pip
    command	./eng/mgmt/automation/generate.py ../azure-sdk-for-java_tmp/generateInput.json ../azure-sdk-for-java_tmp/generateOutput.json
  • ️✔️azure-resourcemanager-storagesync [View full logs]  [Release SDK Changes]
️️✔️ azure-sdk-for-go succeeded [Detail] [Expand]
  • ️✔️Succeeded [Logs]Release - Generate from 077f1f8. SDK Automation 14.0.0
    command	sh ./eng/scripts/automation_init.sh ../../../../../azure-sdk-for-go_tmp/initInput.json ../../../../../azure-sdk-for-go_tmp/initOutput.json
    command	generator automation-v2 ../../../../../azure-sdk-for-go_tmp/generateInput.json ../../../../../azure-sdk-for-go_tmp/generateOutput.json
  • ️✔️sdk/resourcemanager/storagesync/armstoragesync [View full logs]  [Release SDK Changes]
    info	[Changelog] ### Features Added
    info	[Changelog]
    info	[Changelog] - New enum type `CloudTieringLowDiskModeState` with values `CloudTieringLowDiskModeStateDisabled`, `CloudTieringLowDiskModeStateEnabled`
    info	[Changelog] - New enum type `ManagedServiceIdentityType` with values `ManagedServiceIdentityTypeNone`, `ManagedServiceIdentityTypeSystemAssigned`, `ManagedServiceIdentityTypeSystemAssignedUserAssigned`, `ManagedServiceIdentityTypeUserAssigned`
    info	[Changelog] - New enum type `ServerAuthType` with values `ServerAuthTypeCertificate`, `ServerAuthTypeManagedIdentity`
    info	[Changelog] - New enum type `ServerProvisioningStatus` with values `ServerProvisioningStatusError`, `ServerProvisioningStatusInProgress`, `ServerProvisioningStatusNotStarted`, `ServerProvisioningStatusReadySyncFunctional`, `ServerProvisioningStatusReadySyncNotFunctional`
    info	[Changelog] - New function `*CloudEndpointsClient.AfsShareMetadataCertificatePublicKeys(context.Context, string, string, string, string, *CloudEndpointsClientAfsShareMetadataCertificatePublicKeysOptions) (CloudEndpointsClientAfsShareMetadataCertificatePublicKeysResponse, error)`
    info	[Changelog] - New function `*RegisteredServersClient.BeginUpdate(context.Context, string, string, string, RegisteredServerUpdateParameters, *RegisteredServersClientBeginUpdateOptions) (*runtime.Poller[RegisteredServersClientUpdateResponse], error)`
    info	[Changelog] - New struct `CloudEndpointAfsShareMetadataCertificatePublicKeys`
    info	[Changelog] - New struct `CloudTieringLowDiskMode`
    info	[Changelog] - New struct `ErrorAdditionalInfo`
    info	[Changelog] - New struct `ErrorDetail`
    info	[Changelog] - New struct `ErrorResponse`
    info	[Changelog] - New struct `ManagedServiceIdentity`
    info	[Changelog] - New struct `RegisteredServerUpdateParameters`
    info	[Changelog] - New struct `RegisteredServerUpdateProperties`
    info	[Changelog] - New struct `ServerEndpointProvisioningStatus`
    info	[Changelog] - New struct `ServerEndpointProvisioningStepStatus`
    info	[Changelog] - New struct `UserAssignedIdentity`
    info	[Changelog] - New field `LockAggregationType` in struct `OperationResourceMetricSpecification`
    info	[Changelog] - New field `GroupIDs` in struct `PrivateEndpointConnectionProperties`
    info	[Changelog] - New field `ApplicationID`, `Identity` in struct `RegisteredServerCreateParametersProperties`
    info	[Changelog] - New field `ActiveAuthType`, `ApplicationID`, `Identity`, `LatestApplicationID` in struct `RegisteredServerProperties`
    info	[Changelog] - New field `LowDiskMode` in struct `ServerEndpointCloudTieringStatus`
    info	[Changelog] - New field `ServerEndpointProvisioningStatus` in struct `ServerEndpointProperties`
    info	[Changelog] - New field `Identity` in struct `Service`
    info	[Changelog] - New field `ID`, `Identity`, `Name`, `SystemData`, `Type` in struct `ServiceCreateParameters`
    info	[Changelog] - New field `UseIdentity` in struct `ServiceCreateParametersProperties`
    info	[Changelog] - New field `UseIdentity` in struct `ServiceProperties`
    info	[Changelog] - New field `Identity` in struct `ServiceUpdateParameters`
    info	[Changelog] - New field `UseIdentity` in struct `ServiceUpdateProperties`
    info	[Changelog]
    info	[Changelog] Total 0 breaking change(s), 53 additive change(s).
️️✔️ azure-sdk-for-js succeeded [Detail] [Expand]
  • ️✔️Succeeded [Logs]Release - Generate from 077f1f8. SDK Automation 14.0.0
    command	sh .scripts/automation_init.sh ../azure-sdk-for-js_tmp/initInput.json ../azure-sdk-for-js_tmp/initOutput.json
    warn	File azure-sdk-for-js_tmp/initOutput.json not found to read
    command	sh .scripts/automation_generate.sh ../azure-sdk-for-js_tmp/generateInput.json ../azure-sdk-for-js_tmp/generateOutput.json
  • ️✔️@azure/arm-storagesync [View full logs]  [Release SDK Changes] Breaking Change Detected
    info	[Changelog] **Features**
    info	[Changelog]
    info	[Changelog]   - Added operation CloudEndpoints.afsShareMetadataCertificatePublicKeys
    info	[Changelog]   - Added operation RegisteredServers.beginUpdate
    info	[Changelog]   - Added operation RegisteredServers.beginUpdateAndWait
    info	[Changelog]   - Added Interface CloudEndpointAfsShareMetadataCertificatePublicKeys
    info	[Changelog]   - Added Interface CloudEndpointsAfsShareMetadataCertificatePublicKeysHeaders
    info	[Changelog]   - Added Interface CloudEndpointsAfsShareMetadataCertificatePublicKeysOptionalParams
    info	[Changelog]   - Added Interface CloudTieringLowDiskMode
    info	[Changelog]   - Added Interface ErrorAdditionalInfo
    info	[Changelog]   - Added Interface ErrorDetail
    info	[Changelog]   - Added Interface ErrorResponse
    info	[Changelog]   - Added Interface ManagedServiceIdentity
    info	[Changelog]   - Added Interface RegisteredServersUpdateHeaders
    info	[Changelog]   - Added Interface RegisteredServersUpdateOptionalParams
    info	[Changelog]   - Added Interface RegisteredServerUpdateParameters
    info	[Changelog]   - Added Interface ServerEndpointProvisioningStatus
    info	[Changelog]   - Added Interface ServerEndpointProvisioningStepStatus
    info	[Changelog]   - Added Interface SystemData
    info	[Changelog]   - Added Interface UserAssignedIdentity
    info	[Changelog]   - Added Type Alias CloudEndpointsAfsShareMetadataCertificatePublicKeysResponse
    info	[Changelog]   - Added Type Alias CloudTieringLowDiskModeState
    info	[Changelog]   - Added Type Alias CreatedByType
    info	[Changelog]   - Added Type Alias ManagedServiceIdentityType
    info	[Changelog]   - Added Type Alias RegisteredServersUpdateResponse
    info	[Changelog]   - Added Type Alias ServerAuthType
    info	[Changelog]   - Added Type Alias ServerProvisioningStatus
    info	[Changelog]   - Interface OperationResourceMetricSpecification has a new optional parameter lockAggregationType
    info	[Changelog]   - Interface OperationResourceMetricSpecification has a new optional parameter supportedAggregationTypes
    info	[Changelog]   - Interface PrivateEndpointConnection has a new optional parameter groupIds
    info	[Changelog]   - Interface RegisteredServer has a new optional parameter activeAuthType
    info	[Changelog]   - Interface RegisteredServer has a new optional parameter applicationId
    info	[Changelog]   - Interface RegisteredServer has a new optional parameter identity
    info	[Changelog]   - Interface RegisteredServer has a new optional parameter latestApplicationId
    info	[Changelog]   - Interface RegisteredServerCreateParameters has a new optional parameter applicationId
    info	[Changelog]   - Interface RegisteredServerCreateParameters has a new optional parameter identity
    info	[Changelog]   - Interface Resource has a new optional parameter systemData
    info	[Changelog]   - Interface ServerEndpoint has a new optional parameter serverEndpointProvisioningStatus
    info	[Changelog]   - Interface ServerEndpointCloudTieringStatus has a new optional parameter lowDiskMode
    info	[Changelog]   - Interface StorageSyncApiError has a new optional parameter innererror
    info	[Changelog]   - Interface StorageSyncService has a new optional parameter identity
    info	[Changelog]   - Interface StorageSyncService has a new optional parameter useIdentity
    info	[Changelog]   - Interface StorageSyncServiceCreateParameters has a new optional parameter identity
    info	[Changelog]   - Interface StorageSyncServiceCreateParameters has a new optional parameter useIdentity
    info	[Changelog]   - Interface StorageSyncServiceUpdateParameters has a new optional parameter identity
    info	[Changelog]   - Interface StorageSyncServiceUpdateParameters has a new optional parameter useIdentity
    info	[Changelog]   - Added Enum KnownCloudTieringLowDiskModeState
    info	[Changelog]   - Added Enum KnownCreatedByType
    info	[Changelog]   - Added Enum KnownManagedServiceIdentityType
    info	[Changelog]   - Added Enum KnownServerAuthType
    info	[Changelog]   - Added Enum KnownServerProvisioningStatus
    info	[Changelog]
    info	[Changelog] **Breaking Changes**
    info	[Changelog]
    info	[Changelog]   - Interface StorageSyncApiError no longer has parameter innerError
️⚠️ azure-resource-manager-schemas warning [Detail]
  • ⚠️Warning [Logs]Release - Generate from 077f1f8. Schema Automation 14.0.0
    command	.sdkauto/initScript.sh ../azure-resource-manager-schemas_tmp/initInput.json ../azure-resource-manager-schemas_tmp/initOutput.json
    cmderr	[initScript.sh] npm notice
    cmderr	[initScript.sh] npm notice New major version of npm available! 8.19.4 -> 10.2.0
    cmderr	[initScript.sh] npm notice Changelog: <https://github.com/npm/cli/releases/tag/v10.2.0>
    cmderr	[initScript.sh] npm notice Run `npm install -g [email protected]` to update!
    cmderr	[initScript.sh] npm notice
    warn	File azure-resource-manager-schemas_tmp/initOutput.json not found to read
    command	.sdkauto/generateScript.sh ../azure-resource-manager-schemas_tmp/generateInput.json ../azure-resource-manager-schemas_tmp/generateOutput.json
  • ️✔️storagesync [View full logs]  [Release Schema Changes]
️❌ azure-powershell failed [Detail]
  • Pipeline Framework Failed [Logs]Release - Generate from 077f1f8. SDK Automation 14.0.0
    command	sh ./tools/SwaggerCI/init.sh ../azure-powershell_tmp/initInput.json ../azure-powershell_tmp/initOutput.json
    command	pwsh ./tools/SwaggerCI/psci.ps1 ../azure-powershell_tmp/generateInput.json ../azure-powershell_tmp/generateOutput.json
    SSL error: syscall failure: Broken pipe
    Error: SSL error: syscall failure: Broken pipe
  • ️✔️Az.storagesync.DefaultTag [View full logs
    error	Fatal error: SSL error: syscall failure: Broken pipe
    error	The following packages are still pending:
    error		Az.storagesync.DefaultTag
Posted by Swagger Pipeline | How to fix these errors?

@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Oct 4, 2023

Generated ApiView

Language Package Name ApiView Link
Go sdk/resourcemanager/storagesync/armstoragesync https://apiview.dev/Assemblies/Review/529e607168764f5db35ac56591c6adaa
Java azure-resourcemanager-storagesync https://apiview.dev/Assemblies/Review/0371ab2ccf064309b568975fc1551ba7
.Net Azure.ResourceManager.StorageSync There is no API change compared with the previous version
JavaScript @azure/arm-storagesync Create ApiView failed. Please ask PR assignee for help

@ankushbindlish2 ankushbindlish2 changed the title [Draft] microsoft.storage sync 2022 09 01 october [Hub Generated] Review request for Microsoft.StorageSync to add version stable/2022-09-01 Oct 4, 2023
@ankushbindlish2 ankushbindlish2 marked this pull request as ready for review October 4, 2023 09:37
@AzureRestAPISpecReview AzureRestAPISpecReview added ARMReview new-api-version resource-manager WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required labels Oct 4, 2023
cd D:\code\azure-rest-api-specs\specification\storagesync
npm install
npx prettier --check **/*.json
npx prettier --write **/*.json
D:\code\azure-rest-api-specs\specification\storagesync>oav validate-example D:\code\azure-rest-api-specs\specification\storagesync\resource-manager\Microsoft.StorageSync\stable\2022-09-01\storagesync.json
Issues :
Casing in properties
Identity request and response type
incorrect property name
@AzureRestAPISpecReview AzureRestAPISpecReview added ARMChangesRequested <valid label in PR review process>add this label when require changes after ARM review BreakingChangeReviewRequired <valid label in PR review process>add this label when breaking change review is required and removed WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required labels Oct 4, 2023
@ankushbindlish2
Copy link
Member Author

/pr RequestMerge

@ankushbindlish2 ankushbindlish2 removed CI-BreakingChange-Python-Track2 CI-BreakingChange-Go CI-BreakingChange-JavaScript ARMReview BreakingChangeReviewRequired <valid label in PR review process>add this label when breaking change review is required labels Oct 11, 2023
@ankushbindlish2 ankushbindlish2 removed the request for review from anpint October 11, 2023 05:17
@ankushbindlish2
Copy link
Member Author

/pr RequestMerge

@ankushbindlish2
Copy link
Member Author

/pr RequestMerge

@ms-zhenhua ms-zhenhua merged commit 077f1f8 into main Oct 11, 2023
30 of 34 checks passed
@ms-zhenhua ms-zhenhua deleted the ankushbindlish2-storagesync-Microsoft.StorageSync-2022-09-01-October branch October 11, 2023 08:09
@openapi-pipeline-app
Copy link

Swagger pipeline restarted successfully, please wait for status update in this comment.

jnlycklama pushed a commit that referenced this pull request Nov 8, 2023
…on stable/2022-09-01 (#26082)

* Adds base for updating Microsoft.StorageSync from version stable/2022-06-01 to version 2022-09-01

* Updates readme

* Updates API version in new specs and examples

* Update 2022-09-01 Api version with Managed IDentity and FAst Drv2 features

* prettier fix

cd D:\code\azure-rest-api-specs\specification\storagesync
npm install
npx prettier --check **/*.json
npx prettier --write **/*.json

* Model Validation Fix

D:\code\azure-rest-api-specs\specification\storagesync>oav validate-example D:\code\azure-rest-api-specs\specification\storagesync\resource-manager\Microsoft.StorageSync\stable\2022-09-01\storagesync.json
Issues :
Casing in properties
Identity request and response type
incorrect property name

* Location header name fix

* Linter fixes

* Update storagesync.json

* Fix Identity and useidentity linter errors

* Update RegisteredServers_Update.json

* Update

* Fix x-ms-identifiers

* Updating identity to put server to unblock sdk build

* Update storagesync.json

* revert

* Update storagesync.json

* Update storagesync.json

* Update storagesync.json

* Update storagesync.json

* Update storagesync.json

* Update storagesync.json

* Update storagesync.json

* Update storagesync.json

* Update storagesync.json

* Update

* update

* Update storagesync.json

* Update storagesync.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved-BreakingChange DO NOT USE! OBSOLETE label. See https://github.com/Azure/azure-sdk-tools/issues/6374 Approved-SdkBreakingChange-Go Approve the breaking change tracking for azure-sdk-for-go Approved-SdkBreakingChange-JavaScript Approved-SdkBreakingChange-Python ARMReview ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review BreakingChangeReviewRequired <valid label in PR review process>add this label when breaking change review is required CI-BreakingChange-JavaScript new-api-version ReadyForApiTest <valid label in PR review process>add this label when swagger and service APIs are ready for test resource-manager
Projects
None yet
Development

Successfully merging this pull request may close these issues.