Merged
Conversation
…back and live tests.
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates CI pipeline templates to authenticate against the public Azure Artifacts feed (with upstream enabled) so dependency installs can flow through the feed without intermittent 401s.
Changes:
- Move Azure Artifacts feed authentication into the shared
build-test.ymlsteps template. - Remove the now-redundant auth step from the
ci.tests.ymljob template. - Change
auth-dev-feed.ymlso the “UV authentication (disabled)” step is no longer explicitly skipped.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| eng/pipelines/templates/steps/build-test.yml | Adds early feed authentication to the common build/test step template. |
| eng/pipelines/templates/steps/auth-dev-feed.yml | Removes the explicit condition: false that previously prevented the UV-auth step from running. |
| eng/pipelines/templates/jobs/ci.tests.yml | Removes the standalone auth step since auth is now handled by build-test.yml. |
weshaggard
approved these changes
Mar 30, 2026
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…o ensure that no 404s occur using uv
scbedd
requested review from
annatisch,
iscai-msft,
johanste,
kashifkhan,
lmazuel,
pvaneck and
xiangyan99
as code owners
Member
Author
|
/azp run python - pullrequest |
Member
Author
|
/azp run python - cosmos - ci |
|
Azure Pipelines successfully started running 1 pipeline(s). |
1 similar comment
|
Azure Pipelines successfully started running 1 pipeline(s). |
scbedd
commented
Apr 4, 2026
…perly? are we accidentally clearing out variables in the test invocations?
…atchelf and just blowing up on all plats
…fault, and the proxy is not binding to an IPV6 address. As a result, on some mac runs, we are seeing failures to connect. the proxy is STARTED, but we can't see it. at least, so says the suspicions of copilot. let's see
API Change CheckAPIView identified API level changes in this PR and created the following API reviews |
mccoyp
approved these changes
Apr 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Everything going through our public feed, now that it supports upstream.