Skip to content

Fix minimum dependency resolution against configured feeds - #48487

Draft
jenny (JennyPng) wants to merge 1 commit into
Azure:mainfrom
JennyPng:fix-mindep
Draft

Fix minimum dependency resolution against configured feeds#48487
jenny (JennyPng) wants to merge 1 commit into
Azure:mainfrom
JennyPng:fix-mindep

Conversation

@JennyPng

Copy link
Copy Markdown
Member

Description

Fixes minimum-dependency checks when a current local dev package conflicts with exact minimum runtime pins.

  • discovers Python-compatible versions from the configured Azure Artifacts simple index
  • removes the implicit Azure Artifacts-to-public-PyPI release metadata fallback
  • lets pip/uv backtrack published incompatible local dev packages in the same transaction as exact minimum pins
  • preserves compatible local requirements and exact installed-version verification
  • adds deterministic coverage for index compatibility, conflict handling, minimum/latest selection, and unsatisfiable resolution

Validation

  • 35 targeted resolver/index tests passed
  • 306 tool tests passed, 2 skipped, excluding the environment-dependent proxy integration
  • App Configuration mindependency: 181 tests passed with azure-core==1.37.0
  • Cosmos pip and uv minimum smoke runs retained azure-core==1.30.0 and resolved azure-identity==1.17.1
  • Cosmos uv latest smoke selected azure-core==1.40.0

Notes

Package-wide pylint contains pre-existing unrelated diagnostics. Changed files passed formatting and targeted validation. A missing package on the configured feed may currently surface the feed 404 rather than reaching the intended local-requirement fallback; this draft keeps that edge visible for review.

Use configured-feed Python compatibility metadata and delegate incompatible published dev dependency selection to pip or uv.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: d26d1aab-a9d9-4e83-a25d-0320fe04010d
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
10 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

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.

1 participant