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

Dev/dexiang/byoip #9818

Merged
merged 56 commits into from
Jul 14, 2020
Merged

Dev/dexiang/byoip #9818

merged 56 commits into from
Jul 14, 2020

Conversation

dexiang-ms
Copy link
Contributor

@dexiang-ms dexiang-ms commented Jun 11, 2020

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

Contribution checklist:

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

ARM API Review Checklist

  • Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
    • adding/removing APIs.
    • adding/removing properties.
    • adding/removing API-version.
    • adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

  • If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
    Please follow the link to find more details on API review process.

anton-evseev and others added 30 commits May 6, 2020 17:36
…ualApplianceSites child resources (#9357)

* Adds base for updating Microsoft.Network from version stable/2020-03-01 to version 2020-05-01

* Updates readme

* Updates API version in new specs and examples

* Final changes

* Fix formatting issues

* Fix merge and formatting issues

* Fix formatiing

* Ipconfig properties

* Fix build errors

* Lint check

* Fix semantics errors

* Address review comments

* Fix prettier and nextLink checks

* Fix dots

* Address errors

* Fix Ipconfig version

Co-authored-by: dexiang-ms <64503627+dexiang-ms@users.noreply.github.com>
* Adding DnsProxySettings Support in Firewall Policy

dnsServers, dnsEnableProxy, dnsRequireProxyForNetworkRules are added as subproperties.

* spell check

* Removing DNS as prefix on sub properties.

* Syntax error.

* DnsProxySettings to DNSSettings

* Fixing proxy settings name.

Co-authored-by: Ishani Gupta <isgu@microsoft.com>
* fix network

* duplicated publicipprefix

* fixed example in 2018-01-01

* final updates

* missing examples

(cherry picked from commit 42d9639)
* Added new properties to ConnectivityHop

* Fixed spelling

* added x-ms-client-flatten to HopLink.properties.properties
#9356)

* Rename Firewall Policy child resource RuleGroup to RuleCollectionGroup

* rebase with ti whitelist

* rebase with ti whitelist

* rebase with ti whitelist

* prettier run

* making nat rule into a collection

* prettier run
* multi pip for firewall in hub swagger change

* resolving issues
1. Exposed direct PUT/DELETE APIs for hubvnet connections. This is the
   API expected to be used in the future by all consumers (direct rest
   api calls, powershell, portal, azure cli). This avoids needing a put
   on the virtual hub every time a hubvnet connection is to be
   added/changed.
2. Deprecate specifying hubvnet connection in virtualHub PUT. This is to
   avoid having multiple APIs to do the same thing. Also having this API
   introduces other complexities and breaks some scenarios.
3. Deprecate allowHubToRemoteVnetTransit and
   allowRemoteVnetToUseHubVnetGateways flags in hubvnet connection.
   These flags are maintained internally in the implementation and
   shouldn't be exposed to the user. For now, just mark it as such. Will
   be removed in later release.

Co-authored-by: Shyam Dharmarajan <shdharma@microsoft.com>
* v1

* making boolean as nullable

Co-authored-by: Ishani Gupta <isgu@microsoft.com>
* Add Routing State and expose Effective Routes Api

* updated opId name and added examples

* prettier fix done

* rename variable to prevent conflict
* Updating VpnGateway examples

* prettier

Co-authored-by: Khushboo Baheti <khbaheti@microsoft.com>
* Application Gateway Private Link Support

* Fix comments

* Fix comments for including example
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@dexiang-ms
Copy link
Contributor Author

@ravbhatnagar the changes have been applied, please let me know if there are any other requests

@dexiang-ms dexiang-ms requested a review from ravbhatnagar June 25, 2020 19:55
},
"geography": {
"type": "string",
"description": "Geography of edge routers this CIDR can be reflected on. Related to geographies covered by the 5 International RIRs (ARIN/RIPE/APNIC/LACNIC/AFRINIC.",

Choose a reason for hiding this comment

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

The description got cut off - there is no closing parenthesis.

"validationState": {
"readOnly": true,
"type": "string",
"description": "The state of the ARIN / ROA authentication for master custom IP prefix resource.",

Choose a reason for hiding this comment

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

Besides ARIN maybe we should add the other RIRs (RIPE / APNIC / ...). How about this:
"The validation state of Route Origin Authorization for the corresponding RIR (ARIN/RIPE/APNIC/LACNIC/AFRINIC)."

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@dexiang-ms
Copy link
Contributor Author

@Tiano2017 @rajshah11 @vivsriaus please review at your earliest convenience

@dexiang-ms
Copy link
Contributor Author

Hi - please don't merge just yet - there may be one last change coming down the pipeline

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@majastrz majastrz 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 Jul 9, 2020
Copy link
Member

@majastrz majastrz left a comment

Choose a reason for hiding this comment

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

Signed off from ARM side.

@dexiang-ms dexiang-ms requested a review from lirenhe July 10, 2020 19:37
@lirenhe lirenhe merged commit 8fe8bc6 into network-june-release Jul 14, 2020
akning-ms pushed a commit that referenced this pull request Aug 12, 2020
* Adds base for updating Microsoft.Network from version stable/2020-05-01 to version 2020-06-01

* Updates readme

* Updates API version in new specs and examples

* New vWAN resource properties (#9722)

* retarget

* fixed duplicate

* Added code changes for generating letter of authorization (#9829)

Co-authored-by: Umesh Kanodia <umkanodi@microsoft.com>

* Async PATCH operations for Virtual WAN VpnGateways and P2SVpnGateways (#9833)

* Add remoteBgpCommunities to VirtualNetworkPeering resource (#9814)

* Add remoteBgpCommunities to VirtualNetworkPeering resource

* Change VirtualNetworkCommunity in example VirtualNetworkCreateWithBgpCommunities.json to a value that is within the allowed range.

* Application Gateway Mutual Authentication Support (#9965)

* Adding mutual auth api definitions.

* Adding examples.

* Fixing case.

Co-authored-by: Hemant Imudianda <heimudia@microsoft.com>

* Added PATCH flowLog API and example (#9960)

* Added PATCH flowLog API and example

* Fixed example

* Added property to TCPConfiguration

* Fixed prettier warning

* Fixes based on CR

* Add new optional param enableInternetSecurity and new Reset API for P2SVpnGateway resource (#9952)

* Add new optional param enableInternetSecurity and new Reset API for P2SVpnGateway resource

* Fix validation tool fixes

* Remove extra spaces

* Update Network's owners (#10006)

* Add intrusion system configuration (#9912)

* Add intrusion system configuration

* Add ignored traffic configuration

* Fix checkPirvateLinkServiceVisibility in 2020-06-01 (#9821)

* fix checkPirvateLinkServiceVisibility in 2020-06-01

* update examples

* add VpnGatewaypacketCapture api  (#9832)

* add VpnGatewaypacketCapture api

* add custom words

* add vpn connection packet capture

* fix naming errors

* fix error

* fix word

* fixing validation

* fix model error

* finish prettier

* fix test

* expressRouteProviderPort changes (#9909)

* expressRouteProviderPort changes

* fix ref error

* suppressing name id type error

* prettier fix

* locationlist -> listbylocation

* adding filter on location

* ref listbylocation

* fixing id and name

* adding get and bug fixes

* bug fix

* bug fix

* adding proxy resource ref

* fixes for auto checks

* adding proxyresource type to network.json

* fixing description

* fixing description for proxy resource

* Adding swagger updates to support the new NRP resource "DSCP Configuration" (#9954)

* Adding the swagger for the new resource DSCP Configuration. Also updated the Network Interface to return it with its get function

* Updated the definition to the updated version in the source code

* Fix some errors being thrown by the validator (regarding int/ushort typing). Updated the readme.md to include the dscpConfiguration file

* Fixed the erroneous reference within the dscpConfiguration.json to the NetworkInterface definition

* Removed the long running operation flag.
Updated the examples to return two markings

* Added more objets within the lists.

* Removed the duplicated key

* The type 'ushort ' is not accepted. Turn it into an integer. Hopefully this fixes the issue regarding the QosPortRange properties

* Added the custome words dscp and qos in order to fix the error of the words being unkown. Refactor the allocatedNetworkInterfaces to associatedNetworkInterfaces to match the examples with the specification. Removed the QosTrafficProfiler from the examples as it no longer exists. Made sure the delete is using dscpName rather than nicName. Added the period at the end of the description.

* One traffic profiler slipped through the cracks, removed it. Properly formatted the object for the Qos Port/IP ranges. Was placing a string rather than an integer in the get example.

* Well, I was being silly here. Had a typo hahahahaha

* Added the warning suppression to the readme file. Run the prettier on the specification

* Thought I only had to run the prettier on the specification. Did the same for the examples and the NetworkInterfaceGet.json.

* Re-run the command with the full command placed in the documentation. Hopefully this fixes the prettier errors

* Corrected the typo

* Added the proper formatting for the suppression.

* Added the integer format

* Added the api calls for list and list all.
Made sure the name is plural
Added the examples for the new api calls
In Network Interface, now it is using the ID of the DSCP configuration rather than the full payload

* Forgot to switch the operation id from network interface to dscp configuration

* Added the missing next link for the list by resource

Co-authored-by: David Velasco <davelasc@microsoft.com>
Co-authored-by: MyronFanQiu <49134743+MyronFanQiu@users.noreply.github.com>

* Dev/dexiang/byoip (#9818)

* Adds base for updating Microsoft.Network from version stable/2020-04-01 to version 2020-05-01

* Updates readme

* Updates API version in new specs and examples

* Add VirtualHubBgpConnections, VirtualHubIpConfigurations, NetworkVirtualApplianceSites child resources (#9357)

* Adds base for updating Microsoft.Network from version stable/2020-03-01 to version 2020-05-01

* Updates readme

* Updates API version in new specs and examples

* Final changes

* Fix formatting issues

* Fix merge and formatting issues

* Fix formatiing

* Ipconfig properties

* Fix build errors

* Lint check

* Fix semantics errors

* Address review comments

* Fix prettier and nextLink checks

* Fix dots

* Address errors

* Fix Ipconfig version

Co-authored-by: dexiang-ms <64503627+dexiang-ms@users.noreply.github.com>

* Change NRP repo for validation scripts (#9394)

* Move terminateTLS flag to ApplicationRuleCondition (#9539)

* DNS Proxy support for Firewall Policy (#9461)

* Adding DnsProxySettings Support in Firewall Policy

dnsServers, dnsEnableProxy, dnsRequireProxyForNetworkRules are added as subproperties.

* spell check

* Removing DNS as prefix on sub properties.

* Syntax error.

* DnsProxySettings to DNSSettings

* Fixing proxy settings name.

Co-authored-by: Ishani Gupta <isgu@microsoft.com>

* fix network (#9369) (#9547)

* fix network

* duplicated publicipprefix

* fixed example in 2018-01-01

* final updates

* missing examples

(cherry picked from commit 42d9639)

* Added new properties to ConnectivityHop (#9386)

* Added new properties to ConnectivityHop

* Fixed spelling

* added x-ms-client-flatten to HopLink.properties.properties

* Rename Firewall Policy child resource RuleGroup to RuleCollectionGroup (#9356)

* Rename Firewall Policy child resource RuleGroup to RuleCollectionGroup

* rebase with ti whitelist

* rebase with ti whitelist

* rebase with ti whitelist

* prettier run

* making nat rule into a collection

* prettier run

* Multiple Public IPs support for Azure Firewall in Virtual Hub (#9560)

* multi pip for firewall in hub swagger change

* resolving issues

* VirtualWan: Hubvnet Connection related changes (#9536)

1. Exposed direct PUT/DELETE APIs for hubvnet connections. This is the
   API expected to be used in the future by all consumers (direct rest
   api calls, powershell, portal, azure cli). This avoids needing a put
   on the virtual hub every time a hubvnet connection is to be
   added/changed.
2. Deprecate specifying hubvnet connection in virtualHub PUT. This is to
   avoid having multiple APIs to do the same thing. Also having this API
   introduces other complexities and breaks some scenarios.
3. Deprecate allowHubToRemoteVnetTransit and
   allowRemoteVnetToUseHubVnetGateways flags in hubvnet connection.
   These flags are maintained internally in the implementation and
   shouldn't be exposed to the user. For now, just mark it as such. Will
   be removed in later release.

Co-authored-by: Shyam Dharmarajan <shdharma@microsoft.com>

* Destination FQDNs in FirewallPolicy for network rules. (#9609)

* v1

* making boolean as nullable

Co-authored-by: Ishani Gupta <isgu@microsoft.com>

* VirtualHub : Add Routing State and expose Effective Routes Api (#9385)

* Add Routing State and expose Effective Routes Api

* updated opId name and added examples

* prettier fix done

* rename variable to prevent conflict

* Updating VpnGateway examples for bgpSettings (#9656)

* Updating VpnGateway examples

* prettier

Co-authored-by: Khushboo Baheti <khbaheti@microsoft.com>

* Application Gateway Private Link Support (#9490)

* Application Gateway Private Link Support

* Fix comments

* Fix comments for including example

* add byoip resources and examples

* add resources to readme and update version

* fix path

* fix validation errors and remove roaExpirationDate

* remove swagger files to change name

* add the changed file names back

* remove to change name

* re-add changed name

* remove to change name

* re-add changed name

* remove swagger files to change name

* re-add changed name

* fix merge conflicts

* more merge conflicts

* move to june 2020 api version

* add default response to operations

* fix tab/space discrepancy

* fix path names and remove the public ip address version property from mastercustomipprefix

* fix camelCase in paths

* add acronyms to custom words

* fix another camelcase offender

* validation check fixes

* fix some more model validation issues

* fix json formatting

* remove invalid property

* set readonly on etag, fix paths in examples

* fix enum error

* fix network validation errors

* more naming naming convention changes

* change mastercustomipprefix reference from string to subresource

* fix spelling error

* added patch api to tags update

* minor fix

* prettier check

* minor change to force pipeline again

* minor description updates

* remove geography property in masterCustomIpPrefix

* add customIpPrefix property to publicIpPrefix

Co-authored-by: Anton Evseev <anton.evseev@protonmail.com>
Co-authored-by: naveenchekuri <nachekur@microsoft.com>
Co-authored-by: Anton Evseev <v-anevse@microsoft.com>
Co-authored-by: odedlaz <odedlaz@users.noreply.github.com>
Co-authored-by: Ishani Gupta <ishani.gupta27.ig@gmail.com>
Co-authored-by: Ishani Gupta <isgu@microsoft.com>
Co-authored-by: Zim Kalinowski <zikalino@microsoft.com>
Co-authored-by: irrogozh <irrogozh@microsoft.com>
Co-authored-by: Sai Sujith Reddy Mankala <samankal@microsoft.com>
Co-authored-by: Anchal Kapoor <anchalka@usc.edu>
Co-authored-by: shyamshd <64368451+shyamshd@users.noreply.github.com>
Co-authored-by: Shyam Dharmarajan <shdharma@microsoft.com>
Co-authored-by: Ritvika Reddy Nagula <rinagula@microsoft.com>
Co-authored-by: Khushboo Baheti <37917868+Khushboo-Baheti@users.noreply.github.com>
Co-authored-by: Khushboo Baheti <khbaheti@microsoft.com>
Co-authored-by: jaishals <jaishal.shah@gmail.com>

* fix the description (#10151)

* Add support for addressprefix in NVA and enableVirtualRouterRouteProp… (#10153)

* Added O365Policy to VpnSite resource. (#10192)

Co-authored-by: Neel Bhavsar <nebhavsa@microsoft.com>

* Added new properties to connection monitor endpoint (#10197)

* Added new properties to connection monitor endpoint

* Updated description of endpointScopeItem

* MACsec Configuration Edits (#10181)

* Add xpn aes cipher values & rm dashes

* Add sci mode param to macsec

* remove readonly attribute from customIpPrefix commissionedState property (#10246)

* fixing ids in examples (#10221)

* Merge branch 'network-june-release' of https://github.com/adgrieve/azure-rest-api-specs into network-june-release (#10069)

* Network june release revert - reverting changes which are not deployed from NRP side yet.  (#10325)

* reverting providerport changes

* fixing bug

* Remove masterCustomIpPrefix resource from Network-June-Release (does not affect current master branch) (#10338)

* remove masterCustomIpPrefix resource

* delete the MasterCustomIpPrefix swagger examples

Co-authored-by: Anton Evseev <anton.evseev@protonmail.com>
Co-authored-by: Henry Chen <henche@microsoft.com>
Co-authored-by: Umesh Kanodia <61820145+umkanodi@users.noreply.github.com>
Co-authored-by: Umesh Kanodia <umkanodi@microsoft.com>
Co-authored-by: ritwikbasu <ritwikbasu@users.noreply.github.com>
Co-authored-by: Jesus Arango <jesus.arango@gmail.com>
Co-authored-by: Hemant Imudianda <hemant.imudianda@apcera.com>
Co-authored-by: Hemant Imudianda <heimudia@microsoft.com>
Co-authored-by: irrogozh <irrogozh@microsoft.com>
Co-authored-by: Nilambari <nilamd@microsoft.com>
Co-authored-by: Anton Evseev <v-anevse@microsoft.com>
Co-authored-by: adgrieve <62692874+adgrieve@users.noreply.github.com>
Co-authored-by: Kaihui (Kerwin) Sun <sunkaihuisos@gmail.com>
Co-authored-by: tyrannicrex <jiaczh@microsoft.com>
Co-authored-by: dagoutha <64456132+dagoutha@users.noreply.github.com>
Co-authored-by: David Velasco <velascos.davida@gmail.com>
Co-authored-by: David Velasco <davelasc@microsoft.com>
Co-authored-by: MyronFanQiu <49134743+MyronFanQiu@users.noreply.github.com>
Co-authored-by: dexiang-ms <64503627+dexiang-ms@users.noreply.github.com>
Co-authored-by: naveenchekuri <nachekur@microsoft.com>
Co-authored-by: odedlaz <odedlaz@users.noreply.github.com>
Co-authored-by: Ishani Gupta <ishani.gupta27.ig@gmail.com>
Co-authored-by: Ishani Gupta <isgu@microsoft.com>
Co-authored-by: Zim Kalinowski <zikalino@microsoft.com>
Co-authored-by: Sai Sujith Reddy Mankala <samankal@microsoft.com>
Co-authored-by: Anchal Kapoor <anchalka@usc.edu>
Co-authored-by: shyamshd <64368451+shyamshd@users.noreply.github.com>
Co-authored-by: Shyam Dharmarajan <shdharma@microsoft.com>
Co-authored-by: Ritvika Reddy Nagula <rinagula@microsoft.com>
Co-authored-by: Khushboo Baheti <37917868+Khushboo-Baheti@users.noreply.github.com>
Co-authored-by: Khushboo Baheti <khbaheti@microsoft.com>
Co-authored-by: jaishals <jaishal.shah@gmail.com>
Co-authored-by: Anchal Kapoor <ankapoo@microsoft.com>
Co-authored-by: Neel <neel.bhavsar@protonmail.com>
Co-authored-by: Neel Bhavsar <nebhavsa@microsoft.com>
Co-authored-by: francescanarea <francesca@Narea.com>
innosam pushed a commit to innosam/azure-rest-api-specs that referenced this pull request Aug 17, 2020
* Adds base for updating Microsoft.Network from version stable/2020-05-01 to version 2020-06-01

* Updates readme

* Updates API version in new specs and examples

* New vWAN resource properties (Azure#9722)

* retarget

* fixed duplicate

* Added code changes for generating letter of authorization (Azure#9829)

Co-authored-by: Umesh Kanodia <umkanodi@microsoft.com>

* Async PATCH operations for Virtual WAN VpnGateways and P2SVpnGateways (Azure#9833)

* Add remoteBgpCommunities to VirtualNetworkPeering resource (Azure#9814)

* Add remoteBgpCommunities to VirtualNetworkPeering resource

* Change VirtualNetworkCommunity in example VirtualNetworkCreateWithBgpCommunities.json to a value that is within the allowed range.

* Application Gateway Mutual Authentication Support (Azure#9965)

* Adding mutual auth api definitions.

* Adding examples.

* Fixing case.

Co-authored-by: Hemant Imudianda <heimudia@microsoft.com>

* Added PATCH flowLog API and example (Azure#9960)

* Added PATCH flowLog API and example

* Fixed example

* Added property to TCPConfiguration

* Fixed prettier warning

* Fixes based on CR

* Add new optional param enableInternetSecurity and new Reset API for P2SVpnGateway resource (Azure#9952)

* Add new optional param enableInternetSecurity and new Reset API for P2SVpnGateway resource

* Fix validation tool fixes

* Remove extra spaces

* Update Network's owners (Azure#10006)

* Add intrusion system configuration (Azure#9912)

* Add intrusion system configuration

* Add ignored traffic configuration

* Fix checkPirvateLinkServiceVisibility in 2020-06-01 (Azure#9821)

* fix checkPirvateLinkServiceVisibility in 2020-06-01

* update examples

* add VpnGatewaypacketCapture api  (Azure#9832)

* add VpnGatewaypacketCapture api

* add custom words

* add vpn connection packet capture

* fix naming errors

* fix error

* fix word

* fixing validation

* fix model error

* finish prettier

* fix test

* expressRouteProviderPort changes (Azure#9909)

* expressRouteProviderPort changes

* fix ref error

* suppressing name id type error

* prettier fix

* locationlist -> listbylocation

* adding filter on location

* ref listbylocation

* fixing id and name

* adding get and bug fixes

* bug fix

* bug fix

* adding proxy resource ref

* fixes for auto checks

* adding proxyresource type to network.json

* fixing description

* fixing description for proxy resource

* Adding swagger updates to support the new NRP resource "DSCP Configuration" (Azure#9954)

* Adding the swagger for the new resource DSCP Configuration. Also updated the Network Interface to return it with its get function

* Updated the definition to the updated version in the source code

* Fix some errors being thrown by the validator (regarding int/ushort typing). Updated the readme.md to include the dscpConfiguration file

* Fixed the erroneous reference within the dscpConfiguration.json to the NetworkInterface definition

* Removed the long running operation flag.
Updated the examples to return two markings

* Added more objets within the lists.

* Removed the duplicated key

* The type 'ushort ' is not accepted. Turn it into an integer. Hopefully this fixes the issue regarding the QosPortRange properties

* Added the custome words dscp and qos in order to fix the error of the words being unkown. Refactor the allocatedNetworkInterfaces to associatedNetworkInterfaces to match the examples with the specification. Removed the QosTrafficProfiler from the examples as it no longer exists. Made sure the delete is using dscpName rather than nicName. Added the period at the end of the description.

* One traffic profiler slipped through the cracks, removed it. Properly formatted the object for the Qos Port/IP ranges. Was placing a string rather than an integer in the get example.

* Well, I was being silly here. Had a typo hahahahaha

* Added the warning suppression to the readme file. Run the prettier on the specification

* Thought I only had to run the prettier on the specification. Did the same for the examples and the NetworkInterfaceGet.json.

* Re-run the command with the full command placed in the documentation. Hopefully this fixes the prettier errors

* Corrected the typo

* Added the proper formatting for the suppression.

* Added the integer format

* Added the api calls for list and list all.
Made sure the name is plural
Added the examples for the new api calls
In Network Interface, now it is using the ID of the DSCP configuration rather than the full payload

* Forgot to switch the operation id from network interface to dscp configuration

* Added the missing next link for the list by resource

Co-authored-by: David Velasco <davelasc@microsoft.com>
Co-authored-by: MyronFanQiu <49134743+MyronFanQiu@users.noreply.github.com>

* Dev/dexiang/byoip (Azure#9818)

* Adds base for updating Microsoft.Network from version stable/2020-04-01 to version 2020-05-01

* Updates readme

* Updates API version in new specs and examples

* Add VirtualHubBgpConnections, VirtualHubIpConfigurations, NetworkVirtualApplianceSites child resources (Azure#9357)

* Adds base for updating Microsoft.Network from version stable/2020-03-01 to version 2020-05-01

* Updates readme

* Updates API version in new specs and examples

* Final changes

* Fix formatting issues

* Fix merge and formatting issues

* Fix formatiing

* Ipconfig properties

* Fix build errors

* Lint check

* Fix semantics errors

* Address review comments

* Fix prettier and nextLink checks

* Fix dots

* Address errors

* Fix Ipconfig version

Co-authored-by: dexiang-ms <64503627+dexiang-ms@users.noreply.github.com>

* Change NRP repo for validation scripts (Azure#9394)

* Move terminateTLS flag to ApplicationRuleCondition (Azure#9539)

* DNS Proxy support for Firewall Policy (Azure#9461)

* Adding DnsProxySettings Support in Firewall Policy

dnsServers, dnsEnableProxy, dnsRequireProxyForNetworkRules are added as subproperties.

* spell check

* Removing DNS as prefix on sub properties.

* Syntax error.

* DnsProxySettings to DNSSettings

* Fixing proxy settings name.

Co-authored-by: Ishani Gupta <isgu@microsoft.com>

* fix network (Azure#9369) (Azure#9547)

* fix network

* duplicated publicipprefix

* fixed example in 2018-01-01

* final updates

* missing examples

(cherry picked from commit 42d9639)

* Added new properties to ConnectivityHop (Azure#9386)

* Added new properties to ConnectivityHop

* Fixed spelling

* added x-ms-client-flatten to HopLink.properties.properties

* Rename Firewall Policy child resource RuleGroup to RuleCollectionGroup (Azure#9356)

* Rename Firewall Policy child resource RuleGroup to RuleCollectionGroup

* rebase with ti whitelist

* rebase with ti whitelist

* rebase with ti whitelist

* prettier run

* making nat rule into a collection

* prettier run

* Multiple Public IPs support for Azure Firewall in Virtual Hub (Azure#9560)

* multi pip for firewall in hub swagger change

* resolving issues

* VirtualWan: Hubvnet Connection related changes (Azure#9536)

1. Exposed direct PUT/DELETE APIs for hubvnet connections. This is the
   API expected to be used in the future by all consumers (direct rest
   api calls, powershell, portal, azure cli). This avoids needing a put
   on the virtual hub every time a hubvnet connection is to be
   added/changed.
2. Deprecate specifying hubvnet connection in virtualHub PUT. This is to
   avoid having multiple APIs to do the same thing. Also having this API
   introduces other complexities and breaks some scenarios.
3. Deprecate allowHubToRemoteVnetTransit and
   allowRemoteVnetToUseHubVnetGateways flags in hubvnet connection.
   These flags are maintained internally in the implementation and
   shouldn't be exposed to the user. For now, just mark it as such. Will
   be removed in later release.

Co-authored-by: Shyam Dharmarajan <shdharma@microsoft.com>

* Destination FQDNs in FirewallPolicy for network rules. (Azure#9609)

* v1

* making boolean as nullable

Co-authored-by: Ishani Gupta <isgu@microsoft.com>

* VirtualHub : Add Routing State and expose Effective Routes Api (Azure#9385)

* Add Routing State and expose Effective Routes Api

* updated opId name and added examples

* prettier fix done

* rename variable to prevent conflict

* Updating VpnGateway examples for bgpSettings (Azure#9656)

* Updating VpnGateway examples

* prettier

Co-authored-by: Khushboo Baheti <khbaheti@microsoft.com>

* Application Gateway Private Link Support (Azure#9490)

* Application Gateway Private Link Support

* Fix comments

* Fix comments for including example

* add byoip resources and examples

* add resources to readme and update version

* fix path

* fix validation errors and remove roaExpirationDate

* remove swagger files to change name

* add the changed file names back

* remove to change name

* re-add changed name

* remove to change name

* re-add changed name

* remove swagger files to change name

* re-add changed name

* fix merge conflicts

* more merge conflicts

* move to june 2020 api version

* add default response to operations

* fix tab/space discrepancy

* fix path names and remove the public ip address version property from mastercustomipprefix

* fix camelCase in paths

* add acronyms to custom words

* fix another camelcase offender

* validation check fixes

* fix some more model validation issues

* fix json formatting

* remove invalid property

* set readonly on etag, fix paths in examples

* fix enum error

* fix network validation errors

* more naming naming convention changes

* change mastercustomipprefix reference from string to subresource

* fix spelling error

* added patch api to tags update

* minor fix

* prettier check

* minor change to force pipeline again

* minor description updates

* remove geography property in masterCustomIpPrefix

* add customIpPrefix property to publicIpPrefix

Co-authored-by: Anton Evseev <anton.evseev@protonmail.com>
Co-authored-by: naveenchekuri <nachekur@microsoft.com>
Co-authored-by: Anton Evseev <v-anevse@microsoft.com>
Co-authored-by: odedlaz <odedlaz@users.noreply.github.com>
Co-authored-by: Ishani Gupta <ishani.gupta27.ig@gmail.com>
Co-authored-by: Ishani Gupta <isgu@microsoft.com>
Co-authored-by: Zim Kalinowski <zikalino@microsoft.com>
Co-authored-by: irrogozh <irrogozh@microsoft.com>
Co-authored-by: Sai Sujith Reddy Mankala <samankal@microsoft.com>
Co-authored-by: Anchal Kapoor <anchalka@usc.edu>
Co-authored-by: shyamshd <64368451+shyamshd@users.noreply.github.com>
Co-authored-by: Shyam Dharmarajan <shdharma@microsoft.com>
Co-authored-by: Ritvika Reddy Nagula <rinagula@microsoft.com>
Co-authored-by: Khushboo Baheti <37917868+Khushboo-Baheti@users.noreply.github.com>
Co-authored-by: Khushboo Baheti <khbaheti@microsoft.com>
Co-authored-by: jaishals <jaishal.shah@gmail.com>

* fix the description (Azure#10151)

* Add support for addressprefix in NVA and enableVirtualRouterRouteProp… (Azure#10153)

* Added O365Policy to VpnSite resource. (Azure#10192)

Co-authored-by: Neel Bhavsar <nebhavsa@microsoft.com>

* Added new properties to connection monitor endpoint (Azure#10197)

* Added new properties to connection monitor endpoint

* Updated description of endpointScopeItem

* MACsec Configuration Edits (Azure#10181)

* Add xpn aes cipher values & rm dashes

* Add sci mode param to macsec

* remove readonly attribute from customIpPrefix commissionedState property (Azure#10246)

* fixing ids in examples (Azure#10221)

* Merge branch 'network-june-release' of https://github.com/adgrieve/azure-rest-api-specs into network-june-release (Azure#10069)

* Network june release revert - reverting changes which are not deployed from NRP side yet.  (Azure#10325)

* reverting providerport changes

* fixing bug

* Remove masterCustomIpPrefix resource from Network-June-Release (does not affect current master branch) (Azure#10338)

* remove masterCustomIpPrefix resource

* delete the MasterCustomIpPrefix swagger examples

Co-authored-by: Anton Evseev <anton.evseev@protonmail.com>
Co-authored-by: Henry Chen <henche@microsoft.com>
Co-authored-by: Umesh Kanodia <61820145+umkanodi@users.noreply.github.com>
Co-authored-by: Umesh Kanodia <umkanodi@microsoft.com>
Co-authored-by: ritwikbasu <ritwikbasu@users.noreply.github.com>
Co-authored-by: Jesus Arango <jesus.arango@gmail.com>
Co-authored-by: Hemant Imudianda <hemant.imudianda@apcera.com>
Co-authored-by: Hemant Imudianda <heimudia@microsoft.com>
Co-authored-by: irrogozh <irrogozh@microsoft.com>
Co-authored-by: Nilambari <nilamd@microsoft.com>
Co-authored-by: Anton Evseev <v-anevse@microsoft.com>
Co-authored-by: adgrieve <62692874+adgrieve@users.noreply.github.com>
Co-authored-by: Kaihui (Kerwin) Sun <sunkaihuisos@gmail.com>
Co-authored-by: tyrannicrex <jiaczh@microsoft.com>
Co-authored-by: dagoutha <64456132+dagoutha@users.noreply.github.com>
Co-authored-by: David Velasco <velascos.davida@gmail.com>
Co-authored-by: David Velasco <davelasc@microsoft.com>
Co-authored-by: MyronFanQiu <49134743+MyronFanQiu@users.noreply.github.com>
Co-authored-by: dexiang-ms <64503627+dexiang-ms@users.noreply.github.com>
Co-authored-by: naveenchekuri <nachekur@microsoft.com>
Co-authored-by: odedlaz <odedlaz@users.noreply.github.com>
Co-authored-by: Ishani Gupta <ishani.gupta27.ig@gmail.com>
Co-authored-by: Ishani Gupta <isgu@microsoft.com>
Co-authored-by: Zim Kalinowski <zikalino@microsoft.com>
Co-authored-by: Sai Sujith Reddy Mankala <samankal@microsoft.com>
Co-authored-by: Anchal Kapoor <anchalka@usc.edu>
Co-authored-by: shyamshd <64368451+shyamshd@users.noreply.github.com>
Co-authored-by: Shyam Dharmarajan <shdharma@microsoft.com>
Co-authored-by: Ritvika Reddy Nagula <rinagula@microsoft.com>
Co-authored-by: Khushboo Baheti <37917868+Khushboo-Baheti@users.noreply.github.com>
Co-authored-by: Khushboo Baheti <khbaheti@microsoft.com>
Co-authored-by: jaishals <jaishal.shah@gmail.com>
Co-authored-by: Anchal Kapoor <ankapoo@microsoft.com>
Co-authored-by: Neel <neel.bhavsar@protonmail.com>
Co-authored-by: Neel Bhavsar <nebhavsa@microsoft.com>
Co-authored-by: francescanarea <francesca@Narea.com>
@JackTn JackTn deleted the dev/dexiang/byoip branch August 17, 2021 08:09
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.