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

Network SDK update #11769

Merged
merged 8 commits into from
May 1, 2020
Merged

Conversation

anton-evseev
Copy link
Contributor

@anton-evseev anton-evseev commented Apr 30, 2020

Description

This PR update Network SDK to use API version 2020-04-01

  • Skipped one CosmosDB test that uses Network SDK as it requires pre-provisioned resources
  • Skipped 5 PrivateZone tests that use Network SDK in Dns as this API is deprecated
  • Skipped one Insights test that uses Network SDK as there's a backend issue with it unrelated to Network
  • Commented out test for PublicIpPrefix list as there are issue on backend with polling
  • Added change from Fixed property type in PSConnectionMonitorSuccessThreshold object #11630 to make build pass

Checklist

  • I have read the Submitting Changes section of CONTRIBUTING.md
  • The title of the PR is clear and informative
  • The appropriate ChangeLog.md file(s) has been updated:
    • For any service, the ChangeLog.md file can be found at src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
    • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header -- no new version header should be added
  • The PR does not introduce breaking changes
  • If applicable, the changes made in the PR have proper test coverage
  • For public API changes to cmdlets:
    • a cmdlet design review was approved for the changes in this repository (Microsoft internal only)
    • the markdown help files have been regenerated using the commands listed here

@anton-evseev anton-evseev marked this pull request as ready for review May 1, 2020 06:06
@wyunchi-ms wyunchi-ms merged commit e08305b into Azure:network-april May 1, 2020
@anton-evseev anton-evseev deleted the nrp-2020-04-01 branch May 1, 2020 14:05
isra-fel added a commit that referenced this pull request May 8, 2020
* Adding support for ForceFirewallPolicy association on Application Gateway

* updates ?

* CredScan suppression

* Sync Network dev branch with master (#11638)

* Update example 2 with correct command. (#11282)

* Update the filter mechanism to use server side query

* Updated change log

* Update ChangeLog.md

* Add ResourceId related Tags cmdlets.

* Added update cmdlet to the mapping file.

* Update Network.sln

add Sql to Network.sln

* point Sql.dll to local dll

* Updated the help related documents for tags.

* merge upstream master

* fix readme merge

* Add cmdlet to get/set parent device of iot device

* Modified tags related md file.

* Minor changes to tag related md files.

* Fix ScopeDeployment issue.

* Additions the ChangeLog

* Modified Tags.csproj to remove unnecessary reference.

* Remove default from the object.

* Re-record the test session

* Updated changelog and tag md file.

* Record RG filter by tags tests

* Update Invoke-AzStorageSyncChangeDetection.md

* Address the pr review.

* Re-record the tag related tests.

* solve the problem of build stack edge

* update the version of CosmosDB

* remove serialized json file

* Removing client side filtering tests, since we no longer filter on the client.

* Address pr review.

* Fix the indent for tagTest.ps1.

* Update TagTest.ps1 to fix format issue.

* Add tags display format

* fix the problem that stack edge is filtered by stack

* [Storage] Add warning message for table type change in the future release.

* Updated examples on the help page

* Updated the md file for tags.

* minor fix to restore powershell workflow

* update sql server AAD Admin to use 2019-06-01-preview API.

* fix #10849

* update changelog

* fix #11167

* update changelog

* Resolve conflict

* Update ChangeLog.md

* Update ChangeLog.md

* Update ChangeLog.md

* Update ChangeLog.md

* Update ChangeLog.md

* add Disable-AzSqlServerActiveDirectoryOnlyAuthentication cmdlet and help files.

* Add support to manage device parent-child relationship.

* Tasks #10422,#9895,#10482,#10794 changes

* Add Survey Url to Send-Feedback & Resolve-Error

issue: #11020
issue: #11021

* add devportal support

suppress security violations as test violations

* Add breaking change attributes for future changes to policy cmdlets

* Update changelog

* fixes for managed database backups and restore manage db cmdlets

* Address pr comments.

* Fix a typo error.

* add managed instance LTR tests

* Updating the 'Publish-AzWebApp' help document with Force parameter.

* updated help text

* [Storage] Support allowProtectedAppendWrites in ImmutabilityPolicy

* Fix help issues in Resources.

* Make sure CmdletBase produce correct tenant-level resource id.

* Typo

* Update ChangeLog

* Add PublicNetworkAccess to New-AzSqlServer and Set-AzSqlServer

* Use get-azcontext to get the default subscriptionId.

* Fix helper files + references

* Add to ChangeLog.md

* Fix assignment error

* Update help file

* Add breaking change attributes for upcoming cmdlet changes

* Bug Fix: resolve resources test coverage #5216, and update ResourceManagementClient to 2019-10-01.

* Change default parameterset

* update changelog.md

* exclude breaking change issue

* Add Az version into UserAgent

* Correct branding name in changelog

* Removed Verbose

The cmdlet does not seem to have a -Verbose parameter

* fixing PR issues, changing lots of 'servers' to 'managedInstances' where needed

* Update Sql reference to 1.38 in network.csproj

* Delete file due to incorrect commit

Confirmed with original contributor.

* Fix typo

infomation→information

* Fix Accounts changelog error

* Update AzAttestation cmdlets for public preview.
- Added policy signer management cmdlets
- Added support for location and tags

* Updated ChangeLog.md file

* Added online version info for new help markdown files

* Add new common lib which catches exception for test case

* Update changelog

* Documented PSPolicySigners return type, fixed ShouldProcess logic

* Rerecord server tests due to version bump

* Preventing unnecessary tests from failing due to server version bump

* Rerecord test that was having credscan errors

* Manage distributed settings per-device

* Add Minimal Tls Version Optional Parameter for SQLDB/MI

* Polish changelog.md

* Update ChangeLog.md

* 3.6.0 RC1

* updated tests; renamed parameters for consistency; adjusted backups API to be more user-friendly; added session recording for LTR MI Policy

* Update MI and SQL DB Server CRUD tests

* update changelog for Az.Network

* regenerate the help file.

* regenerate blank file and see if that works.

* add online version to help file. Looks like that has become compulsory now.

* update ChangeLog.md

* Added test recordings for Sql DB and bumped version for management.sql

* add mi test

* test udpate

* test fix

* fixed some tests; including all session tests (with mock resources)

* Added test recording

* Update changelog

* trivial fix

* fix helpn files

* fixing test failuires

* some fixes in remove backup; redid session records, minor fixes throughout

* moving seesion record to the right folder

* added az.network to changelog

* bypassing bug in ApiExclusion Logic

* Upgrade SDK to 4.8.0

* bumping .net

* bumping .net

* help files; changelog

* remove api version bypass for a proper fix

* fix test issue

* skipping faulty mock tests

* Show the exact string value for VM PowerState in the table format.

* add SupportedAggregationTypes and Dimentions to PSMetricDefinition

* More updates to help documentation

* add entry in changelog.md

* fix format in changelog.md

* generic names in help files

* adding online versions to help files

* fixing online version

* more tweakes to set-...policy help docs

* Update ChangeLog.md

* Correct version for changelog.md

* Polish content for changelog.md

* Add network release

* For RC2 build

* ps-vsprompt

* removed ps-vsprompt from PR

* add comma

* change dogfood endpoint in session records

* add servers back into vulnerability

* BugFix for PrivateLinkServiceConnectionState

* Added three required assemblies for the AzAttestation PowerShell cmdlets.  (Microsoft.IdentityModel.*)

* ltr policy record

* Updated ChangeLog.md

* re-recorded backup tests

* Get-AzGalleryImageVersion fix example args

Update samples to use proper argument  -GalleryImageDefinitionName not -ImageDefinitionName

* for RC2 release

* update changelog

* Fix typo

infomation→information

* PR comments has been resolved

* Updated help content

* For final RC

* For final RC

* Fix Microsoft.Azure.PowerShell.Cmdlets.ManagedServices.dll.json

#10826

* Fix changelog merge issue

* Update example 2 with correct command.

Updated example 2 to use Get-AzSqlInstanceDatabase instead of Get-AzSqlDatabase.

* remove extra files that were removed on master

Co-authored-by: Hamid Tirmizi <[email protected]>
Co-authored-by: Hamid T <[email protected]>
Co-authored-by: Ran Wang <[email protected]>
Co-authored-by: Yabo Hu <[email protected]>
Co-authored-by: Yunchi Wang <[email protected]>
Co-authored-by: Sapan Saxena <[email protected]>
Co-authored-by: xaliciayang <[email protected]>
Co-authored-by: Daniel Orozco <[email protected]>
Co-authored-by: Daniel Orozco <[email protected]>
Co-authored-by: Meha Kaushik <[email protected]>
Co-authored-by: Yeming Liu <[email protected]>
Co-authored-by: Jeff Patterson <[email protected]>
Co-authored-by: wyunchi-ms <[email protected]>
Co-authored-by: wyunchi <[email protected]>
Co-authored-by: Wei Wei <[email protected]>
Co-authored-by: Amol Agarwal <[email protected]>
Co-authored-by: [email protected] <[email protected]>
Co-authored-by: Kota Sudhakar Reddy <[email protected]>
Co-authored-by: msJinLei <[email protected]>
Co-authored-by: Samir Solanki <[email protected]>
Co-authored-by: Chris Stackhouse <[email protected]>
Co-authored-by: ThejaChoudary <[email protected]>
Co-authored-by: Sambit Rath <[email protected]>
Co-authored-by: He Huang <[email protected]>
Co-authored-by: Emily Gu <[email protected]>
Co-authored-by: sguitardude <[email protected]>
Co-authored-by: Dingmeng Xue <[email protected]>
Co-authored-by: Ikko Ashimine <[email protected]>
Co-authored-by: gkostal <[email protected]>
Co-authored-by: ziwa-msft <[email protected]>
Co-authored-by: ancicovi <[email protected]>
Co-authored-by: xaliciayang <[email protected]>
Co-authored-by: Shawn Xiao (IM) <[email protected]>
Co-authored-by: Hyonho Lee <[email protected]>
Co-authored-by: Bo Wen <[email protected]>
Co-authored-by: Bill Sproule <[email protected]>
Co-authored-by: Yeming Liu <[email protected]>
Co-authored-by: msJinLei <[email protected]>

* [IoT Hub] Invoke a query in an IoT hub. (#11567)

* Invoke a query in an IoT hub.

* Support CosmosDb for PrivateEndpointConnection and PrivateLinkResource operations (#11584)

* Add CosmosDB PrivateEndpointConnection, PrivateLinkResource

* new line at the end of file

* Use correct ProfileName in verbose message (#11580)

Co-authored-by: Steve Strugnell <[email protected]>

* [Blueprint] Add management group level assignment support (#11582)

* Update Blueprint SDK to the latest version.

* Fix a parameter name.

* PSBlueprintAssignment.FromAssignment now takes scope from Assignment.

* Fix a parameter name.

* Add MG level assignment support to existing cmdlets.

* Update help docs.

* Remove netcoreapp2.1 from test target (#11593)

* Remove netcoreapp2.1 from test target

* Remove alias of Resolve-AzError (#11575)

* Remove alias and breaking change warning

* add exception for breaking change issues

* update changelog

* Aladdin generated examples for Accounts, KeyVault, Monitor, Storage (#11569)

* Add new examples to Accounts, KeyVault, Monitor, Storage
	- Add new examples to the following modules:
		Accounts
		KeyVault
		Monitor
		Storage

* Add new examples to Accounts, KeyVault, Monitor, Storage
	- Add new examples to the following modules:
		Accounts
		KeyVault
		Monitor
		Storage

* Fix up the type name.

Co-authored-by: Maoliang Huang <[email protected]>

* use token to get changed files for one PR on macOS (#11588)

* add property TargetModule to filter build/publish module (#11601)

* Fixes #11298 - Example works with multiple subnets (#11599)

* Fixes #11298 - Example works with multiple subnets

* Standardized values in code examples

* Fixes #11300 - Example works with multiple subnets (#11600)

* Update example 2 code snippet in Get-AzKeyVaultCertificate.md (#11598)

Update example 2 code snippet in Get-AzKeyVaultCertificate.md (#11598)

* Removed default population of access policies (#11570)

* Fixed reported broken links (#11612)

* Fixes #10963 - Now reflects intended behavior (#11614)

* Wrong cmdlet used to get the certifacte (#11613)

The example is using Get-AzKeyVaultSecret to get the certificate from Key Vault, Get-AzKeyVaultCertificate is the correct cmdlet to use.

* Updating value of StorageKeyKind when storage account is under VNet. (#11609)

* Add new value to StorageKeyType

* Fix #11597 (#11615)

* [KeyVault] Breaking changes: soft delete and more (#11469)

Breaking changes: soft delete and more

* Merge release-2020-04-21 (#11620)

* Update changelog (#11573)

* Adding support for ForceFirewallPolicy association on Application Gateway

* updates ?

* CredScan suppression

* Publishing release version of Maintenance cmdlets for GA (#11574)

Co-authored-by: Yunchi Wang <[email protected]>

* DPDTimeout core commits

* help files

* changelog

* add change log for azs release (#11594)

* Remove netcoreapp2.1 from test target (#11593)

* Remove netcoreapp2.1 from test target

* Make Maintenance to 1.0.0 GA (#11578)

* update version to 1.0.0

* Remove netcoreapp2.1 from test target (#11593)

* Remove netcoreapp2.1 from test target

Co-authored-by: wyunchi-ms <[email protected]>
Co-authored-by: Dingmeng Xue <[email protected]>

* Check PS Version when importing Az.Storage which using Azure.Core (#11576)

* azure api management breaking change notifications (#11602)

* azure api management breaking change notifications

* notification param fixed

Co-authored-by: vfedonkin <[email protected]>

* release-2020-04-21 (#11619)

Co-authored-by: Venkata Krishnan Anantha Raman <[email protected]>
Co-authored-by: Anton Evseev <[email protected]>
Co-authored-by: gautamd-ms <[email protected]>
Co-authored-by: Yunchi Wang <[email protected]>
Co-authored-by: Henry Chen <[email protected]>
Co-authored-by: erich-wang <[email protected]>
Co-authored-by: wyunchi-ms <[email protected]>
Co-authored-by: Vitaliy Fedonkin <[email protected]>
Co-authored-by: vfedonkin <[email protected]>

* Update Get-AzAutomationJobOutputRecord.md (#11621)

i am proposing to remove this line, as it is not how this cmdlet works. It only returns the first 55 characters, rather than the full summary.

* Add smoke test for set a resource tag case (#11623)

* Add one smoke test for set a resource tag case.

* Implement test case function.

* Article is incorrect (#11625)

This command cannot be used post VM creation.

* [Data Factory] Support CRUD of data flow runtime properties in Managed IR. (#11626)

* support data flow properties in azure ir

* fix md

* [Iot Hub] Generate SAS token for Iot Hub, device or module (#11628)

* Generate SAS token for Iot Hub, device or module

* record test session

* Enhance pefrormance of Data Classification Cmdlets. (#11629)

* Enhance pefrormance of Data Classification Cmdlets.

* Update ChangeLog.md

* Add windows container support in AKS (#11595)

* [AKS] Support creating Windows server container in AKS (#11403)

* support using windows container to create AKS

* upload to signed AKS sdk

* update onlineversion for help of new cmdlets

* add missing dll in psd1

* disable test parallelization in Aks.Test

* update ContainerService SDK

* fix Start-AzAksDashboard and update help files

* update to official nuget package for containerservice

Co-authored-by: DavidMMaxwell <[email protected]>
Co-authored-by: Hamid Tirmizi <[email protected]>
Co-authored-by: Hamid T <[email protected]>
Co-authored-by: Ran Wang <[email protected]>
Co-authored-by: Yabo Hu <[email protected]>
Co-authored-by: Yunchi Wang <[email protected]>
Co-authored-by: Sapan Saxena <[email protected]>
Co-authored-by: xaliciayang <[email protected]>
Co-authored-by: Daniel Orozco <[email protected]>
Co-authored-by: Daniel Orozco <[email protected]>
Co-authored-by: Meha Kaushik <[email protected]>
Co-authored-by: Yeming Liu <[email protected]>
Co-authored-by: Jeff Patterson <[email protected]>
Co-authored-by: wyunchi-ms <[email protected]>
Co-authored-by: wyunchi <[email protected]>
Co-authored-by: Wei Wei <[email protected]>
Co-authored-by: Amol Agarwal <[email protected]>
Co-authored-by: [email protected] <[email protected]>
Co-authored-by: Kota Sudhakar Reddy <[email protected]>
Co-authored-by: msJinLei <[email protected]>
Co-authored-by: Samir Solanki <[email protected]>
Co-authored-by: Chris Stackhouse <[email protected]>
Co-authored-by: ThejaChoudary <[email protected]>
Co-authored-by: Sambit Rath <[email protected]>
Co-authored-by: He Huang <[email protected]>
Co-authored-by: Emily Gu <[email protected]>
Co-authored-by: sguitardude <[email protected]>
Co-authored-by: Dingmeng Xue <[email protected]>
Co-authored-by: Ikko Ashimine <[email protected]>
Co-authored-by: gkostal <[email protected]>
Co-authored-by: ziwa-msft <[email protected]>
Co-authored-by: ancicovi <[email protected]>
Co-authored-by: xaliciayang <[email protected]>
Co-authored-by: Shawn Xiao (IM) <[email protected]>
Co-authored-by: Hyonho Lee <[email protected]>
Co-authored-by: Bo Wen <[email protected]>
Co-authored-by: Bill Sproule <[email protected]>
Co-authored-by: Yeming Liu <[email protected]>
Co-authored-by: msJinLei <[email protected]>
Co-authored-by: Sapan Saxena <[email protected]>
Co-authored-by: Soyoung Eom <[email protected]>
Co-authored-by: Squigg <[email protected]>
Co-authored-by: Steve Strugnell <[email protected]>
Co-authored-by: Shenglong Li <[email protected]>
Co-authored-by: aladdindoc <[email protected]>
Co-authored-by: Maoliang Huang <[email protected]>
Co-authored-by: erich-wang <[email protected]>
Co-authored-by: Mike F. Robbins <[email protected]>
Co-authored-by: Shantanu Chaudhary <[email protected]>
Co-authored-by: Michael Hansen <[email protected]>
Co-authored-by: Stefan Ivemo <[email protected]>
Co-authored-by: bashahee <[email protected]>
Co-authored-by: Venkata Krishnan Anantha Raman <[email protected]>
Co-authored-by: gautamd-ms <[email protected]>
Co-authored-by: Henry Chen <[email protected]>
Co-authored-by: Vitaliy Fedonkin <[email protected]>
Co-authored-by: vfedonkin <[email protected]>
Co-authored-by: Lars Panzerbjørn <[email protected]>
Co-authored-by: Daniel Sol <[email protected]>
Co-authored-by: Jack Ma <[email protected]>

* Network SDK update (#11769)

* Network SDK update

* More recording updates

* CredScan suppression

* Update for NW change

* Test fixes

* Use signed SDK

* Update ChangeLog.md in Network

* Update other RPs

* VirtualWan : Set default value of AllowVnetToVnetTraffic flag to true (#11823)

* set default value to true

* upadate again

* Update after rebase

* Update recordings

* Update test recordings

* Revert recording changes in other RPs

Co-authored-by: Venkata Krishnan Anantha Raman <[email protected]>
Co-authored-by: DavidMMaxwell <[email protected]>
Co-authored-by: Hamid Tirmizi <[email protected]>
Co-authored-by: Hamid T <[email protected]>
Co-authored-by: Ran Wang <[email protected]>
Co-authored-by: Yabo Hu <[email protected]>
Co-authored-by: Yunchi Wang <[email protected]>
Co-authored-by: Sapan Saxena <[email protected]>
Co-authored-by: xaliciayang <[email protected]>
Co-authored-by: Daniel Orozco <[email protected]>
Co-authored-by: Daniel Orozco <[email protected]>
Co-authored-by: Meha Kaushik <[email protected]>
Co-authored-by: Yeming Liu <[email protected]>
Co-authored-by: Jeff Patterson <[email protected]>
Co-authored-by: wyunchi-ms <[email protected]>
Co-authored-by: wyunchi <[email protected]>
Co-authored-by: Wei Wei <[email protected]>
Co-authored-by: Amol Agarwal <[email protected]>
Co-authored-by: [email protected] <[email protected]>
Co-authored-by: Kota Sudhakar Reddy <[email protected]>
Co-authored-by: msJinLei <[email protected]>
Co-authored-by: Samir Solanki <[email protected]>
Co-authored-by: Chris Stackhouse <[email protected]>
Co-authored-by: ThejaChoudary <[email protected]>
Co-authored-by: Sambit Rath <[email protected]>
Co-authored-by: He Huang <[email protected]>
Co-authored-by: Emily Gu <[email protected]>
Co-authored-by: sguitardude <[email protected]>
Co-authored-by: Dingmeng Xue <[email protected]>
Co-authored-by: Ikko Ashimine <[email protected]>
Co-authored-by: gkostal <[email protected]>
Co-authored-by: ziwa-msft <[email protected]>
Co-authored-by: ancicovi <[email protected]>
Co-authored-by: xaliciayang <[email protected]>
Co-authored-by: Shawn Xiao (IM) <[email protected]>
Co-authored-by: Hyonho Lee <[email protected]>
Co-authored-by: Bo Wen <[email protected]>
Co-authored-by: Bill Sproule <[email protected]>
Co-authored-by: Yeming Liu <[email protected]>
Co-authored-by: msJinLei <[email protected]>
Co-authored-by: Sapan Saxena <[email protected]>
Co-authored-by: Soyoung Eom <[email protected]>
Co-authored-by: Squigg <[email protected]>
Co-authored-by: Steve Strugnell <[email protected]>
Co-authored-by: Shenglong Li <[email protected]>
Co-authored-by: aladdindoc <[email protected]>
Co-authored-by: Maoliang Huang <[email protected]>
Co-authored-by: erich-wang <[email protected]>
Co-authored-by: Mike F. Robbins <[email protected]>
Co-authored-by: Shantanu Chaudhary <[email protected]>
Co-authored-by: Michael Hansen <[email protected]>
Co-authored-by: Stefan Ivemo <[email protected]>
Co-authored-by: bashahee <[email protected]>
Co-authored-by: gautamd-ms <[email protected]>
Co-authored-by: Henry Chen <[email protected]>
Co-authored-by: Vitaliy Fedonkin <[email protected]>
Co-authored-by: vfedonkin <[email protected]>
Co-authored-by: Lars Panzerbjørn <[email protected]>
Co-authored-by: Daniel Sol <[email protected]>
Co-authored-by: Jack Ma <[email protected]>
Co-authored-by: Ritvika Reddy Nagula <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants