Skip to content

Conversation

@haitch
Copy link
Contributor

@haitch haitch commented May 11, 2021

Description

To bump AKS SDK dependency version, future PR will consume this newer version of SDK.

Testing Guide

update all recordings, existing test are working fine.

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.


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

@haitch haitch changed the title Haitao/aks sdk update [AKS] sdk update May 11, 2021
@northtyphoon
Copy link
Member

@haitch, does the pr include any change related to ACR? If yes, can you point me to the code?

@haitch
Copy link
Contributor Author

haitch commented May 11, 2021

@haitch, does the pr include any change related to ACR? If yes, can you point me to the code?

nevermind, the initial version have ACR recording file updated as well, I reverted those.

@fengzhou-msft fengzhou-msft merged commit 5434b87 into Azure:dev May 12, 2021
@xfz11
Copy link
Contributor

xfz11 commented May 13, 2021

Hi, the PR caused aks-preview extension test failed. Pls take a look

@FumingZhang
Copy link
Member

@haitch, @xfz11 the updated sdk version (11.2.0) make delete request with api version 2021-03-01 now, however the recordings in aks-preview records request with api version 2021-02-01. So this pr change is breaking aks-preview tests

@xfz11
Copy link
Contributor

xfz11 commented May 14, 2021

@yungezz

@yungezz
Copy link
Member

yungezz commented May 14, 2021

@haitch, @xfz11 the updated sdk version (11.2.0) make delete request with api version 2021-03-01 now, however the recordings in aks-preview records request with api version 2021-02-01. So this pr change is breaking aks-preview tests

hi @haitch this PR caused Azure CLI extension repo CI test failed, it blocks PRs in extension repo. Could you pls fix aks extension tests to make CI pass?

@haitch
Copy link
Contributor Author

haitch commented May 14, 2021

We have a fix PR pending to be merged: Azure/azure-cli-extensions#3378

@haitch, @xfz11 the updated sdk version (11.2.0) make delete request with api version 2021-03-01 now, however the recordings in aks-preview records request with api version 2021-02-01. So this pr change is breaking aks-preview tests

hi @haitch this PR caused Azure CLI extension repo CI test failed, it blocks PRs in extension repo. Could you pls fix aks extension tests to make CI pass?

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.

6 participants