fix(metricAlert_API): add missing properties to web test metric alert…#8033
fix(metricAlert_API): add missing properties to web test metric alert…#8033adibiton wants to merge 1 commit into
Conversation
…#7907) * fix(metricAlert_API): add missing properties to web test metric alert * docs(metricAlert_API): change properties description * fix(metricAlert_API): remove templateType from webtest metric alert rules * fix(metricAlert_API): change subscription description * fix(metricAlert_API): prettier checks * fix(): lint warnings * fix(): lint warnings * fix(): run prettier
|
Azure Pipelines successfully started running 1 pipeline(s). |
1 similar comment
|
Azure Pipelines successfully started running 1 pipeline(s). |
azure-sdk-for-java
|
azure-sdk-for-go
|
azure-sdk-for-net
|
azure-sdk-for-python
|
azure-sdk-for-js
|
|
Can one of the admins verify this patch? |
|
Please trigger validations for these changes since modal validations are already fixed but validations have not triggered yet |
|
@vigaur - model validation run successfully |
|
@adibiton , please fix the avocado CI check failures. |
|
Model validation fails due to a bug with OAV tool (Azure/oav#468) |
|
@myronfanqiu Can you please merge this PR? |
|
@raych1 Could you confirm the OAV bug? If we cannot pass the CI, we need your help to force merge it. @NullMDR could you check SDK automation? error messages are like |
@myronfanqiu , OAV failure is a known issue. @NullMDR , we can force merge this PR once you sign off SDK automation, thanks. |
|
@adibiton Seems like you have some swagger issue here. You have several |
|
Hi @adibiton There are at least two enum definitions with the same name |
|
@ArcturusZhang @NullMDR is adding the missing value ('none') enough? or should i change the x-ms-enum property "modelAsString" to true in order to align the defs? |
|
@adibiton Hello. I saw another PR open, what's the status of this PR? |
|
@myronfanqiu I'm closing this PR since i can't push to this branch - all the changes will be pushed from #7920 |
… (#7907)
fix(metricAlert_API): add missing properties to web test metric alert
docs(metricAlert_API): change properties description
fix(metricAlert_API): remove templateType from webtest metric alert rules
fix(metricAlert_API): change subscription description
fix(metricAlert_API): prettier checks
fix(): lint warnings
fix(): lint warnings
fix(): run prettier
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.