-
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
Adding swagger updates to support the new NRP resource "DSCP Configuration" #9954
Adding swagger updates to support the new NRP resource "DSCP Configuration" #9954
Conversation
…ted the Network Interface to return it with its get function
…yping). Updated the readme.md to include the dscpConfiguration file
[Staging] Swagger Validation Report
️✔️ |
Azure Pipelines successfully started running 1 pipeline(s). |
Can one of the admins verify this patch? |
azure-sdk-for-go
|
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
|
azure-sdk-for-net
|
azure-sdk-for-js
|
azure-sdk-for-python
|
azure-sdk-for-java
|
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-sdk-for-python-track2
|
…e NetworkInterface definition
Azure Pipelines successfully started running 1 pipeline(s). |
...fication/network/resource-manager/Microsoft.Network/stable/2020-06-01/dscpConfiguration.json
Outdated
Show resolved
Hide resolved
...k/resource-manager/Microsoft.Network/stable/2020-06-01/examples/DscpConfigurationCreate.json
Outdated
Show resolved
Hide resolved
Updated the examples to return two markings
Azure Pipelines successfully started running 1 pipeline(s). |
Azure Pipelines successfully started running 1 pipeline(s). |
@davelasc Could you fix CI errors according to the instruction and error mesages? |
@MikhailTryakhov @number213 @myronfanqiu @v-shhatt just a friendly reminder to review the PR. Hopefully it can get approved :D |
"info": { | ||
"title": "NetworkManagementClient", | ||
"description": "The Microsoft Azure Network management API provides a RESTful set of web services that interact with Microsoft Azure Networks service to manage your network resources. The API has entities that capture the relationship between an end user and the Microsoft Azure Networks service.", | ||
"version": "2020-06-01" |
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.
2020-06-01 [](start = 16, length = 10)
Has the 2020-06-01
been released to customers?
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.
By the end of this month it should be supported, as far as I'm aware. Did notice yesterday while doing a little bit of support that NRP is already supporting in production this version.
} | ||
}, | ||
"paths": { | ||
"/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Network/dscpConfiguration/{dscpConfigurationName}": { |
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.
dscpConfiguration [](start = 100, length = 17)
ARM resource type names should be plural. Can you rename to dscpConfigurations
? #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.
Refactor accordingly #Resolved
} | ||
} | ||
}, | ||
"paths": { |
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.
paths [](start = 3, length = 5)
Is this going to be a tracked or proxy resource?
If tracked you will also need to support these list calls:
- GET /subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Network/dscpConfigurations
- GET /subscriptions/{subscriptionId}/providers/Microsoft.Network/dscpConfigurations
If proxy, you will need at least this list call:
- GET /subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Network/dscpConfigurations
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.
This is going to be a tracked resource. Adding the pending calls and examples accordingly
@@ -1139,6 +1139,10 @@ | |||
"readOnly": true, | |||
"description": "A list of references to linked BareMetal resources." | |||
}, | |||
"dscpConfiguration": { | |||
"$ref": "./dscpConfiguration.json#/definitions/DscpConfiguration", |
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.
DscpConfiguration [](start = 57, length = 17)
This will cause the payload of the entire dscpConfiguration resource to be included in the payload of the network interface resource. This is basically an anti-pattern for the following reasons:
- breaks RBAC because a user who has /read permission on a nic but not a dscp configuration should not be able to see the the dscp configuration
- break Policy an unamiguous alias for properties of dscpConfiguration does not exist
- breaks Azure Resource Graph and Change Tracking because a single change notification represents an update of 2 entities
The alternative is to have the dscpConfiguration
property be a string that points to a resource ID of the dscpConfiguration resource or have it be an object with an id property. (NRP uses the latter pattern if I recall correctly in some places.)
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.
On second thought, it appears that your example is following the recommended pattern, but the schema definition implies the whole dscp configuration payload. Can you fix the schema to match the example? (NRP should have a common type that you can reuse for this already.)
In reply to: 451222161 [](ancestors = 451222161)
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.
Did the change so it is now using the sub resource reference rather than the full reference of it.
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.
I added a few comments and questions. Please take a look.
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
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.
Updated the files, please let me know if I successfully address the comments or what needs a little of more polish
@@ -1139,6 +1139,10 @@ | |||
"readOnly": true, | |||
"description": "A list of references to linked BareMetal resources." | |||
}, | |||
"dscpConfiguration": { | |||
"$ref": "./dscpConfiguration.json#/definitions/DscpConfiguration", |
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.
Did the change so it is now using the sub resource reference rather than the full reference of it.
} | ||
}, | ||
"paths": { | ||
"/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Network/dscpConfiguration/{dscpConfigurationName}": { |
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.
Refactor accordingly #Resolved
} | ||
} | ||
}, | ||
"paths": { |
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.
This is going to be a tracked resource. Adding the pending calls and examples accordingly
"info": { | ||
"title": "NetworkManagementClient", | ||
"description": "The Microsoft Azure Network management API provides a RESTful set of web services that interact with Microsoft Azure Networks service to manage your network resources. The API has entities that capture the relationship between an end user and the Microsoft Azure Networks service.", | ||
"version": "2020-06-01" |
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.
By the end of this month it should be supported, as far as I'm aware. Did notice yesterday while doing a little bit of support that NRP is already supporting in production this version.
Azure Pipelines successfully started running 1 pipeline(s). |
} | ||
}, | ||
"/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Network/dscpConfigurations": { | ||
"get": { |
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.
get [](start = 7, length = 3)
You have paging declared on list by Sub but not on this operation. Can you add it here too?
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.
Added the missing link definition
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.
Signed off from ARM side. Please add the missing paging on "list by RG", though.
Azure Pipelines successfully started running 1 pipeline(s). |
Azure Pipelines successfully started running 1 pipeline(s). |
* 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>
* 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>
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
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.