-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Expose new SKUs for VirtualNetworkGateway ER and VPN VMSS #4322
Conversation
Expose new SKUs for VirtualNetworkGateway ER and VPN VMSS
The REST-API PR is Azure/azure-rest-api-specs#3069 |
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.
Please generate the code using generate.ps1 script and commit changes to the .txt log file
@@ -7,7 +7,7 @@ | |||
<PackageId>Microsoft.Azure.Management.Network</PackageId> | |||
<Description>Provides management capabilities for Network services.</Description> | |||
<AssemblyName>Microsoft.Azure.Management.Network</AssemblyName> | |||
<Version>19.0.0-preview</Version> |
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 does not look like a breaking change. You can simply set it as 19.0.1-preview
instead
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.
Also, update the PackageReleaseNotes
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.
Fixed in second commit.
Review Comments
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
Will merge on CIs passing |
|
||
1) azure-rest-api-specs repository information | ||
GitHub user: Azure | ||
Branch: master | ||
Commit: b9721ae4c9f04914cc89260b8339dc33372f835b | ||
Branch: Network-2018-05-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.
Apologize for missing this, we cannot accept PRs for sdks generated from a branch other than master. If your changes have already been merged in the Network May REST spec, you don't need this PR
Please talk to @MikhailTryakhov if you have any doubts
These changes will be introduced with the Network May REST spec sdk |
Expose new SKUs for VirtualNetworkGateway ER and VPN VMSS
Description
This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines
Testing Guidelines
SDK Generation Guidelines
*.csproj
andAssemblyInfo.cs
files have been updated with the new version of the SDK.