-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
DPDTimeout Multiple Radius for VPN Gateways (Network) #8707
Conversation
#8697 Was told to create PR targeting this branch |
Can one of the admins verify this patch? |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
azure-sdk-for-java
|
azure-sdk-for-js
|
azure-sdk-for-go
|
@number213 Can I get a review of this PR please? |
@MikhailTryakhov / @v-shhatt Please review this PR, thanks! |
Azure Pipelines successfully started running 1 pipeline(s). |
...esource-manager/Microsoft.Network/stable/2020-03-01/examples/VpnServerConfigurationList.json
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1 previous comment still open and 1 minor new comment
Azure Pipelines successfully started running 1 pipeline(s). |
@pilor Comments addressed in latest commits |
Azure Pipelines successfully started running 1 pipeline(s). |
@henry416 please resolve CI errors |
Azure Pipelines successfully started running 1 pipeline(s). |
@number213 I have fixed the CI errors in examples jsons. The remaining checks don't display any actionable error that I can find. My local autorest validation doesn't show any error for the changes I made. |
@number213 could you please advise on what the CI errors are? I'm a bit stuck. The pipelines don't show a descriptive error, and autorest/linter validations don't show anything off. |
@henry416 PrettierCheck is throwing errors due to JSON files not being prettified correctly
|
Azure Pipelines successfully started running 1 pipeline(s). |
@number213 I have fixed the prettier output. Is there any way I can run the prettier check on files in local? I cannot find documentation in the repo on how to run it. |
@number213 Ping for visibility |
@zikalino @akning-ms could you please merge this PR? Breaking change job can be ignored since this is a PR for Network's dev branch @henry416 you can run prettier locally with |
* Adds base for updating Microsoft.Network from version stable/2019-12-01 to version 2020-03-01 * Updates readme * Updates API version in new specs and examples * Move ExpressRouteGateway definitions inside of virtualWAN.json to fix… (#8708) * Move ExpressRouteGateway definitions inside of virtualWAN.json to fix documentation issues * Fix readme file Co-authored-by: Vinaya Damle <vinayada@microsoft.com> * Adding SecurityPartnerProvider Resource (#8721) * Adding securityPartnerProvider resource * Adding securityPartnerProvider resource : 2 * Added Definition for SecurityPartnerProviderListResult * Updated Readme.md with securityPartnerProvider * fixing zone.. * Fixing using Prettier * desc fix * Fixing enum conflict * RequiredPropertiesMissingInResourceModel Fix * capitalizing alphabets * renaming list/listall * Updated Tags to Caps Co-authored-by: Ishani Gupta <isgu@microsoft.com> * AzureFirewall User Tags Update (#8748) * AzureFirewall User Tags Update * Prettier Fixes Co-authored-by: Ishani Gupta <isgu@microsoft.com> * Add IpAllocation resource for Hypernet IP management. (#8701) * Add IpAllocation resource for Hypernet IP management. * Fix path issue. * Fix some issues by PR comments. * Fix document issue. * Fix reference with SubResource. * Fix IpAllocation tags name issue. * Add patch operation to IpAllocation resource. * Fix spell issue with pretter. * Introducing force policy association feature (#8738) * Adding privateEndpoint child resource privateDNSZoneGroup and adding customDNSconfig JSON to privateEndpoint. (#8704) * Adding privateEndpoint child resource privateDNSZoneGroup, as well as adding customDNSconfig JSONto privateendpoint * Adding endzone keyword for build * Adding reference to 2020 private endpoint in readme for validation purposes * Misspelled reference to Get file fixed * Ran prettier fix on files * Removing reference in readme * Added some needed capitilization and punctuation, will be a pt2 to this commit * Added descriptions * Rerunning tests. * Updated definitions for customdnsconfig and record set * Rebuilding for pr * Changing word back to original sentence for PR. * Changed name in example file for create, as well as added List operation and fixed pr comments * resolved comments based on pr * Added changes based on PRcomments * Added integer tag to ttl * getting rid of type in private endpoint json * Changed type attribute in privateendpointdnszonegroupget * Added read only attribute to record set array Co-authored-by: Shane Baden <shbaden@microsoft.com> * Adding intrusionSystemMode property to firewallPolicy (#8885) * Add intrusionSystemMode to firewallPolicy * Change intrusionSystemMode to be either Enabled or Disabled * fix pretier * remove white space * remove the examples as this is a preview feature Co-authored-by: Shany Pozin <shanyp@microsoft.com> * DPDTimeout Multiple Radius for VPN Gateways (Network) (#8707) * DPDTimeout Multiple Radius changes * PR feedback, resource changes * PR feedback, example changes * examples fix * PR feedback * CI fix * prettier fix attempt Co-authored-by: Mikhail <mitryakh@microsoft.com> Co-authored-by: vinayada1 <vinayadamle@gmail.com> Co-authored-by: Vinaya Damle <vinayada@microsoft.com> Co-authored-by: Ishani Gupta <ishani.gupta27.ig@gmail.com> Co-authored-by: Ishani Gupta <isgu@microsoft.com> Co-authored-by: liuh-80 <58683130+liuh-80@users.noreply.github.com> Co-authored-by: Venkata Krishnan Anantha Raman <venkatakrishnansvpr@gmail.com> Co-authored-by: Shane Baden <naruto.shane@gmail.com> Co-authored-by: Shane Baden <shbaden@microsoft.com> Co-authored-by: Shany Pozin <shanyp@gmail.com> Co-authored-by: Shany Pozin <shanyp@microsoft.com> Co-authored-by: Henry Chen <henche@microsoft.com>
* Adds base for updating Microsoft.Network from version stable/2019-12-01 to version 2020-03-01 * Updates readme * Updates API version in new specs and examples * Move ExpressRouteGateway definitions inside of virtualWAN.json to fix… (Azure#8708) * Move ExpressRouteGateway definitions inside of virtualWAN.json to fix documentation issues * Fix readme file Co-authored-by: Vinaya Damle <vinayada@microsoft.com> * Adding SecurityPartnerProvider Resource (Azure#8721) * Adding securityPartnerProvider resource * Adding securityPartnerProvider resource : 2 * Added Definition for SecurityPartnerProviderListResult * Updated Readme.md with securityPartnerProvider * fixing zone.. * Fixing using Prettier * desc fix * Fixing enum conflict * RequiredPropertiesMissingInResourceModel Fix * capitalizing alphabets * renaming list/listall * Updated Tags to Caps Co-authored-by: Ishani Gupta <isgu@microsoft.com> * AzureFirewall User Tags Update (Azure#8748) * AzureFirewall User Tags Update * Prettier Fixes Co-authored-by: Ishani Gupta <isgu@microsoft.com> * Add IpAllocation resource for Hypernet IP management. (Azure#8701) * Add IpAllocation resource for Hypernet IP management. * Fix path issue. * Fix some issues by PR comments. * Fix document issue. * Fix reference with SubResource. * Fix IpAllocation tags name issue. * Add patch operation to IpAllocation resource. * Fix spell issue with pretter. * Introducing force policy association feature (Azure#8738) * Adding privateEndpoint child resource privateDNSZoneGroup and adding customDNSconfig JSON to privateEndpoint. (Azure#8704) * Adding privateEndpoint child resource privateDNSZoneGroup, as well as adding customDNSconfig JSONto privateendpoint * Adding endzone keyword for build * Adding reference to 2020 private endpoint in readme for validation purposes * Misspelled reference to Get file fixed * Ran prettier fix on files * Removing reference in readme * Added some needed capitilization and punctuation, will be a pt2 to this commit * Added descriptions * Rerunning tests. * Updated definitions for customdnsconfig and record set * Rebuilding for pr * Changing word back to original sentence for PR. * Changed name in example file for create, as well as added List operation and fixed pr comments * resolved comments based on pr * Added changes based on PRcomments * Added integer tag to ttl * getting rid of type in private endpoint json * Changed type attribute in privateendpointdnszonegroupget * Added read only attribute to record set array Co-authored-by: Shane Baden <shbaden@microsoft.com> * Adding intrusionSystemMode property to firewallPolicy (Azure#8885) * Add intrusionSystemMode to firewallPolicy * Change intrusionSystemMode to be either Enabled or Disabled * fix pretier * remove white space * remove the examples as this is a preview feature Co-authored-by: Shany Pozin <shanyp@microsoft.com> * DPDTimeout Multiple Radius for VPN Gateways (Network) (Azure#8707) * DPDTimeout Multiple Radius changes * PR feedback, resource changes * PR feedback, example changes * examples fix * PR feedback * CI fix * prettier fix attempt Co-authored-by: Mikhail <mitryakh@microsoft.com> Co-authored-by: vinayada1 <vinayadamle@gmail.com> Co-authored-by: Vinaya Damle <vinayada@microsoft.com> Co-authored-by: Ishani Gupta <ishani.gupta27.ig@gmail.com> Co-authored-by: Ishani Gupta <isgu@microsoft.com> Co-authored-by: liuh-80 <58683130+liuh-80@users.noreply.github.com> Co-authored-by: Venkata Krishnan Anantha Raman <venkatakrishnansvpr@gmail.com> Co-authored-by: Shane Baden <naruto.shane@gmail.com> Co-authored-by: Shane Baden <shbaden@microsoft.com> Co-authored-by: Shany Pozin <shanyp@gmail.com> Co-authored-by: Shany Pozin <shanyp@microsoft.com> Co-authored-by: Henry Chen <henche@microsoft.com>
Adding DPDTimeout and Multiple Radius Servers for VPN Gateways
DPDTimeout is an integer field in connection resources.
Multiple Radius is a subobject containing radius sever address/score/secret in gateway resources.
Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub - one location for using our validation tools and finding your workflow.
Contribution checklist:
ARM API Review Checklist
Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.
Please follow the link to find more details on API review process.