-
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
Add IpAllocation resource for Hypernet IP management. #8701
Conversation
Change reviewed in private repo, here is link: https://github.com/Azure/azure-rest-api-specs-pr/pull/1121 |
Azure Pipelines successfully started running 1 pipeline(s). |
azure-sdk-for-go
|
azure-sdk-for-net
|
azure-sdk-for-java
|
azure-sdk-for-js
|
Can one of the admins verify this patch? |
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.
LGTM except a few minor comments
specification/network/resource-manager/Microsoft.Network/stable/2020-03-01/ipAllocation.json
Outdated
Show resolved
Hide resolved
specification/network/resource-manager/Microsoft.Network/stable/2020-03-01/ipAllocation.json
Outdated
Show resolved
Hide resolved
specification/network/resource-manager/Microsoft.Network/stable/2020-03-01/ipAllocation.json
Outdated
Show resolved
Hide resolved
Azure Pipelines successfully started running 1 pipeline(s). |
@raych1 Shall we dismiss breaking change in this case? |
This PR will add new property to VirtualNetwork and Subnet resource, this is by design. |
@NullMDR , we synced up on this and pls label this PR instead. |
specification/network/resource-manager/Microsoft.Network/stable/2020-03-01/ipAllocation.json
Show resolved
Hide resolved
specification/network/resource-manager/Microsoft.Network/stable/2020-03-01/ipAllocation.json
Show resolved
Hide resolved
specification/network/resource-manager/Microsoft.Network/stable/2020-03-01/ipAllocation.json
Show resolved
Hide resolved
specification/network/resource-manager/Microsoft.Network/stable/2020-03-01/ipAllocation.json
Show resolved
Hide resolved
specification/network/resource-manager/Microsoft.Network/stable/2020-03-01/ipAllocation.json
Show resolved
Hide resolved
specification/network/resource-manager/Microsoft.Network/stable/2020-03-01/virtualNetwork.json
Outdated
Show resolved
Hide resolved
specification/network/resource-manager/Microsoft.Network/stable/2020-03-01/virtualNetwork.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.
Comments above
Azure Pipelines successfully started running 1 pipeline(s). |
Azure Pipelines successfully started running 1 pipeline(s). |
Azure Pipelines successfully started running 1 pipeline(s). |
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 unresolved issue
Azure Pipelines successfully started running 1 pipeline(s). |
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.
Azure Pipelines successfully started running 1 pipeline(s). |
@NullMDR , could you please help to merge this PR? because this is a break change so the 'public.rest-api-specs (BreakingChange) ' UT can't pass. |
* 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>
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.