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

s360 fixes for Azure DNS #11591

Merged
merged 9 commits into from
Nov 18, 2020
Merged

s360 fixes for Azure DNS #11591

merged 9 commits into from
Nov 18, 2020

Conversation

vladrai
Copy link
Contributor

@vladrai vladrai commented Nov 9, 2020

MSFT employees can try out our new experience at OpenAPI Hub - one location for using our validation tools and finding your workflow.

Changelog

Please ensure to add changelog with this PR by answering the following questions.

  1. What's the purpose of the update?
    • new service onboarding
    • new API version
    • update existing version for new feature
    • update existing version to fix swagger quality issue in s360
    • Other, please clarify
  2. When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
    N/A Already deployed
  3. When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
    With general release.
  4. If it's an update to existing version, please select SDKs of specific language and CLIs that require refresh after swagger is published.
    • SDK of .NET (need service team to ensure code readiness)
    • SDK of Python
    • SDK of Java
    • SDK of Js
    • SDK of Go
    • PowerShell
    • CLI
    • Terraform
    • No, no need to refresh for updates in this PR

Contribution checklist:

If any further question about AME onboarding or validation tools, please view the FAQ.

ARM API Review Checklist

  • [] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

    • Adding new API(s)
    • Adding a new API version
    • Adding a new service
  • Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

  • [] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist

If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

  • Removing API(s) in stable version
  • Removing properties in stable version
  • Removing API version(s) in stable version
  • Updating API in stable version with Breaking Change Validation errors
  • Updating API(s) in preview over 1 year

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.

Please follow the link to find more details on PR review process.

@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Nov 9, 2020

Swagger Validation Report

️️✔️BreakingChange succeeded [Detail] [Expand]
There are no breaking changes.

️⚠️[Staging] LintDiff: 14 Warnings warning [Detail] Only 10 items are listed, please refer to log for more details.

Rule Message
⚠️ R4009 - RequiredSystemDataInNewApiVersions The response of operation:'RecordSets_List' is defined without 'systemData'. Consider adding the systemData to the response.
New: Microsoft.Network/stable/2020-06-01/privatedns.json#L1302
⚠️ R4009 - RequiredSystemDataInNewApiVersions The response of operation:'PrivateZones_Update' is defined without 'systemData'. Consider adding the systemData to the response.
New: Microsoft.Network/stable/2020-06-01/privatedns.json#L114
⚠️ R4009 - RequiredSystemDataInNewApiVersions The response of operation:'PrivateZones_Get' is defined without 'systemData'. Consider adding the systemData to the response.
New: Microsoft.Network/stable/2020-06-01/privatedns.json#L243
⚠️ R4009 - RequiredSystemDataInNewApiVersions The response of operation:'PrivateZones_List' is defined without 'systemData'. Consider adding the systemData to the response.
New: Microsoft.Network/stable/2020-06-01/privatedns.json#L293
⚠️ R4009 - RequiredSystemDataInNewApiVersions The response of operation:'PrivateZones_ListByResourceGroup' is defined without 'systemData'. Consider adding the systemData to the response.
New: Microsoft.Network/stable/2020-06-01/privatedns.json#L340
⚠️ R4009 - RequiredSystemDataInNewApiVersions The response of operation:'VirtualNetworkLinks_CreateOrUpdate' is defined without 'systemData'. Consider adding the systemData to the response.
New: Microsoft.Network/stable/2020-06-01/privatedns.json#L394
⚠️ R4009 - RequiredSystemDataInNewApiVersions The response of operation:'PrivateZones_CreateOrUpdate' is defined without 'systemData'. Consider adding the systemData to the response.
New: Microsoft.Network/stable/2020-06-01/privatedns.json#L31
⚠️ R4009 - RequiredSystemDataInNewApiVersions The response of operation:'VirtualNetworkLinks_Get' is defined without 'systemData'. Consider adding the systemData to the response.
New: Microsoft.Network/stable/2020-06-01/privatedns.json#L627
⚠️ R4009 - RequiredSystemDataInNewApiVersions The response of operation:'VirtualNetworkLinks_List' is defined without 'systemData'. Consider adding the systemData to the response.
New: Microsoft.Network/stable/2020-06-01/privatedns.json#L684
️️✔️[Staging] Avocado succeeded [Detail] [Expand]
Validation passes for Avocado.
️️✔️[Staging] ModelValidation succeeded [Detail] [Expand]
Validation passes for ModelValidation.
️️✔️[Staging] SemanticValidation succeeded [Detail] [Expand]
Validation passes for SemanticValidation.
️️✔️[Staging] Cross Version BreakingChange (Base on preview version) succeeded [Detail] [Expand]
There are no breaking changes.
️️✔️[Staging] Cross Version BreakingChange (Base on stable version) succeeded [Detail] [Expand]
There are no breaking changes.

Posted by Swagger Pipeline | How to fix these errors?

@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Nov 9, 2020

Swagger Generation Artifacts

️️✔️[Staging] azure-sdk-for-java succeeded [Detail] [Expand]
️❌[Staging] azure-sdk-for-go failed [Detail]
- Breaking Change Detected
  • Failed [Logs]Release - Generate from b47a321. SDK Automation 14.0.0
    command	sh ./initScript.sh ../../../../../azure-sdk-for-go_tmp/initInput.json ../../../../../azure-sdk-for-go_tmp/initOutput.json
    command	go run ./tools/generator/main.go ../../../../../azure-sdk-for-go_tmp/generateInput.json ../../../../../azure-sdk-for-go_tmp/generateOutput.json
    cannot force update branch 'sdkAuto/dns_mgmt/2018-05-01/dns' as it is the current HEAD of the repository.
    Error: cannot force update branch 'sdkAuto/dns_mgmt/2018-05-01/dns' as it is the current HEAD of the repository.
  • ️✔️privatedns/mgmt/2018-09-01/privatedns [View full logs]  [Release SDK Changes] Breaking Change Detected
    Only show 10 items here, please refer to log for details.
    info	[Changelog] 		- To: PrivateZoneListResult, func(context.Context, PrivateZoneListResult) (PrivateZoneListResult, error)
    info	[Changelog] 1. NewRecordSetListResultPage
    info	[Changelog] 	- Params
    info	[Changelog] 		- From: func(context.Context, RecordSetListResult) (RecordSetListResult, error)
    info	[Changelog] 		- To: RecordSetListResult, func(context.Context, RecordSetListResult) (RecordSetListResult, error)
    info	[Changelog] 1. NewVirtualNetworkLinkListResultPage
    info	[Changelog] 	- Params
    info	[Changelog] 		- From: func(context.Context, VirtualNetworkLinkListResult) (VirtualNetworkLinkListResult, error)
    info	[Changelog] 		- To: VirtualNetworkLinkListResult, func(context.Context, VirtualNetworkLinkListResult) (VirtualNetworkLinkListResult, error)
    info	[Changelog]
  • ️✔️dns/mgmt/2018-05-01/dns [View full logs]  [Release SDK Changes] Breaking Change Detected
    Only show 10 items here, please refer to log for details.
    info	[Changelog] 	- Params
    info	[Changelog] 		- From: func(context.Context, ZoneListResult) (ZoneListResult, error)
    info	[Changelog] 		- To: ZoneListResult, func(context.Context, ZoneListResult) (ZoneListResult, error)
    info	[Changelog]
    info	[Changelog] ## Struct Changes
    info	[Changelog]
    info	[Changelog] ### New Struct Fields
    info	[Changelog]
    info	[Changelog] 1. ZoneProperties.MaxNumberOfRecordsPerRecordSet
    info	[Changelog]
  • ️✔️dns/mgmt/2018-05-01/dns [View full logsBreaking Change Detected
    Only show 10 items here, please refer to log for details.
    error		preview/dns/mgmt/2018-03-01-preview/dns
    error		preview/dns/mgmt/2018-03-01-preview/dns
    error		dns/mgmt/2017-10-01/dns
    error		dns/mgmt/2017-10-01/dns
    error		dns/mgmt/2017-09-01/dns
    error		dns/mgmt/2017-09-01/dns
    error		dns/mgmt/2016-04-01/dns
    error		dns/mgmt/2016-04-01/dns
    error		preview/dns/mgmt/2015-05-04-preview/dns
    error		preview/dns/mgmt/2015-05-04-preview/dns
  • ️✔️preview/dns/mgmt/2018-03-01-preview/dnsBreaking Change Detected
  • ️✔️preview/dns/mgmt/2018-03-01-preview/dnsBreaking Change Detected
  • ️✔️dns/mgmt/2017-10-01/dnsBreaking Change Detected
  • ️✔️dns/mgmt/2017-10-01/dnsBreaking Change Detected
  • ️✔️dns/mgmt/2017-09-01/dnsBreaking Change Detected
  • ️✔️dns/mgmt/2017-09-01/dnsBreaking Change Detected
  • ️✔️dns/mgmt/2016-04-01/dnsBreaking Change Detected
  • ️✔️dns/mgmt/2016-04-01/dnsBreaking Change Detected
  • ️✔️preview/dns/mgmt/2015-05-04-preview/dnsBreaking Change Detected
  • ️✔️preview/dns/mgmt/2015-05-04-preview/dnsBreaking Change Detected
️️✔️[Staging] azure-sdk-for-js succeeded [Detail] [Expand]
  • ️✔️Succeeded [Logs]Release - Generate from b47a321. SDK Automation 14.0.0
    warn	Skip initScript due to not configured
    command	autorest --version=V2 --typescript --license-header=MICROSOFT_MIT_NO_VERSION [email protected]/[email protected] --typescript-sdks-folder=/home/vsts/work/1/s/w/azure-sdk-for-js/azure-sdk-for-js ../../azure-rest-api-specs/specification/dns/resource-manager/readme.md
    command	autorest --version=V2 --typescript --license-header=MICROSOFT_MIT_NO_VERSION [email protected]/[email protected] --typescript-sdks-folder=/home/vsts/work/1/s/w/azure-sdk-for-js/azure-sdk-for-js ../../azure-rest-api-specs/specification/privatedns/resource-manager/readme.md
  • ️✔️@azure/arm-dns [View full logs]  [Release SDK Changes]
    cmderr	[npmPack] loaded rollup.config.js with warnings
    cmderr	[npmPack] (!) Unused external imports
    cmderr	[npmPack] default imported from external module 'rollup' but never used
    cmderr	[npmPack] ./esm/dnsManagementClient.js → ./dist/arm-dns.js...
    cmderr	[npmPack] created ./dist/arm-dns.js in 308ms
  • ️✔️@azure/arm-privatedns [View full logs]  [Release SDK Changes]
    cmderr	[npmPack] loaded rollup.config.js with warnings
    cmderr	[npmPack] (!) Unused external imports
    cmderr	[npmPack] default imported from external module 'rollup' but never used
    cmderr	[npmPack] ./esm/privateDnsManagementClient.js → ./dist/arm-privatedns.js...
    cmderr	[npmPack] created ./dist/arm-privatedns.js in 288ms
️️✔️[Staging] azure-sdk-for-python succeeded [Detail] [Expand]
  • ️✔️Succeeded [Logs]Release - Generate from b47a321. SDK Automation 14.0.0
    warn	Skip initScript due to not configured
    command	npx n 12
    command	python3 -m venv venv
    command	pip install setuptools wheel
    command	pip install pathlib jinja2 msrestazure
    command	autorest --version=V2 [email protected]/autorest.python@~4.0.71 --python --python-mode=update --multiapi --keep-version-file --no-async --python-sdks-folder=/home/vsts/work/1/s/w/azure-sdk-for-python/sdk ../azure-rest-api-specs/specification/dns/resource-manager/readme.md
    command	autorest --version=V2 [email protected]/autorest.python@~4.0.71 --python --python-mode=update --multiapi --keep-version-file --no-async --python-sdks-folder=/home/vsts/work/1/s/w/azure-sdk-for-python/sdk ../azure-rest-api-specs/specification/privatedns/resource-manager/readme.md
  • ️✔️azure-mgmt-dns [View full logs]  [Release SDK Changes]
    cmderr	[buildPackage] /usr/lib/python3.6/distutils/dist.py:261: UserWarning: Unknown distribution option: 'long_description_content_type'
    cmderr	[buildPackage]   warnings.warn(msg)
    cmderr	[buildPackage] /usr/lib/python3.6/distutils/dist.py:261: UserWarning: Unknown distribution option: 'long_description_content_type'
    cmderr	[buildPackage]   warnings.warn(msg)
    cmderr	[changelogSetup] Ignoring mock: markers 'python_version <= "2.7"' don't match your environment
    cmderr	[changelogSetup] Cannot uninstall requirement azure-nspkg, not installed
    cmderr	[changelogSetup] Command '['/home/vsts/work/1/s/w/azure-sdk-for-python_tmp/venv/bin/python', '-m', 'pip', 'uninstall', '-y', 'azure-nspkg']' returned non-zero exit status 1.
    cmderr	[changelogReport] INFO:__main__:Looking for Autorest generated package in azure.mgmt.dns
    cmderr	[changelogReport] INFO:__main__:Looking for Autorest generated package in azure.mgmt.dns.v2016_04_01
    cmderr	[changelogReport] INFO:__main__:Found azure.mgmt.dns.v2016_04_01
    cmderr	[changelogReport] INFO:__main__:Looking for Autorest generated package in azure.mgmt.dns.v2018_03_01_preview
    cmderr	[changelogReport] INFO:__main__:Found azure.mgmt.dns.v2018_03_01_preview
    cmderr	[changelogReport] INFO:__main__:Looking for Autorest generated package in azure.mgmt.dns.v2018_05_01
    cmderr	[changelogReport] INFO:__main__:Found azure.mgmt.dns.v2018_05_01
    cmderr	[changelogReport] INFO:__main__:Working on azure.mgmt.dns.v2016_04_01
    cmderr	[changelogReport] INFO:__main__:Report written to sdk/network/azure-mgmt-dns/code_reports/latest/v2016_04_01.json
    cmderr	[changelogReport] INFO:__main__:Working on azure.mgmt.dns.v2018_03_01_preview
    cmderr	[changelogReport] INFO:__main__:Report written to sdk/network/azure-mgmt-dns/code_reports/latest/v2018_03_01_preview.json
    cmderr	[changelogReport] INFO:__main__:Working on azure.mgmt.dns.v2018_05_01
    cmderr	[changelogReport] INFO:__main__:Report written to sdk/network/azure-mgmt-dns/code_reports/latest/v2018_05_01.json
    cmderr	[changelogReport] INFO:__main__:Merged report written to sdk/network/azure-mgmt-dns/code_reports/latest/merged_report.json
    cmderr	[changelogReportLatest] INFO:__main__:Download versions of azure-mgmt-dns on PyPI
    cmderr	[changelogReportLatest] INFO:__main__:Got ['0.30.0rc5', '0.30.0rc6', '1.0.0', '1.0.1', '1.1.0', '1.2.0', '2.0.0rc1', '2.0.0rc2', '2.0.0', '2.1.0', '3.0.0']
    cmderr	[changelogReportLatest] INFO:__main__:Only keep last PyPI version
    cmderr	[changelogReportLatest] INFO:__main__:Installing version 3.0.0 of azure-mgmt-dns in a venv
    cmderr	[changelogReportLatest] INFO:__main__:Looking for Autorest generated package in azure.mgmt.dns
    cmderr	[changelogReportLatest] INFO:__main__:Looking for Autorest generated package in azure.mgmt.dns.v2016_04_01
    cmderr	[changelogReportLatest] INFO:__main__:Found azure.mgmt.dns.v2016_04_01
    cmderr	[changelogReportLatest] INFO:__main__:Looking for Autorest generated package in azure.mgmt.dns.v2018_03_01_preview
    cmderr	[changelogReportLatest] INFO:__main__:Found azure.mgmt.dns.v2018_03_01_preview
    cmderr	[changelogReportLatest] INFO:__main__:Looking for Autorest generated package in azure.mgmt.dns.v2018_05_01
    cmderr	[changelogReportLatest] INFO:__main__:Found azure.mgmt.dns.v2018_05_01
    cmderr	[changelogReportLatest] INFO:__main__:Working on azure.mgmt.dns.v2016_04_01
    cmderr	[changelogReportLatest] INFO:__main__:Report written to sdk/network/azure-mgmt-dns/code_reports/3.0.0/v2016_04_01.json
    cmderr	[changelogReportLatest] INFO:__main__:Working on azure.mgmt.dns.v2018_03_01_preview
    cmderr	[changelogReportLatest] INFO:__main__:Report written to sdk/network/azure-mgmt-dns/code_reports/3.0.0/v2018_03_01_preview.json
    cmderr	[changelogReportLatest] INFO:__main__:Working on azure.mgmt.dns.v2018_05_01
    cmderr	[changelogReportLatest] INFO:__main__:Report written to sdk/network/azure-mgmt-dns/code_reports/3.0.0/v2018_05_01.json
    cmderr	[changelogReportLatest] INFO:__main__:Merged report written to sdk/network/azure-mgmt-dns/code_reports/3.0.0/merged_report.json
    cmdout	[Changelog] Size of delta 1.230% size of original (original: 15607 chars, delta: 192 chars)
    cmdout	[Changelog] **Features**
    cmdout	[Changelog]   - Model Zone has a new parameter max_number_of_records_per_record_set
  • ️✔️azure-mgmt-privatedns [View full logs]  [Release SDK Changes]
    cmderr	[buildPackage] /usr/lib/python3.6/distutils/dist.py:261: UserWarning: Unknown distribution option: 'long_description_content_type'
    cmderr	[buildPackage]   warnings.warn(msg)
    cmderr	[buildPackage] /usr/lib/python3.6/distutils/dist.py:261: UserWarning: Unknown distribution option: 'long_description_content_type'
    cmderr	[buildPackage]   warnings.warn(msg)
    cmderr	[changelogSetup] Ignoring mock: markers 'python_version <= "2.7"' don't match your environment
    cmderr	[changelogSetup] Cannot uninstall requirement azure-nspkg, not installed
    cmderr	[changelogSetup] Command '['/home/vsts/work/1/s/w/azure-sdk-for-python_tmp/venv/bin/python', '-m', 'pip', 'uninstall', '-y', 'azure-nspkg']' returned non-zero exit status 1.
    cmderr	[changelogReport] INFO:__main__:Looking for Autorest generated package in azure.mgmt.privatedns
    cmderr	[changelogReport] INFO:__main__:Found azure.mgmt.privatedns
    cmderr	[changelogReport] INFO:__main__:Working on azure.mgmt.privatedns
    cmderr	[changelogReport] INFO:__main__:Report written to sdk/network/azure-mgmt-privatedns/code_reports/latest/report.json
    cmderr	[changelogReportLatest] INFO:__main__:Download versions of azure-mgmt-privatedns on PyPI
    cmderr	[changelogReportLatest] INFO:__main__:Got ['0.1.0']
    cmderr	[changelogReportLatest] INFO:__main__:Only keep last PyPI version
    cmderr	[changelogReportLatest] INFO:__main__:Installing version 0.1.0 of azure-mgmt-privatedns in a venv
    cmderr	[changelogReportLatest] Cache entry deserialization failed, entry ignored
    cmderr	[changelogReportLatest] INFO:__main__:Looking for Autorest generated package in azure.mgmt.privatedns
    cmderr	[changelogReportLatest] INFO:__main__:Found azure.mgmt.privatedns
    cmderr	[changelogReportLatest] INFO:__main__:Working on azure.mgmt.privatedns
    cmderr	[changelogReportLatest] INFO:__main__:Report written to sdk/network/azure-mgmt-privatedns/code_reports/0.1.0/report.json
    cmdout	[Changelog] Size of delta 0.014% size of original (original: 14755 chars, delta: 2 chars)
️⚠️[Staging] azure-sdk-for-net warning [Detail]
  • ⚠️Warning [Logs]Release - Generate from b47a321. SDK Automation 14.0.0
    warn	Skip initScript due to not configured
    warn		specification/dns/resource-manager/readme.md skipped due to azure-sdk-for-net not found in swagger-to-sdk
    command	autorest --version=V2 --csharp --reflect-api-versions --license-header=MICROSOFT_MIT_NO_VERSION [email protected]/[email protected] --csharp-sdks-folder=/home/vsts/work/1/s/w/azure-sdk-for-net/sdk ../azure-rest-api-specs/specification/privatedns/resource-manager/readme.md
    cmderr	[Autorest] realpath(): Permission denied
    cmderr	[Autorest] realpath(): Permission denied
    cmderr	[Autorest] realpath(): Permission denied
    cmderr	[Autorest] realpath(): Permission denied
    cmderr	[Autorest] realpath(): Permission denied
    cmderr	[Autorest] realpath(): Permission denied
  • ️✔️Microsoft.Azure.Management.PrivateDns [View full logs]  [Release SDK Changes]
️❌[Staging] azure-resource-manager-schemas failed [Detail]
  • Failed [Logs]Release - Generate from b47a321. Schema Automation 14.0.0
    warn	Skip initScript due to not configured
    command	npx n 12
    command	autorest --use=@autorest/[email protected] --version=3.0.6320 --azureresourceschema --multiapi --title=none --pass-thru:subset-reducer --azureresourceschema-folder=/home/vsts/work/1/s/w/azure-resource-manager-schemas/azure-resource-manager-schemas ../../azure-rest-api-specs/specification/dns/resource-manager/readme.md
    command	autorest --use=@autorest/[email protected] --version=3.0.6320 --azureresourceschema --multiapi --title=none --pass-thru:subset-reducer --azureresourceschema-folder=/home/vsts/work/1/s/w/azure-resource-manager-schemas/azure-resource-manager-schemas ../../azure-rest-api-specs/specification/privatedns/resource-manager/readme.md
  • cmderr	[resource-manager] node-pre-gyp
    cmderr	[resource-manager] WARN Using request for node-pre-gyp https download
    cmderr	[resource-manager] WARN [email protected] No repository field.
    cmderr	[resource-manager]  optional SKIPPING OPTIONAL DEPENDENCY: [email protected] (node_modules/fsevents):
    cmderr	[resource-manager] npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for [email protected]: wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
    cmderr	[resource-manager] npm ERR! code ELIFECYCLE
    cmderr	[resource-manager] npm ERR! errno 87
    cmderr	[resource-manager] npm ERR! [email protected] test: `mocha -r ts-node/register tests.ts`
    cmderr	[resource-manager] npm ERR! Exit status 87
    cmderr	[resource-manager] npm ERR!
    cmderr	[resource-manager] npm ERR! Failed at the [email protected] test script.
    cmderr	[resource-manager] npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
    cmderr	[resource-manager] npm ERR! A complete log of this run can be found in:
    cmderr	[resource-manager] npm ERR!     /home/vsts/.npm/_logs/2020-11-18T03_11_53_151Z-debug.log
    cmderr	[resource-manager] child_process.js:674
    cmderr	[resource-manager]     throw err;
    cmderr	[resource-manager]     ^
    cmderr	[resource-manager] Error: Command failed: npm run test --prefix tools
    cmderr	[resource-manager]     at checkExecSyncError (child_process.js:635:11)
    cmderr	[resource-manager]     at Object.execSync (child_process.js:671:15)
    cmderr	[resource-manager]     at Object.<anonymous> (/home/vsts/work/1/s/w/azure-resource-manager-schemas/azure-resource-manager-schemas/sdkauto_afterscript.js:5:26)
    cmderr	[resource-manager]     at Module._compile (internal/modules/cjs/loader.js:1015:30)
    cmderr	[resource-manager]     at Object.Module._extensions..js (internal/modules/cjs/loader.js:1035:10)
    cmderr	[resource-manager]     at Module.load (internal/modules/cjs/loader.js:879:32)
    cmderr	[resource-manager]     at Function.Module._load (internal/modules/cjs/loader.js:724:14)
    cmderr	[resource-manager]     at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:60:12)
    cmderr	[resource-manager]     at internal/main/run_main_module.js:17:47 {
    cmderr	[resource-manager]   status: 87,
    cmderr	[resource-manager]   signal: null,
    cmderr	[resource-manager]   output: [ null, null, null ],
    cmderr	[resource-manager]   pid: 3143,
    cmderr	[resource-manager]   stdout: null,
    cmderr	[resource-manager]   stderr: null
    cmderr	[resource-manager] }
    error	Script return with result [failed] code [1] signal [null] cwd [azure-resource-manager-schemas/azure-resource-manager-schemas]: node sdkauto_afterscript.js dns/resource-manager
    warn	Skip command for failed context: node sdkauto_afterscript.js privatedns/resource-manager
️️✔️[Staging] ApiDocPreview succeeded [Detail] [Expand]
 Please click here to preview with your @microsoft account. 
Posted by Swagger Pipeline | How to fix these errors?

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Nov 9, 2020

Azure CLI Extension Generation

No readme.md specification configuration files were found that are associated with the files modified in this pull request, or swagger_to_sdk section in readme.md is not configured

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Nov 9, 2020

azure-sdk-for-go

️✔️ succeeded [Logs] [Expand Details]

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Nov 9, 2020

azure-sdk-for-js

️✔️ succeeded [Logs] [Expand Details]
  • ️✔️ Generate from 23bcbdb with merge commit f72ef71b8b6e89a9d88f984184e15fc469905696. SDK Automation 13.0.17.20200918.2
  • ️✔️@azure/arm-dns [View full logs]  [Preview SDK Changes]
    [npmPack] npm WARN deprecated [email protected]: This package has been deprecated and is no longer maintained. Please use @rollup/plugin-node-resolve.
    [npmPack] npm WARN deprecated [email protected]: https://github.com/lydell/resolve-url#deprecated
    [npmPack] npm WARN deprecated [email protected]: Please see https://github.com/lydell/urix#deprecated
    [npmPack] npm notice created a lockfile as package-lock.json. You should commit this file.
    [npmPack] loaded rollup.config.js with warnings
    [npmPack] (!) Unused external imports
    [npmPack] default imported from external module 'rollup' but never used
    [npmPack] 
    [npmPack] ./esm/dnsManagementClient.js → ./dist/arm-dns.js...
    [npmPack] created ./dist/arm-dns.js in 295ms
    [npmPack] npm notice 
    [npmPack] npm notice package: @azure/[email protected]
    [npmPack] npm notice === Tarball Contents === 
    [npmPack] npm notice 60.6kB  dist/arm-dns.js                                          
    [npmPack] npm notice 20.7kB  dist/arm-dns.min.js                                      
    [npmPack] npm notice 1.6kB   esm/dnsManagementClient.js                               
    [npmPack] npm notice 2.5kB   esm/dnsManagementClientContext.js                        
    [npmPack] npm notice 2.0kB   esm/operations/dnsResourceReferenceOperations.js         
    [npmPack] npm notice 435B    esm/models/dnsResourceReferenceOperationsMappers.js      
    [npmPack] npm notice 263B    esm/models/index.js                                      
    [npmPack] npm notice 371B    esm/operations/index.js                                  
    [npmPack] npm notice 19.9kB  esm/models/mappers.js                                    
    [npmPack] npm notice 3.3kB   esm/models/parameters.js                                 
    [npmPack] npm notice 11.9kB  esm/operations/recordSets.js                             
    [npmPack] npm notice 507B    esm/models/recordSetsMappers.js                          
    [npmPack] npm notice 889B    rollup.config.js                                         
    [npmPack] npm notice 9.3kB   esm/operations/zones.js                                  
    [npmPack] npm notice 509B    esm/models/zonesMappers.js                               
    [npmPack] npm notice 1.6kB   package.json                                             
    [npmPack] npm notice 457B    tsconfig.json                                            
    [npmPack] npm notice 143.5kB dist/arm-dns.js.map                                      
    [npmPack] npm notice 19.8kB  dist/arm-dns.min.js.map                                  
    [npmPack] npm notice 689B    esm/dnsManagementClient.d.ts.map                         
    [npmPack] npm notice 812B    esm/dnsManagementClient.js.map                           
    [npmPack] npm notice 505B    esm/dnsManagementClientContext.d.ts.map                  
    [npmPack] npm notice 1.4kB   esm/dnsManagementClientContext.js.map                    
    [npmPack] npm notice 810B    esm/operations/dnsResourceReferenceOperations.d.ts.map   
    [npmPack] npm notice 1.1kB   esm/operations/dnsResourceReferenceOperations.js.map     
    [npmPack] npm notice 268B    esm/models/dnsResourceReferenceOperationsMappers.d.ts.map
    [npmPack] npm notice 281B    esm/models/dnsResourceReferenceOperationsMappers.js.map  
    [npmPack] npm notice 9.7kB   esm/models/index.d.ts.map                                
    [npmPack] npm notice 181B    esm/operations/index.d.ts.map                            
    [npmPack] npm notice 126B    esm/models/index.js.map                                  
    [npmPack] npm notice 195B    esm/operations/index.js.map                              
    [npmPack] npm notice 1.2kB   esm/models/mappers.d.ts.map                              
    [npmPack] npm notice 11.1kB  esm/models/mappers.js.map                                
    [npmPack] npm notice 781B    esm/models/parameters.d.ts.map                           
    [npmPack] npm notice 2.5kB   esm/models/parameters.js.map                             
    [npmPack] npm notice 5.5kB   esm/operations/recordSets.d.ts.map                       
    [npmPack] npm notice 7.3kB   esm/operations/recordSets.js.map                         
    [npmPack] npm notice 342B    esm/models/recordSetsMappers.d.ts.map                    
    [npmPack] npm notice 355B    esm/models/recordSetsMappers.js.map                      
    [npmPack] npm notice 3.6kB   esm/operations/zones.d.ts.map                            
    [npmPack] npm notice 5.6kB   esm/operations/zones.js.map                              
    [npmPack] npm notice 339B    esm/models/zonesMappers.d.ts.map                         
    [npmPack] npm notice 352B    esm/models/zonesMappers.js.map                           
    [npmPack] npm notice 3.6kB   README.md                                                
    [npmPack] npm notice 1.1kB   esm/dnsManagementClient.d.ts                             
    [npmPack] npm notice 1.6kB   src/dnsManagementClient.ts                               
    [npmPack] npm notice 863B    esm/dnsManagementClientContext.d.ts                      
    [npmPack] npm notice 2.3kB   src/dnsManagementClientContext.ts                        
    [npmPack] npm notice 1.7kB   esm/operations/dnsResourceReferenceOperations.d.ts       
    [npmPack] npm notice 3.2kB   src/operations/dnsResourceReferenceOperations.ts         
    [npmPack] npm notice 207B    esm/models/dnsResourceReferenceOperationsMappers.d.ts    
    [npmPack] npm notice 381B    src/models/dnsResourceReferenceOperationsMappers.ts      
    [npmPack] npm notice 25.5kB  esm/models/index.d.ts                                    
    [npmPack] npm notice 140B    esm/operations/index.d.ts                                
    [npmPack] npm notice 24.4kB  src/models/index.ts                                      
    [npmPack] npm notice 339B    src/operations/index.ts                                  
    [npmPack] npm notice 1.4kB   esm/models/mappers.d.ts                                  
    [npmPack] npm notice 15.5kB  src/models/mappers.ts                                    
    [npmPack] npm notice 938B    esm/models/parameters.d.ts                               
    [npmPack] npm notice 3.3kB   src/models/parameters.ts                                 
    [npmPack] npm notice 17.8kB  esm/operations/recordSets.d.ts                           
    [npmPack] npm notice 29.8kB  src/operations/recordSets.ts                             
    [npmPack] npm notice 279B    esm/models/recordSetsMappers.d.ts                        
    [npmPack] npm notice 497B    src/models/recordSetsMappers.ts                          
    [npmPack] npm notice 9.4kB   esm/operations/zones.d.ts                                
    [npmPack] npm notice 17.6kB  src/operations/zones.ts                                  
    [npmPack] npm notice 281B    esm/models/zonesMappers.d.ts                             
    [npmPack] npm notice 506B    src/models/zonesMappers.ts                               
    [npmPack] npm notice 1.1kB   LICENSE.txt                                              
    [npmPack] npm notice === Tarball Details === 
    [npmPack] npm notice name:          @azure/arm-dns                          
    [npmPack] npm notice version:       4.0.0                                   
    [npmPack] npm notice filename:      azure-arm-dns-4.0.0.tgz                 
    [npmPack] npm notice package size:  56.3 kB                                 
    [npmPack] npm notice unpacked size: 518.9 kB                                
    [npmPack] npm notice shasum:        0911388b97158ca01f93c86327fc78f62d0f763a
    [npmPack] npm notice integrity:     sha512-qwqMuh5MNxosC[...]I7uz/MEOh+LSg==
    [npmPack] npm notice total files:   69                                      
    [npmPack] npm notice 
  • ️✔️@azure/arm-privatedns [View full logs]  [Preview SDK Changes]
    [npmPack] npm WARN deprecated [email protected]: This package has been deprecated and is no longer maintained. Please use @rollup/plugin-node-resolve.
    [npmPack] npm WARN deprecated [email protected]: https://github.com/lydell/resolve-url#deprecated
    [npmPack] npm WARN deprecated [email protected]: Please see https://github.com/lydell/urix#deprecated
    [npmPack] npm notice created a lockfile as package-lock.json. You should commit this file.
    [npmPack] loaded rollup.config.js with warnings
    [npmPack] (!) Unused external imports
    [npmPack] default imported from external module 'rollup' but never used
    [npmPack] 
    [npmPack] ./esm/privateDnsManagementClient.js → ./dist/arm-privatedns.js...
    [npmPack] created ./dist/arm-privatedns.js in 282ms
    [npmPack] npm notice 
    [npmPack] npm notice package: @azure/[email protected]
    [npmPack] npm notice === Tarball Contents === 
    [npmPack] npm notice 69.3kB  dist/arm-privatedns.js                        
    [npmPack] npm notice 22.6kB  dist/arm-privatedns.min.js                    
    [npmPack] npm notice 263B    esm/models/index.js                           
    [npmPack] npm notice 367B    esm/operations/index.js                       
    [npmPack] npm notice 16.3kB  esm/models/mappers.js                         
    [npmPack] npm notice 3.3kB   esm/models/parameters.js                      
    [npmPack] npm notice 1.7kB   esm/privateDnsManagementClient.js             
    [npmPack] npm notice 2.6kB   esm/privateDnsManagementClientContext.js      
    [npmPack] npm notice 12.4kB  esm/operations/privateZones.js                
    [npmPack] npm notice 549B    esm/models/privateZonesMappers.js             
    [npmPack] npm notice 10.2kB  esm/operations/recordSets.js                  
    [npmPack] npm notice 545B    esm/models/recordSetsMappers.js               
    [npmPack] npm notice 910B    rollup.config.js                              
    [npmPack] npm notice 12.3kB  esm/operations/virtualNetworkLinks.js         
    [npmPack] npm notice 563B    esm/models/virtualNetworkLinksMappers.js      
    [npmPack] npm notice 1.7kB   package.json                                  
    [npmPack] npm notice 457B    tsconfig.json                                 
    [npmPack] npm notice 159.9kB dist/arm-privatedns.js.map                    
    [npmPack] npm notice 20.7kB  dist/arm-privatedns.min.js.map                
    [npmPack] npm notice 11.3kB  esm/models/index.d.ts.map                     
    [npmPack] npm notice 183B    esm/operations/index.d.ts.map                 
    [npmPack] npm notice 126B    esm/models/index.js.map                       
    [npmPack] npm notice 197B    esm/operations/index.js.map                   
    [npmPack] npm notice 1.1kB   esm/models/mappers.d.ts.map                   
    [npmPack] npm notice 9.3kB   esm/models/mappers.js.map                     
    [npmPack] npm notice 781B    esm/models/parameters.d.ts.map                
    [npmPack] npm notice 2.5kB   esm/models/parameters.js.map                  
    [npmPack] npm notice 703B    esm/privateDnsManagementClient.d.ts.map       
    [npmPack] npm notice 826B    esm/privateDnsManagementClient.js.map         
    [npmPack] npm notice 519B    esm/privateDnsManagementClientContext.d.ts.map
    [npmPack] npm notice 1.4kB   esm/privateDnsManagementClientContext.js.map  
    [npmPack] npm notice 3.3kB   esm/operations/privateZones.d.ts.map          
    [npmPack] npm notice 6.2kB   esm/operations/privateZones.js.map            
    [npmPack] npm notice 359B    esm/models/privateZonesMappers.d.ts.map       
    [npmPack] npm notice 372B    esm/models/privateZonesMappers.js.map         
    [npmPack] npm notice 4.7kB   esm/operations/recordSets.d.ts.map            
    [npmPack] npm notice 6.1kB   esm/operations/recordSets.js.map              
    [npmPack] npm notice 355B    esm/models/recordSetsMappers.d.ts.map         
    [npmPack] npm notice 368B    esm/models/recordSetsMappers.js.map           
    [npmPack] npm notice 2.9kB   esm/operations/virtualNetworkLinks.d.ts.map   
    [npmPack] npm notice 5.5kB   esm/operations/virtualNetworkLinks.js.map     
    [npmPack] npm notice 373B    esm/models/virtualNetworkLinksMappers.d.ts.map
    [npmPack] npm notice 386B    esm/models/virtualNetworkLinksMappers.js.map  
    [npmPack] npm notice 3.5kB   README.md                                     
    [npmPack] npm notice 32.5kB  esm/models/index.d.ts                         
    [npmPack] npm notice 136B    esm/operations/index.d.ts                     
    [npmPack] npm notice 31.2kB  src/models/index.ts                           
    [npmPack] npm notice 335B    src/operations/index.ts                       
    [npmPack] npm notice 1.3kB   esm/models/mappers.d.ts                       
    [npmPack] npm notice 13.1kB  src/models/mappers.ts                         
    [npmPack] npm notice 946B    esm/models/parameters.d.ts                    
    [npmPack] npm notice 3.2kB   src/models/parameters.ts                      
    [npmPack] npm notice 1.3kB   esm/privateDnsManagementClient.d.ts           
    [npmPack] npm notice 1.7kB   src/privateDnsManagementClient.ts             
    [npmPack] npm notice 944B    esm/privateDnsManagementClientContext.d.ts    
    [npmPack] npm notice 2.3kB   src/privateDnsManagementClientContext.ts      
    [npmPack] npm notice 9.7kB   esm/operations/privateZones.d.ts              
    [npmPack] npm notice 18.2kB  src/operations/privateZones.ts                
    [npmPack] npm notice 321B    esm/models/privateZonesMappers.d.ts           
    [npmPack] npm notice 539B    src/models/privateZonesMappers.ts             
    [npmPack] npm notice 15.7kB  esm/operations/recordSets.d.ts                
    [npmPack] npm notice 25.6kB  src/operations/recordSets.ts                  
    [npmPack] npm notice 317B    esm/models/recordSetsMappers.d.ts             
    [npmPack] npm notice 537B    src/models/recordSetsMappers.ts               
    [npmPack] npm notice 9.2kB   esm/operations/virtualNetworkLinks.d.ts       
    [npmPack] npm notice 16.6kB  src/operations/virtualNetworkLinks.ts         
    [npmPack] npm notice 335B    esm/models/virtualNetworkLinksMappers.d.ts    
    [npmPack] npm notice 546B    src/models/virtualNetworkLinksMappers.ts      
    [npmPack] npm notice 1.1kB   LICENSE.txt                                   
    [npmPack] npm notice === Tarball Details === 
    [npmPack] npm notice name:          @azure/arm-privatedns                   
    [npmPack] npm notice version:       2.0.0                                   
    [npmPack] npm notice filename:      azure-arm-privatedns-2.0.0.tgz          
    [npmPack] npm notice package size:  60.2 kB                                 
    [npmPack] npm notice unpacked size: 587.4 kB                                
    [npmPack] npm notice shasum:        855eb92acebd9042aa5aec949fd2abe5c45b49c5
    [npmPack] npm notice integrity:     sha512-b77I7s8MkkPn2[...]DHVMTnn33kHNg==
    [npmPack] npm notice total files:   69                                      
    [npmPack] npm notice 

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Nov 9, 2020

azure-sdk-for-net

️✔️ succeeded [Logs] [Expand Details]

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Nov 9, 2020

azure-sdk-for-python

️✔️ succeeded [Logs] [Expand Details]
  • ️✔️ Generate from 23bcbdb with merge commit f72ef71b8b6e89a9d88f984184e15fc469905696. SDK Automation 13.0.17.20200918.2
  • ️✔️azure-mgmt-dns [View full logs]  [Preview SDK Changes]
    [build_conf_init]   Retrying (Retry(total=4, connect=None, read=None, redirect=None, status=None)) after connection broken by 'ConnectTimeoutError(, 'Connection to pypi.python.org timed out. (connect timeout=15)')': /simple/readme-renderer/
    [build_conf_init]   Retrying (Retry(total=4, connect=None, read=None, redirect=None, status=None)) after connection broken by 'ConnectTimeoutError(, 'Connection to pypi.python.org timed out. (connect timeout=15)')': /simple/pyparsing/
    [build_conf] INFO:packaging_tools:Building template azure-mgmt-dns
    [build_conf] INFO:packaging_tools.conf:Skipping default conf since the file exists
    [build_conf] INFO:packaging_tools:Skipping CHANGELOG.md template, since a previous one was found
    [build_conf] INFO:packaging_tools:Template done azure-mgmt-dns
    [build_package] /usr/lib/python3.6/distutils/dist.py:261: UserWarning: Unknown distribution option: 'long_description_content_type'
    [build_package]   warnings.warn(msg)
    [build_package] /usr/lib/python3.6/distutils/dist.py:261: UserWarning: Unknown distribution option: 'long_description_content_type'
    [build_package]   warnings.warn(msg)
    [breaking_change_setup] Ignoring mock: markers 'python_version <= "2.7"' don't match your environment
    [ChangeLog] Size of delta 1.230% size of original (original: 15607 chars, delta: 192 chars)
    [ChangeLog] **Features**
    [ChangeLog] 
    [ChangeLog]   - Model Zone has a new parameter max_number_of_records_per_record_set
  • ️✔️azure-mgmt-privatedns [View full logs]  [Preview SDK Changes]
    [build_conf] INFO:packaging_tools:Building template azure-mgmt-privatedns
    [build_conf] INFO:packaging_tools.conf:Skipping default conf since the file exists
    [build_conf] INFO:packaging_tools:Skipping CHANGELOG.md template, since a previous one was found
    [build_conf] INFO:packaging_tools:Template done azure-mgmt-privatedns
    [build_package] /usr/lib/python3.6/distutils/dist.py:261: UserWarning: Unknown distribution option: 'long_description_content_type'
    [build_package]   warnings.warn(msg)
    [build_package] warning: no files found matching '*.py' under directory 'tests'
    [build_package] warning: no files found matching '*.yaml' under directory 'tests'
    [build_package] /usr/lib/python3.6/distutils/dist.py:261: UserWarning: Unknown distribution option: 'long_description_content_type'
    [build_package]   warnings.warn(msg)
    [build_package] warning: no files found matching '*.py' under directory 'tests'
    [build_package] warning: no files found matching '*.yaml' under directory 'tests'
    [breaking_change_setup] Ignoring mock: markers 'python_version <= "2.7"' don't match your environment
    [breaking_change_setup] Cannot uninstall requirement azure-nspkg, not installed
    [breaking_change_setup] Command '['/z/venv/bin/python', '-m', 'pip', 'uninstall', '-y', 'azure-nspkg']' returned non-zero exit status 1.
    [ChangeLog] Size of delta 0.014% size of original (original: 14755 chars, delta: 2 chars)
    [ChangeLog] 

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Nov 9, 2020

azure-sdk-for-python-track2

No readme.md specification configuration files were found that are associated with the files modified in this pull request, or swagger_to_sdk section in readme.md is not configured

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Nov 9, 2020

azure-resource-manager-schemas

⚠️ warning [Logs] [Expand Details]
  • ⚠️ Generate from 23bcbdb with merge commit f72ef71b8b6e89a9d88f984184e15fc469905696. Schema Automation 13.0.17.20200918.2
    [dns][after_scripts|node] configure: WARNING: secure clearing/zeroing of memory is not supported by the selected crypto backend
    [dns][after_scripts|node] configure: WARNING: secure clearing/zeroing of memory is not supported by the selected crypto backend
    [dns][after_scripts|node] configure: WARNING: secure clearing/zeroing of memory is not supported by the selected crypto backend
    [privatedns][after_scripts|node] configure: WARNING: secure clearing/zeroing of memory is not supported by the selected crypto backend
    [privatedns][after_scripts|node] configure: WARNING: secure clearing/zeroing of memory is not supported by the selected crypto backend
    [privatedns][after_scripts|node] configure: WARNING: secure clearing/zeroing of memory is not supported by the selected crypto backend
    • [after_scripts|node] PostProcessor finished successfully.
      [after_scripts|node]   5739 passing (47s)
      [after_scripts|node]   87 failing
      AzureResourceSchema has no registered package commands.
    • ️✔️privatedns [View full logs]  [Preview Schema Changes]
        [after_scripts|node] PostProcessor finished successfully.
        [after_scripts|node]   5830 passing (1m)
        AzureResourceSchema has no registered package commands.

      @openapi-sdkautomation
      Copy link

      openapi-sdkautomation bot commented Nov 9, 2020

      Trenton Generation

      No readme.md specification configuration files were found that are associated with the files modified in this pull request, or swagger_to_sdk section in readme.md is not configured

      @azure-pipelines
      Copy link

      Azure Pipelines successfully started running 1 pipeline(s).

      @openapi-workflow-bot
      Copy link

      Hi @vladrai, 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 think it is false positive breaking change, please provide the reasons in the PR comment, report to Swagger Tooling Team via https://aka.ma/swaggerfeedback.

      @j5lim
      Copy link
      Contributor

      j5lim commented Nov 16, 2020

      @vladrai Is the added new API 2020-01-01 relevant to s360 fixes? Otherwise, please file a separate PR.

      @j5lim j5lim added the ARMChangesRequested <valid label in PR review process>add this label when require changes after ARM review label Nov 16, 2020
      @vladrai
      Copy link
      Contributor Author

      vladrai commented Nov 16, 2020

      @j5lim API version was used in ARM manifest but did not exist in swagger. So I cloned previous swagger spec with updating API version to mitigate s360.

      @@ -892,6 +892,11 @@
      "format": "int64",
      "description": "The TTL (time-to-live) of the records in the record set."
      },
      "fqdn": {
      "type": "string",
      Copy link
      Contributor

      Choose a reason for hiding this comment

      The reason will be displayed to describe this comment to others. Learn more.

      Adding new properties requires changing the api-version according to Azure API guidline. However, since this is for s360 compliance, it should be acceptable.

      @j5lim j5lim added ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review and removed ARMChangesRequested <valid label in PR review process>add this label when require changes after ARM review WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required labels Nov 16, 2020
      @openapi-workflow-bot
      Copy link

      Hi, @vladrai Thanks for your PR. I am workflow bot for review process. Here are some small tips.

    • Please ensure to do self-check against checklists in first PR comment.
    • PR assignee is the person auto-assigned and responsible for your current PR reviewing and merging.
    • If there is CI failure(s), to fix CI error(s) is mandatory for PR merging; or you need to provide justification in PR comment for explanation. How to fix?

    • Any feedback about review process or workflow bot, pls contact swagger and tools team. [email protected]

      }
      },
      "x-ms-azure-resource": true
      },
      Copy link
      Contributor

      Choose a reason for hiding this comment

      The reason will be displayed to describe this comment to others. Learn more.

      Sorry, I missed from the previous review.
      As of API version 2020-05-01, resources must begin adding the common systemData object property to the top-level property set on read responses (responses for PUT, GET, and PATCH). https://armwiki.azurewebsites.net/api_contracts/guidelines/rpc.html#rpc027-systemdata-support

      You can reference

      Copy link
      Contributor

      Choose a reason for hiding this comment

      The reason will be displayed to describe this comment to others. Learn more.

      Since the api version is already available in public, I think supporting systemData will require a version change. Keeping ARM sign off.

      @j5lim j5lim added ARMChangesRequested <valid label in PR review process>add this label when require changes after ARM review ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review and removed ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review ARMChangesRequested <valid label in PR review process>add this label when require changes after ARM review labels Nov 18, 2020
      @lirenhe lirenhe merged commit b47a321 into Azure:master Nov 18, 2020
      This was referenced Nov 18, 2020
      xiangyan99 added a commit that referenced this pull request Dec 7, 2020
      * CICD Swagger changes for git config APIs (#11528)
      
      * Swagger changes added for workspace & gitintegration specs
      
      	* Updated workspace spec related git repository
      
      	* Added gitintegration spec
      
      	* Updated example files related to workspace
      
      	* Added gitintegration example files
      
      * Remove 'Requires Premium Sku' (#11590)
      
      * Add missing API. (#11548)
      
      * Fix spelling for private link hub path.
      
      * Add missing disableInteractiveQuery API.
      
      * Refactor workspace name to common type.
      
      * Prettier fix.
      
      Co-authored-by: raviarora <[email protected]>
      Co-authored-by: Ravi Arora <[email protected]>
      
      * fix correctness errors for AVS 2019-08-09-preview API (#11545)
      
      * backport Operations_List to AVS 2019-08-09 API
      
      * add deprecated clusterID
      
      * "format": "int32"
      
      * Add Databases APIs to a new PostgreSQL version 2020-11-05-preview (#11432)
      
      * [Hub Generated] Review request for Microsoft.ContainerInstance to add version stable/2020-11-01 (#11449)
      
      * AVS 2020-07-17-preview (#10866)
      
      * feature: Storage Policies contact: David Becher @thinkdavid
       feature: NSX DHCP/Segments & Port Mirroring contacts: Christian Pomales & Weronika Zamlynny & Rebekah Hess
       feature: Credential Rotation, contact: Chenghao Liu @cliu78
       feature: Global Reach, contact: James Spearman @jspearman3
       feature: Private Cloud Addons, contact @leedouglas
      
      * feature: NSX DHCP/Segments & Port Mirroring contacts: Christian Pomales & Weronika Zamlynny & Rebekah Hess
      feature: Credential Rotation, contact: Chenghao Liu @cliu78
      feature: Global Reach, contact: James Spearman @jspearman3
      feature: Private Cloud Addons, contact @leedouglas
      
      * adjusted readme files to reflect new 2020-07-17-preview api
      
      * fixed spelling validation issues
      
      * preview was not specified in path, should fix go sdk validation error
      
      * -preview left of in readme
      
      * typo of preview and spelling  validation fix
      
      * prettier run fixes
      
      * updated swagger because storage policy now will not have the same display around minHosts
      
      * storage policy changed to string
      
      * Update spec
      
      * Add 200 back to nsx patches
      
      * Added 200 back to nsx deletes
      
      * Add provisioningState to nsx models with async operations
      
      * remove arc from api
      
      * remove x-ms-secret for PrivateClouds_ListAdminCredentials
      
      * fix correctness for Operations_list
      
      * pub back x-ms-secret in AdminCredentials
      
      * x-ms-secret on password
      
      * PrivateCloudUpdate can update the resource tags
      
      * Add GETs for gateway and virtual machine resources
      
      * Add virtual machine GET
      
      * Add gateway GET
      
      * minor fixes
      
      * Fixed autorest warnings related to nsx apis
      
      * npx prettier --write specification/vmware/**/*.json
      
      * fixed R2066 name warning
      
      * rename example files as well
      
      Co-authored-by: Christian Pomales <[email protected]>
      Co-authored-by: Cameron Taggart <[email protected]>
      leedouglas/azure-rest-api-specs#5
      
      * different solution to PUT empty body of `{}`
      
      * default for internet is Disabled
      
      * put flatten back
      just need to upgrade pythong autorest to 5.x
      
      * add provisioningState to ManagementCluster
      
      * add readOnly storagePolicy to ManagementCluster
      
      * remove storage policy
      
      * removed all mentions of storage policy
      
      * removed from createor update
      
      * npx prettier --write specification/vmware/**/*.json
      
      Co-authored-by: David Becher <[email protected]>
      Co-authored-by: Cameron Taggart <[email protected]>
      
      * remove examples/DefaultStoragePolicies_List.json
      
      * add Credential Rotation back in
      
      * fully qualified vmware.json
      
      Co-authored-by: David Becher <[email protected]>
      Co-authored-by: Christian Pomales <[email protected]>
      Co-authored-by: david becher <[email protected]>
      Co-authored-by: Cameron Taggart <[email protected]>
      Co-authored-by: Cameron Taggart <[email protected]>
      Co-authored-by: Christian Pomales <[email protected]>
      
      * remove invalid package-2016-08 for RecoveryServices (#11287)
      
      * add notification track2 config (#11439)
      
      * add policyinsights track2 config (#11398)
      
      * Compute: update readme.java.md (#11241)
      
      * [Datadog] Add terms APIs (#11435)
      
      * [Datadog] Add terms APIs
      
      * Feedback
      
      * Feedback
      
      * Update read.cli.md
      
      * Change to PUT
      
      * Feedback
      
      * IoTHub: update readme.java.md (#11244)
      
      * [Storage] File share snapshots CRUD (#11372)
      
      * hybrid-2020-09-01 (#11556)
      
      * hybrid-2020-09-01
      
      * update
      
      * 20201020 regionalmove (#11582)
      
      * regionmove
      
      * resourcemover
      
      * new-typescript-file (#11611)
      
      * [T2]containerservice for track2 python sdk (#11275)
      
      * Added cloud service NIC and PIP APIs specifications (#11568)
      
      * Added cloud service NIC and PIP APIs specifications
      
      * Moved new APIs to 2020-07-01 version
      
      * Updated version to 2020-07-01 in json files
      
      * Updated api version tp 2020-07-01 in examples
      
      Co-authored-by: Richa Jain <[email protected]>
      
      * add action word to pr template (#11623)
      
      * SDK Automation Doc Update (#11422)
      
      * SDK Automation Doc Update
      
      * Use reference file for schema
      
      * Update SwaggerToSdkConfigSchema.json
      
      * Update breakingchangelabel
      
      * Update example
      
      * Add extra envs support
      
      * Add result in generateOutput
      
      * Configurable integrationPR
      
      * Fix description and samples (#11606)
      
      * update samples
      
      * doc link
      
      * apimsi
      
      * Dev containerservice microsoft.container service 2020 11 01 (#11588)
      
      * Adds base for updating Microsoft.ContainerService from version stable/2020-09-01 to version 2020-11-01
      
      * Updates readme
      
      * Updates API version in new specs and examples
      
      * add autoupgradeprofile to 2020-11-01 (#11363)
      
      * add autoupgradeprofile
      
      * add autoupgrader profile to mc
      
      * format change
      
      Co-authored-by: Xiahe Liu <[email protected]>
      
      * add pod subnet id on agentpool (#11310)
      
      * aks: add pod identity profile spec (#11366)
      
      * add private dns zone property in 2020-11-01 (#11311)
      
      Co-authored-by: Li Ma <[email protected]>
      
      * Remove invalid pattern and adjust description (#11312)
      
      * aks: fix pod identity profile field names (#11484)
      
      * aks: add missing `podIdentityProfile.enabled` field
      
      * aks: typo
      
      * add non upgrade channel (#11492)
      
      Co-authored-by: Xiahe Liu <[email protected]>
      
      * Make identityProfile mutable (#11458)
      
      * Add KubeletConfig and LinuxOSConfig specs (#11490)
      
      * aks: update readme tag for 1101 (#11472)
      
      * aks: update readme tag
      
      * aks: add `2020-11-01-only`
      
      * resovle merge conlicts to master, adding recent master change to dev branch (#11604)
      
      Co-authored-by: Xiahe Liu <[email protected]>
      
      Co-authored-by: Xiahe Liu <[email protected]>
      Co-authored-by: Paul Miller <[email protected]>
      Co-authored-by: hbc <[email protected]>
      Co-authored-by: Super <[email protected]>
      Co-authored-by: Li Ma <[email protected]>
      Co-authored-by: Qingchuan Hao <[email protected]>
      Co-authored-by: Tongyao Si <[email protected]>
      Co-authored-by: Bo Wang <[email protected]>
      Co-authored-by: Phoenix He <[email protected]>
      
      * [Synapse] - Update operation id for rename API (#11517)
      
      * [Synapse] - Update operation id for rename API
      
      * Update operation id
      
      Co-authored-by: Dongwei Wang <[email protected]>
      
      * [Text Analytics] rename notstarted to notStarted (#11571)
      
      * rename notstarted to notStarted
      
      * adding rejected
      
      * fix allof
      
      * fix
      
      * fix
      
      * make results required
      
      * name is optional
      
      * Revert: spec changes for Libmary Management in bigDataPool (#11612)
      
      * Revert "attributes added in LibraryInfo in BigDataPool specs (#11415)"
      
      This reverts commit 5d48a7dc324dd221d5e0455e3947c5fc39084132.
      
      * Revert "Custom Library support in Spark pool (#11172)"
      
      This reverts commit c653ee788c4dd74ede34ff17e64e7c0c45681e3e.
      
      Co-authored-by: Navneet Singh <[email protected]>
      
      * fix example (#11631)
      
      Co-authored-by: Ashraf Hamad <[email protected]>
      
      * update (#11641)
      
      * Adding API to support generating access token for ApplicationInsights Profiler (#11393)
      
      * Adds base for updating Microsoft.Insights from version preview/2020-10-05-preview to version 2020-10-26-preview
      
      * Updates readme
      
      * Updates API version in new specs and examples
      
      * Update readme to pointing to profilerToken_API.json
      
      * Update operations list
      
      * Add profiler token getter
      
      * Wire up the defintions
      
      * Fix error: additonal property of liveToken
      
      * Append post action
      
      * Update api-version for example
      
      * Fix some small issues
      
      * Update readme for the resolving autorest check issue
      
      * From profilertoken to profilerToken
      
      * Ran prettier
      
      * Appending back missing readme for 2020-10 tag
      
      * Fix some mistakes
      
      * Use common error response
      
      * Tag secret with x-ms-secret
      
      * Remove list operations
      
      * Clean up packages
      
      * Making 2 post operations for token
      
      * Remove unused operations_list.json example
      
      * Align with official master
      
      * Resolve conflicts
      
      * Add x-ms-secret for the token
      
      * Update error response schema ref
      
      * Update operation ids
      
      * Remove unused error response
      
      * Add SecureScores & SecureScoreControls as new supported datatypes in Automations resource (#11577)
      
      * Add SubAssessment event source type and add more automations examples
      
      * Change Location isReadOnlu to false
      
      * Test
      
      * Add new read/write location and add new type TrackedResourceLocation
      
      * Minor
      
      * minor
      
      * prettier fixes
      
      * Change variable name to TrackedResourceLocation
      
      * Change type name TrackedResourceLocation to AzureTrackedResourceLocation
      
      * Update automations example
      
      * Add SecureScores & SecureScoreControls as new supported datatypes in Automations resource
      
      * add openapi-subtype (#11639)
      
      * #8187884: Swagger Correctness: MICROSOFT.PORTAL(Azure Portal (IbizaFx)) (#11494)
      
      * #8187884: Swagger Correctness: MICROSOFT.PORTAL(Azure Portal (IbizaFx))
      
      * Update examples
      
      * Fix Linter error on integer format
      
      * Fix Prettier
      
      * Fix Linter error
      
      Co-authored-by: Alex Kachanov <[email protected]>
      
      * Revert "Adding API to support generating access token for ApplicationInsights Profiler (#11393)" (#11662)
      
      This reverts commit 80db0e5cef00f0e666c2e1e39d23729eb0c0b0d8.
      
      * Add track2 config to cosmosdb (#10589)
      
      * Fix output-folder (#11645)
      
      * Babylon branding change (#11654)
      
      Rename babylon to purview
      
      * Fix output-folder (#11659)
      
      * [T2] containerservice python automation fix (#11669)
      
      * python automation fix
      
      * Update readme.python.md
      
      * Fix S360 validation issues (#11652)
      
      * FIxing Privatestore ,Offer properties and examples
      
      * Fixing properties types
      
      * Fixing camel cases
      
      * Add Idempotence to custom-words.txt
      
      * prettier check changes
      
      * adding description to isGov
      
      * resolve conflicts
      
      * Fix updateSuppressedDueIdempotence type
      
      * FIx swagger validation issues
      
      * Fix json
      
      * Fix indentation conflicts
      
      * Fix corruption
      
      * Fix validations
      
      * Try renaming preview API version again (#11619)
      
      * Try renaming preview API version again
      
      * Added preview swagger version
      
      * Another try at adding back the preview apis
      
      * Next try at putting -preview API back
      
      * Fixed typo
      
      * Added Java and Typescript generators to resource manager
      
      * Revert typescript.md to original
      
      * Added js and java languages
      
      * Removed SevSnpVm until we have a stronger confidential VM story
      
      * Removed other SevSnpVm vestiges
      
      * update (#11629)
      
      * Fix Swagger Linting issues reported (#11462)
      
      * Fixing linting issues according to report (task 8554751)
      
      * Adding a default response to operations
      
      * Amend error response to indicate that we return additional error info. (#11678)
      
      * Adding Identity to Patch Vault (#11646)
      
      * Create 2020-10-01 version of resources swagger with templateLink.queryString property (#11637)
      
      * Initial commit
      
      * Add templateLink.queryString property into 2020-10 resources version
      
      * Prettier fix
      
      * Fix api-version in examples
      
      * Change description of queryString
      
      * Fixed an undefined tag in readme
      
      * Avocado fixes
      
      * Suppress pre-existing lint errors
      
      * Fix template specs tag
      
      * Revert "Fix template specs tag"
      
      This reverts commit c0a2358b1ff17f1a6e278a79e9a691b4226f1ac4.
      
      * For real this time: Fix template specs tag
      
      Co-authored-by: Filiz Topatan <[email protected]>
      
      * Add WVD Scaling Plan resource definitions (#11224)
      
      Co-authored-by: Jeremy Hurren <[email protected]>
      
      * fix AVS 2020-03-20 correctness errors for empty body and internet default (#11486)
      
      * backport empty body and internet default fixes
      
      * leave flatten on
      
      * hide StorageAccountCredentials put (#11670)
      
      * hide StorageAccountCredentials put
      
      * hide StorageAccounts put
      
      * add traceLevel, continueOnError, and runConcurrently properties to Dataflow activity properties (#11640)
      
      * Fix custom-words.txt (#11696)
      
      * [Synapse] - Fix typo in SqlServer Swagger spec (#11567)
      
      Co-authored-by: Dongwei Wang <[email protected]>
      
      * Removing encryptionprotector APIs as this can land the service in an incosttent state (#11663)
      
      * Update readme.md (#11564)
      
      * Microsoft.OperationalInsights: Updating property name + Adding a new resource (#11649)
      
      * Updating property name.
      
      * example update.
      
      * Adding force cmk to workspace.
      
      * Adding default error response.
      
      * Removing redundant examples.
      
      * adding workspace capping to example per code review request.
      
      * Per review request.
      
      Co-authored-by: Dudi Likvornik <[email protected]>
      
      * Add new api-version for databoxedge (#11694)
      
      * [Hub Generated] Review request for Microsoft.Cache to add version preview/2020-10-01-preview (#11683)
      
      * New Readme Config File
      
      * New Go Language Readme Config File
      
      * New Typescript Language Readme Config File
      
      * New Python Language Readme Config File
      
      * New C# Language Readme Config File
      
      * New Ruby Language Readme Config File
      
      * New Swagger Spec File
      
      * New Swagger Example Spec File
      
      * Initial updates for public preview
      
      * Add default error response. Add integer format.
      
      * Add 200 response to delete operations.
      
      * Add custom words for Redis Enterprise.
      
      * Update whitespace for prettier.
      
      * Update whitespace for prettier.
      
      * Update whitespace for prettier.
      
      * Revert "Update whitespace for prettier."
      
      This reverts commit 4f04b1bfbf35407c783a3249e783ba4b34590ea9.
      
      * Revert "Update whitespace for prettier."
      
      This reverts commit ccca6d5e636f2e3aa38f80c735638e8445a6c42e.
      
      * Revert "Update whitespace for prettier."
      
      This reverts commit 3d72e6bed67117d7a407ba5fd467673f0f12d16b.
      
      * Update whitespace for prettier.
      
      * Update Go settings
      
      * Including .NET SDK in list of SDKs automatically generated.
      
      * Update csharp settings.
      
      * Fix typo.
      
      * Updates addressing comments from reviewer.
      
      * Fix provisioningState errors.
      
      * Update clusteringPolicy description and related examples.
      
      * Remove Enterprise_E1 SKU.
      
      * Updates per comments from reviewer.
      
      * Remove illegal terminal states from provisioningState. Add new resourceState.
      
      * Utilize common properties from updated v2 types.json
      
      * Add 'Deleting' state to ProvisioningState enum.
      
      * Make operationIds of Database operations consistent.
      
      * IoTC 2011 Adding jobs, DTDLv2 templates, groups (#11622)
      
      * Adding jobs, DTDLv2 templates, groups
      
      * Spelling mistakes fix
      
      * Trying to suppress email lint
      
      * Small update to fix linter
      
      * Added additional suppression for long running ops
      
      * Again
      
      * Updated again
      
      * Removing the other suppressors
      
      * New suppression without quotes
      
      * back to just email suppression
      
      * Update to use common type (#11710)
      
      * fix java readme (#11695)
      
      * Fixed some more Swagger issues (#11713)
      
      Co-authored-by: Amogh Natu <[email protected]>
      
      * Microsoft.PolicyInsights/checkPolicyRestrictions API (#10798)
      
      * CheckPolicyRestrictions API
      
      * Run prettier
      
      * Increment package version
      
      * Remove from old tag
      
      * Remove suppression
      
      * Change PartitionOption to object to allow expression (#11613)
      
      * add httpRequestTimeout to odata based connector activities
      
      * add httpRequestTimeout to odata based connector activities
      
      * add sql parallel read properties
      
      * change partitionOption to object
      
      * change ParallelOption to object
      
      * [Monitor] Add DataCollRules section  readme.go.md (#11707)
      
      * 20201113 streamanalytics (#11671)
      
      * sdk-no-flatten
      
      * use unique jobName is examples
      
      * s360 fixes for Azure DNS (#11591)
      
      * s360 fixes for Azure DNS
      
      * Additional property
      
      * Property fix
      
      * Readonly property
      
      * CloudError fix
      
      * Examples. cloud error
      
      * Avocado fix
      
      * read.md
      
      * Prettier
      
      * Threat Intelligence: Update name and descriptions of definitions, parameters (#10949)
      
      * Update API descriptions
      
      * Resolve Comments
      
      * Update parameter name of create/update indicator apis
      
      * Fix example issue
      
      * Fix support swagger correctness violation (#11709)
      
      * updated version, example and schema changes
      
      * fixing examples and updating readme files
      
      * Update support.json by gangan
      
      Removed html tags (kept some <br/> but can remove them as well)
      moved all of the links to md style.
      
      * Update support.json
      
      Added CSP and Sev 1 description
      
      * Updates from swagger json review
      
      * Updates from examples review
      
      * prettier check
      
      * updating custom words and running prettier
      
      * fixing path in readme.go.md
      
      * Reverted change from invalid upstream merfe
      
      * Fix swagger correctness violation for integer type format
      
      * Try to change format to int64 since int32 is flagged as breaking change
      
      * Revert "Try to change format to int64 since int32 is flagged as breaking change"
      
      This reverts commit cb602d9fcec11dee2a5bed30d86f4d329f74d47b.
      
      Co-authored-by: Bhavin Shah <[email protected]>
      Co-authored-by: shahbj79 <[email protected]>
      Co-authored-by: Ganga Narayanan <[email protected]>
      Co-authored-by: Nilay Chheda <[email protected]>
      
      * Add new synapse rbac api spec (#11384)
      
      * Add files via upload
      
      * Fix validation issues
      
      * Fix validation issues
      
      * Fix prettier check issue
      
      * fix validation errors
      
      * Made review fixes
      
      * Run prettier check
      
      * Review fix
      
      Co-authored-by: Dongwei Wang <[email protected]>
      
      * Specifying shipping address required property to autorest. (#11717)
      
      * Ensuring autorest generation follow specific order of required properties in shippingAddress
      
      * added details  sharp yaml
      
      * Removing renameSqlPool api (#11664)
      
      * [ACR] SystemData and new readonly KeyVaultProperties (#11601)
      
      * [Hub Generated] Review request for Microsoft.StoragePool to add version preview/2020-03-15-preview (#11718)
      
      * New Readme Config File
      
      * New Go Language Readme Config File
      
      * New Typescript Language Readme Config File
      
      * New Python Language Readme Config File
      
      * New C# Language Readme Config File
      
      * New Ruby Language Readme Config File
      
      * New Swagger Spec File
      
      * New Swagger Example Spec File
      
      * StoragePool RP swagger definition
      
      * Added missing file
      
      * Added missing title and description
      
      * Prettier fix
      
      * Fixing js code gen issue
      
      Co-authored-by: Harshitha Akkaraju <[email protected]>
      
      * Swagger Correctness for Cognitive Services (#11405)
      
      * Swagger Correctness
      
      * Undo changes in 2016-02-01-preview
      
      * Move location of field
      
      * change enum type to string for Sku.tier
      
      * Microsoft.Capacity 2020-10-01-preview update (#11668)
      
      * Update openapi-authoring-automated-guidelines.md to avoid comfusing (#11208)
      
      * Update openapi-authoring-automated-guidelines.md
      
      * Update openapi-authoring-automated-guidelines.md
      
      * change url of suppressing process
      
      * Reverting the changed made for the address space update in peered vNets. PR: 11090 (#11731)
      
      Co-authored-by: Hari Prasad Perabattula <[email protected]>
      
      * set readonly property of contentStorageAccount to false (#11630)
      
      Co-authored-by: Zhejun Zhao <[email protected]>
      
      * Add Nullable annotation (#11684)
      
      Co-authored-by: Dongwei Wang <[email protected]>
      
      * Revert missing response code. (#11690)
      
      * Revert missing response code.
      
      * Prettier format fix.
      
      Co-authored-by: Ravi Arora <[email protected]>
      
      * Microsoft.OperationalInsights: Property name change. (#11752)
      
      * Microsoft.OperationalInsights: Proerty name change.
      
      * Style fix.
      
      Co-authored-by: Dudi Likvornik <[email protected]>
      
      * [T2] devtestlabs for python track2 (#11724)
      
      * python track2
      
      * python
      
      * Managed Services Swagger Consistency Fixes (#11471)
      
      * Updated 2018-06-01-preview specs
      
      * Updated 2019-04-01-preview specs
      
      * Updated 2018-06-01-preview specs
      
      * Updated 2020-02-01-preview specs
      
      * Updated 2019-06-01 specs
      
      * Updated 2019-09-01 specs
      
      * Updated 2018-06-01-preview examples
      
      * Updated ErrorResponse model
      
      * Model validation fix
      
      * Removed readOnly=true for ErrorResponse
      
      * Updated type specification for error
      
      * Removed old type reference for ErrorResponse
      
      * Changes to avoid Breaking changes for dotnet (#11739)
      
      * Microsoft.Compute: Include CloudServices Resource Swagger under API Version 2020-10-01-preview (#11704)
      
      Co-authored-by: Sanchit Kumar <[email protected]>
      
      * [T2] datalakestore for python track2 (#11722)
      
      * python track2
      
      * python.md
      
      * [Hub Generated] Review request for Microsoft.MixedReality to add version preview/2019-02-28-preview (#11626)
      
      * add isDataAction to listOperations spec
      
      * check if type is wrong
      
      * Type should be boolean
      
      * add identity and kind to the spatial accounts def
      
      * isDataAction should be an enum
      
      * move isDataAction to its own enum
      
      * remove kind from list
      
      * refer to common types instead of defining inline
      
      * update schema to reflect reality
      
      * remove extraneous curly brace
      
      * add curly brace at the correct location
      
      * change nameAvailable to boolean
      
      * add a format for integer type
      
      * change example
      
      * updated wrong file
      
      * remove format per breaking change discussion
      
      * fix prettier error
      
      * update pr assignment (#11750)
      
      * Fix some bad output folder in go SDK (#11781)
      
      * Fix s360 kpis for 2020-03-01. (#11593)
      
      * Fix s360 kpis for 2020-03-01.
      
      * Delete Caches_Delete.json.bak
      
      Remove backup file that was accidentally pushed.
      
      * Delete Caches_Flush.json.bak
      
      Remove backup file that was accidentally pushed.
      
      * Delete Caches_Start.json.bak
      
      Remove backup file that was accidentally pushed.
      
      * Delete Caches_Stop.json.bak
      
      Remove backup file that was accidentally pushed.
      
      * Delete Caches_UpgradeFirmware.json.bak
      
      Remove backup file that was accidentally pushed.
      
      * Delete StorageTargets_Delete.json.bak
      
      Remove backup file that was accidentally pushed.
      
      * Add change directory to reservations spec 2020-10-01-preview (#11555)
      
      * Add change directory to reservations spec 2020-10-01-preview
      
      * refactoring and updating description
      
      * prettier and spellcheck
      
      * Revert "prettier and spellcheck"
      
      This reverts commit b2a9735fe4791b6202083c5d4111a2f588b5b947.
      
      * Revert "refactoring and updating description"
      
      This reverts commit 9dfdd7878629eb83a33070712b302bc1fba94f1e.
      
      * Revert "Add change directory to reservations spec 2020-10-01-preview"
      
      This reverts commit 064f7a9f86a72620d123d69a13c23ef8a61c38b7.
      
      * moving change directory to new api version
      
      * fixing lint error
      
      * update default tag
      
      Co-authored-by: Sean Oh <[email protected]>
      
      * [ACR] New API version 2020-11-01-preview: AZ and On-Prem support (#11412)
      
      * Adding managedBy property that is returned by the API but not in swagger (#11788)
      
      * Updates advisor.Json to improve Documentation for Suppressions_Get operation (#11433)
      
      Co-authored-by: Yen-Chun Chang <[email protected]>
      
      * AssociatedWorkspaces and CapacityReservationProperties added to cluster (#11736)
      
      * AssociatedWorkspaces and CapacityReservationProperties added to cluster. Cluser defualt error bug fixed
      
      * change AssociatedWorkspaces to array
      
      * fix validation errors
      
      * fix validation errors
      
      * missing token added
      
      * read only properties deleted
      
      * read only properties deleted
      
      * LintDIff Errors Fixed
      
      * LACluster added to WorkspaceSku
      
      * update error description
      
      * revert types.json changes
      
      * add prettier
      
      * change default error response to use V2 format
      
      * use V2 types.json file
      
      Co-authored-by: Arie Olshanezky <[email protected]>
      
      * add validate config server(stable) (#10858)
      
      * add validate config server(stable)
      
      * prettier
      
      * typo
      
      * BRP prune backup store API (#11479)
      
      * Add backup external store pruning to the API spec
      
      * Fix CI
      
      * Marketplace miberezi 2020 (#11787)
      
      * FIxing Privatestore ,Offer properties and examples
      
      * Fixing properties types
      
      * Fixing camel cases
      
      * Add Idempotence to custom-words.txt
      
      * prettier check changes
      
      * adding description to isGov
      
      * resolve conflicts
      
      * Fix updateSuppressedDueIdempotence type
      
      * FIx swagger validation issues
      
      * Fix json
      
      * Fix indentation conflicts
      
      * Fix corruption
      
      * Fix validations
      
      * FIx marketplace swagger
      
      * Download files for cm sensors (#11635)
      
      * Added API of DownloadQuotaData to iotDefenderSettings and DownloadResetPassword to iotSensors
      
      * Added reset password api also for the on premise iot sensors
      
      * renamed DownloadQuotaData to DownloadManagerActivation
      
      * Fixed the body in the DownloadResetPassword API
      
      * Azure Healthbot swagger (#11681)
      
      * Azure Healthbot swagger .
      
      * Fixes according to review.
      
      * Fixes according to review.
      
      * Fixes according to review.
      
      * Update samplereadme.md (#11800)
      
      fix language readme path
      
      * Updating Package Manifest Schema in Private Preview API (#11688)
      
      * tried fixing packge infomration flagging (#10870)
      
      * [Hub Generated] Review request for Microsoft.Maps/DEM to add version preview/1.0 (#11410)
      
      * Adds base for updating Microsoft.Maps from version Alias/preview to version 2020-12-01-preview
      
      * Updates readme
      
      * Updates API version in new specs and examples
      
      * Add DEM API reference docs
      
      * Fixing Typos & format
      
      * Fix Typos
      
      * Add dem in render.json
      
      * apply comments
      
      * apply comments
      
      * Apply comments
      
      * Fix validation
      
      * Revert change about version
      
      Co-authored-by: Yuqi Zhang <[email protected]>
      
      * [Hub Generated] Review request for Microsoft.Logic to add version stable/2019-05-01 (#11627)
      
      * Adding integration service environment encryption configuration details to Logic spec
      
      * Correcting json format errors
      
      * Correcting json formats
      
      * Updating examples
      
      * Fixing encryption key properties definition
      
      Co-authored-by: Rama Krishna Rayudu <[email protected]>
      
      * fix powerplatforrm ref & duplicates (#11580)
      
      * Microsoft.ProviderHub GA API version (#11775)
      
      * RPaaS 2020.11.20 GA specs
      
      * update readme
      
      * prettier fix and add custom words
      
      * spelling
      
      * update SDKs
      
      * go sdk
      
      Co-authored-by: Wendy Chang <[email protected]>
      
      * top level name can be 64 char long (#11692)
      
      * Update comment.yml (#11797)
      
      * Update Azure SQLDB 2020-08-01-preview swagger files (#10994)
      
      * Update 2020-08-01 preview swagger files
      and fix lint error
      
      * rollback the sql common type file reference.
      
      * Fix RESPONSE_SCHEMA_NOT_IN_SPEC validation errors in example files
      
      * address part of example files errors.verify the fixes.
      
      * fix LocationCapabilityListByLocation.json
      
      * fix OBJECT_ADDITIONAL_PROPERTIES
      
      * update with most of fix
      
      * BackupShortTermRetentionPolicies fix
      
      * public only
      
      * fix server Key with internal PR
      
      * change resource group name DatabaseVulnerabilityAssessmentScans
      
      * Dev saas microsoft.saa s 2018 03 01 beta (#11680)
      
      * from local
      
      Signed-off-by: Dina Yakobovich <[email protected]>
      
      * add examples
      
      * fix saasDelete
      
      * fix again saasDelete
      
      * change to lowerCase
      
      * second change to lower Case
      
      * fix parameters
      
      * update deleteOptions definition
      
      * add folder
      
      * add new line and cSpell
      
      * typing errors
      
      * change example rsponse
      
      * change response of application example
      
      * add statuses to responses
      
      * change put payload parameters name
      
      * change tag properties
      
      * add readme files
      
      * move x-ms-exaples to the right place
      
      * add defoult response
      
      * fix error responses
      
      * change operation schema
      
      * add 200 to delete response
      
      * fix worning
      
      * readme files
      
      * remove long running
      
      * add space
      
      * spelling errors
      
      * comments fixing
      
      * return operation
      
      * readonly
      
      * add suppression
      
      * [NetAppFiles] Anf 8187 update swagger rest api to 2020 08 01 (#11689)
      
      * Add 2020-08-01 folder
      
      * Update api version
      
      * update readme
      
      * update readme
      
      * issue in readme
      
      * issue in readme
      
      * issue in readme
      
      * touch
      
      Co-authored-by: Audunn Baldvinsson <[email protected]>
      
      * fix typo issue (#11818)
      
      * [Hub Generated] Review request for Microsoft.Maps/Microsoft.Maps to add version preview/1.0 (#11745)
      
      * Adding string type suppport for feature state style rule
      
      * Making the Integer, Boolean and string rule types appear in a strange way. I think this is better than not having them on the doc. Working with the team to figure out a way to do it better.
      
      * Cleaning up the workarounds. Added a link to the reference page.
      
      * Minor change to the text.
      
      * Correctign the stateset description to say that a feature can have only one state set at a given point.
      
      * Prettier changes.
      
      * One more prettier change. By the tool.
      
      Co-authored-by: Subbarayudu Kamma <[email protected]>
      
      * Update pull_request_assignment.yml (#11814)
      
      * correct-package-name (#11666)
      
      * Make DedicatedHostGroup.properties.supportAutomaticPlacement default to false (#11697)
      
      * Removed maximum for DedicatedHostGroup
      
      * Default supportAutomaticPlacement to false
      
      * Updating vmSize description to correct hyperlinks and add a note on deprecating VirtualMachineSizeTypes enum in subsequent api-versions. (#11595)
      
      * Removing vmSizeTypes enum
      
      * Updating description and reverting the enum removal which will be done in a new api-version as a breaking change.
      
      Co-authored-by: Avinash Akka <[email protected]>
      
      * python track2 (#11728)
      
      * Update ci-fix.md (#11828)
      
      * Aggregated swagger changes k8connect (#11817)
      
      * aggregated swagger changes k8conect
      
      * prettier fix
      
      Co-authored-by: Siri Teja Reddy Kasireddy <[email protected]>
      
      * Fix errors in various of readme files encountered in the recent go SDK release (#11826)
      
      * Fix an issue on windows
      
      * Fix cdn
      
      * Fix cognitiveservices/customVision
      
      * Fix cognitiveservices/personalizer
      
      * Fix cognitiveservices/translatorText
      
      * Fix data plane digitaltwins
      
      * Remove problematic tag from release - no impact on sdk, this has never been released
      
      * amend Fix cognitiveservices/translatorText
      
      * Fix imds
      
      * Fix migrate readme that are mistakenly damaged in #10305
      
      * add one api-version back
      
      * Fix digtaltwins data plane again
      
      * Fix hybridnetwork
      
      * Fix streamanlytics
      
      * Revert some changes of digitaltwins
      
      * Revert imds
      
      * Fix migrate
      
      * Fix migrate again
      
      * remove pipeline old un-needed tasks (#11730)
      
      * add privateLinkResources and privateEndpointConnection for 2020-09 API (#11837)
      
      * Updating existing CRR APIs and protection APIs (#10962)
      
      * Fixing bug in PrepareDataMoveResponse Class
      
      * Updating existing CRR and protection APIs
      
      * Adding examples
      
      * Updating APIs to 2020-07-01
      
      * Updating conflict
      
      * Resolving failures
      
      * Update
      
      * Update
      
      * Update
      
      * Update
      
      * Adding workloadCRRAccessToken
      
      * Adding discriminator for CRRAccesToken
      
      * Update
      
      * Reverting change to 2020-07-01
      
      * Update
      
      * Update
      
      * Fixing prettier
      
      * Fix attestation output folder (#11838)
      
      * Combining existing versions in latest version 2020-10-01 (#11733)
      
      * Combining existing versions in latest version 2020-10-01
      
      * Correcting version
      
      * Fixing prettier
      
      * Adding 2018-12-20 in latest package
      
      * ASR: removing blob uri and sas token from examples (#11834)
      
      * removing blob uri and sas token from examplaes
      
      * added place holder value
      
      * Remove api-version in recoveryservicebackup (#11846)
      
      * add deploymentmanager track2 config (#11845)
      
      * [Resources] Add extended locations in the /providers/resourceTypes API in 2020-08-01 version (#10986)
      
      * copy everything from 2020-06-01
      
      * replace 2020-06-01 with 2020-08-01
      
      * add the new API /resourceTypes. add "locationMappings" property.
      
      * add examples
      
      * update readme.md
      
      * run prettier
      
      * fix tag for package-templatespecs
      
      * Add changes to API version 2020-10-01
      
      * [EntityQueries] - Adding EntityQueryKind to EntityQueries and changing Properties to Expansion properties (#11776)
      
      * update swagger - aading LogicAppResourceId to Action put and response
      
      * update swagger - changing to logicAppResourceId style
      
      * adding LogicAppResourceId to Actions Request & Response
      
      * adding LogicAppResourceId to Actions Request & Response
      
      * adding Action property base
      
      * editing examples and adding reauired to logicAppResourceId
      
      * from LogicAppResourceId to logicAppResourceId
      
      * running prettier
      
      * running prettier
      
      * removing required for LogicAppRexourceId
      
      * changing logicApp propery doc
      
      * changing logicApp propery description
      
      * adding kind to entity queries and changing entityQueriesProperties to ExpansionEntityQueriesProperties
      
      * changin return for entityQueryById
      
      * added new rules (#11677)
      
      * Fix the titles of Azure Stack HCI examples (#11853)
      
      * Adding AFDX swagger and its examples (#11655)
      
      * Adding AFDX swagger and its examples
      
      * fix examples and added odata back
      
      * added readme and fixed small sematic problems
      
      * fix log analytics
      
      * updates
      
      * updates
      
      * updates
      
      * updates
      
      * updates
      
      * updates
      
      * updates
      
      * UPDATES
      
      * add secret version back
      
      * lint errors 1
      
      * lint error 2
      
      * lint 3
      
      * lint 4
      
      * updates
      
      * spell check
      
      * add origingroupoverrideaction
      
      * address comments
      
      * fixes
      
      * updates
      
      * remove ruleset update
      
      * updates
      
      Co-authored-by: Shirley Jiang <[email protected]>
      
      * Add some profile tag for go SDK to serve the new profile 2020-09-01 (#10923)
      
      * some update of the tags for profiles
      
      * final update for profiles
      
      * a fix of tag name
      
      * Remove the changes for web readme since it does not work
      
      * Rename tag for compute, revert keyvault tag change
      
      * update
      
      * Adding lint rules: R4028,R4029,R4030 (#11812)
      
      * add R4028
      
      * update doc (#11843)
      
      * update doc
      
      * update description
      
      * format
      
      * add scope
      
      Co-authored-by: jianye xi <[email protected]>
      Co-authored-by: Ruoxuan Wang <[email protected]>
      
      * update examples (#11807)
      
      * adding throttle analytics export API two new filters  (#11806)
      
      * updating with two new filters
      
      * fix typo
      
      * typo
      
      * add timeout support for Synapse notebook and job def (#11860)
      
      * [Synapse] - Add missing API version parameter (#11751)
      
      * [Synapse] - Add missing API version parameter and pageable
      
      * Fix lint
      
      * Fix recent found issues
      
      * Fix lint error
      
      * Fix model validation errors
      
      * Remove x-ms-pageable
      
      * Rename Action to RequiredAction
      
      * Revert x-ms-pageable and AccessDecisions
      
      * Update description for AccessDecisions
      
      * Update checkAccessSynapseRbac.json
      
      Slight wording fix
      
      Co-authored-by: Dongwei Wang <[email protected]>
      Co-authored-by: Jonathan Turner <[email protected]>
      
      * [Synapse] - Update enum name from BlobEventTypes to BlobEventType (#11864)
      
      Co-authored-by: Dongwei Wang <[email protected]>
      
      * Re-order parameters in swagger spec to avoid breaking change in SDK (#11883)
      
      * [Hub Generated] Review request for Microsoft.CostManagement to add version stable/2019-11-01 (#11657)
      
      * Created Separate Swagger for Exports
      
      * add exports swagger to readme.md
      
      * Exports swagger correctness
      
      * Exports swagger correctness
      
      * dataSet in examples
      
      * fix dataSet in examples
      
      * add dataSet to exportList and exportExecutionList examples
      
      * Update ExportCreateOrUpdateByManagementGroup.json
      
      * dataSet property of QueryDefinition costmanagement.json
      
      * undo changes to costmanagement.json
      
      * use ExportDefinition instead of QueryDefinition
      
      * prettier fix
      
      * Include CloudServices resource in API Version '2020-10-01-preview' in ARM Schema (#11874)
      
      Co-authored-by: Sanchit Kumar <[email protected]>
      
      * add maps track2 config (#11867)
      
      * add labservices track2 config (#11866)
      
      * Relay : Removed subscription and entities names from examples (#11891)
      
      * removed subscription and entities names from examples
      
      * example
      
      * Fixing issue 11848 (#11888)
      
      * Fixing issue 11848
      
      * Fixing issue 11848
      
      * Fixing lint errors
      
      * Iot sites and sensor zones (#11296)
      
      * iot-sites-and-zones
      
      * rename displayName and fix missing params
      
      * change caps mistake
      
      * prettier run changes
      
      * Delete Get.json
      
      * Delete Delete.json
      
      * Delete Put.json
      
      * Delete List.json
      
      * remove required tags
      
      * fix merge
      
      Co-authored-by: Gal Moshe <[email protected]>
      Co-authored-by: Gal Moshe <[email protected]>
      
      * Change Hostname per documentation recommendations (#11714)
      
      * Communication common api types (#11749)
      
      * Common API types for Communication Services: Error and Identifier
      
      * Rename type to kind
      
      * Make error, code, message required as specified in guidelines
      
      * Make all props on Identifier read-only
      
      * prettier fix to add an empty line at the end
      
      * add common.json to README
      
      * make identifier id required and fix linter issues
      
      * fix spelling
      
      * make details an array of errors, rename error to CommunicationError
      
      * grammar fix
      
      * remove properties that are identical to id
      
      * add support track2 config (#11894)
      
      * Fix Analysis Services for Swagger Completeness (#11638)
      
      * Fix Analysis Services for Swagger Completeness
      
      * Fix for lintdiff issues
      
      * Change schema to error
      
      * Default schema change
      
      * Adding title and description
      
      * Add separator
      
      * Some changes to address R4018
      
      * Fix parameter
      
      * Move parameter
      
      * Remove a parameter
      
      * Fix quotes bug
      
      * Add required parameters to address R4018
      
      * More updates to Operations API response schema
      
      * Cleaner schema change following the example schema under R4018
      
      * Fix for prettier tool
      
      * fix the output folder for profile 2020-09-01 (#11920)
      
      * Update DomainServices: Add missing property (#11885)
      
      * Update domainservices.json
      
      Add missing migrationProperties property
      Fix casing for ForestTrust
      
      * Update domainservices.json
      
      Add missing migrationProperties property
      Fix casing for ForestTrust
      
      * DomainServices: Update completionPercentage format
      
      * DomainServices: Update completionPercentage format
      
      * DomainServices: Change casing for trustedDomainFqdn and remoteDnsIps
      
      * DomainServices: Change casing for trustedDomainFqdn and remoteDnsIps
      
      DomainServices: Change casing for trustedDomainFqdn and remoteDnsIps
      
      * Microsoft.Authorization 2020-09-01 policy entity APIs (#11712)
      
      * Copy old specs to new path
      
      * Add 2020-09-01 policy entities
      
      * Add description
      
      * Fix prettier
      
      * Fix schema and python
      
      * Remove preview API from stable tag
      
      * Fix go path
      
      * Small amount of swagger validation cleanup
      
      * Fix prettier violation
      
      * Remove obsolete sku property
      
      * Fix GO readme ordering
      
      * Partially revert GO ordering
      
      * add api-version of 2020-10-01 for resources (#11921)
      
      * Defender for iot/8670818 sites and sensors (#11703)
      
      * iot-sites-and-zones
      
      * rename displayName and fix missing params
      
      * change caps mistake
      
      * prettier run changes
      
      * Delete Get.json
      
      * Delete Delete.json
      
      * Delete Put.json
      
      * Delete List.json
      
      * added sensor's properties
      
      * fix specification/security/resource-manager/readme.md
      
      * clean redundant lines in readme
      
      * prettier-fix
      
      * put should return whole object
      
      * indentation fix
      
      * fix enum values
      
      Co-authored-by: Gal Moshe <[email protected]>
      Co-authored-by: galmoshe <[email protected]>
      
      * Fix 2019-06-01-preview/FailoverManagedInstance.json (#11880)
      
      * Change Appearance format to v3.2-preview.2 (#11255)
      
      * v3.1-preview.2
      
      * v3.2-preview change appearance format
      
      * change to solve comments
      
      * typo
      
      * Prettier
      
      * update readme
      
      * update version and solve comment
      
      * change version to v3.2-preview.2
      
      Co-authored-by: Jiayuan Shi <[email protected]>
      
      * Schema Change for New Event Grid Events Emitting from Service Bus (#11935)
      
      * draft for schema changes
      
      * Revert "draft for schema changes"
      
      This reverts commit 04faa0dda5d0c187b9fb485e3b8068e778cfccaf.
      
      * schema change
      
      * modify existing swagger
      
      * Portal revision API (#11738)
      
      * Portal revision API
      
      * fixed spec errors
      
      * fixed references
      
      * fixed return code on create and added long running operation properties
      
      * fixed create response
      
      * fixed comments
      
      * small fix
      
      * [Hub Generated] Review request for Microsoft.CostManagement to add version stable/2020-06-01 (#11753)
      
      * Creating separate swagger file for exports
      
      * add exports swagger to readme
      
      * edit api version
      
      * add "TheLast7Days" to exports timeframes
      
      * remove 'TheLast7Days' from export timeframes
      
      * prettier fix
      
      * prettier fix 2
      
      * Fix for swagger correctness: eTag property missing and recurrence not required
      
      * Update description to indicate force delete is in preview (#11763)
      
      * updating compute.json for force delete
      
      * update description
      
      * update to string
      
      * updating description
      
      * update description
      
      * update comment
      
      * add example
      
      * added example and updated prettier
      
      * updated prettier
      
      * [Hub Generated] Review request for Microsoft.Authorization to add version preview/2020-04-01-preview (#11506)
      
      * [DataFactory]: Add CMK Properties (#10867)
      
      * [DataFactory]: Add CMK Properties
      
      * Prettier validation
      
      * Model Identity Type as string
      
      * RemoveEnumChanges
      
      * Add missing properties in appplatform preview API (#10612)
      
      * add missing properties to 2019-05-01-preview
      
      * update examples
      
      * generate go sdk for authorization (#11942)
      
      * Revert "generate go sdk for authorization (#11942)" (#11944)
      
      This reverts commit c00cdbf22fa96afb7c1ef68aa1f73e9fc52f5ad8.
      
      * fix monitor package name (#11947)
      
      * fix monitor package name
      
      * fix CI
      
      * add storagesync track2 config (#11923)
      
      * Microsoft.key vault dp 7.3 preview (#11464)
      
      * Update Swagger for key rotation
      
      * Update api version
      
      * Fixed build error
      
      * Fixed model validation in keys
      
      * Fixed model validation error in certificates
      
      * Add new version 7.3-preview based on 7.2-preview
      
      * Fixed lint error
      
      * Update api version tag
      
      * Revert changes
      
      * Update keyvault.json (#11040)
      
      * Update keyvault.json
      
      * Update keyvault.json
      
      * Update keyvault.json
      
      * azure-sdk-for-java automation config enable track2 (#11865)
      
      DataPlane not supported
      
      * add attestation track2 config (#11925)
      
      * Update references for OpenAPI (#11859)
      
      * adding the field on which on which summary is done in the response (#11383)
      
      Co-authored-by: Durgesh Kumar <[email protected]>
      
      * Remove the problematic api-version in operationalinsights (#11857)
      
      * add databricks track2 config (#11844)
      
      * Fix problematic inheritance structure in security (#11156)
      
      * Fix problematic inherintance structure
      
      * Add back the missing discriminator
      
      * Fix examples
      
      * rename model
      
      * Update pull_request_assignment.yml (#11974)
      
      * add appplatform track2 config (#11924)
      
      * [Hub Generated] Public private branch 'credscan-fix' (#11939)
      
      * Fix credscan
      
      * Fix more credentials
      
      * [webapp]:Add endpoint for AppService CLI Github token endpoint (#11036)
      
      * Copied over from 2020-06-01
      
      * Changed from 2020-06 to 2020-09
      
      * Added api specs for github auth endpoint
      
      * Copy over Microsoft.CertificateRegistration and Microsoft.DomainRegistration, and small fixes
      
      * Add systemdata
      
      * Add in missing commondefinitions and webapps.json that wasn't copied over from old version
      
      Co-authored-by: Calvin Chan <[email protected]>
      
      Co-authored-by: praveen <[email protected]>
      Co-authored-by: Seth Speaks <[email protected]>
      Co-authored-by: xolve <[email protected]>
      Co-authored-by: raviarora <[email protected]>
      Co-authored-by: Ravi Arora <[email protected]>
      Co-authored-by: Cameron Taggart <[email protected]>
      Co-authored-by: gechris <[email protected]>
      Co-authored-by: worldhello126 <[email protected]>
      Co-authored-by: Douglas Lee <[email protected]>
      Co-authored-by: David Becher <[email protected]>
      Co-authored-by: Christian Pomales <[email protected]>
      Co-authored-by: david becher <[email protected]>
      Co-authored-by: Cameron Taggart <[email protected]>
      Co-authored-by: Christian Pomales <[email protected]>
      Co-authored-by: Kaihui (Kerwin) Sun <[email protected]>
      Co-authored-by: Chuang <[email protected]>
      Co-authored-by: Limin Gu <[email protected]>
      Co-authored-by: Zhenfeng <[email protected]>
      Co-authored-by: colawwj <[email protected]>
      Co-authored-by: changlong-liu <[email protected]>
      Co-authored-by: msyyc <[email protected]>
      Co-authored-by: Richa Jain <[email protected]>
      Co-authored-by: Richa Jain <[email protected]>
      Co-authored-by: Zhenglai Zhang <[email protected]>
      Co-authored-by: Phoenix He <[email protected]>
      Co-authored-by: Samir Solanki <[email protected]>
      Co-authored-by: xhl873 <[email protected]>
      Co-authored-by: Xiahe Liu <[email protected]>
      Co-authored-by: Paul Miller <[email protected]>
      Co-authored-by: hbc <[email protected]>
      Co-authored-by: Super <[email protected]>
      Co-authored-by: Li Ma <[email protected]>
      Co-authored-by: Qingchuan Hao <[email protected]>
      Co-authored-by: Tongyao Si <[email protected]>
      Co-authored-by: Bo Wang <[email protected]>
      Co-authored-by: Dongwei Wang <[email protected]>
      Co-authored-by: Dongwei Wang <[email protected]>
      Co-authored-by: Deyaaeldeen Almahallawi <[email protected]>
      Co-authored-by: Navneet Singh <[email protected]>
      Co-authored-by: Navneet Singh <[email protected]>
      Co-authored-by: Ashraf Hamad <[email protected]>
      Co-authored-by: Ashraf Hamad <[email protected]>
      Co-authored-by: Saar Shen <[email protected]>
      Co-authored-by: surashed <[email protected]>
      Co-authored-by: Sanjai Ganesh Jeenagiri <[email protected]>
      Co-authored-by: AKachan0v <[email protected]>
      Co-authored-by: Alex Kachanov <[email protected]>
      Co-authored-by: kai ru <[email protected]>
      Co-authored-by: Arcturus <[email protected]>
      Co-authored-by: liuyu1981831m <[email protected]>
      Co-authored-by: MiriBerezin <[email protected]>
      Co-authored-by: Larry Osterman <[email protected]>
      Co-authored-by: Ric Serradas <[email protected]>
      Co-authored-by: Jimmy Campbell <[email protected]>
      Co-authored-by: asmaskar <[email protected]>
      Co-authored-by: Gokul Premraj <[email protected]>
      Co-authored-by: Filiz Topatan <[email protected]>
      Co-authored-by: Jeremy Hurren <[email protected]>
      Co-authored-by: Jeremy Hurren <[email protected]>
      Co-authored-by: Adhish Chugh <[email protected]>
      Co-authored-by: Shrecho <[email protected]>
      Co-authored-by: Arik Olsh <[email protected]>
      Co-authored-by: Dudi Likvornik <[email protected]>
      Co-authored-by: Dudi Likvornik <[email protected]>
      Co-authored-by: andrew <[email protected]>
      Co-authored-by: Vivian Liu <[email protected]>
      Co-authored-by: rapatank <[email protected]>
      Co-authored-by: Tanyi Chen <[email protected]>
      Co-authored-by: Amogh Natu <[email protected]>
      Co-authored-by: Amogh Natu <[email protected]>
      Co-authored-by: Chris Eggert <[email protected]>
      Co-authored-by: Zhiyong Yang <[email protected]>
      Co-authored-by: Jorge Herrera <[email protected]>
      Co-authored-by: Vladimir Raikov <[email protected]>
      Co-authored-by: hunngu-ms <[email protected]>
      Co-authored-by: Nilay Chheda <[email protected]>
      Co-authored-by: Bhavin Shah <[email protected]>
      Co-authored-by: shahbj79 <[email protected]>
      Co-authored-by: Ganga Narayanan <[email protected]>
      Co-authored-by: Nilay Chheda <[email protected]>
      Co-authored-by: srkakuru <[email protected]>
      Co-authored-by: Dhawal Jain <[email protected]>
      Co-authored-by: Adithya Upadhya <[email protected]>
      Co-authored-by: Harshitha Akkaraju <[email protected]>
      Co-authored-by: Harshitha Akkaraju <[email protected]>
      Co-authored-by: Yang Yuan <[email protected]>
      Co-authored-by: Ju Hee Lee <[email protected]>
      Co-authored-by: JianyeXi <[email protected]>
      Co-authored-by: Hari Prasad Perabattula <[email protected]>
      Co-authored-by: Hari Prasad Perabattula <[email protected]>
      Co-authored-by: anjou <[email protected]>
      Co-authored-by: Zhejun Zhao <[email protected]>
      Co-authored-by: Saif Kayani <[email protected]>
      Co-authored-by: Sanchit Kumar <[email protected]>
      Co-authored-by: Sanchit Kumar <[email protected]>
      Co-authored-by: roytan-microsoft <[email protected]>
      Co-authored-by: brpanask <[email protected]>
      Co-authored-by: hyo <[email protected]>
      Co-authored-by: Sean Oh <[email protected]>
      Co-authored-by: Jeanine Burke <[email protected]>
      Co-authored-by: yenchunchang <[email protected]>
      Co-authored-by: Yen-Chun Chang <[email protected]>
      Co-authored-by: dhaiman <[email protected]>
      Co-authored-by: Arie Olshanezky <[email protected]>
      Co-authored-by: ninpan-ms <[email protected]>
      Co-authored-by: poetw <[email protected]>
      Co-authored-by: MichalHel <[email protected]>
      Co-authored-by: guy-microsoft <[email protected]>
      Co-authored-by: Tianen <[email protected]>
      Co-authored-by: lavinir <[email protected]>
      Co-authored-by: nivats <[email protected]>
      Co-authored-by: yuqi2 <[email protected]>
      Co-authored-by: Yuqi Zhang <[email protected]>
      Co-authored-by: rarayudu <[email protected]>
      Co-authored-by: Rama Krishna Rayudu <[email protected]>
      Co-authored-by: wwendyc <[email protected]>
      Co-authored-by: Wendy Chang <[email protected]>
      Co-authored-by: datr93 <[email protected]>
      Co-authored-by: Ruoxuan Wang <[email protected]>
      Co-authored-by: Ji Wang <[email protected]>
      Co-authored-by: diyakobo <[email protected]>
      Co-authored-by: Auðunn <[email protected]>
      Co-authored-by: Audunn Baldvinsson <[email protected]>
      Co-authored-by: Subbarayudu Kamma <[email protected]>
      Co-authored-by: Subbarayudu Kamma <[email protected]>
      Co-authored-by: dochung4 <[email protected]>
      Co-authored-by: Avinash <[email protected]>
      Co-authored-by: Avinash Akka <[email protected]>
      Co-authored-by: siriteja <[email protected]>
      Co-authored-by: Siri Teja Reddy Kasireddy <[email protected]>
      Co-authored-by: Pratima Upadhyay <[email protected]>
      Co-authored-by: ayfathim <[email protected]>
      Co-authored-by: Tiano2017 <[email protected]>
      Co-authored-by: ori-licht <[email protected]>
      Co-authored-by: Magdalena Białecka <[email protected]>
      Co-authored-by: Shirley Jiang <[email protected]>
      Co-authored-by: Shirley Jiang <[email protected]>
      Co-authored-by: jianye xi <[email protected]>
      Co-authored-by: Theodore Chang <[email protected]>
      Co-authored-by: Long Tian <[email protected]>
      Co-authored-by: Jonathan Turner <[email protected]>
      Co-authored-by: paornela <[email protected]>
      Co-authored-by: Ajit Navasare <[email protected]>
      Co-authored-by: galmoshe <[email protected]>
      Co-authored-by: Gal Moshe <[email protected]>
      Co-authored-by: Gal Moshe <[email protected]>
      Co-authored-by: John Gallardo <[email protected]>
      Co-authored-by: Dominik <[email protected]>
      Co-authored-by: Suriyha <[email protected]>
      Co-authored-by: jihochang <[email protected]>
      Co-authored-by: Ami Hollander <[email protected]>
      Co-authored-by: Ji Wang <[email protected]>
      Co-authored-by: shijiayuan <[email protected]>
      Co-authored-by: Jiayuan Shi <[email protected]>
      Co-authored-by: DorothySun216 <[email protected]>
      Co-authored-by: Igor O <[email protected]>
      Co-authored-by: Aishwarya Seetharaman <[email protected]>
      Co-authored-by: Daniel Orozco <[email protected]>
      Co-authored-by: Pacodel <[email protected]>
      Co-authored-by: Yuwei Zhou <[email protected]>
      Co-authored-by: njucz <[email protected]>
      Co-authored-by: qinl-li <[email protected]>
      Co-authored-by: Jack Lichwa <[email protected]>
      Co-authored-by: dukum <[email protected]>
      Co-authored-by: Durgesh Kumar <[email protected]>
      Co-authored-by: Calvin <[email protected]>
      Co-authored-by: Calvin Chan <[email protected]>
      giromm2ms pushed a commit to giromm2ms/azure-rest-api-specs that referenced this pull request Dec 20, 2020
      * s360 fixes for Azure DNS
      
      * Additional property
      
      * Property fix
      
      * Readonly property
      
      * CloudError fix
      
      * Examples. cloud error
      
      * Avocado fix
      
      * read.md
      
      * Prettier
      Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
      Labels
      ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review CI-BreakingChange-Go
      Projects
      None yet
      Development

      Successfully merging this pull request may close these issues.

      3 participants