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

{AppConfig} Migrate to Track 2 Mgmt SDK #16376

Merged
merged 1 commit into from
Dec 29, 2020

Conversation

avanigupta
Copy link
Member

Description
This PR upgrades the azure-mgmt-appconfiguration library to version 1.0.1, which is the new track 2 management plane SDK.

Changes in Track 2 SDK:

  • ErrorException has been removed. The SDK now throws HttpResponseError instead.
  • Most client APIs exposed by the SDK now start with begin_.
  • id cannot be passed directly to regenerate_key method. It needs to be wrapped in the new model RegenerateKeyParameters.
  • List/regenerate credentials returns an ApiKey object, which does not need to be converted to JSON object.

Testing Guide

History Notes


This checklist is used to make sure that common guidelines for a pull request are followed.

@avanigupta
Copy link
Member Author

@shenmuxiaosen can you please help review these changes?

Copy link
Contributor

@shenmuxiaosen shenmuxiaosen left a comment

Choose a reason for hiding this comment

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

:shipit:

@yonzhan
Copy link
Collaborator

yonzhan commented Dec 29, 2020

AppConfig

@qwordy
Copy link
Member

qwordy commented Dec 29, 2020

@avanigupta I am writing a document of how to migrate to Track 2 SDK. You are welcome to contribute to it.
#16330

@qwordy qwordy merged commit b9f69b6 into Azure:dev Dec 29, 2020
@avanigupta avanigupta deleted the avanigupta/mgmtsdkmigration branch December 29, 2020 01:45
@qwordy qwordy added the Track2 label Jan 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants