App Service Environment: making workerPools optional#8433
App Service Environment: making workerPools optional#8433tombuildsstuff wants to merge 1 commit intoAzure:masterfrom
workerPools optional#8433Conversation
|
You don't have permission to trigger SDK Automation. |
specification/web/resource-manager/Microsoft.Web/stable/2019-08-01/CommonDefinitions.json
Outdated
Show resolved
Hide resolved
|
And plus I suppose this PR should all get reviewed by someone from the corresponding service team. |
This field is only required when provisioning an ASEv1 - it's optional for an ASEv2
b8cdb23 to
f6cd4a4
Compare
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
azure-sdk-for-go
|
azure-sdk-for-java
|
azure-sdk-for-js
|
azure-sdk-for-net
|
azure-sdk-for-python
|
|
Hi @tombuildsstuff I have emailed the corresponding service team to inform them to have a review on this PR. Once they approve, this PR is good to merge. |
|
Can one of the admins verify this patch? |
|
Hey @tombuildsstuff. Appreciate your effort in trying to make this better. Unfortunately these swagger docs are autogenerated so if you complete this pull request, it will just get wiped out later. We actually need to change our source code to make this happen. I have made a note of removing this from the required properties and we will add it to our backlog. |
|
Thanks @miwebst for the clarification. Hi @tombuildsstuff unfortuantely based on this I will close this PR then, hope you do not mind... |
This field is only required when provisioning an ASEv1 - it's optional for an ASEv2 - as such there's no need for it to be required
Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub - one location for using our validation tools and finding your workflow.
Contribution checklist:
ARM API Review Checklist
Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.
Please follow the link to find more details on API review process.