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.

Sign up for GitHub

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

Jump to bottom

Azure sdk update #6544

Merged
merged 21 commits into from
Mar 18, 2021
Merged

Azure sdk update #6544

merged 21 commits into from
Mar 18, 2021

Conversation

logachev
Copy link
Collaborator

@logachev logachev commented Mar 17, 2021 •
edited
Loading

Moves c7n_azure to the new Azure SDKs (Track2) to ensure we can get latest libraries for the next release.

Major changes:

  • Session is updated to use new Azure Credentials. There is a legacy-compatible method for SDKs that are still Track 1 (mgmt-msi and graph)
  • Storage Utils implement 2 adapters to minimize amount of changes in c7n code. They wrap new Storage clients with old interface.
  • Naming was changed for within Azure SDKs
    • LROPoller methods have begin_ prefix
    • Some clients\libraries\methods were renamed
    • Parameters name changes (credentials -> credential)

closes #6459
closes #6458

logachev added 14 commits March 8, 2021 11:05
@logachev
[Azure SDK integration branch PR] Update KeyVault SDK (#6512)
d164430
@logachev
[Azure SDK integration branch PR] Update majority of SDKs and fix som…
b476d6d 
…e basic name changes (#6513)
@logachev
[Azure SDK integration branch PR] Updates to session.py (#6516)
955431f
@logachev
[Azure SDK integration branch PR] Update Storage SDKs (#6517)
e770ac6
@logachev
[Azure SDK integration branch PR] Minor SDK adjustments (#6518)
d75db82
@logachev
[Azure SDK integration branch PR] Azure Functions tests (#6519)
d6c7e0b 
* Fix Azure Functions Functioanl tests

* Deployment units fixes
@logachev
Fix cosmosdb tests (#6523)
ea95148
@logachev
[Azure SDK integration branch PR] Legacy clients & Storage queue new …
2b3fd11 
…behavior
@logachev
[Azure SDK integration branch PR] Storage Diagnostic Tests pass (#6527)
97ce430
@logachev
azure - custom retries + log headers (#6528)
7464036
@logachev
[Azure SDK integration branch PR] Fixes for live tests (#6533)
5aa057b
@logachev
[Azure SDK integration branch PR] Update DNS, PolicyInsights & Mgmt G…
f61c216 
…roup SDKs (#6539)
@logachev
[Azure SDK integration branch PR] Lint changes (#6540)
5d338e8
@logachev
[Azure SDK integration branch PR] Manual validation fixes (#6543)
c1718e6
@logachev logachev requested a review from kapilt as a code owner March 17, 2021 16:44
@stefangordon
Copy link
Collaborator

🤞

@logachev logachev merged commit 5de00f6 into master Mar 18, 2021
@logachev logachev mentioned this pull request Mar 22, 2021
Closed
@logachev logachev deleted the azure_sdk_update branch March 25, 2021 06:35
@stefangordon stefangordon mentioned this pull request Oct 6, 2021
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stefangordon stefangordon stefangordon approved these changes

@kapilt kapilt Awaiting requested review from kapilt

@erwelch erwelch Awaiting requested review from erwelch

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

azure - migrate azure-mgmt-web to 1.0.0 azure - migrate to azure-keyvault 4.x package
2 participants
@logachev @stefangordon