Skip to content

{Core} Ping azure-mgmt-core version setup.py#18087

Merged
kairu-ms merged 1 commit into
Azure:devfrom
kairu-ms:azure-mgmt-core-ping-to-1-2
May 14, 2021
Merged

{Core} Ping azure-mgmt-core version setup.py#18087
kairu-ms merged 1 commit into
Azure:devfrom
kairu-ms:azure-mgmt-core-ping-to-1-2

Conversation

@kairu-ms
Copy link
Copy Markdown
Contributor

@kairu-ms kairu-ms commented May 14, 2021

Description

The pipeline of extension install pre version of packages https://github.com/Azure/azure-cli-extensions/blob/a0b24e846c3a9d74a5deddac5289753d5187869b/scripts/ci/test_source.sh#L6.

Recently this PR triggered a issue that the azure-mgmt-core 1.3.0b1 (preview version) conflict with azure-core 1.14.0 (stable version).

image

This PR will ping the version of azure-mgmt-core <= 1.3.0 in setup.py to workaround above issue.

Testing Guide

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.

@yonzhan
Copy link
Copy Markdown
Collaborator

yonzhan commented May 14, 2021

Core

@yonzhan yonzhan added this to the S187 milestone May 14, 2021
@kairu-ms kairu-ms merged commit 92100ab into Azure:dev May 14, 2021
@glaubitz
Copy link
Copy Markdown

glaubitz commented Jun 9, 2021

Is there an issue to track the actual fix for this incompatibility?

@kairu-ms
Copy link
Copy Markdown
Contributor Author

kairu-ms commented Jun 9, 2021

Is there an issue to track the actual fix for this incompatibility?

cannot import name `ChallengeAuthenticationPolicy`

@glaubitz
Copy link
Copy Markdown

glaubitz commented Jun 9, 2021

Funny, it seems I already reported this issue myself: #18025

@jiasli
Copy link
Copy Markdown
Member

jiasli commented Jun 15, 2021

- ping
+ pin

@jiasli
Copy link
Copy Markdown
Member

jiasli commented Jun 15, 2021

pip won't install beta versions without --pre. I think we should remove --pre from the extension pipeline instead:

https://github.com/Azure/azure-cli-extensions/blob/a0b24e846c3a9d74a5deddac5289753d5187869b/scripts/ci/test_source.sh#L6

We are already using post instead of dev (which needs --pre) for edge builds:

https://azurecliprod.blob.core.windows.net/edge/azure-cli/

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.

5 participants