Skip to content

Conversation

@austintolani
Copy link
Contributor

@austintolani austintolani commented Oct 13, 2022

Summary
These changes contain the newly generated code for the Microsoft.Azure.Management.Redis package based on the new 2022-06-01 API version. This generation actually includes new code for changes made in both the 2022-06-01 and 2022-05-01 API versions:

Testing

I have added unit tests and functional tests (with updated recorded session results) to test the newly generated code.

Copy link
Member

@Yao725 Yao725 left a comment

Choose a reason for hiding this comment

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

Hi @austintolani, as we announced the deprecation for Track 1 .NET SDK, we can't release Track 1 SDK anymore. If you still need this release, you need to provide the business impact or other strong justification of this release for @ArthurMa1978.

@austintolani
Copy link
Contributor Author

Hi @austintolani, as we announced the deprecation for Track 1 .NET SDK, we can't release Track 1 SDK anymore. If you still need this release, you need to provide the business impact or other strong justification of this release for @ArthurMa1978.

Hello @Yao725. These changes are not for public release, they are for creating a private signed package to be used for our Azure PowerShell module. I spoke to @ArthurMa1978 about this and he said that I should merge my changes into this public repo and then run the net-pr - mgmt pipeline to publish the private signed NuGet package.

Copy link
Member

@Yao725 Yao725 left a comment

Choose a reason for hiding this comment

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

Thanks for the clarification @austintolani, approved with comment.

….Management.Redis.csproj

Co-authored-by: Yao Kou <yaokou@microsoft.com>
@austintolani
Copy link
Contributor Author

@Yao725 thanks for the review. Could you merge the PR as well? Merging is blocked for me.

@Yao725 Yao725 merged commit c3a94f5 into Azure:main Oct 20, 2022
@Yao725
Copy link
Member

Yao725 commented Oct 20, 2022

@austintolani, merged. By the way, I believe if you want to release track 1 SDK, you should follow this guide and go to run the net - mgmt pipeline. Considering it is not a public release, please do not select ShouldPublishToNuget, which will publish the package to nuget.

@austintolani
Copy link
Contributor Author

Will do. Thanks for your help @Yao725

sofiar-msft pushed a commit to sofiar-msft/azure-sdk-for-net that referenced this pull request Dec 7, 2022
* generate Redis SDK

* Update ManagedIdentityTests.cs

* Update BeginCreateFunctionalTests.cs

* use 'latest' as redisVersion

* begin update tests

* Update PatchSchedulesFunctionalTests.cs

* add new properties to CreateOrUpdateTests

* Update GeoReplicationFunctionalTests.cs with async + sync method testing

* Rename BeginCreateFunctionalTests -> BeginCreateBeginUpdateFunctionalTests

* Prevent key sanitization

* disable failing geo-replication tests

* session recordings

* Revert "Prevent key sanitization"

This reverts commit 83c1e40.

* sanitized keys in RegenerateKeyTest.json

* sdk release info

* santize keys

* regenerate session records

* edit key santization in RegenerateKeyTest.json

* Update sdk/redis/Microsoft.Azure.Management.Redis/src/Microsoft.Azure.Management.Redis.csproj

Co-authored-by: Yao Kou <yaokou@microsoft.com>

Co-authored-by: Austin Tolani <austintolani@microsoft.com>
Co-authored-by: Yao Kou <yaokou@microsoft.com>
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.

3 participants