azure-rest-api-specs
azure-rest-api-specs copied to clipboard
recoveryservicesbackup: marking the LRO operations as LROs
These not being marked as an LRO means that consumers of the API need to instead manually parse the header to make these work, as can be seen in this commit:
https://github.com/hashicorp/terraform-provider-azurerm/blob/4d39dde5bd2ff7b40ecbed0db93d35bf4ea5646d/internal/services/recoveryservices/backup_container_storage_account_resource.go#L97-L117
This PR fixes this by correctly marking these operations as LROs - which is a bug as the operation description explicitly defines these operations are an LRO
Hi, @tombuildsstuff Thanks for your PR. I am workflow bot for review process. Here are some small tips.
[Call for Action] To better understand Azure service dev/test scenario, and support Azure service developer better on Swagger and REST API related tests in early phase, please help to fill in with this survey https://aka.ms/SurveyForEarlyPhase. It will take 5 to 10 minutes. If you already complete survey, please neglect this comment. Thanks.
Swagger Validation Report
️❌
BreakingChange: 10 Errors, 0 Warnings failed [Detail]
compared swaggers (via Oad v0.9.6)] | new version | base version |
---|---|---|
bms.json | 2021-07-01(a22f94e) | 2021-07-01(main) |
bms.json | 2021-08-01(a22f94e) | 2021-08-01(main) |
bms.json | 2021-10-01(a22f94e) | 2021-10-01(main) |
bms.json | 2021-12-01(a22f94e) | 2021-12-01(main) |
bms.json | 2022-01-01(a22f94e) | 2022-01-01(main) |
️️✔️
Breaking Change(Cross-Version) succeeded [Detail] [Expand]
There are no breaking changes.
️️✔️
CredScan succeeded [Detail] [Expand]
There is no credential detected.
️🔄
LintDiff inProgress [Detail]
️️✔️
Avocado succeeded [Detail] [Expand]
Validation passes for Avocado.
️️✔️
ApiReadinessCheck succeeded [Detail] [Expand]
️❌
ModelValidation: 7 Errors, 0 Warnings failed [Detail]
️❌
SemanticValidation: 14 Errors, 0 Warnings failed [Detail]
Rule | Message |
---|---|
INVALID_XMS_DISCRIMINATOR_VALUE |
The value of x-ms-dicriminator-value is not in the discriminator enum list: Microsoft.ClassicCompute/virtualMachines JsonUrl: Microsoft.RecoveryServices/stable/2021-07-01/bms.json#L9143:43 |
INVALID_XMS_DISCRIMINATOR_VALUE |
The value of x-ms-dicriminator-value is not in the discriminator enum list: IaaSVMContainer JsonUrl: Microsoft.RecoveryServices/stable/2021-07-01/bms.json#L10848:24 |
INVALID_XMS_DISCRIMINATOR_VALUE |
The value of x-ms-dicriminator-value is not in the discriminator enum list: Microsoft.Compute/virtualMachines JsonUrl: Microsoft.RecoveryServices/stable/2021-07-01/bms.json#L9163:36 |
INVALID_XMS_DISCRIMINATOR_VALUE |
The value of x-ms-dicriminator-value is not in the discriminator enum list: AzureWorkloadContainer JsonUrl: Microsoft.RecoveryServices/stable/2021-07-01/bms.json#L10049:31 |
INVALID_XMS_DISCRIMINATOR_VALUE |
The value of x-ms-dicriminator-value is not in the discriminator enum list: Microsoft.ClassicCompute/virtualMachines JsonUrl: Microsoft.RecoveryServices/stable/2021-08-01/bms.json#L9116:43 |
INVALID_XMS_DISCRIMINATOR_VALUE |
The value of x-ms-dicriminator-value is not in the discriminator enum list: IaaSVMContainer JsonUrl: Microsoft.RecoveryServices/stable/2021-08-01/bms.json#L10821:24 |
INVALID_XMS_DISCRIMINATOR_VALUE |
The value of x-ms-dicriminator-value is not in the discriminator enum list: Microsoft.Compute/virtualMachines JsonUrl: Microsoft.RecoveryServices/stable/2021-08-01/bms.json#L9136:36 |
INVALID_XMS_DISCRIMINATOR_VALUE |
The value of x-ms-dicriminator-value is not in the discriminator enum list: AzureWorkloadContainer JsonUrl: Microsoft.RecoveryServices/stable/2021-08-01/bms.json#L10022:31 |
INVALID_XMS_DISCRIMINATOR_VALUE |
The value of x-ms-dicriminator-value is not in the discriminator enum list: AzureWorkloadAutoProtectionIntent JsonUrl: Microsoft.RecoveryServices/stable/2021-10-01/bms.json#L9947:42 |
INVALID_XMS_DISCRIMINATOR_VALUE |
The value of x-ms-dicriminator-value is not in the discriminator enum list: AzureWorkloadSQLAutoProtectionIntent JsonUrl: Microsoft.RecoveryServices/stable/2021-10-01/bms.json#L9957:45 |
INVALID_XMS_DISCRIMINATOR_VALUE |
The value of x-ms-dicriminator-value is not in the discriminator enum list: Microsoft.ClassicCompute/virtualMachines JsonUrl: Microsoft.RecoveryServices/stable/2021-10-01/bms.json#L9483:43 |
INVALID_XMS_DISCRIMINATOR_VALUE |
The value of x-ms-dicriminator-value is not in the discriminator enum list: IaaSVMContainer JsonUrl: Microsoft.RecoveryServices/stable/2021-10-01/bms.json#L11220:24 |
INVALID_XMS_DISCRIMINATOR_VALUE |
The value of x-ms-dicriminator-value is not in the discriminator enum list: Microsoft.Compute/virtualMachines JsonUrl: Microsoft.RecoveryServices/stable/2021-10-01/bms.json#L9503:36 |
INVALID_XMS_DISCRIMINATOR_VALUE |
The value of x-ms-dicriminator-value is not in the discriminator enum list: AzureWorkloadContainer JsonUrl: Microsoft.RecoveryServices/stable/2021-10-01/bms.json#L10421:31 |
️️✔️
PoliCheck succeeded [Detail] [Expand]
Validation passed for PoliCheck.
️⚠️
SDK Track2 Validation: 10 Warnings warning [Detail]
- The following tags are being changed in this PR
- recoveryservicesbackup/resource-manager/readme.md#package-2022-01
- recoveryservicesbackup/resource-manager/readme.md#package-2021-12
- recoveryservicesbackup/resource-manager/readme.md#package-2021-10
- recoveryservicesbackup/resource-manager/readme.md#package-2021-08
- recoveryservicesbackup/resource-manager/readme.md#package-2021-07
Rule | Message |
---|---|
:warning: OutdatedExtension | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2022-01", "details":"Semantic violation: Extension 'x-ms-code-generation-settings' is not supported in Autorest V3. It will just be ignored. (info > x-ms-code-generation-settings)" |
:warning: IgnoredPropertyNextToRef | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2022-01", "details":"Semantic violation: Sibling values alongside $ref will be ignored. See https://github.com/Azure/autorest/blob/main/docs/openapi/howto/$ref-siblings.md for allowed values (components > schemas > CloudError > properties > error)\n keys: [ \u001b[32m'type'\u001b[39m ]" |
:warning: OutdatedExtension | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-12", "details":"Semantic violation: Extension 'x-ms-code-generation-settings' is not supported in Autorest V3. It will just be ignored. (info > x-ms-code-generation-settings)" |
:warning: IgnoredPropertyNextToRef | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-12", "details":"Semantic violation: Sibling values alongside $ref will be ignored. See https://github.com/Azure/autorest/blob/main/docs/openapi/howto/$ref-siblings.md for allowed values (components > schemas > CloudError > properties > error)\n keys: [ \u001b[32m'type'\u001b[39m ]" |
:warning: OutdatedExtension | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-10", "details":"Semantic violation: Extension 'x-ms-code-generation-settings' is not supported in Autorest V3. It will just be ignored. (info > x-ms-code-generation-settings)" |
:warning: IgnoredPropertyNextToRef | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-10", "details":"Semantic violation: Sibling values alongside $ref will be ignored. See https://github.com/Azure/autorest/blob/main/docs/openapi/howto/$ref-siblings.md for allowed values (components > schemas > CloudError > properties > error)\n keys: [ \u001b[32m'type'\u001b[39m ]" |
:warning: OutdatedExtension | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-08", "details":"Semantic violation: Extension 'x-ms-code-generation-settings' is not supported in Autorest V3. It will just be ignored. (info > x-ms-code-generation-settings)" |
:warning: IgnoredPropertyNextToRef | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-08", "details":"Semantic violation: Sibling values alongside $ref will be ignored. See https://github.com/Azure/autorest/blob/main/docs/openapi/howto/$ref-siblings.md for allowed values (components > schemas > CloudError > properties > error)\n keys: [ \u001b[32m'type'\u001b[39m ]" |
:warning: OutdatedExtension | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-07", "details":"Semantic violation: Extension 'x-ms-code-generation-settings' is not supported in Autorest V3. It will just be ignored. (info > x-ms-code-generation-settings)" |
:warning: IgnoredPropertyNextToRef | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-07", "details":"Semantic violation: Sibling values alongside $ref will be ignored. See https://github.com/Azure/autorest/blob/main/docs/openapi/howto/$ref-siblings.md for allowed values (components > schemas > CloudError > properties > error)\n keys: [ \u001b[32m'type'\u001b[39m ]" |
:speech_balloon: | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2022-01", "details":"> Installing AutoRest extension '@microsoft.azure/openapi-validator' (1.11.0 -> 1.11.0)" |
:speech_balloon: | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2022-01", "details":"> Installed AutoRest extension '@microsoft.azure/openapi-validator' (1.11.0->1.11.0)" |
:speech_balloon: | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2022-01", "details":"> Installing AutoRest extension '@autorest/modelerfour' (4.21.4 -> 4.21.4)" |
:speech_balloon: | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2022-01", "details":"> Installed AutoRest extension '@autorest/modelerfour' (4.21.4->4.21.4)" |
:speech_balloon: | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2022-01", "details":"Autorest completed in 15.26s. 0 files generated." |
:speech_balloon: | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-12", "details":"Autorest completed in 8.48s. 0 files generated." |
:speech_balloon: | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-08", "details":"Autorest completed in 7.94s. 0 files generated." |
:speech_balloon: | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-07", "details":"Autorest completed in 7.99s. 0 files generated." |
The following errors/warnings exist before current PR submission:
Rule | Message |
---|---|
:warning: DeprecatedConfig | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2022-01", "details":"Using directive.0.suppress which is deprecated and will be removed in the future." |
:warning: DeprecatedConfig | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2022-01", "details":"Using directive.1.suppress which is deprecated and will be removed in the future." |
:warning: DeprecatedConfig | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-12", "details":"Using directive.0.suppress which is deprecated and will be removed in the future." |
:warning: DeprecatedConfig | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-12", "details":"Using directive.1.suppress which is deprecated and will be removed in the future." |
:warning: DeprecatedConfig | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-10", "details":"Using directive.0.suppress which is deprecated and will be removed in the future." |
:warning: DeprecatedConfig | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-10", "details":"Using directive.1.suppress which is deprecated and will be removed in the future." |
:warning: DeprecatedConfig | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-08", "details":"Using directive.0.suppress which is deprecated and will be removed in the future." |
:warning: DeprecatedConfig | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-08", "details":"Using directive.1.suppress which is deprecated and will be removed in the future." |
:warning: DeprecatedConfig | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-07", "details":"Using directive.0.suppress which is deprecated and will be removed in the future." |
:warning: DeprecatedConfig | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-07", "details":"Using directive.1.suppress which is deprecated and will be removed in the future." |
:speech_balloon: | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2022-01", "details":"AutoRest core version selected from configuration: ^3.2.0." |
:speech_balloon: | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-12", "details":"AutoRest core version selected from configuration: ^3.2.0." |
:speech_balloon: | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-12", "details":"> Loading AutoRest extension '@microsoft.azure/openapi-validator' (1.11.0->1.11.0)" |
:speech_balloon: | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-12", "details":"> Loading AutoRest extension '@autorest/modelerfour' (4.21.4->4.21.4)" |
:speech_balloon: | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-10", "details":"AutoRest core version selected from configuration: ^3.2.0." |
:speech_balloon: | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-10", "details":"> Loading AutoRest extension '@microsoft.azure/openapi-validator' (1.11.0->1.11.0)" |
:speech_balloon: | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-10", "details":"> Loading AutoRest extension '@autorest/modelerfour' (4.21.4->4.21.4)" |
:speech_balloon: | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-10", "details":"Autorest completed in 8.25s. 0 files generated." |
:speech_balloon: | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-08", "details":"AutoRest core version selected from configuration: ^3.2.0." |
:speech_balloon: | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-08", "details":"> Loading AutoRest extension '@microsoft.azure/openapi-validator' (1.11.0->1.11.0)" |
:speech_balloon: | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-08", "details":"> Loading AutoRest extension '@autorest/modelerfour' (4.21.4->4.21.4)" |
:speech_balloon: | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-07", "details":"AutoRest core version selected from configuration: ^3.2.0." |
:speech_balloon: | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-07", "details":"> Loading AutoRest extension '@microsoft.azure/openapi-validator' (1.11.0->1.11.0)" |
:speech_balloon: | "readme":"recoveryservicesbackup/resource-manager/readme.md", "tag":"package-2021-07", "details":"> Loading AutoRest extension '@autorest/modelerfour' (4.21.4->4.21.4)" |
️️✔️
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).
️️✔️
CadlValidation succeeded [Detail] [Expand]
Validation passes for CadlValidation.
Thank you for your contribution tombuildsstuff! We will review the pull request and get back to you soon.
Swagger Generation Artifacts
️️✔️
ApiDocPreview succeeded [Detail] [Expand]
Only 0 items are rendered, please refer to log for more details.
️❌
SDK Breaking Change Tracking failed [Detail]
Only 0 items are rendered, please refer to log for more details.
️❌
azure-sdk-for-net failed [Detail]
Only 0 items are rendered, please refer to log for more details.
️⚠️
azure-sdk-for-python-track2 warning [Detail]
Only 0 items are rendered, please refer to log for more details.
️️✔️
azure-sdk-for-java succeeded [Detail] [Expand]
Only 0 items are rendered, please refer to log for more details.
️️✔️
azure-sdk-for-go succeeded [Detail] [Expand]
Only 0 items are rendered, please refer to log for more details.
️⚠️
azure-sdk-for-js warning [Detail]
Only 0 items are rendered, please refer to log for more details.
️⚠️
azure-resource-manager-schemas warning [Detail]
Only 0 items are rendered, please refer to log for more details.
️❌
azure-powershell failed [Detail]
Only 0 items are rendered, please refer to log for more details.
Hi @tombuildsstuff, Your PR has some issues. Please fix the CI sequentially by following the order of Avocado, semantic validation, model validation, breaking change, lintDiff
. If you have any questions, please post your questions in this channel https://aka.ms/swaggersupport.
Task | How to fix | Priority |
---|---|---|
Avocado | Fix-Avocado | High |
Semantic validation | Fix-SemanticValidation-Error | High |
Model validation | Fix-ModelValidation-Error | High |
LintDiff | Fix-LintDiff | high |
Hi @tombuildsstuff, one or multiple breaking change(s) is detected in your PR. Please check out the breaking change(s), and provide business justification in the PR comment and @ PR assignee why you must have these change(s), and how external customer impact can be mitigated. Please ensure to follow breaking change policy to request breaking change review and approval before proceeding swagger PR review. Action: To initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki. If you want to know the production traffic statistic, please see ARM Traffic statistic. If you think it is false positive breaking change, please provide the reasons in the PR comment, report to Swagger Tooling Team via https://aka.ms/swaggerfeedback.
Hi, @tombuildsstuff. Your PR has no update for 14 days and it is marked as stale PR. If no further update for over 14 days, the bot will close the PR. If you want to refresh the PR, please remove no-recent-activity
label.
Waiting on the Service Team..
Hi, @tombuildsstuff. Your PR has no update for 14 days and it is marked as stale PR. If no further update for over 14 days, the bot will close the PR. If you want to refresh the PR, please remove no-recent-activity
label.
Waiting on the Service Team..
Hi, @tombuildsstuff. Your PR has no update for 14 days and it is marked as stale PR. If no further update for over 14 days, the bot will close the PR. If you want to refresh the PR, please remove no-recent-activity
label.
/azp run
Azure Pipelines successfully started running 1 pipeline(s).
Hi, @tombuildsstuff. Your PR has no update for 14 days and it is marked as stale PR. If no further update for over 14 days, the bot will close the PR. If you want to refresh the PR, please remove no-recent-activity
label.
Hi, @tombuildsstuff. Your PR has no update for 14 days and it is marked as stale PR. If no further update for over 14 days, the bot will close the PR. If you want to refresh the PR, please remove no-recent-activity
label.
Hi, @tombuildsstuff. Your PR has no update for 14 days and it is marked as stale PR. If no further update for over 14 days, the bot will close the PR. If you want to refresh the PR, please remove no-recent-activity
label.
Hi, @tombuildsstuff. Your PR has no update for 14 days and it is marked as stale PR. If no further update for over 14 days, the bot will close the PR. If you want to refresh the PR, please remove no-recent-activity
label.
Still waiting for a review..
Hi, @tombuildsstuff. Your PR has no update for 14 days and it is marked as stale PR. If no further update for over 14 days, the bot will close the PR. If you want to refresh the PR, please remove no-recent-activity
label.
https://github.com/Azure/azure-rest-api-specs/pull/18269#issuecomment-1280051552