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

[Microsoft.StorageSync] Update SDK to use 2019-06-01 API version #8632

Merged
merged 3 commits into from
Nov 7, 2019
Merged

[Microsoft.StorageSync] Update SDK to use 2019-06-01 API version #8632

merged 3 commits into from
Nov 7, 2019

Conversation

anpint
Copy link
Member

@anpint anpint commented Nov 5, 2019

Check list:

  • Update recorded tests

  • Identity of QA responsible for feature testing (can be conducted post-merging of the pull request):
    Our QA consists of running recorded unit tests, and recorded tests for the Az.StorageSync module that cosumes the SDK (will be updated after SDK is merged and nuget is released).

  • Short description of the payload of pull request.
    This PR updates the StorageSync .NET SDK to consume our new API version - 2019-06-01. This API version introduces some new properties on the server endpoint object to relay cloud tiering and recall status of the server endpoint.

StorageSync 2019-06-01 API version: Azure/azure-rest-api-specs#7337

@anpint anpint requested a review from erich-wang as a code owner November 5, 2019 18:50
Copy link
Member

@isra-fel isra-fel left a comment

Choose a reason for hiding this comment

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

Hi @anpint

  • Please add a link to the swagger spec review PR(s) in the description
  • There are breaking changes, such as renaming a model class, so please update the major version number
  • Please also update AssemblyVersion and AssemblyFileVersion in AssemblyInfo.cs.

Thanks

@isra-fel isra-fel added Mgmt This issue is related to a management-plane library. needs-revision labels Nov 6, 2019
@anpint anpint requested a review from isra-fel November 6, 2019 20:35
@anpint
Copy link
Member Author

anpint commented Nov 6, 2019

Hi @anpint

  • Please add a link to the swagger spec review PR(s) in the description
  • There are breaking changes, such as renaming a model class, so please update the major version number
  • Please also update AssemblyVersion and AssemblyFileVersion in AssemblyInfo.cs.

Thanks

@isra-fel thanks for reviewing the changes. I've made the changes you request.

@isra-fel isra-fel merged commit 92c2177 into Azure:master Nov 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mgmt This issue is related to a management-plane library.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants