Skip to content

PYTHON-5851 Run setup-uv-python.sh before install-dependencies.sh#2842

Merged
blink1073 merged 2 commits into
mongodb:masterfrom
blink1073:PYTHON-5851
Jun 3, 2026
Merged

PYTHON-5851 Run setup-uv-python.sh before install-dependencies.sh#2842
blink1073 merged 2 commits into
mongodb:masterfrom
blink1073:PYTHON-5851

Conversation

@blink1073

Copy link
Copy Markdown
Member

PYTHON-5851

Changes in this PR

Reorder script execution in setup-dev-env.sh so that setup-uv-python.sh (which configures UV_PYTHON) runs before install-dependencies.sh (which uses uv to install dependencies). Previously the ordering caused install-dependencies.sh to run without the correct Python version set.

Test Plan

Verified via Evergreen patch on other-hosts-rhel8-arm64-latest / test-no-toolchain-sync-noauth-nossl-standalone (patch 6a2055295e7fc100078cc5c3) — task that was previously failing now passes.

Checklist

Checklist for Author

  • Did you update the changelog (if necessary)?
  • Is there test coverage?
  • Is any followup work tracked in a JIRA ticket? If so, add link(s).

Checklist for Reviewer

  • Does the title of the PR reference a JIRA Ticket?
  • Do you fully understand the implementation? (Would you be comfortable explaining how this code works to someone else?)
  • Is all relevant documentation (README or docstring) updated?

@blink1073 blink1073 marked this pull request as ready for review June 3, 2026 17:48
@blink1073 blink1073 requested a review from a team as a code owner June 3, 2026 17:48
@blink1073 blink1073 requested a review from sophiayangDB June 3, 2026 17:48
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@blink1073

Copy link
Copy Markdown
Member Author

Link failure is tracked in https://jira.mongodb.org/browse/PYTHON-5859

@sophiayangDB

Copy link
Copy Markdown
Contributor

Is test.asynchronous.test_cursor.TestCursor.test_maxtime_ms_message a known failure? Seems to be failing on master as well.

@blink1073

Copy link
Copy Markdown
Member Author

Is test.asynchronous.test_cursor.TestCursor.test_maxtime_ms_message a known failure? Seems to be failing on master as well.

I fixed it as part of #2841

@sophiayangDB sophiayangDB left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM (assuming all tests pass)

@blink1073 blink1073 merged commit 483a211 into mongodb:master Jun 3, 2026
86 of 87 checks passed
@blink1073 blink1073 deleted the PYTHON-5851 branch June 3, 2026 20:40
aclark4life pushed a commit to aclark4life/mongo-python-driver that referenced this pull request Jun 3, 2026
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.

3 participants