Azure / azure-rest-api-specs

The source for REST API specifications for Microsoft Azure.
MIT License
2.68k stars 5.11k forks source link

Fix default value property error for keda properties - 08.2024 release #30691

Closed tawalke closed 1 month ago

tawalke commented 1 month ago

Purpose:

Files Modified:

ARM (Control Plane) API Specification Update Pull Request

[!TIP] Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram

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

spec_pr_review_workflow_diagram

Purpose of this PR

What's the purpose of this PR? Check the specific option that applies. This is mandatory!

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:

Additional information

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 [suppressions guide](https://aka.ms/azsdk/pr-suppressions) to get approval.

Getting help

openapi-pipeline-app[bot] commented 1 month ago

PR validation pipeline can not start as the pull request is not merged or mergeable - most likely it has merge conflicts.

openapi-pipeline-app[bot] commented 1 month ago

Next Steps to Merge

✅ All automated merging requirements have been met! To get your PR merged, see aka.ms/azsdk/specreview/merge.
openapi-pipeline-app[bot] commented 1 month ago

Generated ApiView

Language Package Name ApiView Link
Go sdk/resourcemanager/appcontainers/armappcontainers https://apiview.dev/Assemblies/Review/77a1e1c7ff4c495e8e1c9cf369fdf74a?revisionId=be252ae816754cc38706a33ac4b1cdb0
JavaScript @azure/arm-appcontainers https://apiview.dev/Assemblies/Review/ba41cd9978b542efb0688a072b2319a1?revisionId=7800f0cee7e944ae8694c037ee3573e1
Java azure-resourcemanager-appcontainers https://apiview.dev/Assemblies/Review/7da9ac1e2db64477ada2ba8bbb71169b?revisionId=990e291e69ca46a7b5ab804499b24e54
Swagger Microsoft.App https://apiview.dev/Assemblies/Review/53999dbad23b45cba7812a00a713cab8?revisionId=df60c25ab3694042a134e5fe48f1b1dc
ms-bogdan-rotaru commented 1 month ago
"/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.App/jobs/{jobName}/resume": {

This new endpoint is expected to be part of this PR?


Refers to: specification/app/resource-manager/Microsoft.App/preview/2024-08-02-preview/Jobs.json:629 in 8d1fc93. [](commit_id = 8d1fc9308c937ee4b9d0b0d7fc47b76bee0ea106, deletion_comment = False)

ms-bogdan-rotaru commented 1 month ago
"/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.App/jobs/{jobName}/suspend": {

This new endpoint is expected to be part of this PR?


Refers to: specification/app/resource-manager/Microsoft.App/preview/2024-08-02-preview/Jobs.json:691 in 8d1fc93. [](commit_id = 8d1fc9308c937ee4b9d0b0d7fc47b76bee0ea106, deletion_comment = False)

jijohn14 commented 1 month ago
"/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.App/jobs/{jobName}/resume": {

This new endpoint is expected to be part of this PR?

Refers to: specification/app/resource-manager/Microsoft.App/preview/2024-08-02-preview/Jobs.json:629 in 8d1fc93. [](commit_id = 8d1fc93, deletion_comment = False)

@ms-bogdan-rotaru These endpoints are already merged to target branch. Refer https://github.com/Azure/azure-rest-api-specs/pull/30304